Skip to content

docs: Clarify instructions for bumping K8s to avoid forks#34791

Merged
christarazi merged 2 commits intocilium:mainfrom
christarazi:pr/christarazi/clarify-k8s-bump-ins
Sep 11, 2024
Merged

docs: Clarify instructions for bumping K8s to avoid forks#34791
christarazi merged 2 commits intocilium:mainfrom
christarazi:pr/christarazi/clarify-k8s-bump-ins

Conversation

@christarazi
Copy link
Copy Markdown
Member

Opening a PR against a branch on a fork won't properly test all of the
changes in the PR because parts of the CI suite don't pick up the
changes in the workflows for security reasons.

Signed-off-by: Chris Tarazi chris@isovalent.com

@christarazi christarazi added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers. release-note/misc This PR makes changes that have no direct user impact. sig/community Impacts contribution workflow, guidelines, and tools. labels Sep 9, 2024
@christarazi christarazi marked this pull request as ready for review September 9, 2024 17:18
@christarazi christarazi requested review from a team as code owners September 9, 2024 17:18
@christarazi christarazi requested a review from qmonnet September 9, 2024 17:18
@christarazi
Copy link
Copy Markdown
Member Author

/test

Copy link
Copy Markdown
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

Looks good! Non-blocking nits below

@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Sep 9, 2024
@qmonnet qmonnet removed the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Sep 9, 2024
@aanm aanm self-requested a review September 10, 2024 08:53
@christarazi christarazi force-pushed the pr/christarazi/clarify-k8s-bump-ins branch from ec5c447 to c03a300 Compare September 10, 2024 19:34
Opening a PR against a branch on a fork won't properly test all of the
changes in the PR because parts of the CI suite don't pick up the
changes in the workflows for security reasons.

Signed-off-by: Chris Tarazi <chris@isovalent.com>
Suggested-by: André Martins  <andre@cilium.io>
Signed-off-by: Chris Tarazi <chris@isovalent.com>
@christarazi christarazi force-pushed the pr/christarazi/clarify-k8s-bump-ins branch from c03a300 to be2ece0 Compare September 10, 2024 19:36
@christarazi christarazi requested a review from aanm September 10, 2024 19:36
@christarazi
Copy link
Copy Markdown
Member Author

/test

@christarazi christarazi added this pull request to the merge queue Sep 11, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Sep 11, 2024
Merged via the queue into cilium:main with commit 5921e46 Sep 11, 2024
@christarazi christarazi deleted the pr/christarazi/clarify-k8s-bump-ins branch September 11, 2024 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/k8s Impacts the kubernetes API, or kubernetes -> cilium internals translation layers. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact. sig/community Impacts contribution workflow, guidelines, and tools.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants