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.

Sign up for GitHub

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

Jump to bottom

feat(Chart): 📦 add optional separated chart for CRDs #1223

Merged
merged 46 commits into from
Jan 10, 2025
Merged

feat(Chart): 📦 add optional separated chart for CRDs #1223

merged 46 commits into from
Jan 10, 2025
+16,123 −30

Conversation

darkweaver87
Copy link
Contributor

@darkweaver87 darkweaver87 commented Oct 14, 2024 •
edited
Loading

What does this PR do?

This PR introduces a separate chart used by the main one to install CRDs.
The dependency between the main chart and the CRDs one will be added once CRDs one will be released.

Motivation

#1141
#1209

More

  • Yes, I updated the tests accordingly
  • Yes, I updated the schema accordingly
  • Yes, I ran make test and all the tests passed

Sorry, something went wrong.

@darkweaver87 darkweaver87 added kind/enhancement New feature or request kind/proposal a proposal that needs to be discussed. labels Oct 14, 2024
@darkweaver87 darkweaver87 force-pushed the feat/crds branch 2 times, most recently from bd35651 to b674cf7 Compare October 14, 2024 08:39
@darkweaver87 darkweaver87 changed the title feat: 📦 add separated chart for CRDs feat!: 📦 add separated chart for CRDs Oct 14, 2024
mloiseleur
mloiseleur reviewed Oct 14, 2024
View reviewed changes
@darkweaver87 darkweaver87 force-pushed the feat/crds branch 9 times, most recently from 9087b4b to e1eb8a0 Compare October 14, 2024 14:53
@darkweaver87 darkweaver87 mentioned this pull request Oct 15, 2024
Merged
3 tasks
@mloiseleur mloiseleur linked an issue Oct 16, 2024 that may be closed by this pull request
CRD only Helm chart for Traefik #1141
Closed
2 tasks
@rgdev rgdev mentioned this pull request Oct 17, 2024
Closed
mloiseleur
mloiseleur requested changes Oct 17, 2024
View reviewed changes
Copy link
Member

@mloiseleur mloiseleur left a comment

Choose a reason for hiding this comment

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

  1. The CI needs to be tested, with a tag on your fork.
  2. The documented CRDs upgrade process should still work. I see (at least) two options:
    a. Keep and copy crd files
    b. Update CRD upgrade instruction to use them in the new directory

@darkweaver87 darkweaver87 force-pushed the feat/crds branch 2 times, most recently from e9619df to 6a67459 Compare October 21, 2024 06:50
@darkweaver87
Copy link
Contributor Author

darkweaver87 commented Oct 21, 2024 •
edited
Loading

Tested on my own account as seen with you 👍

@darkweaver87
fix: remove dependency
Loading
Loading status checks…
4da2fa0
@darkweaver87
fix: doc
Loading
Loading status checks…
92157e4
darkweaver87
darkweaver87 commented Jan 8, 2025
View reviewed changes
darkweaver87 and others added 2 commits January 8, 2025 16:00
@darkweaver87
fix: version

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode
Loading
Loading status checks…
909a277
@mloiseleur
README review
Loading
Loading status checks…
694cfd5
mloiseleur
mloiseleur reviewed Jan 8, 2025
View reviewed changes
@darkweaver87
fix: doc
4640d14
@darkweaver87
fix: CI
Loading
Loading status checks…
069050a
mloiseleur
mloiseleur reviewed Jan 10, 2025
View reviewed changes
darkweaver87 and others added 2 commits January 10, 2025 09:25
@darkweaver87 @mloiseleur
fix: apply suggestions from @mloiseleur's code review
Loading
Loading status checks…
de095f0 
Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com>
@darkweaver87
fix: CI
Loading
Loading status checks…
b6bc446
mloiseleur
mloiseleur reviewed Jan 10, 2025
View reviewed changes
mloiseleur
mloiseleur reviewed Jan 10, 2025
View reviewed changes
darkweaver87 and others added 2 commits January 10, 2025 09:57
@darkweaver87 @mloiseleur
fix: apply suggestions from @mloiseleur's code review
Loading
Loading status checks…
fdbf4b6 
Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com>
@darkweaver87
fix: CI
Loading
Loading status checks…
b99068e
darkweaver87 and others added 3 commits January 10, 2025 10:35
@darkweaver87
fix: doc
Loading
Loading status checks…
d686dd2
@darkweaver87
Merge branch 'master' into feat/crds
Loading
Loading status checks…
71ae3a8
@mloiseleur
add linter on crds consistency
Loading
Loading status checks…
afa08e7
mloiseleur
mloiseleur approved these changes Jan 10, 2025
View reviewed changes
Copy link
Member

@mloiseleur mloiseleur left a comment

Choose a reason for hiding this comment

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

LGTM

@mloiseleur mloiseleur added status/3-needs-merge and removed bot/no-merge kind/proposal a proposal that needs to be discussed. labels Jan 10, 2025
@traefiker traefiker merged commit 057783a into traefik:master Jan 10, 2025
1 check passed
@darkweaver87 darkweaver87 mentioned this pull request Jan 15, 2025
Closed
2 tasks
@Hazmi35 Hazmi35 mentioned this pull request Feb 23, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mloiseleur mloiseleur

Assignees
No one assigned
Labels
kind/enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Don’t bundle Gateway API CRDs CRD only Helm chart for Traefik
5 participants
@darkweaver87 @mloiseleur @jnoordsij @genebean @traefiker