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

Add upgrade doc to v1.12 #6625

Merged
merged 1 commit into from
Aug 10, 2023
Merged

Conversation

allenxu404
Copy link
Contributor

Thank you for contributing to Velero!

Please add a summary of your change

  • Add upgrade-to-1.12 doc

Does your change fix a particular issue?

Please indicate you've done the following:

  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Created a changelog file or added /kind changelog-not-required as a comment on this pull request.
  • Updated the corresponding documentation in site/content/docs/main.

@allenxu404
Copy link
Contributor Author

/kind changelog-not-required

@github-actions github-actions bot added Documentation Website non-docs changes for the website labels Aug 9, 2023
@allenxu404
Copy link
Contributor Author

/kind changelog-not-required

@github-actions github-actions bot added the kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes label Aug 9, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #6625 (f730362) into main (5f463c5) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is n/a.

❗ Current head f730362 differs from pull request most recent head e070890. Consider uploading reports for the commit e070890 to get more accurate results

@@           Coverage Diff           @@
##             main    #6625   +/-   ##
=======================================
  Coverage   60.17%   60.17%           
=======================================
  Files         242      242           
  Lines       25679    25679           
=======================================
  Hits        15452    15452           
  Misses       9150     9150           
  Partials     1077     1077           

qiuming-best
qiuming-best previously approved these changes Aug 9, 2023
Signed-off-by: allenxu404 <qix2@vmware.com>
velero=velero/velero:v1.11.0 \
velero-plugin-for-aws=velero/velero-plugin-for-aws:v1.7.0 \
velero=velero/velero:v1.12.0 \
velero-plugin-for-aws=velero/velero-plugin-for-aws:v1.8.0 \
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
velero-plugin-for-aws=velero/velero-plugin-for-aws:v1.8.0 \
velero-plugin-for-aws=velero/velero-plugin-for-aws:v1.7.1 \

double checked the tags, don't see v1.8 anywhere. Could you help me understand where you got 1.8 version from?

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

@kaovilai 1.7 was released with velero 1.11. 1.8 will be released with 1.12 (Velero 1.12.0 doesn't exist yet either)

Copy link
Contributor

Choose a reason for hiding this comment

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

from discussions I've received with a maintainer, bumping Y version in plugins is typical.

@qiuming-best qiuming-best merged commit 543d8d5 into vmware-tanzu:main Aug 10, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation kind/changelog-not-required PR does not require a user changelog. Often for docs, website, or build changes Website non-docs changes for the website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants