Repository navigation
docs: add v1.0.0 changelog entries - #1715
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. Important Review skippedReview was skipped as selected files did not have any reviewable changes. ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe changelog adds v1.0.0 release notes. It describes optional per-workspace NetworkPolicy provisioning, TLS profile handling, deployment metadata preservation, and a backup Job fix for shared storage on multi-node clusters. Changesv1.0.0 Release Notes
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~4 minutes Change: Other Merge Risk: ⚪ Minimal · up to No actionable release-note issue is established. The changelog is mergeable after the planned rendering check. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 1 system. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: btjd, rohanKanojia 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 |
|
|
||
| ## Bug Fixes & Improvements | ||
|
|
||
| - Fix backup/restore failing for per-user (common) storage on multi-node clusters due to PVC multi-attach errors by pinning backup Jobs to the node where the shared PVC is mounted [#1677](https://github.com/devfile/devworkspace-operator/pull/1677) |
There was a problem hiding this comment.
I have a question about this item, it was merged in July and should have made it in an older DWO version if I'm not mistaken? Was there a reason to add this in the 1.0 changelog?
There was a problem hiding this comment.
Actually, I was also thinking the same thing when I saw for the first time , but when I checked, we released DWO on 29 Jul https://github.com/devfile/devworkspace-operator/releases/tag/v0.43.0 and this PR got merged on 30 Jul
There was a problem hiding this comment.
Correct, as Rohan pointed out, 1677 was merged on July 30th, one day after the completion of 0.43.0.
Add release notes for v1.0.0 covering NetworkPolicy provisioning, OpenShift TLS configuration, deployment label tolerance, and backup/restore fix for multi-node clusters. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Badre Tejado-Imam <btejado@redhat.com>
|
New changes are detected. LGTM label has been removed. |
Summary
CHANGELOG.mdahead of the prerelease branch cutTest plan
1.0.xbranch inherits the changelog🤖 Generated with Claude Code
Summary by CodeRabbit