Docs main placeholder pr concept#1026
Merged
Merged
Conversation
Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com>
Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com>
✅ Deploy Preview for kpt-porch ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds dedicated documentation for “placeholder package revisions” and updates the Concepts index to link to the new page.
Changes:
- Added a new concepts doc page describing placeholder package revision behavior, lifecycle, rollback semantics, and GitOps relationship
- Replaced the inline placeholder description in the concepts index with a relref link + shorter summary
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| docs/content/en/docs/2_concepts/placeholder-package-revision.md | New doc page defining placeholder package revisions and explaining lifecycle/usage constraints |
| docs/content/en/docs/2_concepts/_index.md | Replaces long inline placeholder description with a link to the new dedicated doc page |
Catalin-Stratulat-Ericsson
added
the
documentation
nagygergo
reviewed
Jun 10, 2026
nagygergo
reviewed
Jun 10, 2026
nagygergo
reviewed
Jun 10, 2026
efiacor
requested review from
efiacor,
kispaljr,
liamfallon and
mozesl-nokia
as code owners
|
liamfallon
reviewed
Jun 17, 2026
dosubot
Bot
added
the
size:L
Co-authored-by: Liam Fallon <35595825+liamfallon@users.noreply.github.com> Co-authored-by: JamesMcDermott <james.j.mcdermott@ericsson.com> Co-authored-by: Rendre Greyling <rendre.greyling@nokia.com> Signed-off-by: Catalin Stratulat <159934629+Catalin-Stratulat-Ericsson@users.noreply.github.com>
Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com>
rendre-greyling
previously approved these changes
Jun 24, 2026
…de git without porch on front page Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com>
Contributor
|
Couple more wording adjustment comments |
applied the last of the suggestions Co-authored-by: JamesMcDermott <james.j.mcdermott@ericsson.com> Signed-off-by: Catalin Stratulat <159934629+Catalin-Stratulat-Ericsson@users.noreply.github.com>
JamesMcDermott
approved these changes
Jun 25, 2026
liamfallon
approved these changes
Jun 25, 2026
liamfallon
added a commit
that referenced
this pull request
Jun 25, 2026
* Docs main placeholder pr concept (#1026) * Adding concept for main/placeholder PackageRevision Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com> * missed newline Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com> * Apply suggestions from code review Co-authored-by: Liam Fallon <35595825+liamfallon@users.noreply.github.com> Co-authored-by: JamesMcDermott <james.j.mcdermott@ericsson.com> Co-authored-by: Rendre Greyling <rendre.greyling@nokia.com> Signed-off-by: Catalin Stratulat <159934629+Catalin-Stratulat-Ericsson@users.noreply.github.com> * addressing comments about main deletion + summary table amends Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com> * removing dangerous statement about user modification of packages inside git without porch on front page Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com> * Apply suggestions from code review applied the last of the suggestions Co-authored-by: JamesMcDermott <james.j.mcdermott@ericsson.com> Signed-off-by: Catalin Stratulat <159934629+Catalin-Stratulat-Ericsson@users.noreply.github.com> --------- Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com> Signed-off-by: Catalin Stratulat <159934629+Catalin-Stratulat-Ericsson@users.noreply.github.com> Co-authored-by: Fiachra Corcoran <fiachra.corcoran@est.tech> Co-authored-by: Liam Fallon <35595825+liamfallon@users.noreply.github.com> Co-authored-by: JamesMcDermott <james.j.mcdermott@ericsson.com> Co-authored-by: Rendre Greyling <rendre.greyling@nokia.com> * addressing grammar comments Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com> --------- Signed-off-by: Catalin-Stratulat-Ericsson <catalin.stratulat@ericsson.com> Signed-off-by: Catalin Stratulat <159934629+Catalin-Stratulat-Ericsson@users.noreply.github.com> Co-authored-by: Fiachra Corcoran <fiachra.corcoran@est.tech> Co-authored-by: Liam Fallon <35595825+liamfallon@users.noreply.github.com> Co-authored-by: JamesMcDermott <james.j.mcdermott@ericsson.com> Co-authored-by: Rendre Greyling <rendre.greyling@nokia.com>
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
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.
Title
Adding concept for main/placeholder PackageRevision
Description
latest-1if the latest is deleted where main is pointing atRelated Issue(s)
Type of Change
Checklist
Testing Instructions (Optional)
Additional Notes (Optional)
AI Disclosure
If so, please describe how: