Skip to content

Ironic overcloud hypervisor deployment#1089

Closed
holliefae wants to merge 16 commits into
stackhpc/2024.1from
baremetal-hypervisor
Closed

Ironic overcloud hypervisor deployment#1089
holliefae wants to merge 16 commits into
stackhpc/2024.1from
baremetal-hypervisor

Conversation

@holliefae

@holliefae holliefae commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

Deployment playbook for baremetal instances to be used as hypervisors.

TODO: Documentation refactoring (separate everything prior to hypervisors?)

@holliefae holliefae changed the title Ironic overcloud hypervisor deployment WIP: Ironic overcloud hypervisor deployment Jun 6, 2024
Comment thread etc/kayobe/ansible/deploy-baremetal-instance.yml Outdated
@holliefae holliefae force-pushed the baremetal-hypervisor branch from e2e2213 to e24e4ff Compare June 6, 2024 10:45
@holliefae holliefae self-assigned this Jun 6, 2024
@holliefae holliefae force-pushed the baremetal-hypervisor branch from e24e4ff to 0493247 Compare June 7, 2024 14:50
@holliefae holliefae force-pushed the baremetal-hypervisor branch from c01bda2 to 1218ec8 Compare June 7, 2024 16:07
@holliefae holliefae force-pushed the baremetal-hypervisor branch 2 times, most recently from d02d07d to 7598716 Compare June 19, 2024 11:14
@holliefae holliefae force-pushed the baremetal-hypervisor branch 2 times, most recently from ed6155e to e850079 Compare June 19, 2024 11:40
Comment thread doc/source/configuration/ironic.rst Outdated
Comment thread doc/source/configuration/ironic.rst Outdated
Comment thread doc/source/configuration/ironic.rst Outdated
@holliefae holliefae force-pushed the baremetal-hypervisor branch from e850079 to d7e920a Compare June 27, 2024 11:18
@Alex-Welsh

Copy link
Copy Markdown
Member

@assumptionsandg remember to mark conversations as resolved when you've completed them! Makes it much easier for reviewers to know when to re-review.

Is this still a draft? Who do you need a review from to get it merged?

@holliefae

Copy link
Copy Markdown
Contributor Author

@assumptionsandg remember to mark conversations as resolved when you've completed them! Makes it much easier for reviewers to know when to re-review.

Is this still a draft? Who do you need a review from to get it merged?

It's still WIP on the client side of things, hence leaving it as a draft here. We could merge what's here now and interate/fix things as the client work progresses if what's here will be useful to people.

@Alex-Welsh

Alex-Welsh commented Sep 9, 2024

Copy link
Copy Markdown
Member

@assumptionsandg remember to mark conversations as resolved when you've completed them! Makes it much easier for reviewers to know when to re-review.
Is this still a draft? Who do you need a review from to get it merged?

It's still WIP on the client side of things, hence leaving it as a draft here. We could merge what's here now and interate/fix things as the client work progresses if what's here will be useful to people.

I'd be happy with it as long as you add a big warning at the top saying the docs are still WIP. Something is better than nothing

@holliefae holliefae force-pushed the baremetal-hypervisor branch from 72cf9c8 to eb82723 Compare September 9, 2024 10:52
@holliefae holliefae changed the title WIP: Ironic overcloud hypervisor deployment Ironic overcloud hypervisor deployment Sep 9, 2024
@holliefae holliefae marked this pull request as ready for review September 9, 2024 10:57
@holliefae holliefae requested a review from a team as a code owner September 9, 2024 10:57
@holliefae holliefae force-pushed the baremetal-hypervisor branch from c484916 to 7f42dee Compare September 9, 2024 11:00
@Alex-Welsh

Copy link
Copy Markdown
Member

@assumptionsandg you need to tag people for re-review when you make changes. This has been sat around for two months and I don't think anyone has looked at it

@Alex-Welsh Alex-Welsh 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.

These are good docs. I've got a lot of comments but they're mostly just small formatting and grammar fixes.

Comment thread doc/source/configuration/ironic.rst Outdated
Comment thread doc/source/configuration/ironic.rst
Comment thread doc/source/configuration/ironic.rst Outdated
Comment thread doc/source/configuration/ironic.rst Outdated
Comment thread doc/source/configuration/ironic.rst Outdated
Comment thread doc/source/configuration/ironic.rst Outdated
Comment thread etc/kayobe/ansible/deploy-baremetal-instance.yml Outdated
Comment thread etc/kayobe/ansible/deploy-baremetal-instance.yml
Comment thread etc/kayobe/ansible/deploy-baremetal-instance.yml Outdated
Comment thread releasenotes/notes/baremetal-hypervisors-fc1091bd507e809b.yaml
Co-authored-by: Alex-Welsh <112560678+Alex-Welsh@users.noreply.github.com>
@Alex-Welsh Alex-Welsh added documentation Improvements or additions to documentation antelope labels Nov 15, 2024
@Alex-Welsh

Copy link
Copy Markdown
Member

@assumptionsandg let me know when this is ready for a re-review

@holliefae holliefae changed the base branch from stackhpc/2023.1 to stackhpc/2024.1 February 3, 2025 14:28
@holliefae holliefae added Caracal and removed antelope labels Feb 3, 2025
@holliefae holliefae force-pushed the baremetal-hypervisor branch from 93ea11f to 8f2fac3 Compare February 5, 2025 14:25
@Alex-Welsh

Copy link
Copy Markdown
Member

@assumptionsandg ready for review?

Comment thread etc/kayobe/ansible/deploy-baremetal-instance.yml Outdated
@holliefae holliefae marked this pull request as draft August 13, 2025 08:29
@holliefae

Copy link
Copy Markdown
Contributor Author

Superseded by - #2023

Potentially might be good to add some docs on hypervisor conversion to the stackhpc-baremetal env at some point.

@holliefae holliefae closed this Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants