Release Review: PredictiveConnectivityData r2.1 (rc Sync26) - #64
Release Review: PredictiveConnectivityData r2.1 (rc Sync26)#64camara-release-automation[bot] wants to merge 4 commits into
Conversation
CAMARA Validation — PASS (with warnings)0 errors, 2 warnings, 4 hints | Profile: standard
|
|
Issue created: #65 regarding to the point "Document deferred validation warnings (and hints)". Is there anything left to do before I can click “The release is ready for Release Management review”? WDYT @eric-murray? |
I created PR #66 to use the cloud events format for the notifications, which fixes the validation warnings. I left it as a draft for now, but if you are happy with it, it could be merged for the release candidate. You would need to discard this snapshot and create a new one. I will also need update the test cases. |
Hi @eric-murray, thanks for preparing this and for keeping the PR as a draft. Before merging it into the release candidate, I think we should distinguish between an event notification and an asynchronous response. In this API, the callback delivers the final result of a previously accepted request exactly once; it is not reporting an event that may occur independently or repeatedly. This distinction is now being clarified in Commonalities PR #575, based on Issue #533. The proposed guidance explicitly states that asynchronous responses do not need to use the CAMARA CloudEvents model and should instead reuse the equivalent synchronous response model. Therefore, the current linter warnings do not necessarily indicate that the API design is incorrect; the validation rules may not yet account for this distinction. The existing issue already documents the relevant P-015 and S-035 warnings for release purposes. Under the release-management guidelines, documented warnings and applicable hints are not blockers for the RC. My proposal would be to keep PR #66 in draft, proceed with the current snapshot, and discuss after the release whether the API should remain aligned with the asynchronous-response guidance or deliberately adopt CloudEvents. This also avoids changing the contract and test cases immediately before the RC based solely on a potentially over-general validation rule. WDYT? CC: @PedroDiez |
Release Review: r2.1 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.0Dependencies: 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.1-ecd8a33