Skip to content

Manage manila fork#504

Merged
Alex-Welsh merged 1 commit intomainfrom
manila
Apr 22, 2026
Merged

Manage manila fork#504
Alex-Welsh merged 1 commit intomainfrom
manila

Conversation

@mnasiadka
Copy link
Copy Markdown
Member

No description provided.

@mnasiadka mnasiadka requested a review from a team as a code owner April 21, 2026 16:21
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

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 the manila repository configuration to the Ansible inventory, including ignored releases and workflow settings. A critical issue was identified regarding the community_files structure: it is currently defined as a list, but the associated Ansible task expects a dictionary, which will lead to a runtime failure.

Comment thread ansible/inventory/group_vars/all/source-repositories
@stackhpc-ci
Copy link
Copy Markdown
Contributor

stackhpc-ci commented Apr 21, 2026

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖success

Validation Output


Warning: Deprecated attribute

  on repositories.tf line 20, in resource "github_repository" "repositories":
  20:       has_downloads,

The attribute "has_downloads" is deprecated. Refer to the provider
documentation for details.
Success! The configuration is valid, but there were some validation warnings
as shown above.


Terraform Plan 📖success

Show Plan

undefined

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

@Alex-Welsh Alex-Welsh merged commit 92a3625 into main Apr 22, 2026
4 of 5 checks passed
@Alex-Welsh Alex-Welsh deleted the manila branch April 22, 2026 12:11
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