-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
fix: add support for overwriting the UI extensions path #21027
base: master
Are you sure you want to change the base?
fix: add support for overwriting the UI extensions path #21027
Conversation
🔴 Preview Environment stopped on BunnyshellSee: Environment Details | Pipeline Logs Available commands (reply to this comment):
|
705d1a0
to
852edc8
Compare
Reason for this change:
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #21027 +/- ##
=========================================
Coverage ? 53.46%
=========================================
Files ? 339
Lines ? 57185
Branches ? 0
=========================================
Hits ? 30575
Misses ? 23982
Partials ? 2628 ☔ View full report in Codecov by Sentry. |
852edc8
to
c320c2e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add it along the other params here
command.Flags().BoolVar(&insecure, "insecure", env.ParseBoolFromEnv("ARGOCD_SERVER_INSECURE", false), "Run server without TLS") |
f8a69ae
to
261b0fa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vasilegroza , can you also add the new variable to argocd-server-deployment.yaml similar to
argo-cd/manifests/base/server/argocd-server-deployment.yaml
Lines 265 to 270 in 2f51067
- name: ARGOCD_SERVER_WEBHOOK_PARALLELISM_LIMIT | |
valueFrom: | |
configMapKeyRef: | |
name: argocd-cmd-params-cm | |
key: server.webhook.parallelism.limit | |
optional: true |
via the environment variable ARGOCD_EXTENSIONS_SHARED_PATH Signed-off-by: Vasile Groza <[email protected]>
Signed-off-by: Vasile Groza <[email protected]>
261b0fa
to
cc0e74f
Compare
cc0e74f
to
71edc6b
Compare
patched. |
…nfigMap Signed-off-by: Vasile Groza <[email protected]>
71edc6b
to
d40f2ae
Compare
Signed-off-by: vasilegroza <[email protected]>
e71ab61
to
ce9daf7
Compare
Signed-off-by: vasilegroza <[email protected]>
@ishitasequeira can you review it one more time? |
via the environment variable
ARGOCD_EXTENSIONS_SHARED_PATH
Checklist: