Allow overriding/specifying IRONIC_IP/IRONIC_URL_HOST

[metal3-io]

Allow overriding/specifying IRONIC_IP/IRONIC_URL_HOST

[dtantsur]

Is it something that would go away together with the Inspector anyway?

Yes, but we may hit similar issues with JSON RPC. Which is, whoever, its own ball of wax.

[lentzi90]

Is it something that would go away together with the Inspector anyway?

Yes, but we may hit similar issues with JSON RPC. Which is, whoever, its own ball of wax.

Could you elaborate? I'm afraid I'm lost 😅

[dtantsur]

For JSON RPC to work, Ironic instances have to be able to talk to each other. That requires every Ironic to know its own IP or host name.

[Rozzii]

/triage accepted
/help

[metal3-io-bot]

@Rozzii:
This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

/triage accepted
/help

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[lentzi90]

Based on the discussion in the last community meeting, I'm adding some notes here so I don't forget.

• Ironic can do internal load balancing when running multiple instances
• We could have an external load balancer to provide a fixed endpoint for clients (this was what I was trying to do)
• Perhaps we can get some ideas from how etcd is deployed. It seems to be similar in that it also has one port for clients and another for peers. It needs to know about all peers, while clients only need to know about the single endpoint for clients.

[metal3-io-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues will close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

/lifecycle stale

[lentzi90]

I think we can close this as fixed in #476