Skip to content

Release Review: ModelAsAService r2.2 (rc Sync26) - #86

Merged
lihaojie-CMCC merged 2 commits into
release-snapshot/r2.2-f2a7cdffrom
release-review/r2.2-f2a7cdf
Nov 6, 2026
Merged

lihaojie-CMCC merged 2 commits into
release-snapshot/r2.2-f2a7cdffrom
release-review/r2.2-f2a7cdf

Conversation

@camara-release-automation

@camara-release-automation camara-release-automation Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

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

API Version Status Comparison target
knowledge-base 0.2.0-rc.2 rc 0.2.0-rc.1
qa-assistant-manage 0.2.0-rc.2 rc 0.2.0-rc.1
qa-assistant-service 0.2.0-rc.2 rc 0.2.0-rc.1

Dependencies: 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:

    • Copy all API-consumer-relevant changes from the provided list into the appropriate Breaking changes / Added / Changed / Fixed / Removed sections for each API. List breaking changes both in Breaking changes and in their normal change category.
    • Do not copy administrative, tooling-only, or internal maintenance changes unless they affect API consumers.
    • For each API, fill the CHANGELOG against the stated comparison target, following the release-type rules at the top of the CHANGELOG.
  • Document deferred validation warnings (and hints)

    What to do:

    • Check the CAMARA Validation comment on this PR for warnings and hints.
    • For each warning you do not fix, document it in an issue: include a copy of the validation summary line(s) and the reason the fix is deferred.
    • Document in the same way any validation hint that is applicable to the API and needs to be fixed later.
    • You may group several findings into one issue or split them across issues — either is fine.
    • List the documenting issue(s) in a comment on this PR.
    • Note: documenting deferred warnings is optional but recommended for alpha pre-releases, and mandatory for rc pre-releases and public releases.
  • The release is ready for Release Management review

    Check that:

    • All mandatory release assets for the declared status(es) are present (see the table below "Required release assets per API status" by expanding the arrow);
    • API documentation and test cases are adequate for the target status.

    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:

  • Assign the Release Management reviewer(s) as assignee(s) of this PR
  • CHANGELOG follows the release documentation rules
  • Breaking changes are documented and version updates follow SemVer rules
  • Mandatory release assets are present for each API according to its status
  • All remaining validation warnings are documented in issues and the reasons for deferral are defensible
  • Content findings needing a main PR are collected as sub-issues of one issue titled Release r2.2 review findings (see Recording review findings)
Required release assets per API status
Nr Asset alpha rc initial
public
stable
public
1 Release Plan M M M M
2 API Definition(s) M M M M
3 Commonalities compliance O M M M
4 API Documentation M M M M
5 User Stories O O O M
6 Test Cases (basic) O M M M
7 Test Cases (enhanced) O O O M
8 API Description O O M M

M = Mandatory, O = Optional — Full documentation

Valid next actions for codeowners

  • Merge this PR when all Codeowner Actions and Release Management Actions are complete and the required approvals are present — creates the draft release
  • Use /discard-snapshot <reason> in the Release Issue to discard this snapshot, return to planned, and update content on main

Snapshot: r2.2-f2a7cdf

@camara-release-automation
camara-release-automation Bot requested a review from a team November 2, 2026 11:52
@camara-release-automation camara-release-automation Bot mentioned this pull request Nov 2, 2026
6 tasks done
@camara-validation

camara-validation Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

CAMARA Validation — PASS

0 errors, 0 warnings, 31 hints | Profile: standard

View full results

@lihaojie-CMCC

Copy link
Copy Markdown
Contributor

@camaraproject/release-management_reviewers Thank you for your review and for documenting the findings in #74.

We have completed the necessary updates to address these issues, and the snapshot has been restarted to reflect the latest changes. Please let us know if everything looks good on your end.

Thanks again for your efforts and guidance!

@tanjadegroot tanjadegroot self-assigned this Nov 2, 2026

@tanjadegroot tanjadegroot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi team,
you went a bit too quick as I had not completed the review :-)
Please see 2 additional sub issues in #78.

I have now completed the review, but you will need to do a new snapshot.

@tanjadegroot tanjadegroot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi team, many thanks for the quick update, it looks good !

As the 2 additional review findings concern only test cases (most of which are enhanced cases), you may decide to address these now or later.
I can approve the release-candidate as it stands.

/LGTM

@lihaojie-CMCC

Copy link
Copy Markdown
Contributor

Hi @tanjadegroot,

Thank you for the review and for your flexibility! We really appreciate your approval.

Those are indeed great suggestions regarding the test cases. We will definitely address them as soon as possible.

We will go ahead and merge this PR now. Thanks again for all your help and guidance throughout this review : )

@lihaojie-CMCC
lihaojie-CMCC merged commit 83abe1d into release-snapshot/r2.2-f2a7cdf Nov 6, 2026
3 checks passed
@tanjadegroot

Copy link
Copy Markdown
Contributor

This release-candidate is ready for merge !

Then please prepare the public release where the main work is

  • ensure the API description (on the wiki) is aligned with the API
  • ensure the user stories are aligned with the API (e.g. 5xx errors were removed)
  • ensure any other supporting documents are aligned with the API
  • once the snapshot is created, to update the changelog with all (developer focused) changes since the previous public release.

@camara-release-automation
camara-release-automation Bot deleted the release-review/r2.2-f2a7cdf branch November 6, 2026 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants