Skip to content

Add magnum-capi-helm repository#536

Closed
priteau wants to merge 2 commits into
mainfrom
magnum-capi-helm
Closed

Add magnum-capi-helm repository#536
priteau wants to merge 2 commits into
mainfrom
magnum-capi-helm

Conversation

@priteau

@priteau priteau commented Jun 11, 2026

Copy link
Copy Markdown
Member

No description provided.

@priteau priteau self-assigned this Jun 11, 2026
@priteau priteau requested a review from a team as a code owner June 11, 2026 15:38

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request adds configuration for the magnum-capi-helm repository in the Ansible inventory. The reviewer suggested also ignoring the master branch to prevent sync issues, and adding the repository to terraform/github/terraform.tfvars.json to avoid CI validation failures.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment on lines +263 to +269
ignored_releases:
- yoga
- zed
- 2023.1
- 2024.1
- 2025.1
- 2026.1

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Since magnum-capi-helm ignores all stable release branches, it should also ignore master to prevent the sync role from attempting to manage a stackhpc/master branch. This is consistent with other repositories in this file that ignore all stable releases (e.g., magnum-ui, barbican, ironic-ui).

Additionally, please ensure that magnum-capi-helm is also added to terraform/github/terraform.tfvars.json under the appropriate team/group to prevent the validate-source-repos.py script from failing in CI.

    ignored_releases:
      - yoga
      - zed
      - 2023.1
      - 2024.1
      - 2025.1
      - 2026.1
      - master

@priteau priteau force-pushed the magnum-capi-helm branch from 4599de8 to 80f0ced Compare June 11, 2026 15:40
@priteau priteau force-pushed the magnum-capi-helm branch from 80f0ced to 650ec36 Compare June 11, 2026 15:43
@priteau priteau force-pushed the magnum-capi-helm branch from 7c97bbf to 8a9c97f Compare June 11, 2026 15:47
@stackhpc-ci

Copy link
Copy Markdown
Contributor

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖success

Validation Output

Success! The configuration is valid.


Terraform Plan 📖success

Show Plan

undefined

Pusher: @priteau, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub

@priteau

priteau commented Jun 11, 2026

Copy link
Copy Markdown
Member Author

Not needed

@priteau priteau closed this Jun 11, 2026
@priteau priteau deleted the magnum-capi-helm branch June 11, 2026 16:02
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