Skip to content

Conversation

ritazh
Copy link
Collaborator

@ritazh ritazh commented Aug 5, 2025

Reason for Change:

Requirements

  • added unit tests and e2e tests (if applicable).

Issue Fixed:

Fixes #1298
Related: #1299

Notes for Reviewers:

Signed-off-by: Rita Zhang <rita.z.zhang@gmail.com>
Copy link

kaito-pr-agent bot commented Aug 5, 2025

Title

Add detailed release management documentation


Description

  • Added comprehensive release management documentation

  • Covered versioning, supported releases, and upgrades


Changes walkthrough 📝

Relevant files
Documentation
Release_Management.md
Create release management documentation                                   

docs/Release_Management.md

  • Created new file with overview of KAITO release management
  • Included sections on versioning, milestones, and patch releases
  • Described supported releases and Kubernetes compatibility
  • Added information on upgrades and acknowledged sources
  • +68/-0   

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • Copy link

    kaito-pr-agent bot commented Aug 5, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Clarification Needed

    The statement "No plan to move to 1.0.0 unless there is a major design change like an incompatible API change in the project" might need clarification. It's unclear what constitutes a "major design change" or an "incompatible API change." Consider providing more details or examples.

    No plan to move to 1.0.0 unless there is a major design change like an incompatible API change in the project
    


    **Major Releases**

    No plan to move to 1.0.0 unless there is a major design change like an incompatible API change in the project
    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    Should we move to 1.0.0 when we graduate the API from v1beta1 to v1?

    Copy link
    Collaborator Author

    Choose a reason for hiding this comment

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

    IMO we should create a separate issue to track KAITO v1 graduation, which should include graduation criteria and blockers. In general, we should decouple API graduations from KAITO core maturity (e.g. Kubernetes versions are tracked separately from Kubernetes resources' APIs) as there are different features in KAITO and they will/should have their own version and graduation status. e.g. workspace and raengine

    Copy link
    Collaborator Author

    Choose a reason for hiding this comment

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

    @Fei-Guo thoughts?

    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    I don't have strong opinion here. Either works for me.


    **Minor Releases**

    - X.Y.0-rc.W, W >= 0 (Branch: release-X.Y)
    Copy link
    Collaborator Author

    Choose a reason for hiding this comment

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

    IMO rc releases are good to allow users to test and give feedback to ensure new releases at least get soaked for 1 week before official releases are cut.

    Copy link
    Collaborator Author

    Choose a reason for hiding this comment

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

    @chewong @Fei-Guo thoughts? if we agree, this needs to be part of the release process

    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    Works for me. Currently, without release branches, we have to hold off from merging certain PRs to the main branch until the release is cut. With release branches, we can merge PRs to main branch without affecting upcoming release.

    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    I am ok as long as RC is not mandatory per release.

    Signed-off-by: Rita Zhang <rita.z.zhang@gmail.com>
    @ritazh ritazh force-pushed the docs-release-management branch from b727f75 to 9c21131 Compare August 6, 2025 18:47
    - **Breaking changes** refer to schema changes, flag changes, and behavior changes of KAITO that may require a clean installation during upgrade, and it may introduce changes that could break backward compatibility.
    - **Milestone** should be designed to include feature sets to accommodate 2 months release cycles including test gates. GitHub's milestones are used by maintainers to manage each release. PRs and Issues for each release should be created as part of a corresponding milestone.
    - **Patch releases** refer to applicable fixes, including security fixes, may be backported to support releases, depending on severity and feasibility.
    - **Test gates** should include soak tests and upgrade tests from the last minor version.
    Copy link
    Collaborator Author

    Choose a reason for hiding this comment

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

    We need to create an issue to do this as part of CI as part of the release job.

    Copy link
    Collaborator

    Choose a reason for hiding this comment

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

    #1366 added.

    @Fei-Guo Fei-Guo merged commit 5fe9a44 into kaito-project:main Aug 6, 2025
    12 checks passed
    @ritazh ritazh deleted the docs-release-management branch August 6, 2025 19:51
    farooqameen pushed a commit to farooqameen/kaito that referenced this pull request Aug 7, 2025
    **Reason for Change**:
    <!-- What does this PR improve or fix in KAITO? Why is it needed? -->
    
    **Requirements**
    
    - [ ] added unit tests and e2e tests (if applicable).
    
    **Issue Fixed**:
    <!-- If this PR fixes GitHub issue 4321, add "Fixes #4321" to the next
    line. -->
    
    Fixes kaito-project#1298 
    Related: kaito-project#1299 
    
    **Notes for Reviewers**:
    
    ---------
    
    Signed-off-by: Rita Zhang <rita.z.zhang@gmail.com>
    farooqameen pushed a commit to farooqameen/kaito that referenced this pull request Aug 7, 2025
    **Reason for Change**:
    <!-- What does this PR improve or fix in KAITO? Why is it needed? -->
    
    **Requirements**
    
    - [ ] added unit tests and e2e tests (if applicable).
    
    **Issue Fixed**:
    <!-- If this PR fixes GitHub issue 4321, add "Fixes #4321" to the next
    line. -->
    
    Fixes kaito-project#1298 
    Related: kaito-project#1299 
    
    **Notes for Reviewers**:
    
    ---------
    
    Signed-off-by: Rita Zhang <rita.z.zhang@gmail.com>
    farooqameen pushed a commit to farooqameen/kaito that referenced this pull request Aug 7, 2025
    **Reason for Change**:
    <!-- What does this PR improve or fix in KAITO? Why is it needed? -->
    
    **Requirements**
    
    - [ ] added unit tests and e2e tests (if applicable).
    
    **Issue Fixed**:
    <!-- If this PR fixes GitHub issue 4321, add "Fixes #4321" to the next
    line. -->
    
    Fixes kaito-project#1298
    Related: kaito-project#1299
    
    **Notes for Reviewers**:
    
    ---------
    
    Signed-off-by: Rita Zhang <rita.z.zhang@gmail.com>
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    Status: Done
    Development

    Successfully merging this pull request may close these issues.

    [Docs] for release management
    3 participants