-
Notifications
You must be signed in to change notification settings - Fork 507
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MCO-52: add MCO API into openshift/api #1453
Merged
openshift-merge-robot
merged 10 commits into
openshift:master
from
jkyros:migrate-mco-api
Sep 29, 2023
Merged
MCO-52: add MCO API into openshift/api #1453
openshift-merge-robot
merged 10 commits into
openshift:master
from
jkyros:migrate-mco-api
Sep 29, 2023
Commits on Sep 28, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 19779aa - Browse repository at this point
Copy the full SHA 19779aaView commit details -
Configuration menu - View commit details
-
Copy full SHA for edca7b8 - Browse repository at this point
Copy the full SHA edca7b8View commit details -
annotate all required/optional fields properly
This explicitly marks fields with optional/required in a way that matches the original "requiredness" of the API in the MCO repo. Some of these fields should probably be changed to required -- KubeletConfigCondition members for example should not be entirely optional, but they always have been, and so for now we're going to leave them set how they were. Later we can come back and fix them and see what breaks. This does set +kubebuilder:validation:Optional, but that's because we historically have a field in ContinerRuntimeConfigSpec that was both required in the CRD and tagged as omitempty in types.go and for now I'm trying to preserve that until we can figure out if it's still necessary.
Configuration menu - View commit details
-
Copy full SHA for 659dcd1 - Browse repository at this point
Copy the full SHA 659dcd1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 93d740a - Browse repository at this point
Copy the full SHA 93d740aView commit details -
add mco types.go fields from recent MCO merges
- Adds cert details from machine-config-operator#3756 - Adds buildcontroller state from machine-config-operator#3731 - Adds image registry bundles from machine-config-operator#3770 - Adds internal registry pull secret from machine-config-operator#3806
Configuration menu - View commit details
-
Copy full SHA for c247522 - Browse repository at this point
Copy the full SHA c247522View commit details -
Configuration menu - View commit details
-
Copy full SHA for a7b5374 - Browse repository at this point
Copy the full SHA a7b5374View commit details -
Configuration menu - View commit details
-
Copy full SHA for ad88a43 - Browse repository at this point
Copy the full SHA ad88a43View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6365217 - Browse repository at this point
Copy the full SHA 6365217View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0d78e89 - Browse repository at this point
Copy the full SHA 0d78e89View commit details -
Configuration menu - View commit details
-
Copy full SHA for 545417a - Browse repository at this point
Copy the full SHA 545417aView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.