-
Notifications
You must be signed in to change notification settings - Fork 8
feat: support promotion with pull request #880
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
Changes from 5 commits
23b896c
7669aa4
ce93352
95345cd
458594d
cab5b98
6ed12d1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -189,11 +189,10 @@ global: | |
| scrapeTimeout: 10s | ||
| labels: {} | ||
| config: {} | ||
|
|
||
| anchors: | ||
| common-envs: | ||
| # -- Telemetry configuration | ||
| - &otel-config | ||
| # -- Telemetry configuration | ||
| # -- Base endpoint URL for all OpenTelemetry signals. | ||
| # Ref: https://opentelemetry.io/docs/languages/sdk-configuration/otlp-exporter/ | ||
| OTEL_EXPORTER_OTLP_ENDPOINT: 'http://localhost:4317' | ||
|
|
@@ -227,7 +226,6 @@ anchors: | |
| OTEL_EXPORTER_PROMETHEUS_PORT: '9464' | ||
| # -- Emit the stable HTTP and networking OTel conventions if CF_TELEMETRY_OTEL_ALLOW_HTTP_INSTRUMENTATION=true. | ||
| OTEL_SEMCONV_STABILITY_OPT_IN: 'http' | ||
|
|
||
| # ------------------------------------------------------------------------------------------------------------------------- | ||
| # Installer | ||
| # ------------------------------------------------------------------------------------------------------------------------- | ||
|
|
@@ -272,26 +270,22 @@ sealed-secrets: | |
| argo-cd: | ||
| enabled: true | ||
| fullnameOverride: argo-cd | ||
|
|
||
| notifications: | ||
| enabled: false | ||
|
|
||
| redis: | ||
| ## Redis image | ||
| image: | ||
| # -- Redis repository | ||
| repository: ecr-public.aws.com/docker/library/redis | ||
| # -- Redis tag | ||
| tag: 8.2.2-alpine | ||
|
|
||
| redis-ha: | ||
| ## Redis-ha image | ||
| image: | ||
| # -- Redis repository | ||
| repository: ecr-public.aws.com/docker/library/redis | ||
| # -- Redis tag | ||
| tag: 8.2.2-alpine | ||
|
|
||
| configs: | ||
| cm: | ||
| timeout.reconciliation: 20s | ||
|
|
@@ -570,7 +564,7 @@ app-proxy: | |
| # -- Cors settings for app-proxy. This is the list of allowed domains for platform (comma separated). | ||
| cors: "https://g.codefresh.io" | ||
| env: | ||
| <<: | ||
| !!merge <<: | ||
| - *otel-config | ||
| # -- Level of logging for app-proxy | ||
| CF_TELEMETRY_LOGS_LEVEL: 'info' | ||
|
|
@@ -688,9 +682,9 @@ gitops-operator: | |
| # -- defaults | ||
| registry: quay.io | ||
| repository: codefresh/codefresh-gitops-operator | ||
| tag: "293f24f" | ||
| tag: "6882325" | ||
| env: | ||
| <<: | ||
| !!merge <<: | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. what is this?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. i'm also wondering. this all came from the automation commit that bumped the operator image tag.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
| - *otel-config | ||
| GITOPS_OPERATOR_VERSION: 0.11.1 | ||
| serviceAccount: | ||
|
|
@@ -859,11 +853,11 @@ redis: | |
| event-reporters: | ||
| cluster-event-reporter: | ||
| env: | ||
| <<: | ||
| !!merge <<: | ||
| - *otel-config | ||
| runtime-event-reporter: | ||
| env: | ||
| <<: | ||
| !!merge <<: | ||
| - *otel-config | ||
| # -- Redis-HA subchart replaces custom redis deployment when `redis-ha.enabled=true` | ||
| # Ref: https://github.com/DandyDeveloper/charts/blob/master/charts/redis-ha/values.yaml | ||
|
|
||
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.
remove this