[docs] Changelogs for the release series v0.34.x - #1192
Conversation
WalkthroughAdded and updated changelog files for Cozystack v0.34.0 and subsequent patches v0.34.1–v0.34.8 (plus template); entries record features, fixes, security notes, regressions/upgrade guidance, dependency updates, and documentation/CI changes. No code or public API declarations were changed. Changes
Sequence Diagram(s)(omitted — changes are documentation-only and do not modify control flow) Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Suggested labels
Suggested reviewers
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches🧪 Generate unit tests
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. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
6da364c to
b76edbf
Compare
b76edbf to
885ab0f
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (3)
docs/changelogs/v0.34.0.md (3)
22-22: Typo in contributor handle – drop the extra “l”.
@lllamnyphas three leading “l”s, whereas earlier entries use@llamnyp. Align them for consistency.-* [kubevirt-csi] Fix a regression by updating the role of the CSI controller. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1165) +* [kubevirt-csi] Fix a regression by updating the role of the CSI controller. (@llamnyp in https://github.com/cozystack/cozystack/pull/1165)
18-18: Tone polish – replace “Fixed” with a neutral verb.For formal release notes, “Resolves” or “Fixes” reads better than the past-tense “Fixed”.
-* [seaweedfs] Ensured JWT signing keys in the SeaweedFS security configuration remain consistent across Helm upgrades. Fixed an upstream issue. ... +* [seaweedfs] Ensures JWT signing keys in the SeaweedFS security configuration remain consistent across Helm upgrades and resolves an upstream issue. ...
6-15: Bare URLs trigger MD034 – consider Markdown link syntax or angle brackets.
markdownlintflags every raw URL (MD034). Two quick fixes:
- Wrap URLs in angle brackets (
<…>) – minimal disruption.- Prefer full Markdown links
[text](url)– improves readability.Example:
-* [seaweedfs] Support multi-zone configuration for S3 storage. (@kvaps in https://github.com/cozystack/cozystack/pull/1194) +* [seaweedfs] Support multi-zone configuration for S3 storage. (@kvaps in <https://github.com/cozystack/cozystack/pull/1194>)If you keep the current style, add an exception for MD034 in
.markdownlint.json.Also applies to: 18-37, 42-67, 71-71
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/changelogs/v0.34.0.md(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
🪛 LanguageTool
docs/changelogs/v0.34.0.md
[style] ~18-~18: Consider using a different verb for a more formal wording.
Context: ...remain consistent across Helm upgrades. Fixed an upstream issue. (Andrei Kvapil (@kvaps) in https://g...
(FIX_RESOLVE)
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
6-6: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
10-10: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
12-12: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
23-23: Bare URL used
(MD034, no-bare-urls)
24-24: Bare URL used
(MD034, no-bare-urls)
24-24: Bare URL used
(MD034, no-bare-urls)
25-25: Bare URL used
(MD034, no-bare-urls)
26-26: Bare URL used
(MD034, no-bare-urls)
27-27: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
29-29: Bare URL used
(MD034, no-bare-urls)
30-30: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
43-43: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
52-52: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
64-64: Bare URL used
(MD034, no-bare-urls)
65-65: Bare URL used
(MD034, no-bare-urls)
66-66: Bare URL used
(MD034, no-bare-urls)
67-67: Bare URL used
(MD034, no-bare-urls)
67-67: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
🔇 Additional comments (1)
docs/changelogs/v0.34.0.md (1)
6-6: Verify upper-bound Kubernetes version (1.33).Version 1.33 is not yet GA upstream. Confirm this is intentional (e.g., future-proofing) or adjust the stated range.
ee96053 to
213355b
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (1)
docs/changelogs/v0.34.0.md (1)
47-47: Fix misspelling:startegic-json-patch→strategic-json-patchRepeated typo from earlier changelogs.
-* [cozystack-api] Disable `startegic-json-patch` support. (@kvaps in https://github.com/cozystack/cozystack/pull/1179) +* [cozystack-api] Disable `strategic-json-patch` support. (@kvaps in https://github.com/cozystack/cozystack/pull/1179)
🧹 Nitpick comments (2)
docs/changelogs/v0.34.0.md (2)
20-20: Trim “able to” for a crisper sentence-Now VPA is able to manage itself through an ancillary VPA instance. +Now VPA can manage itself through an ancillary VPA instance.
37-37: Use present-tense “Fixes” for consistency with other bullets-* [seaweedfs] Ensured JWT signing keys in the SeaweedFS security configuration remain consistent across Helm upgrades. Fixed an upstream issue. (@kvaps in https://github.com/cozystack/cozystack/pull/1193 and https://github.com/seaweedfs/seaweedfs/pull/6967) +* [seaweedfs] Ensure JWT signing keys in the SeaweedFS security configuration remain consistent across Helm upgrades, fixing an upstream issue. (@kvaps in https://github.com/cozystack/cozystack/pull/1193 and https://github.com/seaweedfs/seaweedfs/pull/6967)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/changelogs/v0.34.0.md(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
docs/changelogs/v0.34.0.md (1)
Learnt from: NickVolynkin
PR: #1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
🪛 LanguageTool
docs/changelogs/v0.34.0.md
[style] ~20-~20: As a shorter alternative for ‘able to’, consider using “can”.
Context: ...pt itself — until this release. Now VPA is able to manage itself through an ancillary VPA ...
(BE_ABLE_TO)
[style] ~37-~37: Consider using a different verb for a more formal wording.
Context: ...remain consistent across Helm upgrades. Fixed an upstream issue. (Andrei Kvapil (@kvaps) in https://g...
(FIX_RESOLVE)
[grammar] ~47-~47: Ensure spelling is correct
Context: ...ck/pull/1171) * [cozystack-api] Disable startegic-json-patch support. (Andrei Kvapil (@kvaps) in https:/...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
25-25: Bare URL used
(MD034, no-bare-urls)
26-26: Bare URL used
(MD034, no-bare-urls)
27-27: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
29-29: Bare URL used
(MD034, no-bare-urls)
30-30: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
43-43: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
52-52: Bare URL used
(MD034, no-bare-urls)
53-53: Bare URL used
(MD034, no-bare-urls)
54-54: Bare URL used
(MD034, no-bare-urls)
55-55: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
65-65: Bare URL used
(MD034, no-bare-urls)
66-66: Bare URL used
(MD034, no-bare-urls)
67-67: Bare URL used
(MD034, no-bare-urls)
68-68: Bare URL used
(MD034, no-bare-urls)
69-69: Bare URL used
(MD034, no-bare-urls)
70-70: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
80-80: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
82-82: Bare URL used
(MD034, no-bare-urls)
83-83: Bare URL used
(MD034, no-bare-urls)
84-84: Bare URL used
(MD034, no-bare-urls)
85-85: Bare URL used
(MD034, no-bare-urls)
86-86: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
88-88: Bare URL used
(MD034, no-bare-urls)
89-89: Bare URL used
(MD034, no-bare-urls)
90-90: Bare URL used
(MD034, no-bare-urls)
90-90: Bare URL used
(MD034, no-bare-urls)
94-94: Bare URL used
(MD034, no-bare-urls)
213355b to
7501acb
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
docs/changelogs/v0.34.0.md (1)
27-29: Correct the@lllamnyphandle (triple-L) everywhereThe author handle is spelled with two leading
l’s (@llamnyp).
Broken mentions hamper attribution and link rendering.-(@lllamnyp +(@llamnypApply the replacement on lines 27, 29, 44, 46 and 97.
Also applies to: 44-46, 97-97
🧹 Nitpick comments (3)
docs/changelogs/v0.34.0.md (3)
60-60: Remove duplicate “and” in the tenant bullet-(@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) +(@klinch0 and @kvaps in https://github.com/cozystack/cozystack/pull/1162)
54-54: Use plural “READMEs” instead of possessive-Reorder configuration values in application README's for better readability. +Reorder configuration values in application READMEs for better readability.
23-23: Terminate the sentence with a period-Now VPA can manage itself through an ancillary VPA instance +Now VPA can manage itself through an ancillary VPA instance.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/changelogs/v0.34.0.md(1 hunks)
🧠 Learnings (2)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
docs/changelogs/v0.34.0.md (2)
Learnt from: NickVolynkin
PR: #1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
Learnt from: NickVolynkin
PR: #1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
27-27: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
29-29: Bare URL used
(MD034, no-bare-urls)
30-30: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
52-52: Bare URL used
(MD034, no-bare-urls)
53-53: Bare URL used
(MD034, no-bare-urls)
54-54: Bare URL used
(MD034, no-bare-urls)
55-55: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
68-68: Bare URL used
(MD034, no-bare-urls)
69-69: Bare URL used
(MD034, no-bare-urls)
70-70: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
83-83: Bare URL used
(MD034, no-bare-urls)
84-84: Bare URL used
(MD034, no-bare-urls)
85-85: Bare URL used
(MD034, no-bare-urls)
86-86: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
88-88: Bare URL used
(MD034, no-bare-urls)
89-89: Bare URL used
(MD034, no-bare-urls)
90-90: Bare URL used
(MD034, no-bare-urls)
91-91: Bare URL used
(MD034, no-bare-urls)
92-92: Bare URL used
(MD034, no-bare-urls)
93-93: Bare URL used
(MD034, no-bare-urls)
93-93: Bare URL used
(MD034, no-bare-urls)
97-97: Bare URL used
(MD034, no-bare-urls)
99-99: Bare URL used
(MD034, no-bare-urls)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
docs/changelogs/v0.34.0.md (2)
Learnt from: NickVolynkin
PR: #1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
Learnt from: NickVolynkin
PR: #1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
27-27: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
29-29: Bare URL used
(MD034, no-bare-urls)
30-30: Bare URL used
(MD034, no-bare-urls)
31-31: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
52-52: Bare URL used
(MD034, no-bare-urls)
53-53: Bare URL used
(MD034, no-bare-urls)
54-54: Bare URL used
(MD034, no-bare-urls)
55-55: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
68-68: Bare URL used
(MD034, no-bare-urls)
69-69: Bare URL used
(MD034, no-bare-urls)
70-70: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
83-83: Bare URL used
(MD034, no-bare-urls)
84-84: Bare URL used
(MD034, no-bare-urls)
85-85: Bare URL used
(MD034, no-bare-urls)
86-86: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
88-88: Bare URL used
(MD034, no-bare-urls)
89-89: Bare URL used
(MD034, no-bare-urls)
90-90: Bare URL used
(MD034, no-bare-urls)
91-91: Bare URL used
(MD034, no-bare-urls)
92-92: Bare URL used
(MD034, no-bare-urls)
93-93: Bare URL used
(MD034, no-bare-urls)
93-93: Bare URL used
(MD034, no-bare-urls)
97-97: Bare URL used
(MD034, no-bare-urls)
99-99: Bare URL used
(MD034, no-bare-urls)
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
docs/changelogs/v0.34.2.md (1)
3-5: Replace bare URLs with Markdown links for lint-compliance and consistency
markdownlintflags MD034 on these lines. Converting the raw URLs to link syntax keeps the entry readable and satisfies the linter without changing content.-* [objectstorage] Fix recording image in objectstorage. (@kvaps in https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637) +* [objectstorage] Fix recording image in objectstorage. (@kvaps in [4d9a838](https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637)) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2 +**Full Changelog**: [v0.34.1…v0.34.2](https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2)docs/changelogs/v0.34.3.md (1)
3-5: Convert raw URLs to links to silence MD034Same lint rule and style concern as the previous file; recommend link syntax.
-* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in https://github.com/cozystack/cozystack/pull/1232, backported in https://github.com/cozystack/cozystack/pull/1272) +* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in [#1232](https://github.com/cozystack/cozystack/pull/1232), backported in [#1272](https://github.com/cozystack/cozystack/pull/1272)) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3 +**Full Changelog**: [v0.34.2…v0.34.3](https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3)docs/changelogs/v0.34.1.md (1)
3-6: Use link syntax for pull-request / commit references and full-changelog URLApplying the same tweak here keeps the whole v0.34.x series consistent and linter-clean.
-* [kubernetes] Fix regression in `volumesnapshotclass` installation from https://github.com/cozystack/cozystack/pull/1203. (@kvaps in https://github.com/cozystack/cozystack/pull/1238) -* [objectstorage] Fix building objectstorage images. (@kvaps in https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923) +* [kubernetes] Fix regression in `volumesnapshotclass` installation from [#1203](https://github.com/cozystack/cozystack/pull/1203). (@kvaps in [#1238](https://github.com/cozystack/cozystack/pull/1238)) +* [objectstorage] Fix building objectstorage images. (@kvaps in [a9e9dfc](https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923)) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1 +**Full Changelog**: [v0.34.0…v0.34.1](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.1.md
3-3: Bare URL used
(MD034, no-bare-urls)
3-3: Bare URL used
(MD034, no-bare-urls)
4-4: Bare URL used
(MD034, no-bare-urls)
6-6: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.2.md
3-3: Bare URL used
(MD034, no-bare-urls)
5-5: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.3.md
3-3: Bare URL used
(MD034, no-bare-urls)
5-5: Bare URL used
(MD034, no-bare-urls)
6c660c0 to
e804f40
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
docs/changelogs/v0.34.3.md (1)
3-3: Convert bare URLs to proper Markdown links to satisfy MD034 and improve readabilityThe changelog currently contains bare URLs, which trigger markdownlint violations and look less polished. Wrapping them in Markdown link syntax (or angle brackets) resolves the issue and keeps the style consistent with other changelogs.
-* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in https://github.com/cozystack/cozystack/pull/1232, backported in https://github.com/cozystack/cozystack/pull/1272) +* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in [#1232](https://github.com/cozystack/cozystack/pull/1232), backported in [#1272](https://github.com/cozystack/cozystack/pull/1272)) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3>Also applies to: 5-5
docs/changelogs/v0.34.2.md (2)
4-5: Addhttps://to release links for correctnessThe links lack a scheme, so they render as relative URLs. Prefix them with
https://to ensure they work everywhere.-> A patch release [0.34.3](github.com/cozystack/cozystack/releases/tag/v0.34.3) fixing a regression in 0.34.0 has been released. -> It is recommended to skip this version and upgrade directly to [0.34.3](github.com/cozystack/cozystack/releases/tag/v0.34.3). +> A patch release [0.34.3](https://github.com/cozystack/cozystack/releases/tag/v0.34.3) fixing a regression in 0.34.0 has been released. +> It is recommended to skip this version and upgrade directly to [0.34.3](https://github.com/cozystack/cozystack/releases/tag/v0.34.3).
7-7: Wrap bare URLs to comply with MD034Like the other changelogs, wrap commit and compare links to avoid bare-URL lint hits.
-* [objectstorage] Fix recording image in objectstorage. (@kvaps in https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637) +* [objectstorage] Fix recording image in objectstorage. (@kvaps in <https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637>) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2>Also applies to: 9-9
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- docs/changelogs/template.md
🚧 Files skipped from review as they are similar to previous changes (2)
- docs/changelogs/v0.34.1.md
- docs/changelogs/v0.34.0.md
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.2.md
7-7: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.3.md
3-3: Bare URL used
(MD034, no-bare-urls)
3-3: Bare URL used
(MD034, no-bare-urls)
5-5: Bare URL used
(MD034, no-bare-urls)
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (2)
docs/changelogs/v0.34.0.md (1)
8-11: Prefix release links withhttps://The two links to
0.34.3lack the URL scheme and render as plain text rather than hyperlinks.-> [!WARNING] -> A patch release [0.34.3](github.com/cozystack/cozystack/releases/tag/v0.34.3) fixing a regression in 0.34.0 has been released. -> It is recommended to skip this version and upgrade directly to [0.34.3](github.com/cozystack/cozystack/releases/tag/v0.34.3). +> [!WARNING] +> A patch release [0.34.3](https://github.com/cozystack/cozystack/releases/tag/v0.34.3) fixing a regression in 0.34.0 has been released. +> It is recommended to skip this version and upgrade directly to [0.34.3](https://github.com/cozystack/cozystack/releases/tag/v0.34.3).docs/changelogs/v0.34.1.md (1)
7-10: Prefix release links withhttps://The admonition’s links are missing the URL scheme.
-> [!WARNING] -> A patch release [0.34.3](github.com/cozystack/cozystack/releases/tag/v0.34.3) fixing a regression in 0.34.0 has been released. -> It is recommended to skip this version and upgrade directly to [0.34.3](github.com/cozystack/cozystack/releases/tag/v0.34.3). +> [!WARNING] +> A patch release [0.34.3](https://github.com/cozystack/cozystack/releases/tag/v0.34.3) fixing a regression in 0.34.0 has been released. +> It is recommended to skip this version and upgrade directly to [0.34.3](https://github.com/cozystack/cozystack/releases/tag/v0.34.3).
🧹 Nitpick comments (2)
docs/changelogs/v0.34.0.md (2)
48-48: Remove duplicate “and” in author attribution-* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) +* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps in https://github.com/cozystack/cozystack/pull/1162)
50-50: Add missing “in” before PR link-* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 https://github.com/cozystack/cozystack/pull/1204) +* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 in https://github.com/cozystack/cozystack/pull/1204)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)
✅ Files skipped from review due to trivial changes (2)
- docs/changelogs/v0.34.3.md
- docs/changelogs/v0.34.2.md
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/changelogs/template.md
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
docs/changelogs/v0.34.0.md (2)
Learnt from: NickVolynkin
PR: #1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
Learnt from: NickVolynkin
PR: #1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
15-15: Bare URL used
(MD034, no-bare-urls)
16-16: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
20-20: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
23-23: Bare URL used
(MD034, no-bare-urls)
24-24: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
38-38: Bare URL used
(MD034, no-bare-urls)
39-39: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
41-41: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
43-43: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
64-64: Bare URL used
(MD034, no-bare-urls)
65-65: Bare URL used
(MD034, no-bare-urls)
66-66: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
79-79: Bare URL used
(MD034, no-bare-urls)
80-80: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
85-85: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.1.md
11-11: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
e804f40 to
1c23e04
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (9)
docs/changelogs/v0.34.0.md (5)
8-10: Add missinghttps://scheme to avoid a broken linkThe target URL in the warning block is rendered as a relative path.
Use an absolute URL so users can click it directly.-> A regression was found in this release and fixed in a subsequent [0.34.1](github.com/cozystack/cozystack/releases/tag/v0.34.1). +> A regression was found in this release and fixed in a subsequent [0.34.1](https://github.com/cozystack/cozystack/releases/tag/v0.34.1).
15-15: Correct the author handle typo (lllamnyp→llamnyp)The triple-
lhandle breaks the attribution link and was fixed elsewhere in the same file series.-* [kubernetes] … (@lllamnyp and @IvanHunters in … +* [kubernetes] … (@llamnyp and @IvanHunters in …
24-24: Minor grammar: add missing preposition “of”-* [dashboard] … Fix handling multiline strings. … +* [dashboard] … Fix handling of multiline strings. …
48-48: Remove duplicated conjunction in bullet-* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) +* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps in https://github.com/cozystack/cozystack/pull/1162)
50-50: Insert missing “in” before the pull-request link-* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 https://github.com/cozystack/cozystack/pull/1204) +* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 in https://github.com/cozystack/cozystack/pull/1204)docs/changelogs/v0.34.2.md (2)
1-1: Remove stray “34” at the start of the fileLooks like a leftover line-number that ended up in the markdown.
-34<!-- +<!--
7-7: Tiny wording tweak: “objectstorage” → “object storage”-* [objectstorage] Fix recording image in objectstorage. (@kvaps … +* [objectstorage] Fix recording image in object storage. (@kvaps …docs/changelogs/v0.34.1.md (1)
7-8: Optional: keep terminology consistent (“objectstorage” vs “object storage”)Earlier changelogs sometimes use a space. Consider aligning for searchability, but feel free to ignore if “objectstorage” is the formal component name.
docs/changelogs/v0.34.3.md (1)
7-7: Optional consistency: “objectstorage” vs “object storage”Same note as in 0.34.1/0.34.2—align wording if there isn’t a strong reason to keep it concatenated.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/changelogs/template.md
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
docs/changelogs/v0.34.0.md (2)
Learnt from: NickVolynkin
PR: #1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
Learnt from: NickVolynkin
PR: #1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
10-10: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.1.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
10-10: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.2.md
7-7: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.3.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
1c23e04 to
005cb08
Compare
005cb08 to
272a9fb
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
docs/changelogs/v0.34.1.md (1)
12-15: Convert bare URLs to Markdown links for lint-clean output
markdownlintflags the bare URLs here (MD034). Converting them to standard Markdown links improves readability and lets the file pass CI linting.-* [kubernetes] Fix regression in `volumesnapshotclass` installation from https://github.com/cozystack/cozystack/pull/1203. (@kvaps in https://github.com/cozystack/cozystack/pull/1238) -* [objectstorage] Fix building objectstorage images. (@kvaps in https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923) +* [kubernetes] Fix regression in `volumesnapshotclass` installation from [#1203](https://github.com/cozystack/cozystack/pull/1203). (@kvaps in [#1238](https://github.com/cozystack/cozystack/pull/1238)) +* [objectstorage] Fix building objectstorage images. (@kvaps in [commit a9e9dfc](https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923))A similar tweak for the “Full Changelog” line keeps the section consistent.
docs/changelogs/v0.34.0.md (2)
15-15: Misspelled author handle@lllamnyp→@llamnypThe handle appears with three consecutive “l” characters on these lines; the correct spelling uses two. Fixing it keeps attribution accurate and prevents broken profile links.
-(@lllamnyp +(@llamnypApply to all occurrences in this file.
Also applies to: 32-32, 85-85
48-48: Remove extraneous word “and” in parentheticalMinor grammar polish—double “and” before the PR link.
-(@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) +(@klinch0 and @kvaps in https://github.com/cozystack/cozystack/pull/1162)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (3)
- docs/changelogs/v0.34.3.md
- docs/changelogs/v0.34.2.md
- docs/changelogs/template.md
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
📚 Learning: the cozystack project uses yq v4+ on their ci runner, so yq v4 syntax (-o json --indent 4) is compat...
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
Applied to files:
docs/changelogs/v0.34.0.md
📚 Learning: in the cozystack repository, the maintainer nickvolynkin prefers to keep realistic-looking example c...
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Applied to files:
docs/changelogs/v0.34.0.md
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
12-12: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.1.md
12-12: Bare URL used
(MD034, no-bare-urls)
12-12: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
15-15: Bare URL used
(MD034, no-bare-urls)
272a9fb to
1391a5f
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (6)
docs/changelogs/v0.34.2.md (2)
12-12: Convert bare commit URL into a Markdown linkExplicit URLs trigger MD034 warnings and look noisy. Suggest wrapping the SHA in link syntax:
-* [objectstorage] Fix recording image in objectstorage. (@kvaps in https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637) +* [objectstorage] Fix recording image in objectstorage. (@kvaps in [4d9a8389](https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637))
14-14: Replace bare “Full Changelog” URL with a labelled linkKeeps the document tidy and silences MD034:
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2 +**Full Changelog**: [`v0.34.1...v0.34.2`](https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2)docs/changelogs/v0.34.1.md (2)
4-4: Add an explicit H1 version header for consistencyOther changelog files usually begin with a top-level heading that contains the version and release date. Adding it here keeps the navigation uniform and helps readers immediately identify the release.
+ +# v0.34.1 – 2025-08-?? <!-- adjust date as needed --> +
15-15: Replace bare URL with Markdown link to satisfy MD034markdownlint flags the naked URL; wrapping it in link syntax improves readability and passes automated checks.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1 +[**Full Changelog**](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1)docs/changelogs/v0.34.0.md (2)
42-42: Typo: apostrophe in plural “READMEs”.Use the simple plural rather than the possessive.
-* [applications] Reorder configuration values in application README's for better readability. (@NickVolynkin in https://github.com/cozystack/cozystack/pull/1214) +* [applications] Reorder configuration values in application READMEs for better readability. (@NickVolynkin in https://github.com/cozystack/cozystack/pull/1214)
48-48: Redundant “and” in author list.Remove the extra conjunction for clarity.
-* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) +* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps in https://github.com/cozystack/cozystack/pull/1162)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)docs/changelogs/v0.34.4.md(1 hunks)docs/changelogs/v0.34.5.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- docs/changelogs/template.md
- docs/changelogs/v0.34.3.md
🧰 Additional context used
🧠 Learnings (4)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
📚 Learning: the cozystack project uses yq v4+ on their ci runner, so yq v4 syntax (-o json --indent 4) is compat...
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
Applied to files:
docs/changelogs/v0.34.0.mddocs/changelogs/v0.34.5.md
📚 Learning: in the cozystack repository, the maintainer nickvolynkin prefers to keep realistic-looking example c...
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Applied to files:
docs/changelogs/v0.34.0.mddocs/changelogs/v0.34.4.md
📚 Learning: in the cozystack repository, for the virtual-machine app's resources.sockets parameter, the value is...
Learnt from: NickVolynkin
PR: cozystack/cozystack#1216
File: packages/apps/virtual-machine/values.yaml:0-0
Timestamp: 2025-07-26T18:12:05.641Z
Learning: In the cozystack repository, for the virtual-machine app's resources.sockets parameter, the value is intentionally kept as a string in values.yaml despite being documented as {int} type, because the string-to-integer parsing happens elsewhere in the system pipeline rather than at the YAML validation level.
Applied to files:
docs/changelogs/v0.34.5.md
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.1.md
15-15: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.2.md
12-12: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.4.md
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.5.md
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
10-10: Bare URL used
(MD034, no-bare-urls)
🔇 Additional comments (3)
docs/changelogs/v0.34.2.md (1)
5-7: Verify admonition style renders on GitHub
> [!WARNING]is a Markdown extension that isn’t natively rendered on GitHub web UI. Unless a Docs-specific renderer is in use, the block will appear as plain text.
Ensure other changelog files use the same pattern or consider a fallback such as:> **Warning** > …docs/changelogs/v0.34.4.md (1)
1-21: Changelog looks good – no issues found.Content, links, and grammar read well and follow the project’s established style.
Nothing to change here.docs/changelogs/v0.34.5.md (1)
1-10: LGTM – entry is clear and accurate.No wording or link issues detected.
1391a5f to
4c5ddba
Compare
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (4)
docs/changelogs/v0.34.0.md (1)
48-48: Remove redundant conjunction in authors list
and @kvaps andcontains an extra “and”.- (@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) + (@klinch0 and @kvaps in https://github.com/cozystack/cozystack/pull/1162)docs/changelogs/v0.34.3.md (1)
10-10: Convert bare commit hash to a clickable linkThe KubeVirt CCM fix currently shows a raw hash, breaking readability.
-* [kubevirt] Fix building Kubevirt CCM (@kvaps in 3c7e256906e1dbb0f957dc3a205fa77a147d419d) +* [kubevirt] Fix building Kubevirt CCM (@kvaps in https://github.com/cozystack/cozystack/commit/3c7e256906e1dbb0f957dc3a205fa77a147d419d)docs/changelogs/v0.34.1.md (1)
12-15: Convert bare URLs to Markdown links for lint-clean docs
markdownlintraises MD034 on these lines. Switching to inline link syntax improves readability and keeps the repo’s docs lint-clean and stylistically consistent with earlier changelog entries.-* [kubernetes] Fix regression in `volumesnapshotclass` installation from https://github.com/cozystack/cozystack/pull/1203. (@kvaps in https://github.com/cozystack/cozystack/pull/1238) -* [objectstorage] Fix building objectstorage images. (@kvaps in https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1 +* [kubernetes] Fix regression in `volumesnapshotclass` installation from [#1203](https://github.com/cozystack/cozystack/pull/1203). (@kvaps in [#1238](https://github.com/cozystack/cozystack/pull/1238)) +* [objectstorage] Fix building object-storage images. (@kvaps in [commit a9e9dfc](https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923)) + +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1>docs/changelogs/v0.34.4.md (1)
7-8: Replace raw URLs with inline links to resolve MD034 and unify styleThe same bare-URL pattern crops up throughout this file. Converting them to Markdown links satisfies
markdownlintand keeps the changelog style uniform.-* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1286) -* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in https://github.com/cozystack/cozystack/pull/1287, backported in https://github.com/cozystack/cozystack/pull/1291) +* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in [#1286](https://github.com/cozystack/cozystack/pull/1286)) +* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP and enable `cookie-secure`. (@klinch0 in [#1287](https://github.com/cozystack/cozystack/pull/1287), backported in [#1291](https://github.com/cozystack/cozystack/pull/1291)) @@ -* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in https://github.com/cozystack/cozystack/pull/1284) -* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in https://github.com/cozystack/cozystack/pull/1290, backported in https://github.com/cozystack/cozystack/pull/1293) +* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the volume-snapshot CRD name from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. After upgrading Cozystack, update each affected tenant Kubernetes cluster. (@klinch0 in [#1284](https://github.com/cozystack/cozystack/pull/1284)) +* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in [#1290](https://github.com/cozystack/cozystack/pull/1290), backported in [#1293](https://github.com/cozystack/cozystack/pull/1293)) @@ -* [tests] Add tests for S3 buckets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1283, backported in https://github.com/cozystack/cozystack/pull/1292) +* [tests] Add tests for S3 buckets. (@IvanHunters in [#1283](https://github.com/cozystack/cozystack/pull/1283), backported in [#1292](https://github.com/cozystack/cozystack/pull/1292)) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4>Also applies to: 13-14, 19-21
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)docs/changelogs/v0.34.4.md(1 hunks)docs/changelogs/v0.34.5.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- docs/changelogs/v0.34.2.md
- docs/changelogs/template.md
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
📚 Learning: the cozystack project uses yq v4+ on their ci runner, so yq v4 syntax (-o json --indent 4) is compat...
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
Applied to files:
docs/changelogs/v0.34.0.md
📚 Learning: in the cozystack repository, the maintainer nickvolynkin prefers to keep realistic-looking example c...
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Applied to files:
docs/changelogs/v0.34.0.mddocs/changelogs/v0.34.4.md
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
15-15: Bare URL used
(MD034, no-bare-urls)
16-16: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
20-20: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
23-23: Bare URL used
(MD034, no-bare-urls)
24-24: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
38-38: Bare URL used
(MD034, no-bare-urls)
39-39: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
41-41: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
43-43: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
64-64: Bare URL used
(MD034, no-bare-urls)
65-65: Bare URL used
(MD034, no-bare-urls)
66-66: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
79-79: Bare URL used
(MD034, no-bare-urls)
80-80: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
85-85: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.1.md
12-12: Bare URL used
(MD034, no-bare-urls)
12-12: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
15-15: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.3.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.4.md
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.5.md
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
🪛 LanguageTool
docs/changelogs/v0.34.5.md
[style] ~9-~9: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (Timofei Larkin (@lllamnyp) in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
4c5ddba to
575a5ec
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
docs/changelogs/v0.34.5.md (1)
8-9: Author-handle typo and “VPA for VPA” phrasing still present
Same feedback was given earlier and remains unresolved.
🧹 Nitpick comments (5)
docs/changelogs/v0.34.2.md (1)
12-13: Tighten wording & avoid repetition in the fix descriptionThe current phrasing is a bit clunky — “recording image in objectstorage”. Consider a more idiomatic formulation and drop the repeated “objectstorage”.
-* [objectstorage] Fix recording image in objectstorage. (@kvaps in https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637) +* [objectstorage] Fix image recording in the object-storage component. (@kvaps in https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637)docs/changelogs/v0.34.0.md (3)
48-48: Remove duplicated conjunction in contributor lineThere's an extra “and” before the PR link.
-* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) +* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps in https://github.com/cozystack/cozystack/pull/1162)
50-50: Add missing “in” before the PR link-* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 https://github.com/cozystack/cozystack/pull/1204) +* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 in https://github.com/cozystack/cozystack/pull/1204)
85-85: Capitalise “CI” for consistency with common terminology-* [ci] Improve workflow for contributors submitting PRs from forks. Use Oracle Cloud Infrastructure Registry for non-release PRs, bypassing restrictions preventing pushing to ghcr.io with default GitHub token. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1226) +* [CI] Improve workflow for contributors submitting PRs from forks. Use Oracle Cloud Infrastructure Registry for non-release PRs, bypassing restrictions preventing pushing to ghcr.io with the default GitHub token. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1226)docs/changelogs/v0.34.1.md (1)
12-13: Minor wording polish for the fix entries
- Capitalise the CRD kind and wrap it in back-ticks for readability.
- “objectstorage images” → “object-storage images” for consistency.
-* [kubernetes] Fix regression in `volumesnapshotclass` installation from https://github.com/cozystack/cozystack/pull/1203. (@kvaps in https://github.com/cozystack/cozystack/pull/1238) +* [kubernetes] Fix regression in installing `VolumeSnapshotClass` CRDs introduced by https://github.com/cozystack/cozystack/pull/1203. (@kvaps in https://github.com/cozystack/cozystack/pull/1238) -* [objectstorage] Fix building objectstorage images. (@kvaps in https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923) +* [objectstorage] Fix building object-storage images. (@kvaps in https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)docs/changelogs/v0.34.4.md(1 hunks)docs/changelogs/v0.34.5.md(1 hunks)docs/changelogs/v0.34.6.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- docs/changelogs/template.md
- docs/changelogs/v0.34.3.md
🧰 Additional context used
🧠 Learnings (5)
📓 Common learnings
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
📚 Learning: the cozystack project uses yq v4+ on their ci runner, so yq v4 syntax (-o json --indent 4) is compat...
Learnt from: NickVolynkin
PR: cozystack/cozystack#1117
File: packages/apps/mysql/Makefile:8-8
Timestamp: 2025-06-26T04:29:24.830Z
Learning: The cozystack project uses yq v4+ on their CI runner, so yq v4 syntax (-o json --indent 4) is compatible and version checks are not needed.
Applied to files:
docs/changelogs/v0.34.0.md
📚 Learning: the github username Timofei Larkin (@lllamnyp) is spelled with three 'l's, not two. this is the correct spelling for ...
Learnt from: NickVolynkin
PR: cozystack/cozystack#1192
File: docs/changelogs/v0.34.0.md:15-17
Timestamp: 2025-08-05T07:17:29.755Z
Learning: The GitHub username @lllamnyp is spelled with three 'l's, not two. This is the correct spelling for attribution in changelogs and documentation.
Applied to files:
docs/changelogs/v0.34.0.md
📚 Learning: in the cozystack repository, the maintainer nickvolynkin prefers to keep realistic-looking example c...
Learnt from: NickVolynkin
PR: cozystack/cozystack#1120
File: packages/apps/ferretdb/README.md:35-37
Timestamp: 2025-07-02T09:58:11.406Z
Learning: In the cozystack repository, the maintainer NickVolynkin prefers to keep realistic-looking example credentials in README documentation rather than using generic placeholders like <ACCESS_KEY>, even though they are just examples and not real secrets.
Applied to files:
docs/changelogs/v0.34.0.mddocs/changelogs/v0.34.4.md
📚 Learning: in cozystack, the plural form for the kubernetes custom resource is `kuberneteses.apps.cozystack.io`...
Learnt from: lllamnyp
PR: cozystack/cozystack#1130
File: hack/e2e-apps/kubernetes.bats:101-101
Timestamp: 2025-07-11T06:11:25.438Z
Learning: In cozystack, the plural form for the Kubernetes custom resource is `kuberneteses.apps.cozystack.io`, not `kubernetes.apps.cozystack.io`. This is defined in the API schema even though it's not grammatically perfect.
Applied to files:
docs/changelogs/v0.34.6.md
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
15-15: Bare URL used
(MD034, no-bare-urls)
16-16: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
20-20: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
23-23: Bare URL used
(MD034, no-bare-urls)
24-24: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
38-38: Bare URL used
(MD034, no-bare-urls)
39-39: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
41-41: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
43-43: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
64-64: Bare URL used
(MD034, no-bare-urls)
65-65: Bare URL used
(MD034, no-bare-urls)
66-66: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
79-79: Bare URL used
(MD034, no-bare-urls)
80-80: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
85-85: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.1.md
12-12: Bare URL used
(MD034, no-bare-urls)
12-12: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
15-15: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.2.md
12-12: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.4.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.5.md
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.6.md
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
🪛 LanguageTool
docs/changelogs/v0.34.5.md
[style] ~9-~9: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (Timofei Larkin (@lllamnyp) in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
🔇 Additional comments (2)
docs/changelogs/v0.34.6.md (1)
5-9: Changelog entry LGTMClear, concise, and follows the existing changelog style. No issues spotted.
docs/changelogs/v0.34.4.md (1)
1-21: No additional issues found beyond previously-flagged bare-URL lint warnings, which are acceptable in this repo’s changelog style.
3aa1d36 to
1508fe0
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
docs/changelogs/v0.34.5.md (1)
9-9: Clarify phrasing: “Disable VPA for VPA” reads awkwardly.Improve readability; keep correct author handle spelling (Timofei Larkin (@lllamnyp)).
-* [kubernetes] Disable VPA for VPA in tenant Kubernetes clusters. Tenant clusters have no need for this feature, and it was not designed to work in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1301, backported in https://github.com/cozystack/cozystack/pull/1305) +* [kubernetes] Disable the Vertical Pod Autoscaler in tenant Kubernetes clusters; it is not intended for tenant workloads and was enabled unintentionally. (@lllamnyp in <https://github.com/cozystack/cozystack/pull/1301>, backported in <https://github.com/cozystack/cozystack/pull/1305>)
🧹 Nitpick comments (14)
docs/changelogs/v0.34.6.md (2)
7-9: Wrap bare URLs to satisfy MD034 (no-bare-urls).Consistently wrap URLs with angle brackets or convert them to Markdown links.
-* [dashboard] Fix filling multiline values in the visual editor. (@kvaps in https://github.com/cozystack/cozystack/commit/56fca9bd75efeca25f9483f6c514b6fec26d5d22 and https://github.com/cozystack/kubeapps/commit/4926bc68fabb0914afab574006643c85a597b371) +* [dashboard] Fix filling multiline values in the visual editor. (@kvaps in <https://github.com/cozystack/cozystack/commit/56fca9bd75efeca25f9483f6c514b6fec26d5d22> and <https://github.com/cozystack/kubeapps/commit/4926bc68fabb0914afab574006643c85a597b371>) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.5...v0.34.6 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.5...v0.34.6>
9-9: Add trailing newline at EOF.Some linters/tools prefer files to end with a newline.
-**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.5...v0.34.6> +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.5...v0.34.6> +docs/changelogs/v0.34.2.md (2)
12-12: Polish phrasing for clarity.“Fix recording image in objectstorage” reads awkwardly. Consider “image recording.”
-* [objectstorage] Fix recording image in objectstorage. (@kvaps in https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637) +* [objectstorage] Fix image recording in objectstorage. (@kvaps in <https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637>)
14-14: Wrap bare URL to satisfy MD034 (no-bare-urls).-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2>docs/changelogs/v0.34.5.md (1)
8-11: Wrap bare URLs to satisfy MD034 (no-bare-urls).Apply consistently to this file.
-* [virtual-machine] Enable using custom `instanceType` values in `virtual-machine` and `vm-instance` by disabling field validation. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1300, backported in https://github.com/cozystack/cozystack/pull/1303) +* [virtual-machine] Enable using custom `instanceType` values in `virtual-machine` and `vm-instance` by disabling field validation. (@lllamnyp in <https://github.com/cozystack/cozystack/pull/1300>, backported in <https://github.com/cozystack/cozystack/pull/1303>) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.4...v0.34.5 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.4...v0.34.5>docs/changelogs/v0.34.3.md (2)
10-10: Link the commit hash and fix product capitalization.Make the commit clickable and use standard “KubeVirt” capitalization.
-* [kubevirt] Fix building Kubevirt CCM (@kvaps in 3c7e256906e1dbb0f957dc3a205fa77a147d419d) +* [kubevirt] Fix building KubeVirt CCM (@kvaps in <https://github.com/cozystack/cozystack/commit/3c7e256906e1dbb0f957dc3a205fa77a147d419d>)
7-9: Wrap bare URLs to satisfy MD034 (no-bare-urls).-* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in https://github.com/cozystack/cozystack/pull/1232, backported in https://github.com/cozystack/cozystack/pull/1272) +* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in <https://github.com/cozystack/cozystack/pull/1232>, backported in <https://github.com/cozystack/cozystack/pull/1272>) -* [kubernetes] Add dependency for snapshot CRD and migration to latest version. (@kvaps in https://github.com/cozystack/cozystack/pull/1275, backported in https://github.com/cozystack/cozystack/pull/1279) +* [kubernetes] Add dependency for snapshot CRD and migration to latest version. (@kvaps in <https://github.com/cozystack/cozystack/pull/1275>, backported in <https://github.com/cozystack/cozystack/pull/1279>) -* [seaweedfs] Add support for whitelisting and exporting via nginx-ingress. Update cosi-driver. (@kvaps in https://github.com/cozystack/cozystack/pull/1277) +* [seaweedfs] Add support for whitelisting and exporting via nginx-ingress. Update cosi-driver. (@kvaps in <https://github.com/cozystack/cozystack/pull/1277>) -* [virtual-machine] Fix a regression with field `optional=true`. (@kvaps in https://github.com/cozystack/cozystack/commit/01053f7c3180d1bd045d7c5fb949984c2bdaf19d) +* [virtual-machine] Fix a regression with field `optional=true`. (@kvaps in <https://github.com/cozystack/cozystack/commit/01053f7c3180d1bd045d7c5fb949984c2bdaf19d>) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3>Also applies to: 11-11, 13-13
docs/changelogs/v0.34.4.md (2)
7-9: Link wrapping for MD034 and consistency.Wrap the PR links in angle brackets.
-* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1286) +* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in <https://github.com/cozystack/cozystack/pull/1286>) -* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in https://github.com/cozystack/cozystack/pull/1287, backported in https://github.com/cozystack/cozystack/pull/1291) +* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in <https://github.com/cozystack/cozystack/pull/1287>, backported in <https://github.com/cozystack/cozystack/pull/1291>)
13-15: Apply MD034 fix throughout the file.Pattern example shown below; apply similarly to the remaining bare URLs.
-* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in https://github.com/cozystack/cozystack/pull/1284) +* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in <https://github.com/cozystack/cozystack/pull/1284>) -* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in https://github.com/cozystack/cozystack/pull/1290, backported in https://github.com/cozystack/cozystack/pull/1293) +* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in <https://github.com/cozystack/cozystack/pull/1290>, backported in <https://github.com/cozystack/cozystack/pull/1293>) -* [tests] Add tests for S3 buckets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1283, backported in https://github.com/cozystack/cozystack/pull/1292) +* [tests] Add tests for S3 buckets. (@IvanHunters in <https://github.com/cozystack/cozystack/pull/1283>, backported in <https://github.com/cozystack/cozystack/pull/1292>) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4>Also applies to: 19-19, 21-21
docs/changelogs/v0.34.0.md (5)
24-24: Minor grammar: “handling of multiline strings.”-* [dashboard] Put YAML editor first when deploying and upgrading applications, as a more powerful option. Fix handling multiline strings. (@kvaps in https://github.com/cozystack/cozystack/pull/1227) +* [dashboard] Put YAML editor first when deploying and upgrading applications, as a more powerful option. Fix handling of multiline strings. (@kvaps in <https://github.com/cozystack/cozystack/pull/1227>)
42-42: Pluralization: “READMEs” (no apostrophe).-* [applications] Reorder configuration values in application README's for better readability. (@NickVolynkin in https://github.com/cozystack/cozystack/pull/1214) +* [applications] Reorder configuration values in application READMEs for better readability. (@NickVolynkin in <https://github.com/cozystack/cozystack/pull/1214>)
48-48: Remove duplicate “and”.-* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) +* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps in <https://github.com/cozystack/cozystack/pull/1162>)
50-50: Add missing “in” before the PR link and wrap URL.-* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 https://github.com/cozystack/cozystack/pull/1204) +* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 in <https://github.com/cozystack/cozystack/pull/1204>)
15-23: Optionally wrap remaining bare URLs (MD034).There are many bare URLs throughout; consider consistently using angle brackets or Markdown links across the whole file (examples above already show the pattern).
Also applies to: 28-28, 32-41, 45-47, 49-51, 56-67, 71-81, 85-87
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (9)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)docs/changelogs/v0.34.4.md(1 hunks)docs/changelogs/v0.34.5.md(1 hunks)docs/changelogs/v0.34.6.md(1 hunks)docs/changelogs/v0.34.7.md(1 hunks)
✅ Files skipped from review due to trivial changes (2)
- docs/changelogs/v0.34.7.md
- docs/changelogs/template.md
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/changelogs/v0.34.1.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
15-15: Bare URL used
(MD034, no-bare-urls)
16-16: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
20-20: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
23-23: Bare URL used
(MD034, no-bare-urls)
24-24: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
38-38: Bare URL used
(MD034, no-bare-urls)
39-39: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
41-41: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
43-43: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
64-64: Bare URL used
(MD034, no-bare-urls)
65-65: Bare URL used
(MD034, no-bare-urls)
66-66: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
79-79: Bare URL used
(MD034, no-bare-urls)
80-80: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
85-85: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.2.md
12-12: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.3.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.4.md
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.5.md
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.6.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
🪛 LanguageTool
docs/changelogs/v0.34.5.md
[style] ~9-~9: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (Timofei Larkin (@lllamnyp) in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
docs/changelogs/v0.34.5.md (1)
8-11: Rephrase VPA item and wrap bare URLs.
- Avoid “Disable VPA for VPA …” repetition; clarify scope.
- Keep the correct handle Timofei Larkin (@lllamnyp) (three l’s).
- Wrap raw URLs in angle brackets.
-* [virtual-machine] Enable using custom `instanceType` values in `virtual-machine` and `vm-instance` by disabling field validation. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1300, backported in https://github.com/cozystack/cozystack/pull/1303) -* [kubernetes] Disable VPA for VPA in tenant Kubernetes clusters. Tenant clusters have no need for this feature, and it was not designed to work in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1301, backported in https://github.com/cozystack/cozystack/pull/1305) +* [virtual-machine] Enable using custom `instanceType` values in `virtual-machine` and `vm-instance` by disabling field validation. (@lllamnyp in <https://github.com/cozystack/cozystack/pull/1300>, backported in <https://github.com/cozystack/cozystack/pull/1303>) +* [kubernetes] Disable the Vertical Pod Autoscaler in tenant Kubernetes clusters; it is not needed and not designed for tenant workloads. (@lllamnyp in <https://github.com/cozystack/cozystack/pull/1301>, backported in <https://github.com/cozystack/cozystack/pull/1305>) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.4...v0.34.5 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.4...v0.34.5>
🧹 Nitpick comments (9)
docs/changelogs/v0.34.1.md (1)
12-15: Wrap bare URLs to satisfy markdownlint MD034 and improve readability.Convert raw URLs to autolinks or Markdown links.
Apply examples like these:
-* [kubernetes] Fix regression in `volumesnapshotclass` installation from https://github.com/cozystack/cozystack/pull/1203. (@kvaps in https://github.com/cozystack/cozystack/pull/1238) +* [kubernetes] Fix regression in `volumesnapshotclass` installation from <https://github.com/cozystack/cozystack/pull/1203>. (@kvaps in <https://github.com/cozystack/cozystack/pull/1238>) -* [objectstorage] Fix building objectstorage images. (@kvaps in https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923) +* [objectstorage] Fix building objectstorage images. (@kvaps in <https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923>) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1>docs/changelogs/v0.34.3.md (1)
7-13: Linkify commit hash and wrap bare URLs (markdownlint MD034).
- Make the bare commit hash clickable.
- Wrap all raw URLs with angle brackets to pass MD034.
-* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in https://github.com/cozystack/cozystack/pull/1232, backported in https://github.com/cozystack/cozystack/pull/1272) +* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in <https://github.com/cozystack/cozystack/pull/1232>, backported in <https://github.com/cozystack/cozystack/pull/1272>) -* [kubernetes] Add dependency for snapshot CRD and migration to latest version. (@kvaps in https://github.com/cozystack/cozystack/pull/1275, backported in https://github.com/cozystack/cozystack/pull/1279) +* [kubernetes] Add dependency for snapshot CRD and migration to latest version. (@kvaps in <https://github.com/cozystack/cozystack/pull/1275>, backported in <https://github.com/cozystack/cozystack/pull/1279>) -* [seaweedfs] Add support for whitelisting and exporting via nginx-ingress. Update cosi-driver. (@kvaps in https://github.com/cozystack/cozystack/pull/1277) +* [seaweedfs] Add support for whitelisting and exporting via nginx-ingress. Update cosi-driver. (@kvaps in <https://github.com/cozystack/cozystack/pull/1277>) -* [kubevirt] Fix building Kubevirt CCM (@kvaps in 3c7e256906e1dbb0f957dc3a205fa77a147d419d) +* [kubevirt] Fix building Kubevirt CCM (@kvaps in <https://github.com/cozystack/cozystack/commit/3c7e256906e1dbb0f957dc3a205fa77a147d419d>) -* [virtual-machine] Fix a regression with field `optional=true`. (@kvaps in https://github.com/cozystack/cozystack/commit/01053f7c3180d1bd045d7c5fb949984c2bdaf19d) +* [virtual-machine] Fix a regression with field `optional=true`. (@kvaps in <https://github.com/cozystack/cozystack/commit/01053f7c3180d1bd045d7c5fb949984c2bdaf19d>) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3>docs/changelogs/v0.34.0.md (5)
24-24: Grammar: add “of” — “handling of multiline strings.”-* [dashboard] Put YAML editor first when deploying and upgrading applications, as a more powerful option. Fix handling multiline strings. (@kvaps in https://github.com/cozystack/cozystack/pull/1227) +* [dashboard] Put YAML editor first when deploying and upgrading applications, as a more powerful option. Fix handling of multiline strings. (@kvaps in <https://github.com/cozystack/cozystack/pull/1227>)
42-42: Pluralization: “READMEs” instead of “README's”.-* [applications] Reorder configuration values in application README's for better readability. (@NickVolynkin in https://github.com/cozystack/cozystack/pull/1214) +* [applications] Reorder configuration values in application READMEs for better readability. (@NickVolynkin in <https://github.com/cozystack/cozystack/pull/1214>)
48-48: Fix duplicated conjunction.Remove the extra “and”.
-* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) +* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps in <https://github.com/cozystack/cozystack/pull/1162>)
50-50: Missing preposition before URL.Add “in” before the PR link.
-* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 https://github.com/cozystack/cozystack/pull/1204) +* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 in <https://github.com/cozystack/cozystack/pull/1204>)
15-87: Wrap bare URLs to satisfy markdownlint MD034 (representative examples).There are many raw URLs across the file. Wrap them in angle brackets or use Markdown links. Example adjustments:
-* [kubernetes] Enable users to select Kubernetes versions in tenant clusters. Supported versions range from 1.28 to 1.33, updated to the latest patches. (@lllamnyp and @IvanHunters in https://github.com/cozystack/cozystack/pull/1202) +* [kubernetes] Enable users to select Kubernetes versions in tenant clusters. Supported versions range from 1.28 to 1.33, updated to the latest patches. (@lllamnyp and @IvanHunters in <https://github.com/cozystack/cozystack/pull/1202>) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.33.0...v0.34.0 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.33.0...v0.34.0>Apply similarly to other items in this file.
docs/changelogs/v0.34.6.md (1)
7-9: Wrap bare URLs and add a trailing newline.
- Angle-bracket the raw URLs to satisfy MD034.
- Ensure the file ends with a newline.
-* [dashboard] Fix filling multiline values in the visual editor. (@kvaps in https://github.com/cozystack/cozystack/commit/56fca9bd75efeca25f9483f6c514b6fec26d5d22 and https://github.com/cozystack/kubeapps/commit/4926bc68fabb0914afab574006643c85a597b371) +* [dashboard] Fix filling multiline values in the visual editor. (@kvaps in <https://github.com/cozystack/cozystack/commit/56fca9bd75efeca25f9483f6c514b6fec26d5d22> and <https://github.com/cozystack/kubeapps/commit/4926bc68fabb0914afab574006643c85a597b371>) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.5...v0.34.6 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.5...v0.34.6> +docs/changelogs/v0.34.4.md (1)
7-21: Wrap bare URLs in angle brackets to pass markdownlint MD034 (examples).Helps with consistency and linter cleanliness.
-* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1286) +* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in <https://github.com/cozystack/cozystack/pull/1286>) -* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in https://github.com/cozystack/cozystack/pull/1287, backported in https://github.com/cozystack/cozystack/pull/1291) +* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in <https://github.com/cozystack/cozystack/pull/1287>, backported in <https://github.com/cozystack/cozystack/pull/1291>) -* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in https://github.com/cozystack/cozystack/pull/1284) +* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in <https://github.com/cozystack/cozystack/pull/1284>) -* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in https://github.com/cozystack/cozystack/pull/1290, backported in https://github.com/cozystack/cozystack/pull/1293) +* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in <https://github.com/cozystack/cozystack/pull/1290>, backported in <https://github.com/cozystack/cozystack/pull/1293>) -* [tests] Add tests for S3 buckets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1283, backported in https://github.com/cozystack/cozystack/pull/1292) +* [tests] Add tests for S3 buckets. (@IvanHunters in <https://github.com/cozystack/cozystack/pull/1283>, backported in <https://github.com/cozystack/cozystack/pull/1292>) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4 +**Full Changelog**: <https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4>
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (9)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)docs/changelogs/v0.34.4.md(1 hunks)docs/changelogs/v0.34.5.md(1 hunks)docs/changelogs/v0.34.6.md(1 hunks)docs/changelogs/v0.34.7.md(1 hunks)
✅ Files skipped from review due to trivial changes (2)
- docs/changelogs/v0.34.7.md
- docs/changelogs/template.md
🚧 Files skipped from review as they are similar to previous changes (1)
- docs/changelogs/v0.34.2.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
15-15: Bare URL used
(MD034, no-bare-urls)
16-16: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
20-20: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
23-23: Bare URL used
(MD034, no-bare-urls)
24-24: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
38-38: Bare URL used
(MD034, no-bare-urls)
39-39: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
41-41: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
43-43: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
64-64: Bare URL used
(MD034, no-bare-urls)
65-65: Bare URL used
(MD034, no-bare-urls)
66-66: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
79-79: Bare URL used
(MD034, no-bare-urls)
80-80: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
85-85: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.1.md
12-12: Bare URL used
(MD034, no-bare-urls)
12-12: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
15-15: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.3.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.4.md
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.5.md
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.6.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
🪛 LanguageTool
docs/changelogs/v0.34.5.md
[style] ~9-~9: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (Timofei Larkin (@lllamnyp) in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
🔇 Additional comments (2)
docs/changelogs/v0.34.1.md (1)
5-7: Upgrade guidance looks good and consistent with 0.34.0/0.34.2.Clear recommendation to skip directly to 0.34.3. No changes needed.
docs/changelogs/v0.34.0.md (1)
8-11: Upgrade warning is clear and matches the patch guidance elsewhere.Consistent with 0.34.1 advising upgrades to 0.34.3. Good callout.
3f85e91 to
897a796
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
♻️ Duplicate comments (2)
docs/changelogs/v0.34.5.md (1)
8-9: Fix author handle, rephrase VPA sentence, and replace bare URLs with Markdown linksCorrects the handle typo and removes the “VPA for VPA” repetition. Also addresses MD034.
-* [virtual-machine] Enable using custom `instanceType` values in `virtual-machine` and `vm-instance` by disabling field validation. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1300, backported in https://github.com/cozystack/cozystack/pull/1303) -* [kubernetes] Disable VPA for VPA in tenant Kubernetes clusters. Tenant clusters have no need for this feature, and it was not designed to work in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1301, backported in https://github.com/cozystack/cozystack/pull/1305) +* [virtual-machine] Enable using custom `instanceType` values in `virtual-machine` and `vm-instance` by disabling field validation. (@llamnyp in [#1300](https://github.com/cozystack/cozystack/pull/1300), backported in [#1303](https://github.com/cozystack/cozystack/pull/1303)) +* [kubernetes] Disable the Vertical Pod Autoscaler in tenant Kubernetes clusters; it is neither intended for nor required by tenant workloads. (@llamnyp in [#1301](https://github.com/cozystack/cozystack/pull/1301), backported in [#1305](https://github.com/cozystack/cozystack/pull/1305))docs/changelogs/v0.34.0.md (1)
15-18: Username Timofei Larkin (@lllamnyp) (triple‑l) is correct — no change needed.Acknowledging the prior clarification: the GitHub handle is indeed Timofei Larkin (@lllamnyp) (three l’s). The attributions here look correct.
🧹 Nitpick comments (11)
docs/changelogs/v0.34.6.md (1)
7-9: Replace bare URLs with Markdown links and add a trailing newlineThis resolves markdownlint MD034 and improves readability. Also consider adding a trailing newline at EOF.
-* [dashboard] Fix filling multiline values in the visual editor. (@kvaps in https://github.com/cozystack/cozystack/commit/56fca9bd75efeca25f9483f6c514b6fec26d5d22 and https://github.com/cozystack/kubeapps/commit/4926bc68fabb0914afab574006643c85a597b371) +* [dashboard] Fix filling multiline values in the visual editor. (@kvaps in [cozystack commit 56fca9b](https://github.com/cozystack/cozystack/commit/56fca9bd75efeca25f9483f6c514b6fec26d5d22) and [kubeapps commit 4926bc6](https://github.com/cozystack/kubeapps/commit/4926bc68fabb0914afab574006643c85a597b371)) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.5...v0.34.6 +**Full Changelog**: [v0.34.5...v0.34.6](https://github.com/cozystack/cozystack/compare/v0.34.5...v0.34.6)Note: Add a newline at end of file.
docs/changelogs/v0.34.3.md (1)
7-13: Convert bare URLs to Markdown links and linkify the bare commit hashImproves readability and satisfies markdownlint MD034.
-* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in https://github.com/cozystack/cozystack/pull/1232, backported in https://github.com/cozystack/cozystack/pull/1272) -* [kubernetes] Add dependency for snapshot CRD and migration to latest version. (@kvaps in https://github.com/cozystack/cozystack/pull/1275, backported in https://github.com/cozystack/cozystack/pull/1279) -* [seaweedfs] Add support for whitelisting and exporting via nginx-ingress. Update cosi-driver. (@kvaps in https://github.com/cozystack/cozystack/pull/1277) -* [kubevirt] Fix building Kubevirt CCM (@kvaps in 3c7e256906e1dbb0f957dc3a205fa77a147d419d) -* [virtual-machine] Fix a regression with field `optional=true`. (@kvaps in https://github.com/cozystack/cozystack/commit/01053f7c3180d1bd045d7c5fb949984c2bdaf19d) +* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in [#1232](https://github.com/cozystack/cozystack/pull/1232), backported in [#1272](https://github.com/cozystack/cozystack/pull/1272)) +* [kubernetes] Add dependency for snapshot CRD and migrate it to the latest version. (@kvaps in [#1275](https://github.com/cozystack/cozystack/pull/1275), backported in [#1279](https://github.com/cozystack/cozystack/pull/1279)) +* [seaweedfs] Add support for whitelisting and exporting via nginx-ingress. Update cosi-driver. (@kvaps in [#1277](https://github.com/cozystack/cozystack/pull/1277)) +* [kubevirt] Fix building Kubevirt CCM. (@kvaps in [3c7e2569](https://github.com/cozystack/cozystack/commit/3c7e256906e1dbb0f957dc3a205fa77a147d419d)) +* [virtual-machine] Fix a regression with field `optional=true`. (@kvaps in [01053f7c](https://github.com/cozystack/cozystack/commit/01053f7c3180d1bd045d7c5fb949984c2bdaf19d)) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3 +**Full Changelog**: [v0.34.2...v0.34.3](https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3)docs/changelogs/v0.34.1.md (1)
12-15: Replace bare URLs with Markdown links; minor wording polishImproves readability and fixes MD034. Clarified “from” to “introduced in”.
-* [kubernetes] Fix regression in `volumesnapshotclass` installation from https://github.com/cozystack/cozystack/pull/1203. (@kvaps in https://github.com/cozystack/cozystack/pull/1238) -* [objectstorage] Fix building objectstorage images. (@kvaps in https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923) +* [kubernetes] Fix regression in `volumesnapshotclass` installation introduced in [#1203](https://github.com/cozystack/cozystack/pull/1203). (@kvaps in [#1238](https://github.com/cozystack/cozystack/pull/1238)) +* [objectstorage] Fix building objectstorage images. (@kvaps in [a9e9dfc](https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923)) -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1 +**Full Changelog**: [v0.34.0...v0.34.1](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1)docs/changelogs/v0.34.4.md (1)
7-21: Replace bare URLs with Markdown links across sectionsImproves readability and satisfies markdownlint MD034. No content changes.
-* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1286) -* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in https://github.com/cozystack/cozystack/pull/1287, backported in https://github.com/cozystack/cozystack/pull/1291) +* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in [#1286](https://github.com/cozystack/cozystack/pull/1286)) +* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in [#1287](https://github.com/cozystack/cozystack/pull/1287), backported in [#1291](https://github.com/cozystack/cozystack/pull/1291)) @@ -* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in https://github.com/cozystack/cozystack/pull/1284) -* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in https://github.com/cozystack/cozystack/pull/1290, backported in https://github.com/cozystack/cozystack/pull/1293) +* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in [#1284](https://github.com/cozystack/cozystack/pull/1284)) +* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in [#1290](https://github.com/cozystack/cozystack/pull/1290), backported in [#1293](https://github.com/cozystack/cozystack/pull/1293)) @@ -* [tests] Add tests for S3 buckets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1283, backported in https://github.com/cozystack/cozystack/pull/1292) +* [tests] Add tests for S3 buckets. (@IvanHunters in [#1283](https://github.com/cozystack/cozystack/pull/1283), backported in [#1292](https://github.com/cozystack/cozystack/pull/1292)) @@ -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4 +**Full Changelog**: [v0.34.3...v0.34.4](https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4)docs/changelogs/v0.34.0.md (7)
20-20: Consistency: use “schema” (lowercase) for OpenAPI.Align capitalization with line 19 and typical usage.
-* [cozystack-api] Refactor OpenAPI Schema and support reading it from config. (@kvaps in https://github.com/cozystack/cozystack/pull/1173) +* [cozystack-api] Refactor OpenAPI schema and support reading it from config. (@kvaps in https://github.com/cozystack/cozystack/pull/1173)
24-24: Grammar: “handling of multiline strings.”Minor readability improvement.
-* [dashboard] Put YAML editor first when deploying and upgrading applications, as a more powerful option. Fix handling multiline strings. (@kvaps in https://github.com/cozystack/cozystack/pull/1227) +* [dashboard] Put YAML editor first when deploying and upgrading applications, as a more powerful option. Fix handling of multiline strings. (@kvaps in https://github.com/cozystack/cozystack/pull/1227)
50-50: Missing “in” before PR link.Keeps phrasing consistent across entries.
-* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 https://github.com/cozystack/cozystack/pull/1204) +* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 in https://github.com/cozystack/cozystack/pull/1204)
56-56: Sentence case after line break.Capitalize “Before” at the start of the sentence.
-* Update FerretDB from v1 to v2.4.0.<br>**Breaking change:** before upgrading FerretDB instances, back up and restore the data following the [migration guide](https://docs.ferretdb.io/migration/migrating-from-v1/). (@kvaps in https://github.com/cozystack/cozystack/pull/1206) +* Update FerretDB from v1 to v2.4.0.<br>**Breaking change:** Before upgrading FerretDB instances, back up and restore the data following the [migration guide](https://docs.ferretdb.io/migration/migrating-from-v1/). (@kvaps in https://github.com/cozystack/cozystack/pull/1206)
61-61: Version format consistency: add “v” prefix.Most dependency versions use the “v” prefix; align Flux Operator accordingly.
-* Update Flux Operator to 0.24.0. (@kingdonb in https://github.com/cozystack/cozystack/pull/1167) +* Update Flux Operator to v0.24.0. (@kingdonb in https://github.com/cozystack/cozystack/pull/1167)
87-87: Fix markdownlint MD034 by linking the bare URL.Converts the bare URL into a Markdown link.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.33.0...v0.34.0 +**Full Changelog**: [v0.33.0...v0.34.0](https://github.com/cozystack/cozystack/compare/v0.33.0...v0.34.0)
15-45: Optional: convert remaining bare URLs to Markdown links (markdownlint MD034).markdownlint flags many bare URLs. If you want a clean lint pass, consider wrapping “in https://github...” occurrences as links, e.g. “in cozystack/cozystack#1203”. This keeps visual noise low and satisfies MD034. Alternatively, disable MD034 for changelogs in your lint config.
Happy to mass-prepare a patch that converts all “in https://github.com/cozystack/cozystack/pull/NNNN” to “in cozystack/cozystack#NNNN” across this file (or the changelogs directory) — say the word.
Also applies to: 47-52, 56-66, 71-81, 85-85
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (10)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)docs/changelogs/v0.34.4.md(1 hunks)docs/changelogs/v0.34.5.md(1 hunks)docs/changelogs/v0.34.6.md(1 hunks)docs/changelogs/v0.34.7.md(1 hunks)docs/changelogs/v0.34.8.md(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- docs/changelogs/v0.34.8.md
🚧 Files skipped from review as they are similar to previous changes (3)
- docs/changelogs/template.md
- docs/changelogs/v0.34.7.md
- docs/changelogs/v0.34.2.md
🧰 Additional context used
🪛 LanguageTool
docs/changelogs/v0.34.5.md
[style] ~9-~9: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (Timofei Larkin (@lllamnyp) in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.34.0.md
15-15: Bare URL used
(MD034, no-bare-urls)
16-16: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
20-20: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
23-23: Bare URL used
(MD034, no-bare-urls)
24-24: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
38-38: Bare URL used
(MD034, no-bare-urls)
39-39: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
41-41: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
43-43: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
64-64: Bare URL used
(MD034, no-bare-urls)
65-65: Bare URL used
(MD034, no-bare-urls)
66-66: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
79-79: Bare URL used
(MD034, no-bare-urls)
80-80: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
85-85: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.1.md
15-15: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.3.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.4.md
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.6.md
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
897a796 to
b04c828
Compare
Signed-off-by: Nick Volynkin <nick.volynkin@gmail.com>
b04c828 to
64a8a15
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (2)
docs/changelogs/v0.34.5.md (1)
8-11: Rephrase awkward duplication and replace bare URLs; keep handle as Timofei Larkin (@lllamnyp) (consistent elsewhere)The “Disable VPA for VPA” repetition reads awkwardly; also convert URLs to labeled links. Handle spelling with three leading l’s matches other changelogs.
-* [virtual-machine] Enable using custom `instanceType` values in `virtual-machine` and `vm-instance` by disabling field validation. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1300, backported in https://github.com/cozystack/cozystack/pull/1303) -* [kubernetes] Disable VPA for VPA in tenant Kubernetes clusters. Tenant clusters have no need for this feature, and it was not designed to work in a tenant cluster, but was enabled by mistake. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1301, backported in https://github.com/cozystack/cozystack/pull/1305) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.4...v0.34.5 +* [virtual-machine] Enable using custom `instanceType` values in `virtual-machine` and `vm-instance` by disabling field validation. (@lllamnyp in [#1300](https://github.com/cozystack/cozystack/pull/1300), backported in [#1303](https://github.com/cozystack/cozystack/pull/1303)) +* [kubernetes] Disable the Vertical Pod Autoscaler in tenant Kubernetes clusters; it is not intended for tenant workloads and was enabled unintentionally. (@lllamnyp in [#1301](https://github.com/cozystack/cozystack/pull/1301), backported in [#1305](https://github.com/cozystack/cozystack/pull/1305)) + +**Full Changelog**: [v0.34.4...v0.34.5](https://github.com/cozystack/cozystack/compare/v0.34.4...v0.34.5)docs/changelogs/v0.34.0.md (1)
48-48: Remove redundant “and”This duplication was flagged earlier; still present here.
-* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps and in https://github.com/cozystack/cozystack/pull/1162) +* [tenant] Enable deleting extra applications from a tenant. (@klinch0 and @kvaps in https://github.com/cozystack/cozystack/pull/1162)
🧹 Nitpick comments (17)
docs/changelogs/v0.31.0.md (1)
132-132: Handle correction looks good; convert bare URLs to Markdown links to satisfy MD034The contributor handle is now consistent across changelogs. To quiet markdownlint and improve readability, wrap the PR URLs as labeled links.
-* [platform] Refactor dashboard values. (@kvaps in https://github.com/cozystack/cozystack/pull/928, patched by @lllamnyp in https://github.com/cozystack/cozystack/pull/952) +* [platform] Refactor dashboard values. (@kvaps in [#928](https://github.com/cozystack/cozystack/pull/928), patched by @lllamnyp in [#952](https://github.com/cozystack/cozystack/pull/952))docs/changelogs/v0.34.2.md (1)
12-14: Replace bare URLs with labeled links; minor phrasing polishAddresses MD034 and tightens the sentence slightly.
-* [objectstorage] Fix recording image in objectstorage. (@kvaps in https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2 +* [objectstorage] Fix image recording in objectstorage. (@kvaps in [commit 4d9a8389](https://github.com/cozystack/cozystack/commit/4d9a8389d6bc7e86d63dd976ec853b374a91a637)) + +**Full Changelog**: [v0.34.1...v0.34.2](https://github.com/cozystack/cozystack/compare/v0.34.1...v0.34.2)docs/changelogs/v0.34.8.md (1)
7-11: Use labeled links for PRs and the compare URLThis removes bare URLs (MD034) and improves readability.
-* [etcd] Fix `topologySpreadConstraints`. (@klinch0 in https://github.com/cozystack/cozystack/pull/1331, backported in https://github.com/cozystack/cozystack/pull/1332) -* [linstor] Update LINSTOR monitoring configuration: switch labels on `linstor-satellite` and `linstor-controller`. (@kvaps in https://github.com/cozystack/cozystack/pull/1335, backported in https://github.com/cozystack/cozystack/pull/1336) -* [kamaji] Fix broken migration jobs originating from missing environment variables in the in-tree build. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1338, backported in https://github.com/cozystack/cozystack/pull/1340) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.7...v0.34.8 +* [etcd] Fix `topologySpreadConstraints`. (@klinch0 in [#1331](https://github.com/cozystack/cozystack/pull/1331), backported in [#1332](https://github.com/cozystack/cozystack/pull/1332)) +* [linstor] Update LINSTOR monitoring configuration: switch labels on `linstor-satellite` and `linstor-controller`. (@kvaps in [#1335](https://github.com/cozystack/cozystack/pull/1335), backported in [#1336](https://github.com/cozystack/cozystack/pull/1336)) +* [kamaji] Fix broken migration jobs originating from missing environment variables in the in-tree build. (@lllamnyp in [#1338](https://github.com/cozystack/cozystack/pull/1338), backported in [#1340](https://github.com/cozystack/cozystack/pull/1340)) + +**Full Changelog**: [v0.34.7...v0.34.8](https://github.com/cozystack/cozystack/compare/v0.34.7...v0.34.8)docs/changelogs/v0.34.3.md (1)
7-13: Standardize capitalization (KubeVirt), add punctuation, and replace bare URLs with labeled linksImproves consistency and resolves MD034 by avoiding bare URLs.
-* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in https://github.com/cozystack/cozystack/pull/1232, backported in https://github.com/cozystack/cozystack/pull/1272) -* [kubernetes] Add dependency for snapshot CRD and migration to latest version. (@kvaps in https://github.com/cozystack/cozystack/pull/1275, backported in https://github.com/cozystack/cozystack/pull/1279) -* [seaweedfs] Add support for whitelisting and exporting via nginx-ingress. Update cosi-driver. (@kvaps in https://github.com/cozystack/cozystack/pull/1277) -* [kubevirt] Fix building Kubevirt CCM (@kvaps in 3c7e256906e1dbb0f957dc3a205fa77a147d419d) -* [virtual-machine] Fix a regression with field `optional=true`. (@kvaps in https://github.com/cozystack/cozystack/commit/01053f7c3180d1bd045d7c5fb949984c2bdaf19d) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3 +* [tenant] Fix tenant network policy to allow traffic to additional tenant-related services across namespace hierarchies. (@klinch0 in [#1232](https://github.com/cozystack/cozystack/pull/1232), backported in [#1272](https://github.com/cozystack/cozystack/pull/1272)) +* [kubernetes] Add dependency for snapshot CRD and migrate to the latest version. (@kvaps in [#1275](https://github.com/cozystack/cozystack/pull/1275), backported in [#1279](https://github.com/cozystack/cozystack/pull/1279)) +* [seaweedfs] Add support for whitelisting and exporting via nginx-ingress. Update cosi-driver. (@kvaps in [#1277](https://github.com/cozystack/cozystack/pull/1277)) +* [kubevirt] Fix building KubeVirt CCM. (@kvaps in [commit 3c7e2569](https://github.com/cozystack/cozystack/commit/3c7e256906e1dbb0f957dc3a205fa77a147d419d)) +* [virtual-machine] Fix a regression with field `optional=true`. (@kvaps in [commit 01053f7c](https://github.com/cozystack/cozystack/commit/01053f7c3180d1bd045d7c5fb949984c2bdaf19d)) + +**Full Changelog**: [v0.34.2...v0.34.3](https://github.com/cozystack/cozystack/compare/v0.34.2...v0.34.3)docs/changelogs/v0.34.1.md (1)
12-15: Address markdownlint MD034 by replacing bare URLs with proper linksConvert bare URLs to markdown links for consistency and to satisfy the linter.
Apply this diff:
-* [kubernetes] Fix regression in `volumesnapshotclass` installation from https://github.com/cozystack/cozystack/pull/1203. (@kvaps in https://github.com/cozystack/cozystack/pull/1238) +* [kubernetes] Fix regression in `volumesnapshotclass` installation from [#1203](https://github.com/cozystack/cozystack/pull/1203). (@kvaps in [#1238](https://github.com/cozystack/cozystack/pull/1238)) -* [objectstorage] Fix building objectstorage images. (@kvaps in https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923) +* [objectstorage] Fix building objectstorage images. (@kvaps in [a9e9dfc](https://github.com/cozystack/cozystack/commit/a9e9dfca1fadde1bf2b4e100753e0731bbcfe923)) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1 +**Full Changelog**: [v0.34.0...v0.34.1](https://github.com/cozystack/cozystack/compare/v0.34.0...v0.34.1)docs/changelogs/v0.34.4.md (3)
7-9: Convert PR references to markdown links; keep “cookie-secure” as-isThis avoids MD034 and standardizes link rendering.
-* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1286) -* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in https://github.com/cozystack/cozystack/pull/1287, backported in https://github.com/cozystack/cozystack/pull/1291) +* [keycloak] Store administrative passwords in the management cluster's secrets. (@IvanHunters in [#1286](https://github.com/cozystack/cozystack/pull/1286)) +* [keycloak] Update Keycloak client redirect URI to use HTTPS instead of HTTP. Enable `cookie-secure`. (@klinch0 in [#1287](https://github.com/cozystack/cozystack/pull/1287), backported in [#1291](https://github.com/cozystack/cozystack/pull/1291))
13-15: Shorten CRD name note reads well; convert bare URLs to linksReplace bare URLs per markdownlint MD034.
-* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in https://github.com/cozystack/cozystack/pull/1284) -* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in https://github.com/cozystack/cozystack/pull/1290, backported in https://github.com/cozystack/cozystack/pull/1293) +* [kubernetes] Resolve problems with pod names exceeding allowed length by shortening the name of volume snapshot CRD from `*-volumesnapshot-crd-for-tenant-k8s` to `*-vsnap-crd`. To apply this change, update each affected tenant Kubernetes cluster after updating Cozystack. (@klinch0 in [#1284](https://github.com/cozystack/cozystack/pull/1284)) +* [cozystack-api] Show correct `kind` values of `ApplicationList`. (@kvaps in [#1290](https://github.com/cozystack/cozystack/pull/1290), backported in [#1293](https://github.com/cozystack/cozystack/pull/1293))
19-21: Address MD034 for remaining items and Full Changelog-* [tests] Add tests for S3 buckets. (@IvanHunters in https://github.com/cozystack/cozystack/pull/1283, backported in https://github.com/cozystack/cozystack/pull/1292) +* [tests] Add tests for S3 buckets. (@IvanHunters in [#1283](https://github.com/cozystack/cozystack/pull/1283), backported in [#1292](https://github.com/cozystack/cozystack/pull/1292)) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4 +**Full Changelog**: [v0.34.3...v0.34.4](https://github.com/cozystack/cozystack/compare/v0.34.3...v0.34.4)docs/changelogs/v0.34.7.md (1)
7-11: Replace bare URLs with markdown links across the Fixes and Full ChangelogConforms to MD034 and keeps formatting consistent with other changelogs.
-* [seaweedfs] Disable proxy buffering and proxy request buffering for ingress. (@kvaps in https://github.com/cozystack/cozystack/pull/1330, backported in https://github.com/cozystack/cozystack/commit/96d462e911d4458704b596533d3f10e4b5e80862) -* [linstor] Update LINSTOR monitoring configuration to use label `controller_node` instead of `node`. (@kvaps in https://github.com/cozystack/cozystack/pull/1326, backported in https://github.com/cozystack/cozystack/pull/1327) -* [kubernetes] Disable VPA for VPA in tenant Kubernetes clusters, patched a fix from v0.34.5. (@lllamnyp in https://github.com/cozystack/cozystack/pull/1318, backported in https://github.com/cozystack/cozystack/pull/1319) +* [seaweedfs] Disable proxy buffering and proxy request buffering for ingress. (@kvaps in [#1330](https://github.com/cozystack/cozystack/pull/1330), backported in [96d462e](https://github.com/cozystack/cozystack/commit/96d462e911d4458704b596533d3f10e4b5e80862)) +* [linstor] Update LINSTOR monitoring configuration to use label `controller_node` instead of `node`. (@kvaps in [#1326](https://github.com/cozystack/cozystack/pull/1326), backported in [#1327](https://github.com/cozystack/cozystack/pull/1327)) +* [kubernetes] Disable VPA for VPA in tenant Kubernetes clusters, patched a fix from v0.34.5. (@lllamnyp in [#1318](https://github.com/cozystack/cozystack/pull/1318), backported in [#1319](https://github.com/cozystack/cozystack/pull/1319)) - -**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.34.6...v0.34.7 +**Full Changelog**: [v0.34.6...v0.34.7](https://github.com/cozystack/cozystack/compare/v0.34.6...v0.34.7)docs/changelogs/v0.34.0.md (8)
24-24: Grammar: “Fix handling of multiline strings.”Add “of” for correct phrasing.
-* [dashboard] Put YAML editor first when deploying and upgrading applications, as a more powerful option. Fix handling multiline strings. (@kvaps in https://github.com/cozystack/cozystack/pull/1227) +* [dashboard] Put YAML editor first when deploying and upgrading applications, as a more powerful option. Fix handling of multiline strings. (@kvaps in https://github.com/cozystack/cozystack/pull/1227)
20-20: Consistency: “OpenAPI schema” (lowercase ‘schema’)Line 19 uses “schema” lowercase; make line 20 consistent.
-* [cozystack-api] Refactor OpenAPI Schema and support reading it from config. (@kvaps in https://github.com/cozystack/cozystack/pull/1173) +* [cozystack-api] Refactor OpenAPI schema and support reading it from config. (@kvaps in https://github.com/cozystack/cozystack/pull/1173)
28-28: Replace trailing bare URL with a labeled linkMinor readability improvement and satisfies MD034.
-* [seaweedfs] Ensure that JWT signing keys in the SeaweedFS security configuration remain consistent across Helm upgrades. Resolve an upstream issue. (@kvaps in https://github.com/cozystack/cozystack/pull/1193 and https://github.com/seaweedfs/seaweedfs/pull/6967) +* [seaweedfs] Ensure that JWT signing keys in the SeaweedFS security configuration remain consistent across Helm upgrades. Resolve an upstream issue. (@kvaps in [#1193](https://github.com/cozystack/cozystack/pull/1193) and [seaweedfs#6967](https://github.com/seaweedfs/seaweedfs/pull/6967))
35-35: Voice consistency: use imperative mood like other bulletsMost entries use “Enable/Fix/Improve…”. Align this one.
-* [virtual-machine,vm-instance] Adjusted RBAC role to let users read the service associated with the VMs they create. Consequently, users can now see details of the service in the dashboard and therefore read the IP address of the VM. (@klinch0 in https://github.com/cozystack/cozystack/pull/1161) +* [virtual-machine,vm-instance] Adjust RBAC role to let users read the service associated with the VMs they create. Consequently, users can now see details of the service in the dashboard and therefore read the IP address of the VM. (@klinch0 in https://github.com/cozystack/cozystack/pull/1161)
42-42: Typo: “README’s” → “READMEs”Plural, not possessive.
-* [applications] Reorder configuration values in application README's for better readability. (@NickVolynkin in https://github.com/cozystack/cozystack/pull/1214) +* [applications] Reorder configuration values in application READMEs for better readability. (@NickVolynkin in https://github.com/cozystack/cozystack/pull/1214)
50-50: Missing preposition “in” before PR linkReadability fix.
-* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 https://github.com/cozystack/cozystack/pull/1204) +* [postgres] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboard. (@klinch0 in https://github.com/cozystack/cozystack/pull/1204)
87-87: Convert Full Changelog to a markdown linkAlign with other files and avoid MD034.
-**Full Changelog**: https://github.com/cozystack/cozystack/compare/v0.33.0...v0.34.0 +**Full Changelog**: [v0.33.0...v0.34.0](https://github.com/cozystack/cozystack/compare/v0.33.0...v0.34.0)
15-85: Bulk URL formatting in v0.34.x changelogs
There are many bare URLs in docs/changelogs/v0.34.0.md (and its patch-level siblings), which trigger MD034. To automate wrapping them in<…>more reliably thangawk -i inplace, you can run for example:#!/usr/bin/env bash set -euo pipefail # Wrap bare http(s) URLs in angle brackets across all v0.34.x changelogs find docs/changelogs -regextype posix-extended -regex '.*v0\.34\.[0-9]+\.md$' \ -exec sed -E -i \ 's@(^|[^([<])(https?://[[:alnum:]/._#?=&%?&=-]+)@\1<\2>@g' {} +• This preserves existing
[text](url)links unchanged.
• After running, please verify there are no unintended brackets around already-wrapped URLs or within links.Let me know if you’d like me to prepare a follow-up PR; otherwise you can run this locally to satisfy MD034.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (11)
docs/changelogs/template.md(1 hunks)docs/changelogs/v0.31.0.md(1 hunks)docs/changelogs/v0.34.0.md(1 hunks)docs/changelogs/v0.34.1.md(1 hunks)docs/changelogs/v0.34.2.md(1 hunks)docs/changelogs/v0.34.3.md(1 hunks)docs/changelogs/v0.34.4.md(1 hunks)docs/changelogs/v0.34.5.md(1 hunks)docs/changelogs/v0.34.6.md(1 hunks)docs/changelogs/v0.34.7.md(1 hunks)docs/changelogs/v0.34.8.md(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- docs/changelogs/template.md
- docs/changelogs/v0.34.6.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
docs/changelogs/v0.31.0.md
132-132: Bare URL used
(MD034, no-bare-urls)
132-132: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.0.md
15-15: Bare URL used
(MD034, no-bare-urls)
16-16: Bare URL used
(MD034, no-bare-urls)
17-17: Bare URL used
(MD034, no-bare-urls)
18-18: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
20-20: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
22-22: Bare URL used
(MD034, no-bare-urls)
23-23: Bare URL used
(MD034, no-bare-urls)
24-24: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
28-28: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
32-32: Bare URL used
(MD034, no-bare-urls)
33-33: Bare URL used
(MD034, no-bare-urls)
34-34: Bare URL used
(MD034, no-bare-urls)
35-35: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
36-36: Bare URL used
(MD034, no-bare-urls)
37-37: Bare URL used
(MD034, no-bare-urls)
38-38: Bare URL used
(MD034, no-bare-urls)
39-39: Bare URL used
(MD034, no-bare-urls)
40-40: Bare URL used
(MD034, no-bare-urls)
41-41: Bare URL used
(MD034, no-bare-urls)
42-42: Bare URL used
(MD034, no-bare-urls)
43-43: Bare URL used
(MD034, no-bare-urls)
44-44: Bare URL used
(MD034, no-bare-urls)
45-45: Bare URL used
(MD034, no-bare-urls)
46-46: Bare URL used
(MD034, no-bare-urls)
47-47: Bare URL used
(MD034, no-bare-urls)
48-48: Bare URL used
(MD034, no-bare-urls)
49-49: Bare URL used
(MD034, no-bare-urls)
50-50: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
51-51: Bare URL used
(MD034, no-bare-urls)
56-56: Bare URL used
(MD034, no-bare-urls)
57-57: Bare URL used
(MD034, no-bare-urls)
58-58: Bare URL used
(MD034, no-bare-urls)
59-59: Bare URL used
(MD034, no-bare-urls)
60-60: Bare URL used
(MD034, no-bare-urls)
61-61: Bare URL used
(MD034, no-bare-urls)
62-62: Bare URL used
(MD034, no-bare-urls)
63-63: Bare URL used
(MD034, no-bare-urls)
64-64: Bare URL used
(MD034, no-bare-urls)
65-65: Bare URL used
(MD034, no-bare-urls)
66-66: Bare URL used
(MD034, no-bare-urls)
71-71: Bare URL used
(MD034, no-bare-urls)
72-72: Bare URL used
(MD034, no-bare-urls)
73-73: Bare URL used
(MD034, no-bare-urls)
74-74: Bare URL used
(MD034, no-bare-urls)
75-75: Bare URL used
(MD034, no-bare-urls)
76-76: Bare URL used
(MD034, no-bare-urls)
77-77: Bare URL used
(MD034, no-bare-urls)
78-78: Bare URL used
(MD034, no-bare-urls)
79-79: Bare URL used
(MD034, no-bare-urls)
80-80: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
81-81: Bare URL used
(MD034, no-bare-urls)
85-85: Bare URL used
(MD034, no-bare-urls)
87-87: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.1.md
12-12: Bare URL used
(MD034, no-bare-urls)
12-12: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
15-15: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.2.md
12-12: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.3.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.4.md
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
13-13: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
14-14: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
19-19: Bare URL used
(MD034, no-bare-urls)
21-21: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.5.md
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.7.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
docs/changelogs/v0.34.8.md
7-7: Bare URL used
(MD034, no-bare-urls)
7-7: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
8-8: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
9-9: Bare URL used
(MD034, no-bare-urls)
11-11: Bare URL used
(MD034, no-bare-urls)
🪛 LanguageTool
docs/changelogs/v0.34.0.md
[grammar] ~34-~34: There might be a mistake here.
Context: ...ithub.com//pull/1165) * [virtual-machine,vm-instance] Adjusted R...
(QB_NEW_EN)
[grammar] ~36-~36: There might be a mistake here.
Context: ...a3ee5d0ea9e87529c8adc4fcccffabe8782292e) * [cozystack-api] Fix an error with `resou...
(QB_NEW_EN)
[grammar] ~37-~37: There might be a mistake here.
Context: ...r with resourceVersion which resulted in message 'failed to update HelmRelease: ...
(QB_NEW_EN)
[grammar] ~37-~37: There might be a mistake here.
Context: ...ithub.com//pull/1170) * [cozystack-api] Fix an error in updating...
(QB_NEW_EN)
[grammar] ~38-~38: There might be a mistake here.
Context: ...ts in Cozystack objects, which resulted in message "Warning: resource ... is missi...
(QB_NEW_EN)
[grammar] ~40-~40: There might be a mistake here.
Context: ...ithub.com//pull/1208) * [dashboard] Fix the code for removing da...
(QB_NEW_EN)
[grammar] ~44-~44: There might be a mistake here.
Context: ...ithub.com//pull/1172) * [kubernetes] Enable deleting Velero comp...
(QB_NEW_EN)
[grammar] ~45-~45: There might be a mistake here.
Context: ...ystack/pull/1172) * [kubernetes] Enable deleting Velero component from a tenant Kubernet...
(QB_NEW_EN)
[grammar] ~49-~49: There might be a mistake here.
Context: ...ithub.com//pull/1195) * [postgres] Resolve an issue with the vis...
(QB_NEW_EN)
[grammar] ~50-~50: There might be a mistake here.
Context: ...s] Resolve an issue with the visibility of PostgreSQL load balancer on the dashboa...
(QB_NEW_EN)
[grammar] ~85-~85: There might be a mistake here.
Context: ...trictions preventing pushing to ghcr.io with default GitHub token. (Timofei Larkin (@lllamnyp) in htt...
(QB_NEW_EN)
docs/changelogs/v0.34.5.md
[style] ~9-~9: ‘by mistake’ might be wordy. Consider a shorter alternative.
Context: ...rk in a tenant cluster, but was enabled by mistake. (Timofei Larkin (@lllamnyp) in https://github.com/cozys...
(EN_WORDINESS_PREMIUM_BY_MISTAKE)
🔇 Additional comments (1)
docs/changelogs/v0.34.1.md (1)
5-7: Warning block is clear and actionable — LGTMThe guidance to skip 0.34.1 and upgrade to 0.34.3 is explicit and links are correct.
No description provided.