Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clarify how to proceed with requirement "Test result statement" for stable APIs #89

Closed
jlurien opened this issue Aug 29, 2024 · 5 comments
Labels
enhancement New feature or request Fall24 Issues relevant for Fall24 meta-release (without review issues)

Comments

@jlurien
Copy link
Collaborator

jlurien commented Aug 29, 2024

Problem description

API-Readiness-Checklist indicates that for stable APIs a mandatory check is

Nr API release assets Explanation
9 Test result statement A statement in a discussion issue of the API Sub Project by at least one of the API Sub Project members that the Gherkin feature files have been successfully executed against their (lab) API implementation.

Which would imply passing the enhanced test plan to the last RC after M3 in a lab environment, but this is not really feasible for this meta-release, due to the reduced period between M3 and M4, and the delay in the availability of enhanced test plans.

Possible evolution

Discuss at release and project management level how to deal with the exceptionality for this first meta-release, and give guidelines to APIs intending to release stable versions about how to proceed and fill the API-Readiness-Checklist

@tanjadegroot
Copy link
Collaborator

tanjadegroot commented Sep 5, 2024

In the TSC of 05/09/24, it was decided that EXCEPTIONALLY the 5 stable APIs can be published without having done the test plan execution, as they have been implemented in various industry contexts.
Not having done the execution of the test plans, means that there can be bugs found in the test plans. In this case, these errors should be notified to the Sub-Project which should then prepare a patch release of the API.

The following information is proposed t be added in the M4 release PR in the API Readiness checklist:

  • In row with point 9 "Test result statement":
    - in Status column put "N"
    - in Comments column add: "Fall24 EXCEPTION: Test results not available (*)"
  • Add the following footnote under the table: "(*) If you encounter issues with the provided test files (.feature), please create an issue in the API Sub-Project to signal these issues so they can be fixed in a patch release."

Please comment on the proposal here, by latest tomorrow friday EOB

@jlurien
Copy link
Collaborator Author

jlurien commented Sep 5, 2024

Thanks @tanjadegroot, what about the Status column? Should APIs fill it with "N"?

@hdamker
Copy link
Collaborator

hdamker commented Sep 5, 2024

Thanks @tanjadegroot, what about the Status column? Should APIs fill it with "N"?

My take would fill it with "N", allows later in a patch release to change it to "Y" (if bugs had to be fixed).

@tanjadegroot
Copy link
Collaborator

Thanks @tanjadegroot, what about the Status column? Should APIs fill it with "N"?

yes, let me update the description accordingly

bigludo7 added a commit to camaraproject/NumberVerification that referenced this issue Sep 5, 2024
Taking care of the Test result statement accordingly to camaraproject/ReleaseManagement#89
bigludo7 added a commit to camaraproject/OTPValidation that referenced this issue Sep 5, 2024
bigludo7 added a commit to camaraproject/SimSwap that referenced this issue Sep 5, 2024
Taking care of the Test result statement accordingly to camaraproject/ReleaseManagement#89
jlurien added a commit to camaraproject/DeviceLocation that referenced this issue Sep 6, 2024
Kevsy added a commit to camaraproject/SimpleEdgeDiscovery that referenced this issue Sep 6, 2024
@tanjadegroot
Copy link
Collaborator

tanjadegroot commented Sep 12, 2024

Implemented in the API-Readiness-Checklist.md of the Fall24 meta-release stable APIs and closed as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Fall24 Issues relevant for Fall24 meta-release (without review issues)
Projects
None yet
Development

No branches or pull requests

3 participants