CORS-3702: Add api and api-int ips to bootstrap node

[barbacbd]

** The api and api-int addresses are coming from the capi gcp provider. The addresses must be added to the Load Balancer Config and to the ignition config for the bootstrap node.

[openshift-ci-robot]

@barbacbd: This pull request explicitly references no jira issue.

In response to this:

** The api and api-int addresses are coming from the capi gcp provider. The addresses must be added to the Load Balancer Config and to the ignition config for the bootstrap node.

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 openshift-eng/jira-lifecycle-plugin repository.

[barbacbd]

/label platform/google

[barbacbd]

/cc @bfournie
/cc @patrickdillon
/cc @sadasu

[sadasu]

What is the format of the API Server IP address in the Cluster Status? Could add sample output here?
Why do we need to do:

apiIPAddress := *gcpCluster.Status.Network.APIServerAddress
addressCut := apiIPAddress[strings.LastIndex(apiIPAddress, "/")+1:]

[barbacbd]

What is the format of the API Server IP address in the Cluster Status? Could add sample output here? Why do we need to do:

apiIPAddress := *gcpCluster.Status.Network.APIServerAddress
addressCut := apiIPAddress[strings.LastIndex(apiIPAddress, "/")+1:]

WARNING api address: https://www.googleapis.com/compute/v1/projects/openshift-dev-installer/global/addresses/bbarbach-gcp-oa-scc2d-apiserver 
WARNING api int address: https://www.googleapis.com/compute/v1/projects/openshift-dev-installer/regions/us-east1/addresses/bbarbach-gcp-oa-scc2d-api-internal

[sadasu]

What is the format of the API Server IP address in the Cluster Status? Could add sample output here? Why do we need to do:

apiIPAddress := *gcpCluster.Status.Network.APIServerAddress
addressCut := apiIPAddress[strings.LastIndex(apiIPAddress, "/")+1:]

WARNING api address: https://www.googleapis.com/compute/v1/projects/openshift-dev-installer/global/addresses/bbarbach-gcp-oa-scc2d-apiserver 
WARNING api int address: https://www.googleapis.com/compute/v1/projects/openshift-dev-installer/regions/us-east1/addresses/bbarbach-gcp-oa-scc2d-api-internal

We should be able to get to the actual IP address.

[barbacbd]

What is the format of the API Server IP address in the Cluster Status? Could add sample output here? Why do we need to do:

apiIPAddress := *gcpCluster.Status.Network.APIServerAddress
addressCut := apiIPAddress[strings.LastIndex(apiIPAddress, "/")+1:]

WARNING api address: https://www.googleapis.com/compute/v1/projects/openshift-dev-installer/global/addresses/bbarbach-gcp-oa-scc2d-apiserver 
WARNING api int address: https://www.googleapis.com/compute/v1/projects/openshift-dev-installer/regions/us-east1/addresses/bbarbach-gcp-oa-scc2d-api-internal

We should be able to get to the actual IP address.

@sadasu We do get the actual IP address. You asked what the output above was, so I provided that output but maybe I should have provided the other data too. If you look at the changes to the PR we then take that information above and query the API to get the IP Address from the compute address.

[sadasu]

What is the format of the API Server IP address in the Cluster Status? Could add sample output here? Why do we need to do:

apiIPAddress := *gcpCluster.Status.Network.APIServerAddress
addressCut := apiIPAddress[strings.LastIndex(apiIPAddress, "/")+1:]

WARNING api address: https://www.googleapis.com/compute/v1/projects/openshift-dev-installer/global/addresses/bbarbach-gcp-oa-scc2d-apiserver 
WARNING api int address: https://www.googleapis.com/compute/v1/projects/openshift-dev-installer/regions/us-east1/addresses/bbarbach-gcp-oa-scc2d-api-internal

We should be able to get to the actual IP address.

@sadasu We do get the actual IP address. You asked what the output above was, so I provided that output but maybe I should have provided the other data too. If you look at the changes to the PR we then take that information above and query the API to get the IP Address from the compute address.

Thanks! That is neat.

[sadasu]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sadasu

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:

• OWNERS [sadasu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[sadasu]

/retitle CORS-3702: Add api and api-int ips to bootstrap node

[openshift-ci-robot]

@barbacbd: This pull request references CORS-3702 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.18.0" version, but no target version was set.

In response to this:

** The api and api-int addresses are coming from the capi gcp provider. The addresses must be added to the Load Balancer Config and to the ignition config for the bootstrap node.

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 openshift-eng/jira-lifecycle-plugin repository.

[bfournie]

/lgtm

[openshift-ci]

@barbacbd: all tests passed!

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

[openshift-bot]

[ART PR BUILD NOTIFIER]

Distgit: ose-installer-altinfra
This PR has been included in build ose-installer-altinfra-container-v4.18.0-202410020939.p0.gb4ccd77.assembly.stream.el9.
All builds following this will include this PR.

[openshift-bot]

[ART PR BUILD NOTIFIER]

Distgit: ose-baremetal-installer
This PR has been included in build ose-baremetal-installer-container-v4.18.0-202410020939.p0.gb4ccd77.assembly.stream.el9.
All builds following this will include this PR.

[openshift-bot]

[ART PR BUILD NOTIFIER]

Distgit: ose-installer-terraform-providers
This PR has been included in build ose-installer-terraform-providers-container-v4.18.0-202410020939.p0.gb4ccd77.assembly.stream.el9.
All builds following this will include this PR.

[openshift-bot]

[ART PR BUILD NOTIFIER]

Distgit: ose-installer-artifacts
This PR has been included in build ose-installer-artifacts-container-v4.18.0-202410020939.p0.gb4ccd77.assembly.stream.el9.
All builds following this will include this PR.