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

Documentation: add NuttX-12.7.0 release notes #13785

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

jerpelea
Copy link
Contributor

@jerpelea jerpelea commented Oct 2, 2024

Summary

Add release notes for 12.7.0 release

NOTE
Ddue to the quality of commit titles new unsorted sections were added.
If the titles and commit descriptions will improve those sections will disappear in future releases.

Impact

Release

Testing

CI

Add release notes for 12.7.0 release

Signed-off-by: Alin Jerpelea <[email protected]>
@github-actions github-actions bot added Area: Documentation Improvements or additions to documentation Size: L The size of the change in this PR is large labels Oct 2, 2024
@jerpelea jerpelea requested a review from cederom October 2, 2024 08:27
@nuttxpr
Copy link

nuttxpr commented Oct 2, 2024

[Experimental Bot, please feedback here]

This PR appears to meet the basic NuttX requirements for a release notes update. Here's a breakdown:

Meets Requirements:

  • Summary: Clearly states the purpose of the PR is to add release notes. The note about commit titles provides helpful context.
  • Impact: Correctly identifies this as a "Release" impact, signifying a documentation update for a specific NuttX version.
  • Testing: "CI" implies the PR will be validated through the Continuous Integration system, which is standard practice.

Potential Improvements (Based on Limited Information):

  • Testing (Specificity): While "CI" is sufficient, providing more details about the specific CI checks relevant to release notes would be beneficial. This could include:
    • Confirmation that the release notes build correctly (e.g., format, links).
    • Mention any automated checks for completeness against the changelog or commit history.

Overall: This PR seems aligned with NuttX requirements. Adding more specifics about the CI testing would further strengthen it.

@lupyuen
Copy link
Member

lupyuen commented Oct 2, 2024

Hi @jerpelea: Sorry I believe nuttx/releases/12.7 (a537ecd) has major issues with RISC-V vork() and Signals. Here's my test log:

signest_test: Started interfere_main pid=52
[   55.182000] riscv_exception: EXCEPTION: Instruction page fault. MCAUSE: 000000000000000c, EPC: 0000000000000000, MTVAL: 0000000000000000
[   55.182000] riscv_exception: Segmentation fault in PID 52: ostest

We fixed the RISC-V issues recently. Could you add these to the 12.7 release? Thanks!

@jerpelea
Copy link
Contributor Author

jerpelea commented Oct 2, 2024

@lupyuen they will be added to the release

@jerpelea
Copy link
Contributor Author

jerpelea commented Oct 2, 2024

@lupyuen

#13788
#13789

@xiaoxiang781216 xiaoxiang781216 merged commit 2ec6c6e into apache:releases/12.7 Oct 2, 2024
3 checks passed
@cederom
Copy link
Contributor

cederom commented Oct 2, 2024

12.7 release?? did i miss anything? o_O

@lupyuen
Copy link
Member

lupyuen commented Oct 3, 2024

FYI nuttx/releases/12.7 (10e44f8) was tested successfully with QEMU RISC-V and OSTest for rv-virt:knsh64. Stress Test of rv-virt:knsh64 was also successful for 20 iterations. Thanks! 🎉

Update: Oops we're having build issues with a typo:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Documentation Improvements or additions to documentation Size: L The size of the change in this PR is large
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants