Skip to content

fix(deployment-matrix): deploy mock-btg-server to anacleto - #987

Merged
fredcamaral merged 1 commit into
developfrom
fix/anacleto-mock-btg-matrix
Oct 27, 2026
Merged

fredcamaral merged 1 commit into
developfrom
fix/anacleto-mock-btg-matrix

Conversation

@fredcamaral

Copy link
Copy Markdown
Member
Lerian

GitHub Actions Shared Workflows


Description

Every develop, release-candidate and main release of the Pix BTG plugin (plugin-br-pix-indirect-btg) now bumps the mock BTG server on the six anacleto chaos and fuzzing lanes too, not only on benedita.

LerianStudio/lerian-internal-gitops#3754 restored mock-btg-server in the anacleto chaos and fuzzing lanes (dev-st, stg-st, prd-st, namespaces mock-btg-server-<lane>-<tier>). The plugin's update_mock_gitops job resolves its targets from config/deployment-matrix.yml, where the app was listed under benedita only, so those lanes were never bumped and kept the tags they were created with.

Affected: config/deployment-matrix.yml only. mock-btg-server joins clusters.anacleto.apps, and the registry comment that said "benedita only" now says benedita and anacleto. No workflow changes.

What each release type writes after this change (key .mock.image.tag, the plugin's yaml_key_mappings):

Release anacleto values files written benedita (unchanged)
beta chaos/dev-st, fuzzing/dev-st dev-st
rc chaos/stg-st, fuzzing/stg-st stg-st
stable chaos/prd-st, fuzzing/prd-st none (no prd-st values file, as before)

All paths are environments/anacleto/helmfile/applications/<lane>/<tier>/mock-btg-server/values.yaml. The -mt expansions have no values file, so they are skipped with a warning, as they are for the plugin itself on anacleto. ArgoCD sync targets resolve to anacleto-mock-btg-server-<lane>-<tier>, which match the six Application names added in #3754.

Takes effect for callers once this reaches main: gitops-update.yml reads the matrix from main by default (deployment_matrix_ref). The plugin pins gitops-update.yml@v1.75.0 on all three release branches, so it needs no pin bump.

Type of Change

  • feat: New workflow or new input/output/step in an existing workflow
  • fix: Bug fix in a workflow (incorrect behavior, broken step, wrong condition)
  • perf: Performance improvement (e.g. caching, parallelism, reduced steps)
  • refactor: Internal restructuring with no behavior change
  • docs: Documentation only (README, docs/, inline comments)
  • ci: Changes to self-CI (workflows under .github/workflows/ that run on this repo)
  • chore: Dependency bumps, config updates, maintenance
  • test: Adding or updating tests
  • BREAKING CHANGE: Callers must update their configuration after this PR

Breaking Changes

None.

Testing

  • YAML syntax validated locally
  • Triggered a real workflow run on a caller repository using @this-branch or the beta tag
  • Verified all existing inputs still work with default values
  • Confirmed no secrets or tokens are printed in logs
  • Checked that unrelated workflows are not affected

Checks run locally:

  • Deployment Matrix Lint (the src/lint/deployment-matrix script): Deployment matrix is valid (49 apps registered, 2 clusters defined).
  • yamllint -c .yamllint.yml config/deployment-matrix.yml: exit 0; one existing line-length warning on line 115, not on the lines this PR touches.
  • typos config/deployment-matrix.yml: clean.
  • Consumer dry run: I ran the real resolve_clusters and apply_tags step scripts from gitops-update.yml@v1.75.0 against this matrix and a snapshot of lerian-internal-gitops main (929a02d2), with a fake mock-btg-server.tag artifact and the plugin's inputs, for a beta, an rc and a stable tag. Each run changed exactly the files in the table above, one tag: line each, and produced the sync targets listed. Nothing else in the gitops tree changed.
  • Other matrix consumer: gitops-chart-update (chart pins). It has no caller for helm-internal, where the mock-btg-server-helm chart lives, so this entry does not change any chart pin.

Caller repo / workflow run: none. The real path runs on the next Pix BTG plugin tag after this reaches main.

Related Issues

Follows LerianStudio/lerian-internal-gitops#3754.

lerian-internal-gitops#3754 restored mock-btg-server in the six anacleto
chaos and fuzzing -st lanes, but the matrix listed it on benedita only, so
the Pix BTG plugin's mock image bump never reached them. Listing it under
anacleto makes every beta, rc and stable tag write .mock.image.tag in
chaos/<env>-st and fuzzing/<env>-st; the -mt paths have no values file and
are skipped with a warning, as for the plugin itself.
@fredcamaral
fredcamaral requested a review from a team as a code owner October 27, 2026 18:01
@coderabbitai

coderabbitai Bot commented Oct 27, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@lerian-studio

Copy link
Copy Markdown
Contributor

🔍 Lint Analysis

Check Files Scanned Status
YAML Lint 1 file(s) ✅ success
Action Lint no changes ⏭️ skipped
Pinned Actions no changes ⏭️ skipped
Markdown Link Check no changes ⏭️ skipped
Spelling Check 1 file(s) ✅ success
Shell Check no changes ⏭️ skipped
README Check no changes ⏭️ skipped
Composite Schema no changes ⏭️ skipped
Deployment Matrix 1 file(s) ✅ success

🔍 View full scan logs

@lerian-studio lerian-studio added size/XS PR changes < 50 lines deployment-matrix Changes to the canonical deployment matrix (config/deployment-matrix.yml) labels Oct 27, 2026
@lerian-studio

Copy link
Copy Markdown
Contributor

🔍 PR Validation Summary

✅ PR Mergeable — no blocking failures

Check Status Blocking
Source Branch ✅ success yes
PR Title ✅ success yes
PR Description ✅ success yes
Breaking Change Guard ✅ success yes
Commit Signatures ✅ success yes
PR Size ✅ success no
Auto Labels ✅ success no
PR Metadata ✅ success no

🔍 View workflow run

@radagast-lerian radagast-lerian Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Radagast · approved · 5f207a1 · nothing blocking.

@fredcamaral
fredcamaral merged commit 7264e32 into develop Oct 27, 2026
47 of 48 checks passed
@github-actions
github-actions Bot deleted the fix/anacleto-mock-btg-matrix branch October 27, 2026 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deployment-matrix Changes to the canonical deployment matrix (config/deployment-matrix.yml) size/XS PR changes < 50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants