Skip to content

Add Ironic repo#517

Merged
mnasiadka merged 1 commit into
mainfrom
conf/import_ironic
May 18, 2026
Merged

Add Ironic repo#517
mnasiadka merged 1 commit into
mainfrom
conf/import_ironic

Conversation

@dougszumski
Copy link
Copy Markdown
Member

This is for back-porting to epoxy the ramdisk deploy feature (diskless boot)

@dougszumski dougszumski requested a review from a team as a code owner May 18, 2026 14:01
@dougszumski dougszumski force-pushed the conf/import_ironic branch from 6b603d6 to e8623a9 Compare May 18, 2026 14:02
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 ironic repository to the Ansible source repositories and Terraform configuration. The reviewer suggests reordering the ironic entry in the Ansible configuration to maintain alphabetical consistency with the Terraform file for better maintainability.

Comment thread ansible/inventory/group_vars/all/source-repositories Outdated
@dougszumski dougszumski force-pushed the conf/import_ironic branch from e8623a9 to 5bdff5f Compare May 18, 2026 14:03
@stackhpc-ci
Copy link
Copy Markdown
Contributor

stackhpc-ci commented May 18, 2026

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: @dougszumski, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub

@Alex-Welsh
Copy link
Copy Markdown
Member

Needs #516 first, otherwise I think it's going to break things

@Alex-Welsh
Copy link
Copy Markdown
Member

But other than that, lgtm

@Alex-Welsh
Copy link
Copy Markdown
Member

#516 merged, just want to wait until CI finishes now https://github.com/stackhpc/stackhpc-release-train/actions/runs/26039990575/job/76548597370
Parallel runs break things

@mnasiadka mnasiadka merged commit 0d49233 into main May 18, 2026
5 of 7 checks passed
@mnasiadka mnasiadka deleted the conf/import_ironic branch May 18, 2026 14:48
@Alex-Welsh Alex-Welsh mentioned this pull request May 18, 2026
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.

4 participants