Skip to content

Commit 4b3b5b4

Browse files
committed
Merge remote-tracking branch 'origin/feat/CR-30516-promote-app-with-pr' into fix/CR-32117-update-status-err-handling
2 parents ef667d2 + cab5b98 commit 4b3b5b4

File tree

2 files changed

+67
-16
lines changed

2 files changed

+67
-16
lines changed

charts/gitops-runtime/templates/_components/gitops-operator/crds/promotiontasks.yaml

Lines changed: 61 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -561,6 +561,8 @@ spec:
561561
promoteAppWithPR:
562562
properties:
563563
appStatus:
564+
description: APPStatus holds the status of the application
565+
after the PR was merged
564566
properties:
565567
health:
566568
description: Represents resource health status
@@ -583,6 +585,8 @@ spec:
583585
- sync
584586
type: object
585587
commitInfo:
588+
description: CommitInfo holds information about the commit
589+
that was created when the PR was merged
586590
properties:
587591
commitAuthorId:
588592
properties:
@@ -609,13 +613,66 @@ spec:
609613
description: PromoteAppWithPRPhase
610614
enum:
611615
- Pending
612-
- PRCreated
616+
- Suspended
613617
- Syncing
614618
- Complete
615619
type: string
616-
prUrl:
617-
format: uri
618-
type: string
620+
prInfo:
621+
description: PRInfo holds information about the created
622+
pull request
623+
properties:
624+
author:
625+
description: Pull request author
626+
type: string
627+
avatarUrl:
628+
description: Pull request author avatar url
629+
type: string
630+
baseBranch:
631+
description: Pull request base branch
632+
type: string
633+
createdAt:
634+
description: Pull request created at
635+
format: date-time
636+
type: string
637+
description:
638+
description: Pull request description
639+
type: string
640+
headBranch:
641+
description: Pull request head branch
642+
type: string
643+
id:
644+
description: Pull request id
645+
type: integer
646+
isMerged:
647+
description: Is pull request merged
648+
type: boolean
649+
mergeCommitSha:
650+
description: Commit sha from the pull request merge
651+
maxLength: 40
652+
minLength: 7
653+
pattern: ^[A-Fa-f0-9]+$
654+
type: string
655+
repo:
656+
description: Pull request repo name
657+
type: string
658+
state:
659+
description: Pull request state
660+
enum:
661+
- Open
662+
- Closed
663+
type: string
664+
title:
665+
description: Pull request title
666+
type: string
667+
url:
668+
description: Pull request url
669+
format: uri
670+
type: string
671+
required:
672+
- isMerged
673+
- state
674+
- url
675+
type: object
619676
required:
620677
- phase
621678
type: object

charts/gitops-runtime/values.yaml

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -189,11 +189,10 @@ global:
189189
scrapeTimeout: 10s
190190
labels: {}
191191
config: {}
192-
193192
anchors:
194193
common-envs:
195-
# -- Telemetry configuration
196194
- &otel-config
195+
# -- Telemetry configuration
197196
# -- Base endpoint URL for all OpenTelemetry signals.
198197
# Ref: https://opentelemetry.io/docs/languages/sdk-configuration/otlp-exporter/
199198
OTEL_EXPORTER_OTLP_ENDPOINT: 'http://localhost:4317'
@@ -227,7 +226,6 @@ anchors:
227226
OTEL_EXPORTER_PROMETHEUS_PORT: '9464'
228227
# -- Emit the stable HTTP and networking OTel conventions if CF_TELEMETRY_OTEL_ALLOW_HTTP_INSTRUMENTATION=true.
229228
OTEL_SEMCONV_STABILITY_OPT_IN: 'http'
230-
231229
# -------------------------------------------------------------------------------------------------------------------------
232230
# Installer
233231
# -------------------------------------------------------------------------------------------------------------------------
@@ -272,26 +270,22 @@ sealed-secrets:
272270
argo-cd:
273271
enabled: true
274272
fullnameOverride: argo-cd
275-
276273
notifications:
277274
enabled: false
278-
279275
redis:
280276
## Redis image
281277
image:
282278
# -- Redis repository
283279
repository: ecr-public.aws.com/docker/library/redis
284280
# -- Redis tag
285281
tag: 8.2.2-alpine
286-
287282
redis-ha:
288283
## Redis-ha image
289284
image:
290285
# -- Redis repository
291286
repository: ecr-public.aws.com/docker/library/redis
292287
# -- Redis tag
293288
tag: 8.2.2-alpine
294-
295289
configs:
296290
cm:
297291
timeout.reconciliation: 20s
@@ -570,7 +564,7 @@ app-proxy:
570564
# -- Cors settings for app-proxy. This is the list of allowed domains for platform (comma separated).
571565
cors: "https://g.codefresh.io"
572566
env:
573-
<<:
567+
!!merge <<:
574568
- *otel-config
575569
# -- Level of logging for app-proxy
576570
CF_TELEMETRY_LOGS_LEVEL: 'info'
@@ -688,9 +682,9 @@ gitops-operator:
688682
# -- defaults
689683
registry: quay.io
690684
repository: codefresh/codefresh-gitops-operator
691-
tag: "293f24f"
685+
tag: "6882325"
692686
env:
693-
<<:
687+
!!merge <<:
694688
- *otel-config
695689
GITOPS_OPERATOR_VERSION: 0.11.1
696690
serviceAccount:
@@ -859,11 +853,11 @@ redis:
859853
event-reporters:
860854
cluster-event-reporter:
861855
env:
862-
<<:
856+
!!merge <<:
863857
- *otel-config
864858
runtime-event-reporter:
865859
env:
866-
<<:
860+
!!merge <<:
867861
- *otel-config
868862
# -- Redis-HA subchart replaces custom redis deployment when `redis-ha.enabled=true`
869863
# Ref: https://github.com/DandyDeveloper/charts/blob/master/charts/redis-ha/values.yaml

0 commit comments

Comments
 (0)