Merged
Conversation
This updates provider-helm from v0 to v1, enabling generation of v1 namespaced models with the .m API group suffix. The v1 models are generated in .up/kcl/models/io/crossplane/helmm/ (helmm with double 'm') and will be used in subsequent KCL function updates for Crossplane v2 migration. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Update apiVersion from v1 to v2 - Add required scope: Namespaced field - Remove "X" prefix from resource kind (XFlux → Flux) - Update metadata.name to match new plural (xflux → flux) - Replace deletionPolicy parameter with managementPolicies array - Update kubeConfigSecretRef description for v2 namespace handling - Maintain all existing parameters and schema structure
- Update metadata.name: xflux.gitops.platform.upbound.io → flux.gitops.platform.upbound.io - Update compositeTypeRef.kind: XFlux → Flux - Update step name: xflux → flux - Keep functionRef.name as upbound-configuration-gitops-fluxxflux (matches function directory)
…ed resources - Update import from helm to helmm (namespaced models) - Change oxr type from XFlux to Flux - Replace deletionPolicy with managementPolicies - Add providerConfigRef.kind to both Helm Release resources - Use conditional field assignment pattern for kubeConfig to avoid null values
- Changed kind from XFlux to Flux - Added namespace: default field (required for namespaced resources) - Replaced deletionPolicy: Delete with managementPolicies: ["*"]
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Renamed directory apis/xflux to apis/flux for consistency with v2 resource kind - Updated README to reflect current Crossplane v2 state: - Added Crossplane v2 section explaining key characteristics - Updated file paths from /apis/xflux/ to /apis/flux/ - Added example usage section with complete Flux XR example - Clarified namespaced resource requirements - Documented managementPolicies and provider-helm v1 usage - Updated test references in tests/test-xflux/main.k The directory rename ensures consistency between the resource kind (Flux) and the directory structure, improving developer experience. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Signed-off-by: Yury Tsarev <yury@upbound.io>
condition Workaround to mitigate race condition like ``` up: error: unable to execute e2e tests: failed to apply extra resources: failed to apply resource ProviderConfig/default: providerconfigs.aws.m.***.io "default" is forbidden: User "***:robot:***-bot" cannot patch resource "providerconfigs" in API group "aws.m.***.io" in the namespace "default" ``` Signed-off-by: Yury Tsarev <yury@upbound.io>
3 tasks
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Description of your changes
Migrates configuration-gitops-flux from v1 to v2 with namespaced resources.
Changes:
Breaking Changes:
⚠️ Existing v1 XFlux resources will not auto-migrate. Users must:
I have:
make reviewableto ensure this PR is ready for review.backport release-x.ylabels to auto-backport this PR, as appropriate.How has this code been tested
✅ All composition tests passing
✅ E2E tests updated with Crossplane v2.1.3-up.2
✅ Build and linting successful