Skip to content

feat: update setup URL in NOTES.txt to use dynamic values #449

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 23, 2025

Conversation

plakyda-codefresh
Copy link
Member

What

This PR updates the GitOps Runtime installation completion URL to open a specific drawer instead of just the runtime list, and it includes additional parameters to manage the installation flow.

Notes

Drawer opened: The URL now opens the "install-runtime-wizard" drawer instead of the runtime list.

Waiting for events: The waitEvents=true parameter is added to signal that the process should wait for the runtime events to be completed.

Dynamic URL: The URL dynamically includes:

{{ .Values.global.codefresh.url }} – The base URL for the Codefresh platform.

{{ .Values.global.codefresh.accountId }} – The account ID for user-specific routing.

{{ .Values.global.runtime.name }} – The name of the runtime being installed.

Conditional installation mode: Based on whether argo-cd is enabled or not, the installation mode is adjusted:

If argo-cd is enabled: mode=new

Otherwise: mode=byoa

@plakyda-codefresh plakyda-codefresh requested a review from a team as a code owner April 15, 2025 09:50
@plakyda-codefresh plakyda-codefresh changed the title fix: update setup URL in NOTES.txt to use dynamic values feat: update setup URL in NOTES.txt to use dynamic values Apr 15, 2025
@plakyda-codefresh
Copy link
Member Author

/e2e

@plakyda-codefresh plakyda-codefresh merged commit 834dfe2 into main Apr 23, 2025
2 checks passed
@plakyda-codefresh plakyda-codefresh deleted the feat/post-install-notes branch April 23, 2025 09:01
ilia-medvedev-codefresh pushed a commit that referenced this pull request May 14, 2025
* fix: update setup URL in NOTES.txt to use dynamic values

* fix: correct formatting in setup URL in NOTES.txt

* fix: update conditional check for argo-cd in NOTES.txt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants