-
Notifications
You must be signed in to change notification settings - Fork 138
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
Update the documents for the release 1.3.0 #88
Conversation
README.md
Outdated
@@ -301,8 +302,8 @@ It is not possible to use different credentials for additional Backup Storage Lo | |||
|
|||
### Prerequisites | |||
|
|||
* Velero 1.6.0 or later | |||
* AWS plugin must be installed, either at install time, or by running `velero plugin add velero/velero-plugin-for-aws:v1.2.1` | |||
* Velero 1.7.0 or later |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't be updated, the addition bsl with their own credential is supported from v1.6.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it, thanks
README.md
Outdated
* Velero 1.6.0 or later | ||
* AWS plugin must be installed, either at install time, or by running `velero plugin add velero/velero-plugin-for-aws:v1.2.1` | ||
* Velero 1.7.0 or later | ||
* AWS plugin must be installed, either at install time, or by running `velero plugin add velero/velero-plugin-for-aws:v1.3.0` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here, maybe we can use the same style as vmware-tanzu/velero-plugin-for-microsoft-azure#107 ?
changelogs/CHANGELOG-1.3.0.md
Outdated
## All changes | ||
|
||
This release contains no user facing changes but includes fixes for Change the base image to distroless. | ||
CVE-2021-3121 and CVE-2021-3580 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm curious about it that the PR for cve was just merged a few hours ago
changelogs/CHANGELOG-1.3.0.md
Outdated
@@ -0,0 +1,4 @@ | |||
## All changes | |||
|
|||
This release contains no user facing changes but includes fixes for Change the base image to distroless. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change
-> changing
f440355
to
68abc21
Compare
Signed-off-by: danfengl <[email protected]>
68abc21
to
3f42d5d
Compare
Update the documents for the release 1.3.0
Signed-off-by: danfengl [email protected]