forked from crossplane-contrib/function-patch-and-transform
-
Notifications
You must be signed in to change notification settings - Fork 1
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 Conditionals via CEL, change name of repo #2
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This makes these fields effectively required. Previously they had defaults, but our Input isn't a real CRD with API server validation, so we don't enjoy that anymore. Signed-off-by: Nic Cope <[email protected]>
Require string and math transform types
I reversed this compared to almost every other test we have by mistake. Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Patch to and from an Environment loaded from Function context
Signed-off-by: Steven Borrelli <[email protected]>
change list toType to array
Signed-off-by: Nic Cope <[email protected]>
Setup GitHub Actions
It needs setup at the repo level, which might be a bit much to expect new Functions to do by default. Signed-off-by: Nic Cope <[email protected]>
Looks like distroless doesn't have a PPC target. Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
We build inside Docker, so don't need it. Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
We just need this to get the short revision SHA. Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Nic Cope <[email protected]>
Repo already contains repo owner Signed-off-by: Nic Cope <[email protected]>
Build a package using GitHub Actions
Not entirely sure what's going on, but /function is definitely compiled for amd64 inside an image that is otherwise arm64. Signed-off-by: Nic Cope <[email protected]>
Don't build an amd64 binary in the arm64 image
Signed-off-by: Steven Borrelli <[email protected]>
…vate/github.com-alecthomas-kong-0.x Update module github.com/alecthomas/kong to v0.8.1
Including replacing the newly deprecated pointer package with the ptr package. Signed-off-by: Nic Cope <[email protected]>
Bump k8s.io/utils
This requires running go generate to update some generated files. Signed-off-by: Nic Cope <[email protected]>
Bump to controller-tools v0.13.0
…vate/github.com-crossplane-function-sdk-go-0.x Update module github.com/crossplane/function-sdk-go to v0.1.0
See crossplane/function-template-go#38. Signed-off-by: Nic Cope <[email protected]>
Use Go style pseudo-versions when there is no semver supplied details.
Signed-off-by: Nic Cope <[email protected]>
Slim down the README
…vate/github.com-crossplane-crossplane-runtime-1.x Update module github.com/crossplane/crossplane-runtime to v1.14.1
Signed-off-by: Nic Cope <[email protected]>
Add useful package metadata
Signed-off-by: Nic Cope <[email protected]>
Annotations plural, not annotation. Signed-off-by: Nic Cope <[email protected]>
Fix package metadata
Annotations plural, not annotation. Signed-off-by: Nic Cope <[email protected]>
Backport package metadata
Third time's the charm! Signed-off-by: Nic Cope <[email protected]>
Fix typos in the package metadata
Adds conditional to the resources
Signed-off-by: Steven Borrelli <[email protected]>
Signed-off-by: Steven Borrelli <[email protected]>
Signed-off-by: Steven Borrelli <[email protected]>
Signed-off-by: Steven Borrelli <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This kicks off the fork of function-patch-and-transform. This fork will support conditionals at the top level and for resources.