Skip to content

Conversation

@dgoodwin
Copy link
Contributor

The installer's InstallConfig is not currently usable as a Kube type, so it had to be copied in, with a few minor modifications. (remove some objectmeta, change the IPNet usage to string, and critically ensure the auto-generated methods are present for Kube such as DeepCopyInto) I am pursuing what we should do with this long term with installer team, but this gets us moving now and we can change easily in future.

ClusterDeployment template allows creating with your desired admin info and SSH key.

Controller updated to create a dummy job that just passes, associated with the cluster deployment. (automatically cleaned up and triggering resyncs of the cluster deployment on job updates changes)

@openshift-ci-robot openshift-ci-robot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Sep 18, 2018
@abutcher
Copy link
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Sep 18, 2018
@abutcher abutcher merged commit a1ad220 into openshift:master Sep 18, 2018
@2uasimojo 2uasimojo mentioned this pull request Mar 22, 2022
abutcher added a commit that referenced this pull request Dec 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants