Skip to content
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

Sync upstream master #107

Merged
merged 513 commits into from
Nov 2, 2023
Merged

Sync upstream master #107

merged 513 commits into from
Nov 2, 2023

Conversation

turkenh
Copy link
Member

@turkenh turkenh commented Nov 1, 2023

Description of your changes

Sync upstream master.

Different from the previous syncs, this PR does the followings:

  • Revert Add support for alpha provider identity support #69 with this commit. That PR touching the code that was refactored heavily and causing a lot of conflicts. I will open a fresh PR for that functionality against this PR before merging this. This is mostly for visibility but also to be able to cherry-pick to somewhere later if needed.
  • Adds xpkg.upbound.io/upbound to REGISTRY_ORGS and updates the CI to push to Upbound Marketplace as well. This is to be consistent with Crossplane regarding pulling the image from the Upbound Marketplace.

I have:

  • Read and followed Crossplane's contribution process.
  • Added or updated unit and E2E tests for my change.
  • Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR, if necessary.
  • Opened a PR updating the docs, if necessary.

renovate bot and others added 30 commits October 13, 2023 13:54
…l-action-digest

chore(deps): update github/codeql-action digest to 0116bc2 (master)
Context can be loaded from either values, or files. Either way, the
value must be JSON.

Signed-off-by: Nic Cope <[email protected]>
Signed-off-by: Philippe Scorsolini <[email protected]>
Support loading context when running `crossplane beta render`
…-rbac-manager-pods

fix(helm): Add custom annotation for rbac manager pods
…` field

* claim CRD contains now the default value for `.spec.compositeDeletePolicy` field
  copied from XRD `spec.defaultCompositeDeletePolicy`. If not specified at claim,
  it will be set by k8s API
* claim controller does not set the value for `spec.compositeDeletePolicy`,
  leading to one less claim update, and hence one less no-op reconcile loop
* added unit and e2e tests
* removed `APIDefaultSelector` as unneeded

Signed-off-by: Predrag Knezevic <[email protected]>
k8s API sets the default value of claim's `spec.compositeDeletePolicy` field
Signed-off-by: Philippe Scorsolini <[email protected]>
Signed-off-by: Philippe Scorsolini <[email protected]>
feat(crank): add init function subcommand
Signed-off-by: Dr. Stefan Schimanski <[email protected]>
Signed-off-by: Dr. Stefan Schimanski <[email protected]>
test/e2e: print related object events in kubectl style
feat(crank): drop duplicated build/push commands
e2e: wait for all nop resource deleted before deleting prerequisites
negz and others added 14 commits October 27, 2023 15:23
Push Protobufs to Buf Schema Registry
Handle externally managed Provider service accounts properly
…sion-syncpackage-events

fix: avoid spamming SyncPackage and InstallPackageRevision events
Update releases table and base branches for v1.14
This reverts commit e9d31c7.
To be re-introduced with a seperate PR based on the latest
codebase
@turkenh
Copy link
Member Author

turkenh commented Nov 2, 2023

PR reimplementing alpha provider identity support: turkenh#1

@turkenh turkenh marked this pull request as ready for review November 2, 2023 13:02
@turkenh turkenh requested a review from phisco November 2, 2023 13:02
.github/workflows/ci.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@phisco phisco left a comment

Choose a reason for hiding this comment

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

Double checked the uxp specific parts and all looks good

@turkenh turkenh merged commit 2402241 into upbound:master Nov 2, 2023
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants