-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
Add a blog post about Kluctl and its use of SSA #37563
Conversation
Welcome @codablock! |
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
Some early feedback; I hope that it's useful. None of these are must-fix-right-now things.
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
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.
Just some small suggestions in places where I spotted typos or stumbled over very (to me) German-in-style formulated passages (I know the pain...).
Apart from that, from the perspective of someone who's still discovering new features and tools in and around Kubernetes everyday, this is a very good blog post imo. Concise, diving into the history of things in the right places, with a convincing motivation.
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
Pushed a new version with all the review comments being handled. |
d2da4dc
to
2c50f75
Compare
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
content/en/blog/_posts/2022-10-27-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
@codablock please set the publish date for the article to be the 4th of November (you need to change it in 2 places, the front matter and the filename) /hold |
Handled remaining review comments and changed the publish date to 2022-11-04. I've also temporarily changed the links to the podtato-head delivery scenario to be pointing to my fork until it gets merged (can't say how long this will take). |
content/en/blog/_posts/2022-11-04-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
/hold cancel |
…l-and-ssa.md Co-authored-by: Tim Bannister <[email protected]>
content/en/blog/_posts/2022-11-04-live-and-let-live-with-kluctl-and-ssa.md
Outdated
Show resolved
Hide resolved
/lgtm Would be nice to make the byline match how we usually do them. |
LGTM label has been added. Git tree hash: 25ad82b9f2e5b79e6375ca93c56e83c97c2b54dd
|
…l-and-ssa.md Co-authored-by: Tim Bannister <[email protected]>
The way SSA is implemented in `kubectl` does not allow enough | ||
control over conflict resolution as it can only switch between | ||
"not force-applying anything and erroring out" and "force-applying everything | ||
without showing any mercy!". |
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.
@apelisse and I have talked about adding a mechanism to tell the server which fields you want to force apply, thanks for the evidence that we would have at least one user ;)
I think this is a great case study, thank you very much for submitting it! |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sftim The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/lgtm |
LGTM label has been added. Git tree hash: c3de447e73f3b45d2891b728971c576f1250c8ee
|
This PR proposes a blog post about the use of SSA in Kluctl. Creating this post was suggested to me when I contacted #sig-api-machinery after reading the recent "Server Side Apply Is Great And You Should Be Using It" blog post.
I hope this post is of some value for the Kubernetes blog. I'm not sure how reviews are usually performed here and hope that it's ok that I used this early draft to directly create the PR. If preferred, I can also copy this into a Google Doc and let review happen there first.
The post also contains a link to the podtato-head repository. This link is currently dead as it refers to a directory that did not get merged yet. The PR for this can be found here. Until then, you can check this link for now to see what will be shown behind that link in the future.