Skip to content

[wip] [vsphere] modify the cloud-config to support multiple datacenters#5356

Closed
rvanderp3 wants to merge 1 commit intoopenshift:masterfrom
rvanderp3:cloud-config-multiple-datacenters
Closed

[wip] [vsphere] modify the cloud-config to support multiple datacenters#5356
rvanderp3 wants to merge 1 commit intoopenshift:masterfrom
rvanderp3:cloud-config-multiple-datacenters

Conversation

@rvanderp3
Copy link
Contributor

The intent of this PR is to configure the cloud-config to support multiple datacenters if configured appropriately in install-config.yaml. See openshift/enhancements#918 for details.

cc: @jcpowermac

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 4, 2021
@openshift-ci openshift-ci bot requested review from e-tienne and jhixson74 November 4, 2021 20:18
@rvanderp3
Copy link
Contributor Author

This PR depends on #5342 #5347 #5352 and #5353

@rvanderp3 rvanderp3 force-pushed the cloud-config-multiple-datacenters branch 3 times, most recently from dda4b56 to 6af6727 Compare November 5, 2021 18:31
@openshift-ci openshift-ci bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 21, 2021
@rvanderp3 rvanderp3 force-pushed the cloud-config-multiple-datacenters branch from 6af6727 to 0279056 Compare November 30, 2021 19:33
@openshift-ci openshift-ci bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 30, 2021
@rvanderp3 rvanderp3 force-pushed the cloud-config-multiple-datacenters branch 2 times, most recently from e42f6e1 to cf8d354 Compare November 30, 2021 19:48
@rvanderp3 rvanderp3 force-pushed the cloud-config-multiple-datacenters branch from cf8d354 to 3c31588 Compare December 7, 2021 20:59
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Dec 7, 2021

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign smarterclayton after the PR has been reviewed.
You can assign the PR to them by writing /assign @smarterclayton 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
Copy link
Contributor

openshift-ci bot commented Apr 1, 2022

@rvanderp3: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-metal-ipi-ovn-ipv6 3c31588 link false /test e2e-metal-ipi-ovn-ipv6
ci/prow/e2e-aws-single-node 3c31588 link false /test e2e-aws-single-node
ci/prow/e2e-libvirt 3c31588 link false /test e2e-libvirt
ci/prow/e2e-aws-fips 3c31588 link false /test e2e-aws-fips
ci/prow/e2e-metal-single-node-live-iso 3c31588 link false /test e2e-metal-single-node-live-iso
ci/prow/e2e-openstack 3c31588 link false /test e2e-openstack
ci/prow/e2e-crc 3c31588 link false /test e2e-crc
ci/prow/verify-codegen 3c31588 link true /test verify-codegen
ci/prow/e2e-aws-workers-rhel8 3c31588 link false /test e2e-aws-workers-rhel8
ci/prow/e2e-openstack-kuryr 3c31588 link false /test e2e-openstack-kuryr
ci/prow/unit 3c31588 link true /test unit
ci/prow/e2e-aws-workers-rhel7 3c31588 link false /test e2e-aws-workers-rhel7
ci/prow/verify-vendor 3c31588 link true /test verify-vendor
ci/prow/gofmt 3c31588 link true /test gofmt
ci/prow/e2e-ovirt 3c31588 link false /test e2e-ovirt
ci/prow/golint 3c31588 link true /test golint
ci/prow/govet 3c31588 link true /test govet
ci/prow/images 3c31588 link true /test images
ci/prow/okd-e2e-aws 3c31588 link false /test okd-e2e-aws
ci/prow/okd-images 3c31588 link true /test okd-images
ci/prow/okd-e2e-aws-upgrade 3c31588 link false /test okd-e2e-aws-upgrade
ci/prow/okd-unit 3c31588 link true /test okd-unit
ci/prow/e2e-alibaba 3c31588 link true /test e2e-alibaba
ci/prow/e2e-aws 3c31588 link true /test e2e-aws
ci/prow/openstack-manifests 3c31588 link true /test openstack-manifests
ci/prow/e2e-gcp-upgrade 3c31588 link true /test e2e-gcp-upgrade
ci/prow/e2e-aws-upgrade 3c31588 link true /test e2e-aws-upgrade

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.

@rvanderp3
Copy link
Contributor Author

closing in preference to #5911

@rvanderp3 rvanderp3 closed this May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant