Repository navigation
Release Review: DedicatedNetworks r2.2 (rc Sync26) - #150
Closed
camara-release-automation[bot] wants to merge 6 commits into
Closed
camara-release-automation[bot] wants to merge 6 commits into
camara-release-automation[bot] wants to merge 6 commits into
Conversation
camara-release-automation
Bot
requested review from
PeterKovacs2,
carlosromerog,
eric-murray and
tlohmar
as code owners
August 14, 2026 10:52
6 tasks done
CAMARA Validation — PASS0 errors, 0 warnings, 14 hints | Profile: standard |
hdamker
reviewed
Aug 18, 2026
Contributor
There was a problem hiding this comment.
A deeper review of the four APIs has revealed several issues which I collected in #156. Especially the shape of dedicated-network-accesses needs some rework to be functional.
If you want go forward with the PR and release there are some points to be fixed in the CHANGELOG:
- PR#141 duplicated — listed twice, verbatim, under dedicated-network-accesses → Fixed.
- Breaking changes not repeated in their normal category — eight entries (accesses #98/#122/#149, profiles #83/#125, network #87/#83/#149) appear only under Breaking changes, though the PR template requires listing them there and in their normal section too.
- Wrong API attribution — PR#81 (409 INCOMPATIBLE_STATE) is listed under dedicated-network → Added, but it only touched dedicated-network-accesses.yaml; that error code appears in no other file.
- Raw PR titles instead of consumer-facing prose — entries are just "title by @author in URL", several tooling-only (#115/#118/#124/#148/#102); and dedicated-network-areas has Changed/Fixed populated despite its comparison target being N/A, where Added-only is expected.
Clarifying, whether breaking changes are because of new functions (added) or because of changes existing functions.
Removing PR#81 from dedicated-network Bit more reader friendly change description.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release Review: r2.2 rc
This PR finalizes the reviewable release content for the active snapshot.
Edit and review this PR before merging it into the release snapshot. After Codeowner and Release Management approval, merging this PR creates the draft release.
Release contents
0.2.0-rc.10.1.00.2.0-rc.10.1.00.2.0-rc.10.1.00.1.0-rc.1N/ADependencies: Commonalities r4.3, ICM r4.2
Codeowner Actions
Tick each box once done. Ticking the last box — "The release is ready for Release Management review" — starts the Release Management review.
Update the CHANGELOG
What to do:
Document deferred validation warnings (and hints)
What to do:
The release is ready for Release Management review
Check that:
Tick this box to confirm readiness and to start the Release Management review.
Release Management Actions
The following actions and checks are done by a Release Management reviewer before approving the PR:
Required release assets per API status
public
public
M = Mandatory, O = Optional — Full documentation
Valid next actions for codeowners
/discard-snapshot <reason>in the Release Issue to discard this snapshot, return toplanned, and update content onmainSnapshot:
r2.2-1d16652