User guide: first take at documenting IrSO #493
Conversation
metal3-io-bot
added
the
size/L
dtantsur
force-pushed
the
ironic-standalone-operator
branch
from
f9ea033 to
436f879
Compare
| - Which physical network interface will be used for provisioning? Without any | ||
| configuration, Ironic will use the host cluster networking. | ||
|
|
||
| - If you use a dedicated network interface, are you going to use the built-in |
There was a problem hiding this comment.
A little bit confused about this: if we have multiple control plane nodes, should we make sure one interface on each control plane node has an IP address (which means one ironic per control plane node using that IP address)? Shouldn't that IP address be the same, or shouldn't we have a proxy?
There was a problem hiding this comment.
The role of keepalived is to maintain the same IP address, regardless of which node is hosting Ironic. I'll try to clarify a bit.
There was a problem hiding this comment.
So the bottom line is: Make sure you have a usable IP address on the control plane node hosting ironic. It's possible to use keepalived for this.
Its clearer now, thank you!
| git clone https://github.com/metal3-io/ironic-standalone-operator | ||
| cd ironic-standalone-operator | ||
| git checkout -b <DESIRED BRANCH OR main> | ||
|
|
||
| make install deploy |
There was a problem hiding this comment.
In the future we should start publishing the output of kustomize build as a release artifact so users can just do a kubectl apply -f [URL]. This is fine for now though, no need to change!
There was a problem hiding this comment.
Yeah, I totally agree. Do we already do it for CAPM3?
There was a problem hiding this comment.
Yes, it is part of how CAPI expects things to work. So it is published as infrastructure-components.yaml on every release. That allows clusterctl init --infrastructure=metal3 to automatically find the latest and install it 🙂
There was a problem hiding this comment.
Nice! So, we'll eventually need infrastructure-components.yaml to include IrSO as well (once we switch to it by default)?
Signed-off-by: Dmitry Tantsur <[email protected]>
dtantsur
force-pushed
the
ironic-standalone-operator
branch
from
436f879 to
fb72493
Compare
|
Hopefully addresses all the comments. Also added information about versions. |
|
LGTM! |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: lentzi90 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
metal3-io-bot
added
the
approved
Signed-off-by: Dmitry Tantsur [email protected]