-
Notifications
You must be signed in to change notification settings - Fork 905
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
Bump crossplane-runtime and build submodule #3183
Conversation
Setting this to draft - looks like the below PR updated some reference types to add a policy field. We use those reference types in some places in core that won't honor the policy. I don't think we can use these types in core. |
Can we also include upbound/build#192 once its merged? |
Yes, will do. Edit: Done. |
Notably this doesn't make assumptions about BASH existing at /bin/bash, which it does not on my NixOS machine. Signed-off-by: Nic Cope <nicc@rk0n.org>
Notably this bumps the client-side rate-limits up fairly significantly (to ~5x per reconcile w/ ~10x burst). This should help us get packages with large payloads installed faster. Signed-off-by: Nic Cope <nicc@rk0n.org>
Signed-off-by: Nic Cope <nicc@rk0n.org>
1d2c729
to
ad7ae76
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@negz two small nitpicks (which I think were unintentional but perhaps this is an annotation I'm not familiar with) -- otherwise LGTM!
Previously we used *xpv1.Reference (from crossplane-runtime) in a few places in this codebase. This type was recently updated to add a 'policy' field that isn't applicable to core Crossplane's uses. Reference structs are simple enough that it seemed better to define our own than to depend on (and have to name!) a new kind of reference in crossplane-runtime. Signed-off-by: Nic Cope <nicc@rk0n.org>
Description of your changes
I believe this will get us to sub 2 mins to install the "big three" jet providers (~1,800 CRDs).
Note that the
xpv1.Reference
type changed in the latest crossplane-runtime to include aPolicy
field. We don't use this field in core Crossplane so I've replaced our handful ofxpv1.Reference
uses with local structs. These reference structs are simple enough that it seemed better to define our own than to depend on (and have to name!) a new kind of reference in crossplane-runtime.I have:
make reviewable
to ensure this PR is ready for review.backport release-x.y
labels to auto-backport this PR if necessary.How has this code been tested
I've successfully deployed the "getting started with GCP" configuration from the https://crossplane.io documentation (with composition revisions enabled).