Skip to content

Add categories to source controller CRDs#2086

Open
shruthihub wants to merge 3 commits into
fluxcd:mainfrom
shruthihub:add-crd-categories
Open

Add categories to source controller CRDs#2086
shruthihub wants to merge 3 commits into
fluxcd:mainfrom
shruthihub:add-crd-categories

Conversation

@shruthihub

Copy link
Copy Markdown

No description provided.

@matheuscscp matheuscscp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please undo the changes in non-v1 files.

Also, please add categories to Bucket and ExternalArtifact.

Also, please add the category all to all CRDs.

Signed-off-by: kavirakesh14 <kavirakesh007@gmail.com>
@shruthihub shruthihub force-pushed the add-crd-categories branch from 9677b3a to 488455a Compare June 21, 2026 05:38
@shruthihub

Copy link
Copy Markdown
Author

Thanks for the review.

I've updated the PR to:

  • Remove the changes from the non-v1 API versions
  • Add categories to Bucket and ExternalArtifact
  • Add the all category to all source-controller CRDs

The CRDs have been regenerated accordingly.

@Iam-Karan-Suresh

Iam-Karan-Suresh commented Jun 21, 2026

Copy link
Copy Markdown

Hey @shruthihub Thanks for amazing contribution 🎉. please always run this command locally before raising pull request make tidy fmt vet && make test. It ensure every test cases pass and formatted and nothing goes wrong.

Comment thread api/v1/externalartifact_types.go Outdated
kavirakesh14 and others added 2 commits June 21, 2026 09:32
Signed-off-by: kavirakesh14 <kavirakesh007@gmail.com>
Signed-off-by: Matheus Pimenta <matheuscscp@gmail.com>

@matheuscscp matheuscscp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! 🚀

Looks like the +kubebuilder:resource marker was causing the old versions to influence the generated CRDs. I removed them and then all CRDs are now correct.

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.

5 participants