-
Notifications
You must be signed in to change notification settings - Fork 70
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
Add v1alpha2 API version #139
Merged
Merged
Commits on Apr 13, 2021
-
This adds a v1alpha2 API, without changing the controller or tests to use it (yet). The new API has roughly the desired shape, per fluxcd/flux2#1124, but supporting only things that the controller supports now. It's necessary to give the v1alpha1 type a `storageversion` marker so that 1. code generation keeps working, and 2. tests still work, since they still expect v1alphav1 types. v1alpha1 will be removed once the controller and tests are ported to v1alpha2. Signed-off-by: Michael Bridgen <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for f28a0c4 - Browse repository at this point
Copy the full SHA f28a0c4View commit details -
Flip the controller and tests to API v1alpha2
This finishes the v1alpha2 API, and rewrites everything needed so that the controller supports it and the tests pass. For the most part, that is just changing the location of fields. However, there's a few notable extras: - check that the `sourceRef` is a git repo (that's the default), and that a `.spec.git` is supplied; - change a test that blindly patched an update object, so that it first gets the object it's patching. Previously, it succeeded because it was OK to patch everything to empty strings, but that's no longer the case since SourceReference.Kind is an enum. Signed-off-by: Michael Bridgen <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for cd8a2e9 - Browse repository at this point
Copy the full SHA cd8a2e9View commit details -
Split author email and name into separate fields
This is a bit neater to read and write, and since I'm making breaking changes anyway. The name is now optional; an email is enough. Signed-off-by: Michael Bridgen <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for b28c5cc - Browse repository at this point
Copy the full SHA b28c5ccView commit details -
This changes the API so that the checkout field has a ref, the same as GItRepository. This means you can check out a branch or a tag or a particular commit. Most of these won't work unless you supply a branch to push to as well. An addtional change is that you can leave out the checkout altogether, and the ref will default to that given in the GitRepository, or its default. In the latter case, again you will need to provide a push branch. Signed-off-by: Michael Bridgen <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for ebb2d6a - Browse repository at this point
Copy the full SHA ebb2d6aView commit details -
This switches the API doc generation from v1alpha1, to v1alpha2. Signed-off-by: Michael Bridgen <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 1c33a2d - Browse repository at this point
Copy the full SHA 1c33a2dView commit details -
Write a specification explainer for v1alpha2
This mostly adapts the material in the v1alpha1 spec explainer doc to the different structure of the v1alpha2 types. Signed-off-by: Michael Bridgen <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for b0fc415 - Browse repository at this point
Copy the full SHA b0fc415View commit details -
Add guide to migrating to v1alpha2
This commit adds a step-by-step guide to rewriting specs for v1alpha2. Signed-off-by: Michael Bridgen <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for f746a11 - Browse repository at this point
Copy the full SHA f746a11View 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.