Skip to content
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

[GitOps] Add kbc-stacks image tag update trigger, cleanup GCP provisioning #2174

Merged
merged 1 commit into from
Dec 9, 2024

Conversation

wokoman
Copy link
Contributor

@wokoman wokoman commented Dec 9, 2024

Relates to ST-2491.

Po vzoru predchozich GitOpsu: #2158 a #2171

  • Meni se tag imagu 🫡
  • Pridavam trigger pro update Helm charty v kbc-stacks
  • A mazu starej GCP provisioning.

@wokoman wokoman requested a review from kacurez December 9, 2024 08:19
@wokoman wokoman self-assigned this Dec 9, 2024
Copy link

github-actions bot commented Dec 9, 2024

Apps Proxy Kubernetes Diff [CI]

Between base 6a1d822 ⬅️ head 8900112.

Expand
--- /tmp/artifacts/test-k8s-state.old.json.processed.kv	2024-12-09 08:26:13.642425518 +0000
+++ /tmp/artifacts/test-k8s-state.new.json.processed.kv	2024-12-09 08:26:13.762426173 +0000
@@ -114 +114 @@
-<Deployment/apps-proxy>.spec.template.spec.containers[0].image = "docker.io/keboola/apps-proxy:6a1d822-1733732435";
+<Deployment/apps-proxy>.spec.template.spec.containers[0].image = "docker.io/keboola/apps-proxy:8900112-1733732711";
@@ -497,3 +497,3 @@
-<Pod/apps-proxy-<hash>>.spec.containers[0].image = "docker.io/keboola/apps-proxy:6a1d822-1733732435";
-<Pod/apps-proxy-<hash>>.spec.containers[0].image = "docker.io/keboola/apps-proxy:6a1d822-1733732435";
-<Pod/apps-proxy-<hash>>.spec.containers[0].image = "docker.io/keboola/apps-proxy:6a1d822-1733732435";
+<Pod/apps-proxy-<hash>>.spec.containers[0].image = "docker.io/keboola/apps-proxy:8900112-1733732711";
+<Pod/apps-proxy-<hash>>.spec.containers[0].image = "docker.io/keboola/apps-proxy:8900112-1733732711";
+<Pod/apps-proxy-<hash>>.spec.containers[0].image = "docker.io/keboola/apps-proxy:8900112-1733732711";
@@ -924 +924 @@
-<ReplicaSet/apps-proxy-<hash>>.spec.template.spec.containers[0].image = "docker.io/keboola/apps-proxy:6a1d822-1733732435";
+<ReplicaSet/apps-proxy-<hash>>.spec.template.spec.containers[0].image = "docker.io/keboola/apps-proxy:8900112-1733732711";


(see artifacts in the Github Action for more information)

Copy link
Member

@kacurez kacurez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, pls potom nezabudni updatnut AzureDevOps pipeline trigger pre AWS a Azure kde ten trigger je nastavene na stary tag apps-proxy-v*

@wokoman wokoman marked this pull request as ready for review December 9, 2024 08:36
Copy link
Contributor

@Matovidlo Matovidlo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Mate si nastavit .editorconfig 😄

@wokoman wokoman merged commit 95cb46b into main Dec 9, 2024
13 checks passed
@wokoman wokoman deleted the michal-st-2491-apps-proxy-cd-cleanup branch December 9, 2024 09:58
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.

3 participants