Skip to content

Conversation

@staebler
Copy link
Contributor

@staebler staebler commented Feb 5, 2019

The machines field in install-config.yaml will be replaced with controlPlane and compute fields. This PR brings in the latest openshift/installer version, which includes those changes.

- replace machines field in install-config.yaml with controlPlane and compute
- remove iamRoleName from machine pools in install-config.yaml
- replace netopv1.ClusterNetwork with ClusterNetworkEntry in InstallConfig
- remove ClusterName from ClusterUninstaller
@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Feb 5, 2019
@staebler
Copy link
Contributor Author

staebler commented Feb 5, 2019

/hold

Requires openshift/installer#1157

@openshift-ci-robot openshift-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 5, 2019
@staebler
Copy link
Contributor Author

staebler commented Feb 5, 2019

There are other changes that we may want to do to the Hive API that are not covered by this PR.

  1. Change the name of the network type to reflect the updated name of OpenShiftSDN used by the cluster-network-operator.
  2. Remove IAMRoleName from AWSMachinePoolPlatform.

@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 23, 2019
@openshift-ci-robot
Copy link

@staebler: 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.

@staebler
Copy link
Contributor Author

/close

Obsolete by #228.

@openshift-ci-robot
Copy link

@staebler: Closed this PR.

Details

In response to this:

/close

Obsolete by #228.

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.

@staebler staebler deleted the install-config_controlplane-compute branch December 7, 2019 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants