Make the JSON RPC port configurable

[dtantsur]

The default port of 8089 may conflict with other workloads. At the very least, we need to be able to change it similarly to other ports (see the Networking structure).

There is no suitable option in ironic-image, so use OS_JSON_RPC__PORT to pass it.

/kind feature
/help

[metal3-io-bot]

@dtantsur:
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:

The default port of 8089 may conflict with other workloads. At the very least, we need to be able to change it similarly to other ports (see the Networking structure).

There is no suitable option in ironic-image, so use OS_JSON_RPC__PORT to pass it.

/kind feature
/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-sigs/prow repository.

[winiciusallan]

Hi @dtantsur! The user can not already pass it with extraConfig?

spec:
  extraConfig:
  - group: json_rpc
    name: port
    value: "8090"

[Rozzii]

@dtantsur will take a look, we have agreed that I won't accept this Issue for now.

[dtantsur]

The user can not already pass it with extraConfig?

True. It will change, however, once we start supporting TLS on RPC. If we do it the usual path - using a reverse proxy - we need to pass the same value to httpd and ironic.

I guess this request is a bit premature. Do we have a status "on hold" for bugs?

[Rozzii]

/triage accepted