search

open-cluster-management-io / api

Core APIs for open cluster management
https://open-cluster-management.io
Apache License 2.0
251 stars 77 forks source link

:seedling: add api upgrade flow #265

Closed haoqing0110 closed 11 months ago

haoqing0110 commented 1 year ago

Summary

Related issue(s)

Fixes #

haoqing0110 commented 1 year ago

cc @zhujian7 we might work together to refine the API migration flow.

haoqing0110 commented 1 year ago

cc @qiujian16 @elgnay

haoqing0110 commented 1 year ago

/assign @deads2k

Hello @deads2k , we have a summary of the API upgrade flow based on our experience in previous releases, hope you can help review and provide some comments, thanks a lot!

qiujian16 commented 1 year ago

also need to check here https://github.com/open-cluster-management-io/community/blob/main/sig-architecture/api_changes.md on what is updated.

haoqing0110 commented 1 year ago

https://github.com/open-cluster-management-io/community/blob/main/sig-architecture/api_changes.md is a useful addition to this flow, append some URLs to the related part.

haoqing0110 commented 11 months ago

@qiujian16 @elgnay @xuezhaojun plz help do a final review.

qiujian16 commented 11 months ago

/approve

@xuezhaojun leave to you for final review

openshift-ci[bot] commented 11 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: haoqing0110, qiujian16

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/open-cluster-management-io/api/blob/main/OWNERS)~~ [qiujian16] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
xuezhaojun commented 11 months ago

/lgtm