Skip to content

Add CISInstanceCRN to IBMCloudPlatformStatus#951

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
BobbyRadford:ibmcis
Jun 25, 2021
Merged

Add CISInstanceCRN to IBMCloudPlatformStatus#951
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
BobbyRadford:ibmcis

Conversation

@BobbyRadford
Copy link
Copy Markdown
Contributor

CISInstanceCRN is the Cloud Resource Name (CRN) referencing the instance of IBM Cloud Internet Services that is setup to manage the DNS zone of the cluster's base domain. This can be guaranteed in an IPI/UPI installation since this configuration is a pre-requisite. Components like the Cluster Ingress Operator will use this CRN in API calls to update DNS records.

This PR is required to implement IBMCloud IPI/UPI.

Enhancement doc: openshift/enhancements#773
Relevant Installer PR: openshift/installer#4923

CISInstanceCRN is the Cloud Resource Name (CRN) referencing the instance of
IBM Cloud Internet Services that is setup to manage the DNS zone of the
cluster's base domain. This can be guaranteed in an IPI/UPI installation
since this configuration is a pre-requisite. Components like the Cluster
Ingress Operator will use this CRN in API calls to update DNS records.
@fabianofranz
Copy link
Copy Markdown
Contributor

@sttts @mfojtik can you please ptal? tks!

@fabianofranz
Copy link
Copy Markdown
Contributor

/assign @sttts @mfojtik @deads2k

@sttts
Copy link
Copy Markdown
Contributor

sttts commented Jun 24, 2021

This needs review from the owning team first.

@fabianofranz
Copy link
Copy Markdown
Contributor

@staebler @jstuever mind taking a look?

Copy link
Copy Markdown
Contributor

@staebler staebler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 25, 2021
@jeffnowicki
Copy link
Copy Markdown

@deads2k can you approve/merge? that will help us with work that is dependent on this. thank you

@sttts
Copy link
Copy Markdown
Contributor

sttts commented Jun 25, 2021

/approve

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci Bot commented Jun 25, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: BobbyRadford, staebler, sttts

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

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

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 25, 2021
@openshift-merge-robot openshift-merge-robot merged commit 02b64ca into openshift:master Jun 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants