Conversation
✅ Deploy Preview for kubestellarconsole ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
👋 Hey @clubanderson — thanks for opening this PR!
This is an automated message. |
There was a problem hiding this comment.
Pull request overview
Adds Longhorn to the repository’s adopters list, reflecting its use of KubeStellar Console’s guided install mission and linking to the relevant install mission page.
Changes:
- Add a new ADOPTERS table entry for Longhorn.
- Include links to the Longhorn project site, install mission, and collaboration reference issue.
ADOPTERS.MD
Outdated
| | Open Cluster Management | Guided install mission for OCM via KubeStellar Console | Sandbox | [Link](https://console.kubestellar.io/missions/install-open-cluster-management) | | ||
| | Notary Project | Guided install mission for Ratify with automated pre-flight checks, Gatekeeper and Ratify deployment via helmfile, signed/unsigned image validation, troubleshooting, and rollback support | Incubating | [Notary Project/Ratify](https://github.com/notaryproject/ratify) | | ||
| | OpenCost | Guided install mission for OpenCost via KubeStellar Console, endorsed by OpenCost contributor peatey in [opencost/opencost#3649](https://github.com/opencost/opencost/issues/3649) | Sandbox | [OpenCost](https://opencost.io) · [Install Mission](https://console.kubestellar.io/missions/install-opencost) | | ||
| | Longhorn | Guided install mission for Longhorn via KubeStellar Console, with active collaboration from Longhorn maintainers @derekbit and @PhanLe1010 in [longhorn/longhorn#12809](https://github.com/longhorn/longhorn/issues/12809) | Incubating | [Longhorn](https://longhorn.io) · [Install Mission](https://console.kubestellar.io/missions/install-longhorn) | |
There was a problem hiding this comment.
The PR body doesn’t follow the repo requirement that the first line must be Fixes #ISSUE_NUMBER (so the linked issue auto-closes on merge). Please update the PR description to add that line at the very top.
ADOPTERS.MD
Outdated
| | Open Cluster Management | Guided install mission for OCM via KubeStellar Console | Sandbox | [Link](https://console.kubestellar.io/missions/install-open-cluster-management) | | ||
| | Notary Project | Guided install mission for Ratify with automated pre-flight checks, Gatekeeper and Ratify deployment via helmfile, signed/unsigned image validation, troubleshooting, and rollback support | Incubating | [Notary Project/Ratify](https://github.com/notaryproject/ratify) | | ||
| | OpenCost | Guided install mission for OpenCost via KubeStellar Console, endorsed by OpenCost contributor peatey in [opencost/opencost#3649](https://github.com/opencost/opencost/issues/3649) | Sandbox | [OpenCost](https://opencost.io) · [Install Mission](https://console.kubestellar.io/missions/install-opencost) | | ||
| | Longhorn | Guided install mission for Longhorn via KubeStellar Console, with active collaboration from Longhorn maintainers @derekbit and @PhanLe1010 in [longhorn/longhorn#12809](https://github.com/longhorn/longhorn/issues/12809) | Incubating | [Longhorn](https://longhorn.io) · [Install Mission](https://console.kubestellar.io/missions/install-longhorn) | |
There was a problem hiding this comment.
The new ADOPTERS entry includes GitHub @ mentions for individuals. This can create unintended notifications and may get neutralized by repo sanitization tooling; consider replacing the mentions with profile links or plain names (consistent with the OpenCost row which doesn’t use @).
| | Longhorn | Guided install mission for Longhorn via KubeStellar Console, with active collaboration from Longhorn maintainers @derekbit and @PhanLe1010 in [longhorn/longhorn#12809](https://github.com/longhorn/longhorn/issues/12809) | Incubating | [Longhorn](https://longhorn.io) · [Install Mission](https://console.kubestellar.io/missions/install-longhorn) | | |
| | Longhorn | Guided install mission for Longhorn via KubeStellar Console, with active collaboration from Longhorn maintainers [derekbit](https://github.com/derekbit) and [PhanLe1010](https://github.com/PhanLe1010) in [longhorn/longhorn#12809](https://github.com/longhorn/longhorn/issues/12809) | Incubating | [Longhorn](https://longhorn.io) · [Install Mission](https://console.kubestellar.io/missions/install-longhorn) | |
🔄 Auto-Applying Copilot Code ReviewCopilot code review found 1 code suggestion(s) and 1 general comment(s). @copilot Please apply all of the following code review suggestions:
Also address these general comments:
Push all fixes in a single commit. Run Auto-generated by copilot-review-apply workflow. |
Applied in commit 98c950a — replaced |
Signed-off-by: Andy Anderson <andy@clubanderson.com>
98c950a to
a319ef0
Compare
|
Thank you for your contribution! Your PR has been merged. Check out what's new:
Stay connected: Slack #kubestellar-dev | Multi-Cluster Survey |
… approval Reverts #3987, #3988, #3989, #3990, #3991, #3992, #3993, #3994, #3995, #3996, #3999 These PRs were merged prematurely by automation. The adopter entries require approval from the external CNCF project contributors before merging. Each PR will be re-opened for the original contributors to review and approve. Signed-off-by: Andrew Anderson <andy@clubanderson.com>
… approval (#4034) Reverts #3987, #3988, #3989, #3990, #3991, #3992, #3993, #3994, #3995, #3996, #3999 These PRs were merged prematurely by automation. The adopter entries require approval from the external CNCF project contributors before merging. Each PR will be re-opened for the original contributors to review and approve. Signed-off-by: Andrew Anderson <andy@clubanderson.com>
|
Apologies — this PR was merged in error by automation before receiving approval from the external project contributor. We've reverted it (PR #4034). The adopter entry will need to be re-submitted and merged only after receiving approval from a contributor on the respective CNCF project. Sorry for the confusion. |
|
Replacement PR created: #4046 — this one will remain open until approved by the external project contributor. |
📌 Fixes
📝 Summary of Changes
@mentions with profile links to avoid unintended notifications, consistent with the OpenCost row styleChanges Made
[derekbit](https://github.com/derekbit),[PhanLe1010](https://github.com/PhanLe1010)) instead of@mentionsChecklist
Please ensure the following before submitting your PR:
git commit -s)Screenshots or Logs (if applicable)
N/A — documentation-only change.
👀 Reviewer Notes
This is a documentation-only change to ADOPTERS.MD. No code, tests, or build artifacts are affected.