Skip to content

ci: write image tag to global.image.tag#3081

Merged
levivannoort merged 1 commit into
mainfrom
chore/global-image-tag
Jul 3, 2026
Merged

ci: write image tag to global.image.tag#3081
levivannoort merged 1 commit into
mainfrom
chore/global-image-tag

Conversation

@levivannoort

Copy link
Copy Markdown
Member

The applications repo moves image to global.image (generic chart 1.7.0 — see appwrite-labs/helm-charts#17); this updates the tag-bump automation to write the single canonical path .global.image.tag in default.yaml.

Merge order: land appwrite-labs/assets-applications#29 first. Until then, an old-style per-alias write still overrides global, so deploys stay correct in either interim state (a stale per-alias tag key would just need removing).

🤖 Generated with Claude Code

The applications repo now sets the image once under `global.image` (generic
chart 1.7.0 fallbacks); the bump writes the single canonical path
`.global.image.tag` in default.yaml instead of per-alias keys.

Merge AFTER appwrite-labs/assets-applications#29 — until then per-alias writes
still override global, so deploys stay correct either way.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@appwrite

appwrite Bot commented Jul 3, 2026

Copy link
Copy Markdown

Appwrite Website

Project ID: 69d7efb00023389e8d27

Sites (1)
Site Status Logs Preview QR
 website
69d7f2670014e24571ca
Ready Ready View Logs Preview URL QR Code

Website (appwrite/website)

Project ID: 684969cb000a2f6c0a02

Sites (1)
Site Status Logs Preview QR
 website
68496a17000f03d62013
Processing Processing View Logs Preview URL QR Code


Tip

Silent mode disables those chatty PR comments if you prefer peace and quiet

@levivannoort levivannoort marked this pull request as ready for review July 3, 2026 11:47
@greptile-apps

greptile-apps Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR updates the image tag write path in both staging and production deployment workflows from .website.image.tag to .global.image.tag in default.yaml, aligning with the restructured helm chart (generic chart 1.7.0) that moves image under global.

  • Both staging.yml and production.yml receive the identical one-line change to the yq command — no other workflow logic is affected.
  • As noted in the PR description, assets-applications#29 must land before this PR to ensure the target YAML structure exists; until then the old per-alias key remains in place and deploys continue to function correctly.

Confidence Score: 5/5

The change is a single-path rename in a yq write command, applied symmetrically to both staging and production; the PR description clearly documents the required merge order and backward-compatibility guarantees.

Both workflows receive the same minimal, well-scoped update. The yq syntax is correct, the deployment logic around it is untouched, and the PR description explains exactly how interim state is handled while the dependent repo PR is pending.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/production.yml Updates the yq path from .website.image.tag to .global.image.tag to match the new canonical key in the helm chart (generic chart 1.7.0); all other workflow steps are unchanged.
.github/workflows/staging.yml Identical one-line update to yq path from .website.image.tag to .global.image.tag; mirrors the production.yml change exactly.

Reviews (1): Last reviewed commit: "ci: write image tag to global.image.tag" | Re-trigger Greptile

@ChiragAgg5k ChiragAgg5k left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reviewed the deployment workflow updates. Both environments now write the image tag to the global chart path while keeping the existing commit flow intact. Looks good.

@levivannoort levivannoort merged commit a7fd8ad into main Jul 3, 2026
6 of 7 checks passed
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