Release Notes for 0.31.0-RC.2 - #965
Conversation
|
Warning Rate limit exceededNick Volynkin (@NickVolynkin) has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 12 minutes and 38 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
WalkthroughThe changelog for Cozystack v0.31.0 release candidate 2 documents new features, enhancements, bug fixes, dependency updates, and maintenance activities. Major highlights include GPU and ARM support, Kubernetes cluster improvements, dashboard introduction, new controllers, policy updates, and a comprehensive list of fixes and dependency upgrades. Changes
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
|
||
| Cozystack 0.31.0 further advances GPU support, monitoring, and all-around convenience features. | ||
|
|
||
| ## New Features and Changes |
There was a problem hiding this comment.
Let's take an example from kubernetes' changelog. It would be nice to have a header somewhere stating that these are changes since v0.30.0 or v0.31.0-rc.1 or whatever. As a maintainer, I would actually prefer to have the UNRELEASED.md document list changes since v0.31.0-rc.1. This would make it easier for me to compile the next release by just picking the specific PRs that are only present in main and not in the latest release candidate.
There was a problem hiding this comment.
I have the following workflow in my mind:
- When we publish a stable release and stop merging things to
main, make this fileunreleased.mdthere - All the way to a new stable version, add lines in the file.
- On a release candidate, take what we have and use it as release notes. Then, add more lines.
- On a stable release, update the file and rename it to
v0.31.0.md, for example
|
I think |
9896ea0 to
f282d1f
Compare
|
This PR probably needs to be merged into |
f282d1f to
aac52e9
Compare
Add to v0.31.0.md, as we're acuumulating changelogs until a stable release. See https://github.com/cozystack/cozystack/blob/main/docs/release.md#release-candidates Signed-off-by: Nick Volynkin <nick.volynkin@gmail.com>
19d254a to
b327f47
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (4)
docs/changelogs/v0.31.0.md (4)
1-2: Merge into bothrelease-0.31andmainbranches
To ensure clear traceability of changes between RC.1, RC.2, and since v0.30, please merge this changelog into both therelease-0.31branch and themainbranch.
1-3: Standardize changelog file naming & location
The directorydocs/changelogsand version-specific file name may be too generic. Common conventions use a top-levelCHANGELOG.mdor anUNRELEASED.mdunderdocs/releases/(in all-caps) for easier discovery and consistency.
67-67: Correct grammar and version for MetalLB entry
The entry* MetalLB s now included directly as a patched image based on version 14.9. (@lllamnyp in https://github.com/cozystack/cozystack/pull/945)contains a stray “s” and omits the leading zero in “0.14.9.” It should be:
* MetalLB is now included directly as a patched image based on version 0.14.9. (@lllamnyp in https://github.com/cozystack/cozystack/pull/945)🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
67-67: Bare URL used
null(MD034, no-bare-urls)
84-88: Consistent contributor handle formatting
The line* gwynbleidd2106 made their first contribution in https://github.com/cozystack/cozystack/pull/962is missing an “@” before the GitHub handle and should link explicitly. It should read:
* @gwynbleidd2106 made their first contribution in https://github.com/cozystack/cozystack/pull/962🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
84-84: Bare URL used
null(MD034, no-bare-urls)
85-85: Bare URL used
null(MD034, no-bare-urls)
86-86: Bare URL used
null(MD034, no-bare-urls)
87-87: Bare URL used
null(MD034, no-bare-urls)
🧹 Nitpick comments (3)
docs/changelogs/v0.31.0.md (3)
9-11: Fix nested list indentation
Several nested list items are indented by 4 spaces instead of the expected 2 spaces, causing inconsistent markdown formatting (MD007). Please adjust these to 2-space indents.Also applies to: 13-14, 23-27
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
9-9: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
9-9: Bare URL used
null(MD034, no-bare-urls)
10-10: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
10-10: Bare URL used
null(MD034, no-bare-urls)
11-11: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
11-11: Bare URL used
null(MD034, no-bare-urls)
7-27: Replace bare URLs with Markdown links
Numerous list items include bare URLs (MD034). Consider using reference-style links, angle-bracketed URLs, or descriptive link text to improve readability.Also applies to: 31-63, 67-76, 84-88, 89-89
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
7-7: Bare URL used
null(MD034, no-bare-urls)
9-9: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
9-9: Bare URL used
null(MD034, no-bare-urls)
10-10: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
10-10: Bare URL used
null(MD034, no-bare-urls)
11-11: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
11-11: Bare URL used
null(MD034, no-bare-urls)
13-13: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
13-13: Bare URL used
null(MD034, no-bare-urls)
14-14: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
14-14: Bare URL used
null(MD034, no-bare-urls)
15-15: Bare URL used
null(MD034, no-bare-urls)
16-16: Bare URL used
null(MD034, no-bare-urls)
17-17: Bare URL used
null(MD034, no-bare-urls)
18-18: Bare URL used
null(MD034, no-bare-urls)
19-19: Bare URL used
null(MD034, no-bare-urls)
20-20: Bare URL used
null(MD034, no-bare-urls)
21-21: Bare URL used
null(MD034, no-bare-urls)
23-23: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
23-23: Bare URL used
null(MD034, no-bare-urls)
24-24: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
24-24: Bare URL used
null(MD034, no-bare-urls)
25-25: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
25-25: Bare URL used
null(MD034, no-bare-urls)
25-25: Bare URL used
null(MD034, no-bare-urls)
25-25: Bare URL used
null(MD034, no-bare-urls)
26-26: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
26-26: Bare URL used
null(MD034, no-bare-urls)
27-27: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
27-27: Bare URL used
null(MD034, no-bare-urls)
27-27: Bare URL used
null(MD034, no-bare-urls)
46-46: Add missing article
The line* [monitoring] Remove legacy label "POD" from the exclude filter in metrics.should read
* [monitoring] Remove the legacy label "POD" from the exclude filter in metrics.to include the article “the.”
🧰 Tools
🪛 LanguageTool
[uncategorized] ~46-~46: You might be missing the article “the” here.
Context: ...ystack/pull/956) * [monitoring] Remove legacy label "POD" from the exclude filter in ...(AI_EN_LECTOR_MISSING_DETERMINER_THE)
🪛 markdownlint-cli2 (0.17.2)
46-46: Bare URL used
null(MD034, no-bare-urls)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting
📒 Files selected for processing (1)
docs/changelogs/v0.31.0.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/changelogs/v0.31.0.md
[uncategorized] ~46-~46: You might be missing the article “the” here.
Context: ...ystack/pull/956) * [monitoring] Remove legacy label "POD" from the exclude filter in ...
(AI_EN_LECTOR_MISSING_DETERMINER_THE)
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.31.0.md
7-7: Bare URL used
null
(MD034, no-bare-urls)
9-9: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
9-9: Bare URL used
null
(MD034, no-bare-urls)
10-10: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
10-10: Bare URL used
null
(MD034, no-bare-urls)
11-11: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
11-11: Bare URL used
null
(MD034, no-bare-urls)
13-13: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
13-13: Bare URL used
null
(MD034, no-bare-urls)
14-14: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
14-14: Bare URL used
null
(MD034, no-bare-urls)
15-15: Bare URL used
null
(MD034, no-bare-urls)
16-16: Bare URL used
null
(MD034, no-bare-urls)
17-17: Bare URL used
null
(MD034, no-bare-urls)
18-18: Bare URL used
null
(MD034, no-bare-urls)
19-19: Bare URL used
null
(MD034, no-bare-urls)
20-20: Bare URL used
null
(MD034, no-bare-urls)
21-21: Bare URL used
null
(MD034, no-bare-urls)
23-23: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
23-23: Bare URL used
null
(MD034, no-bare-urls)
24-24: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
24-24: Bare URL used
null
(MD034, no-bare-urls)
25-25: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
25-25: Bare URL used
null
(MD034, no-bare-urls)
25-25: Bare URL used
null
(MD034, no-bare-urls)
25-25: Bare URL used
null
(MD034, no-bare-urls)
26-26: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
26-26: Bare URL used
null
(MD034, no-bare-urls)
27-27: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
27-27: Bare URL used
null
(MD034, no-bare-urls)
27-27: Bare URL used
null
(MD034, no-bare-urls)
31-31: Bare URL used
null
(MD034, no-bare-urls)
32-32: Bare URL used
null
(MD034, no-bare-urls)
33-33: Bare URL used
null
(MD034, no-bare-urls)
34-34: Bare URL used
null
(MD034, no-bare-urls)
35-35: Bare URL used
null
(MD034, no-bare-urls)
36-36: Bare URL used
null
(MD034, no-bare-urls)
36-36: Bare URL used
null
(MD034, no-bare-urls)
36-36: Bare URL used
null
(MD034, no-bare-urls)
37-37: Bare URL used
null
(MD034, no-bare-urls)
38-38: Bare URL used
null
(MD034, no-bare-urls)
38-38: Bare URL used
null
(MD034, no-bare-urls)
39-39: Bare URL used
null
(MD034, no-bare-urls)
40-40: Bare URL used
null
(MD034, no-bare-urls)
41-41: Bare URL used
null
(MD034, no-bare-urls)
42-42: Bare URL used
null
(MD034, no-bare-urls)
43-43: Bare URL used
null
(MD034, no-bare-urls)
44-44: Bare URL used
null
(MD034, no-bare-urls)
45-45: Bare URL used
null
(MD034, no-bare-urls)
45-45: Bare URL used
null
(MD034, no-bare-urls)
46-46: Bare URL used
null
(MD034, no-bare-urls)
47-47: Bare URL used
null
(MD034, no-bare-urls)
47-47: Bare URL used
null
(MD034, no-bare-urls)
48-48: Bare URL used
null
(MD034, no-bare-urls)
49-49: Bare URL used
null
(MD034, no-bare-urls)
50-50: Bare URL used
null
(MD034, no-bare-urls)
51-51: Bare URL used
null
(MD034, no-bare-urls)
52-52: Bare URL used
null
(MD034, no-bare-urls)
53-53: Bare URL used
null
(MD034, no-bare-urls)
54-54: Bare URL used
null
(MD034, no-bare-urls)
55-55: Bare URL used
null
(MD034, no-bare-urls)
55-55: Bare URL used
null
(MD034, no-bare-urls)
55-55: Bare URL used
null
(MD034, no-bare-urls)
56-56: Bare URL used
null
(MD034, no-bare-urls)
57-57: Bare URL used
null
(MD034, no-bare-urls)
58-58: Bare URL used
null
(MD034, no-bare-urls)
59-59: Bare URL used
null
(MD034, no-bare-urls)
60-60: Bare URL used
null
(MD034, no-bare-urls)
61-61: Bare URL used
null
(MD034, no-bare-urls)
62-62: Bare URL used
null
(MD034, no-bare-urls)
63-63: Bare URL used
null
(MD034, no-bare-urls)
67-67: Bare URL used
null
(MD034, no-bare-urls)
68-68: Bare URL used
null
(MD034, no-bare-urls)
69-69: Bare URL used
null
(MD034, no-bare-urls)
70-70: Bare URL used
null
(MD034, no-bare-urls)
71-71: Bare URL used
null
(MD034, no-bare-urls)
72-72: Bare URL used
null
(MD034, no-bare-urls)
72-72: Bare URL used
null
(MD034, no-bare-urls)
73-73: Bare URL used
null
(MD034, no-bare-urls)
74-74: Bare URL used
null
(MD034, no-bare-urls)
74-74: Bare URL used
null
(MD034, no-bare-urls)
75-75: Bare URL used
null
(MD034, no-bare-urls)
75-75: Bare URL used
null
(MD034, no-bare-urls)
76-76: Bare URL used
null
(MD034, no-bare-urls)
80-80: Bare URL used
null
(MD034, no-bare-urls)
84-84: Bare URL used
null
(MD034, no-bare-urls)
85-85: Bare URL used
null
(MD034, no-bare-urls)
86-86: Bare URL used
null
(MD034, no-bare-urls)
87-87: Bare URL used
null
(MD034, no-bare-urls)
89-89: Bare URL used
null
(MD034, no-bare-urls)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Build and Test
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
docs/changelogs/v0.31.0.md (1)
67-67: Duplicate: Fix grammar and version in MetalLB entry
This suggestion matches lllamny’s earlier feedback.Example fix:
-* MetalLB s now included directly as a patched image based on version 14.9. (@lllamnyp in https://github.com/cozystack/cozystack/pull/945) +* MetalLB is now included directly as a patched image based on version 0.14.9. (@lllamnyp in https://github.com/cozystack/cozystack/pull/945)🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
67-67: Bare URL used
null(MD034, no-bare-urls)
🧹 Nitpick comments (5)
docs/changelogs/v0.31.0.md (5)
1-4: Add a top-level header for clarity
The file begins with prose but lacks a clear title. Introducing an H1 header (e.g., “Cozystack v0.31.0-rc.2 Release Notes”) will make it easier to locate and render correctly in generated docs.Example:
+ # Cozystack v0.31.0-rc.2 Release Notes This is the second release candidate for the upcoming Cozystack v0.31.0 release. Cozystack 0.31.0 further advances GPU support, monitoring, and all-around convenience features.
8-11: Fix nested list indentation to conform with markdownlint (MD007)
Nested bullets are indented with four spaces but the lint rule expects two. Please adjust all second-level list items accordingly.Example patch:
-* Add VerticalPodAutoscaler to a few more components: - * [kubernetes] Kubernetes clusters in user tenants. (@klinch0 in https://github.com/cozystack/cozystack/pull/806) - * [platform] Cozystack dashboard. (@klinch0 in https://github.com/cozystack/cozystack/pull/828) - * [platform] Cozystack etcd-operator (@klinch0 in https://github.com/cozystack/cozystack/pull/850) +* Add VerticalPodAutoscaler to a few more components: + * [kubernetes] Kubernetes clusters in user tenants. (@klinch0 in https://github.com/cozystack/cozystack/pull/806) + * [platform] Cozystack dashboard. (@klinch0 in https://github.com/cozystack/cozystack/pull/828) + * [platform] Cozystack etcd-operator (@klinch0 in https://github.com/cozystack/cozystack/pull/850)Apply the same two-space indent to the other nested groups under cross-architecture builds (lines 13–14) and release policy (lines 23–27).
Also applies to: 13-14, 23-27
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
9-9: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
9-9: Bare URL used
null(MD034, no-bare-urls)
10-10: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
10-10: Bare URL used
null(MD034, no-bare-urls)
11-11: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
11-11: Bare URL used
null(MD034, no-bare-urls)
7-89: Convert bare URLs to inline markdown links for readability (MD034)
There are many bare URLs throughout the lists. Consider using link text or PR numbers, for example[PR #834](https://github.com/…/834), and wrapping full-changelog URLs in descriptive link text. This improves readability and prevents lint errors.Example for a feature entry and the full-changelog link:
-* [kubernetes] Introduce GPU support for tenant Kubernetes clusters. (@kvaps in https://github.com/cozystack/cozystack/pull/834) +* [kubernetes] Introduce GPU support for tenant Kubernetes clusters. (@kvaps in [#834](https://github.com/cozystack/cozystack/pull/834)) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.30.0...v0.31.0-rc.2 +**Full Changelog**: [Compare v0.30.0 → v0.31.0-rc.2](https://github.com/cozystack/cozystack/compare/v0.30.0...v0.31.0-rc.2)Please apply this pattern consistently to all entries.
🧰 Tools
🪛 LanguageTool
[uncategorized] ~46-~46: You might be missing the article “the” here.
Context: ...ystack/pull/956) * [monitoring] Remove legacy label "POD" from the exclude filter in ...(AI_EN_LECTOR_MISSING_DETERMINER_THE)
🪛 markdownlint-cli2 (0.17.2)
7-7: Bare URL used
null(MD034, no-bare-urls)
9-9: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
9-9: Bare URL used
null(MD034, no-bare-urls)
10-10: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
10-10: Bare URL used
null(MD034, no-bare-urls)
11-11: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
11-11: Bare URL used
null(MD034, no-bare-urls)
13-13: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
13-13: Bare URL used
null(MD034, no-bare-urls)
14-14: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
14-14: Bare URL used
null(MD034, no-bare-urls)
15-15: Bare URL used
null(MD034, no-bare-urls)
16-16: Bare URL used
null(MD034, no-bare-urls)
17-17: Bare URL used
null(MD034, no-bare-urls)
18-18: Bare URL used
null(MD034, no-bare-urls)
19-19: Bare URL used
null(MD034, no-bare-urls)
20-20: Bare URL used
null(MD034, no-bare-urls)
21-21: Bare URL used
null(MD034, no-bare-urls)
23-23: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
23-23: Bare URL used
null(MD034, no-bare-urls)
24-24: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
24-24: Bare URL used
null(MD034, no-bare-urls)
25-25: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
25-25: Bare URL used
null(MD034, no-bare-urls)
25-25: Bare URL used
null(MD034, no-bare-urls)
25-25: Bare URL used
null(MD034, no-bare-urls)
26-26: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
26-26: Bare URL used
null(MD034, no-bare-urls)
27-27: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
27-27: Bare URL used
null(MD034, no-bare-urls)
27-27: Bare URL used
null(MD034, no-bare-urls)
31-31: Bare URL used
null(MD034, no-bare-urls)
32-32: Bare URL used
null(MD034, no-bare-urls)
33-33: Bare URL used
null(MD034, no-bare-urls)
34-34: Bare URL used
null(MD034, no-bare-urls)
35-35: Bare URL used
null(MD034, no-bare-urls)
36-36: Bare URL used
null(MD034, no-bare-urls)
36-36: Bare URL used
null(MD034, no-bare-urls)
36-36: Bare URL used
null(MD034, no-bare-urls)
37-37: Bare URL used
null(MD034, no-bare-urls)
38-38: Bare URL used
null(MD034, no-bare-urls)
38-38: Bare URL used
null(MD034, no-bare-urls)
39-39: Bare URL used
null(MD034, no-bare-urls)
40-40: Bare URL used
null(MD034, no-bare-urls)
41-41: Bare URL used
null(MD034, no-bare-urls)
42-42: Bare URL used
null(MD034, no-bare-urls)
43-43: Bare URL used
null(MD034, no-bare-urls)
44-44: Bare URL used
null(MD034, no-bare-urls)
45-45: Bare URL used
null(MD034, no-bare-urls)
45-45: Bare URL used
null(MD034, no-bare-urls)
46-46: Bare URL used
null(MD034, no-bare-urls)
47-47: Bare URL used
null(MD034, no-bare-urls)
47-47: Bare URL used
null(MD034, no-bare-urls)
48-48: Bare URL used
null(MD034, no-bare-urls)
49-49: Bare URL used
null(MD034, no-bare-urls)
50-50: Bare URL used
null(MD034, no-bare-urls)
51-51: Bare URL used
null(MD034, no-bare-urls)
52-52: Bare URL used
null(MD034, no-bare-urls)
53-53: Bare URL used
null(MD034, no-bare-urls)
54-54: Bare URL used
null(MD034, no-bare-urls)
55-55: Bare URL used
null(MD034, no-bare-urls)
55-55: Bare URL used
null(MD034, no-bare-urls)
55-55: Bare URL used
null(MD034, no-bare-urls)
56-56: Bare URL used
null(MD034, no-bare-urls)
57-57: Bare URL used
null(MD034, no-bare-urls)
58-58: Bare URL used
null(MD034, no-bare-urls)
59-59: Bare URL used
null(MD034, no-bare-urls)
60-60: Bare URL used
null(MD034, no-bare-urls)
61-61: Bare URL used
null(MD034, no-bare-urls)
62-62: Bare URL used
null(MD034, no-bare-urls)
63-63: Bare URL used
null(MD034, no-bare-urls)
67-67: Bare URL used
null(MD034, no-bare-urls)
68-68: Bare URL used
null(MD034, no-bare-urls)
69-69: Bare URL used
null(MD034, no-bare-urls)
70-70: Bare URL used
null(MD034, no-bare-urls)
71-71: Bare URL used
null(MD034, no-bare-urls)
72-72: Bare URL used
null(MD034, no-bare-urls)
72-72: Bare URL used
null(MD034, no-bare-urls)
73-73: Bare URL used
null(MD034, no-bare-urls)
74-74: Bare URL used
null(MD034, no-bare-urls)
74-74: Bare URL used
null(MD034, no-bare-urls)
75-75: Bare URL used
null(MD034, no-bare-urls)
75-75: Bare URL used
null(MD034, no-bare-urls)
76-76: Bare URL used
null(MD034, no-bare-urls)
80-80: Bare URL used
null(MD034, no-bare-urls)
84-84: Bare URL used
null(MD034, no-bare-urls)
85-85: Bare URL used
null(MD034, no-bare-urls)
86-86: Bare URL used
null(MD034, no-bare-urls)
87-87: Bare URL used
null(MD034, no-bare-urls)
89-89: Bare URL used
null(MD034, no-bare-urls)
46-46: Add missing article “the” for clarity
Small grammatical tweak to improve readability.-* [monitoring] Remove legacy label "POD" from the exclude filter in metrics. (@xy2 in https://github.com/cozystack/cozystack/pull/826) +* [monitoring] Remove legacy label "POD" from the exclude filter in the metrics. (@xy2 in https://github.com/cozystack/cozystack/pull/826)🧰 Tools
🪛 LanguageTool
[uncategorized] ~46-~46: You might be missing the article “the” here.
Context: ...ystack/pull/956) * [monitoring] Remove legacy label "POD" from the exclude filter in ...(AI_EN_LECTOR_MISSING_DETERMINER_THE)
🪛 markdownlint-cli2 (0.17.2)
46-46: Bare URL used
null(MD034, no-bare-urls)
87-87: Add missing “@” for consistent username formatting
All other contributors are prefixed with@. Please update this entry.-* gwynbleidd2106 made their first contribution in https://github.com/cozystack/cozystack/pull/962 +* @gwynbleidd2106 made their first contribution in https://github.com/cozystack/cozystack/pull/962🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
87-87: Bare URL used
null(MD034, no-bare-urls)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Cache: Disabled due to data retention organization setting
Knowledge Base: Disabled due to data retention organization setting
📒 Files selected for processing (1)
docs/changelogs/v0.31.0.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/changelogs/v0.31.0.md
[uncategorized] ~46-~46: You might be missing the article “the” here.
Context: ...ystack/pull/956) * [monitoring] Remove legacy label "POD" from the exclude filter in ...
(AI_EN_LECTOR_MISSING_DETERMINER_THE)
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.31.0.md
7-7: Bare URL used
null
(MD034, no-bare-urls)
9-9: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
9-9: Bare URL used
null
(MD034, no-bare-urls)
10-10: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
10-10: Bare URL used
null
(MD034, no-bare-urls)
11-11: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
11-11: Bare URL used
null
(MD034, no-bare-urls)
13-13: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
13-13: Bare URL used
null
(MD034, no-bare-urls)
14-14: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
14-14: Bare URL used
null
(MD034, no-bare-urls)
15-15: Bare URL used
null
(MD034, no-bare-urls)
16-16: Bare URL used
null
(MD034, no-bare-urls)
17-17: Bare URL used
null
(MD034, no-bare-urls)
18-18: Bare URL used
null
(MD034, no-bare-urls)
19-19: Bare URL used
null
(MD034, no-bare-urls)
20-20: Bare URL used
null
(MD034, no-bare-urls)
21-21: Bare URL used
null
(MD034, no-bare-urls)
23-23: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
23-23: Bare URL used
null
(MD034, no-bare-urls)
24-24: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
24-24: Bare URL used
null
(MD034, no-bare-urls)
25-25: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
25-25: Bare URL used
null
(MD034, no-bare-urls)
25-25: Bare URL used
null
(MD034, no-bare-urls)
25-25: Bare URL used
null
(MD034, no-bare-urls)
26-26: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
26-26: Bare URL used
null
(MD034, no-bare-urls)
27-27: Unordered list indentation
Expected: 2; Actual: 4
(MD007, ul-indent)
27-27: Bare URL used
null
(MD034, no-bare-urls)
27-27: Bare URL used
null
(MD034, no-bare-urls)
31-31: Bare URL used
null
(MD034, no-bare-urls)
32-32: Bare URL used
null
(MD034, no-bare-urls)
33-33: Bare URL used
null
(MD034, no-bare-urls)
34-34: Bare URL used
null
(MD034, no-bare-urls)
35-35: Bare URL used
null
(MD034, no-bare-urls)
36-36: Bare URL used
null
(MD034, no-bare-urls)
36-36: Bare URL used
null
(MD034, no-bare-urls)
36-36: Bare URL used
null
(MD034, no-bare-urls)
37-37: Bare URL used
null
(MD034, no-bare-urls)
38-38: Bare URL used
null
(MD034, no-bare-urls)
38-38: Bare URL used
null
(MD034, no-bare-urls)
39-39: Bare URL used
null
(MD034, no-bare-urls)
40-40: Bare URL used
null
(MD034, no-bare-urls)
41-41: Bare URL used
null
(MD034, no-bare-urls)
42-42: Bare URL used
null
(MD034, no-bare-urls)
43-43: Bare URL used
null
(MD034, no-bare-urls)
44-44: Bare URL used
null
(MD034, no-bare-urls)
45-45: Bare URL used
null
(MD034, no-bare-urls)
45-45: Bare URL used
null
(MD034, no-bare-urls)
46-46: Bare URL used
null
(MD034, no-bare-urls)
47-47: Bare URL used
null
(MD034, no-bare-urls)
47-47: Bare URL used
null
(MD034, no-bare-urls)
48-48: Bare URL used
null
(MD034, no-bare-urls)
49-49: Bare URL used
null
(MD034, no-bare-urls)
50-50: Bare URL used
null
(MD034, no-bare-urls)
51-51: Bare URL used
null
(MD034, no-bare-urls)
52-52: Bare URL used
null
(MD034, no-bare-urls)
53-53: Bare URL used
null
(MD034, no-bare-urls)
54-54: Bare URL used
null
(MD034, no-bare-urls)
55-55: Bare URL used
null
(MD034, no-bare-urls)
55-55: Bare URL used
null
(MD034, no-bare-urls)
55-55: Bare URL used
null
(MD034, no-bare-urls)
56-56: Bare URL used
null
(MD034, no-bare-urls)
57-57: Bare URL used
null
(MD034, no-bare-urls)
58-58: Bare URL used
null
(MD034, no-bare-urls)
59-59: Bare URL used
null
(MD034, no-bare-urls)
60-60: Bare URL used
null
(MD034, no-bare-urls)
61-61: Bare URL used
null
(MD034, no-bare-urls)
62-62: Bare URL used
null
(MD034, no-bare-urls)
63-63: Bare URL used
null
(MD034, no-bare-urls)
67-67: Bare URL used
null
(MD034, no-bare-urls)
68-68: Bare URL used
null
(MD034, no-bare-urls)
69-69: Bare URL used
null
(MD034, no-bare-urls)
70-70: Bare URL used
null
(MD034, no-bare-urls)
71-71: Bare URL used
null
(MD034, no-bare-urls)
72-72: Bare URL used
null
(MD034, no-bare-urls)
72-72: Bare URL used
null
(MD034, no-bare-urls)
73-73: Bare URL used
null
(MD034, no-bare-urls)
74-74: Bare URL used
null
(MD034, no-bare-urls)
74-74: Bare URL used
null
(MD034, no-bare-urls)
75-75: Bare URL used
null
(MD034, no-bare-urls)
75-75: Bare URL used
null
(MD034, no-bare-urls)
76-76: Bare URL used
null
(MD034, no-bare-urls)
80-80: Bare URL used
null
(MD034, no-bare-urls)
84-84: Bare URL used
null
(MD034, no-bare-urls)
85-85: Bare URL used
null
(MD034, no-bare-urls)
86-86: Bare URL used
null
(MD034, no-bare-urls)
87-87: Bare URL used
null
(MD034, no-bare-urls)
89-89: Bare URL used
null
(MD034, no-bare-urls)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: Build and Test
🔇 Additional comments (1)
docs/changelogs/v0.31.0.md (1)
90-90: Ensure this changelog is merged into the main branch as well
Per discussion, these release-candidate updates should land in bothrelease-0.31andmainto keep the master view up to date.
8e7357d to
aa84c93
Compare
Add the changes accumulated since RC.1 to the changelog. Signed-off-by: Nick Volynkin <nick.volynkin@gmail.com>
aa84c93 to
013e193
Compare
There was a problem hiding this comment.
Caution
Inline review comments failed to post. This is likely due to GitHub's limits when posting large numbers of comments. If you are seeing this consistently it is likely a permissions issue. Please check "Moderation" -> "Code review limits" under your organization settings.
Actionable comments posted: 1
♻️ Duplicate comments (2)
docs/changelogs/v0.31.0.md (2)
68-68: Fix grammar and version notation in the MetalLB entry.The current line reads:
* MetalLB s now included directly as a patched image based on version 14.9. (@lllamnyp in https://github.com/cozystack/cozystack/pull/945)It should be:
-* MetalLB s now included directly as a patched image based on version 14.9. (@lllamnyp in https://github.com/cozystack/cozystack/pull/945) +* MetalLB is now included directly as a patched image based on version 0.14.9. (@lllamnyp in https://github.com/cozystack/cozystack/pull/945)🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
68-68: Bare URL used
null(MD034, no-bare-urls)
88-88: Prefix the GitHub username with@.Ensure the contributor’s username includes the
@symbol:-* gwynbleidd2106 made their first contribution in https://github.com/cozystack/cozystack/pull/962 +* @gwynbleidd2106 made their first contribution in https://github.com/cozystack/cozystack/pull/962🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
88-88: Bare URL used
null(MD034, no-bare-urls)
🧹 Nitpick comments (5)
docs/changelogs/v0.31.0.md (5)
6-29: Resolve list indentation and convert bare URLs to markdown links.Markdown linting reports incorrect indentation for nested lists (expected 2 spaces, actual 4) and numerous bare URLs in this section. Please align nested items with two spaces and convert links like
https://github.com/cozystack/cozystack/pull/834into[PR #834](https://github.com/cozystack/cozystack/pull/834).🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
8-8: Bare URL used
null(MD034, no-bare-urls)
10-10: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
10-10: Bare URL used
null(MD034, no-bare-urls)
11-11: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
11-11: Bare URL used
null(MD034, no-bare-urls)
12-12: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
12-12: Bare URL used
null(MD034, no-bare-urls)
14-14: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
14-14: Bare URL used
null(MD034, no-bare-urls)
15-15: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
15-15: Bare URL used
null(MD034, no-bare-urls)
16-16: Bare URL used
null(MD034, no-bare-urls)
17-17: Bare URL used
null(MD034, no-bare-urls)
18-18: Bare URL used
null(MD034, no-bare-urls)
19-19: Bare URL used
null(MD034, no-bare-urls)
20-20: Bare URL used
null(MD034, no-bare-urls)
21-21: Bare URL used
null(MD034, no-bare-urls)
22-22: Bare URL used
null(MD034, no-bare-urls)
24-24: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
24-24: Bare URL used
null(MD034, no-bare-urls)
25-25: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
25-25: Bare URL used
null(MD034, no-bare-urls)
26-26: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
26-26: Bare URL used
null(MD034, no-bare-urls)
26-26: Bare URL used
null(MD034, no-bare-urls)
26-26: Bare URL used
null(MD034, no-bare-urls)
27-27: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
27-27: Bare URL used
null(MD034, no-bare-urls)
28-28: Unordered list indentation
Expected: 2; Actual: 4(MD007, ul-indent)
28-28: Bare URL used
null(MD034, no-bare-urls)
28-28: Bare URL used
null(MD034, no-bare-urls)
30-64: Apply markdown best practices for the Fixes section.This block also contains bare URLs and inconsistent list indentation for nested items. Adjust to a two-space indent and wrap each URL in link text (e.g.,
[PR #833](https://github.com/cozystack/cozystack/pull/833)).🧰 Tools
🪛 LanguageTool
[uncategorized] ~47-~47: You might be missing the article “the” here.
Context: ...ystack/pull/956) * [monitoring] Remove legacy label "POD" from the exclude filter in ...(AI_EN_LECTOR_MISSING_DETERMINER_THE)
🪛 markdownlint-cli2 (0.17.2)
32-32: Bare URL used
null(MD034, no-bare-urls)
33-33: Bare URL used
null(MD034, no-bare-urls)
34-34: Bare URL used
null(MD034, no-bare-urls)
35-35: Bare URL used
null(MD034, no-bare-urls)
36-36: Bare URL used
null(MD034, no-bare-urls)
37-37: Bare URL used
null(MD034, no-bare-urls)
37-37: Bare URL used
null(MD034, no-bare-urls)
37-37: Bare URL used
null(MD034, no-bare-urls)
38-38: Bare URL used
null(MD034, no-bare-urls)
39-39: Bare URL used
null(MD034, no-bare-urls)
39-39: Bare URL used
null(MD034, no-bare-urls)
40-40: Bare URL used
null(MD034, no-bare-urls)
41-41: Bare URL used
null(MD034, no-bare-urls)
42-42: Bare URL used
null(MD034, no-bare-urls)
43-43: Bare URL used
null(MD034, no-bare-urls)
44-44: Bare URL used
null(MD034, no-bare-urls)
45-45: Bare URL used
null(MD034, no-bare-urls)
46-46: Bare URL used
null(MD034, no-bare-urls)
46-46: Bare URL used
null(MD034, no-bare-urls)
47-47: Bare URL used
null(MD034, no-bare-urls)
48-48: Bare URL used
null(MD034, no-bare-urls)
48-48: Bare URL used
null(MD034, no-bare-urls)
49-49: Bare URL used
null(MD034, no-bare-urls)
50-50: Bare URL used
null(MD034, no-bare-urls)
51-51: Bare URL used
null(MD034, no-bare-urls)
52-52: Bare URL used
null(MD034, no-bare-urls)
53-53: Bare URL used
null(MD034, no-bare-urls)
54-54: Bare URL used
null(MD034, no-bare-urls)
55-55: Bare URL used
null(MD034, no-bare-urls)
56-56: Bare URL used
null(MD034, no-bare-urls)
56-56: Bare URL used
null(MD034, no-bare-urls)
56-56: Bare URL used
null(MD034, no-bare-urls)
57-57: Bare URL used
null(MD034, no-bare-urls)
58-58: Bare URL used
null(MD034, no-bare-urls)
59-59: Bare URL used
null(MD034, no-bare-urls)
60-60: Bare URL used
null(MD034, no-bare-urls)
61-61: Bare URL used
null(MD034, no-bare-urls)
62-62: Bare URL used
null(MD034, no-bare-urls)
63-63: Bare URL used
null(MD034, no-bare-urls)
64-64: Bare URL used
null(MD034, no-bare-urls)
66-77: Standardize the Dependencies section formatting.The dependencies list mixes bare URLs and improper indentation. Align nested entries with two spaces and use link text for each URL (for example,
[v1.32.4](https://github.com/cozystack/cozystack/pull/949)).🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
68-68: Bare URL used
null(MD034, no-bare-urls)
69-69: Bare URL used
null(MD034, no-bare-urls)
70-70: Bare URL used
null(MD034, no-bare-urls)
71-71: Bare URL used
null(MD034, no-bare-urls)
72-72: Bare URL used
null(MD034, no-bare-urls)
73-73: Bare URL used
null(MD034, no-bare-urls)
73-73: Bare URL used
null(MD034, no-bare-urls)
74-74: Bare URL used
null(MD034, no-bare-urls)
75-75: Bare URL used
null(MD034, no-bare-urls)
75-75: Bare URL used
null(MD034, no-bare-urls)
76-76: Bare URL used
null(MD034, no-bare-urls)
76-76: Bare URL used
null(MD034, no-bare-urls)
77-77: Bare URL used
null(MD034, no-bare-urls)
79-89: Apply markdown best practices in Maintenance and New Contributors sections.This block uses bare URLs and inconsistent list indentation. Update nested items to a two-space indent and convert bare links (e.g.,
https://github.com/cozystack/cozystack/pull/872) into markdown links.🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
81-81: Bare URL used
null(MD034, no-bare-urls)
85-85: Bare URL used
null(MD034, no-bare-urls)
86-86: Bare URL used
null(MD034, no-bare-urls)
87-87: Bare URL used
null(MD034, no-bare-urls)
88-88: Bare URL used
null(MD034, no-bare-urls)
90-91: Convert the full changelog URL to markdown link syntax.Replace the bare URL with:
**Full Changelog**: [v0.30.0…v0.31.0-rc.2](https://github.com/cozystack/cozystack/compare/v0.30.0...v0.31.0-rc.2)🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
90-90: Bare URL used
null(MD034, no-bare-urls)
🛑 Comments failed to post (1)
docs/changelogs/v0.31.0.md (1)
1-5: 🛠️ Refactor suggestion
Consider renaming and relocating the changelog file.
The path
docs/changelogs/v0.31.0.mdis somewhat verbose and the top-leveldocsdirectory is too generic for release notes. A more conventional approach is to maintain a singleCHANGELOG.mdat the repository root or use a dedicatedchangelogs/folder with uppercase filenames (e.g.,CHANGELOG-0.31.0-RC.2.md) or anUNRELEASED.mdfor upcoming versions.🤖 Prompt for AI Agents
In docs/changelogs/v0.31.0.md lines 1 to 5, the changelog file is placed in a verbose and generic path. Rename the file to follow a conventional naming scheme such as CHANGELOG-0.31.0-RC.2.md and move it to a dedicated changelogs/ folder at the repository root, or alternatively consolidate all release notes into a single CHANGELOG.md at the root. This improves clarity and aligns with common project structures.
Summary by CodeRabbit
New Features
Bug Fixes
Dependency Updates
Documentation