Skip to content

Conversation

@BobbyRadford
Copy link
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.
@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 23, 2021
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 23, 2021

@BobbyRadford: PR needs rebase.

Details

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.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 23, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BobbyRadford
To complete the pull request process, please assign mfojtik after the PR has been reviewed.
You can assign the PR to them by writing /assign @mfojtik in a comment when ready.

The full list of commands accepted by this bot can be found 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 requested review from abhinavdahiya and sttts June 23, 2021 13:07
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 23, 2021

@BobbyRadford: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/unit aa9dd33 link /test unit
ci/prow/build aa9dd33 link /test build
ci/prow/verify-client-go aa9dd33 link /test verify-client-go
ci/prow/verify aa9dd33 link /test verify

Full PR test history. Your PR dashboard.

Details

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

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 23, 2021

@BobbyRadford: The following test failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
ci/prow/verify-deps aa9dd33 link /test verify-deps

Full PR test history. Your PR dashboard.

Details

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant