Repository navigation
Release Review: DeviceLocation r4.2 (rc Sync26) - #440
camara-release-automation[bot] wants to merge 5 commits into
Conversation
CAMARA Validation — PASS0 errors, 0 warnings, 1 hints | Profile: standard |
|
The only remaining hint is not to be fixed as that definition comes from the artifact in Commonalities |
hdamker
left a comment
There was a problem hiding this comment.
The CHANGELOG for geofencing-subscriptions 0.6.0-rc.2 lists no breaking changes, but #437 declares two, and oasdiff between r4.1 and the snapshot confirms them: two error codes added to createGeofencingSubscription (400 OUT_OF_RANGE, 422 MULTIEVENT_COMBINATION_TEMPORARILY_NOT_SUPPORTED). The API is at initial status (0.y.z), so the changes are allowed, but they need to be listed under Breaking changes and under Changed. #437 also removes error codes that cannot occur; OUT_OF_RANGE on the list and callback 400 responses was added in 0.6.0-rc.1 and is now reverted to the 0.5.0 state. Suggestions inline.
location-retrieval and location-verification: oasdiff reports no wire-level changes against r4.1, so the single Changed entry for each is accurate.
The remaining S-313 hint needs no deferral issue.
|
BTW: as I contributed some of the fixes of this release it would make sense to get another Release Management reviewer involved here for the final approval. @tanjadegroot @eric-murray? |
Co-authored-by: Herbert Damker <herbert.damker@telekom.de>
Co-authored-by: Herbert Damker <herbert.damker@telekom.de>
Co-authored-by: Herbert Damker <herbert.damker@telekom.de>
Release Review: r4.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.6.0-rc.20.6.0-rc.11.0.0-rc.21.0.0-rc.13.1.0-rc.33.1.0-rc.2Dependencies: Commonalities r4.4, 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:
mainPR are collected as sub-issues of one issue titledRelease r4.2 review findings(see Recording review findings)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:
r4.2-0b155f3