Skip to content

Conversation

@abhinavdahiya
Copy link
Contributor

@abhinavdahiya abhinavdahiya commented Jul 26, 2018

  • Adds basic scripts in hack for code-gen and verify
  • Adds the machineconfiguration.openshift.io apis
  • Adds the generated clients

@openshift-ci-robot openshift-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Jul 26, 2018
@abhinavdahiya abhinavdahiya changed the title Init apis and client *: init apis and client Jul 26, 2018
Copy link
Member

@ashcrow ashcrow left a comment

Choose a reason for hiding this comment

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

There is too much to really review in depth, but I perused some of the non generated go code, configs, and shell scripts and they look good.

Copy link
Contributor

Choose a reason for hiding this comment

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

if len(configs) == 0 {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@abhinavdahiya abhinavdahiya merged commit 1f45d4b into openshift:master Jul 26, 2018
@abhinavdahiya abhinavdahiya deleted the add_apis_client branch July 26, 2018 19:10
wking added a commit to wking/machine-config-operator that referenced this pull request Feb 28, 2019
There seems to have been some initial uncertainty about what this
would be called, with MachinePool first landing in 30d522f (docs: add
initial documentation of design, 2018-07-25) and MachineConfigPool
first landing in cfc9660 (pkg/apis: add
machineconfiguration.openshift.io apis, 2018-07-26, openshift#2).  This commit
consolidates us around MachineConfigPool, except for the
machine-pools/ bootstrap path (since changing that would require
consumer updates).
alaypatel07 pushed a commit to alaypatel07/machine-config-operator that referenced this pull request Sep 30, 2019
*: use kubecsr mount, add readiness probe to etcd pod
osherdp pushed a commit to osherdp/machine-config-operator that referenced this pull request Apr 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

4 participants