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

[velero] Add helm annotation for velero.io/v1 schedule CRD #153

Merged
merged 3 commits into from
Oct 7, 2020
Merged

[velero] Add helm annotation for velero.io/v1 schedule CRD #153

merged 3 commits into from
Oct 7, 2020

Conversation

jenting
Copy link
Collaborator

@jenting jenting commented Aug 31, 2020

Special notes for your reviewer:

fixes #151
fixes #152
fixes #160
fixes #165
fixes #167

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • DCO signed
  • Chart Version bumped
  • Variables are documented in the README.md
  • Title of the PR starts with chart name (e.g. [velero])

@jenting jenting self-assigned this Aug 31, 2020
@jenting jenting marked this pull request as ready for review August 31, 2020 04:17
@jenting
Copy link
Collaborator Author

jenting commented Aug 31, 2020

@namm2, @reddiablo85 PTAL 🙇

@namm2
Copy link

namm2 commented Aug 31, 2020

I tested and it works fine. Thanks for the PR.

carlisia
carlisia previously approved these changes Sep 3, 2020
Copy link
Contributor

@carlisia carlisia left a comment

Choose a reason for hiding this comment

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

👍 thank you for this fix!

@carlisia
Copy link
Contributor

carlisia commented Sep 3, 2020

@jenting this will need a version bump please.

@jenting
Copy link
Collaborator Author

jenting commented Sep 21, 2020

@carlisia @ashish-amarnath @nrb @cpanato PTAL 🙇

@@ -2,7 +2,7 @@ apiVersion: v1
appVersion: 1.4.2
description: A Helm chart for velero
name: velero
version: 2.12.17
version: 2.12.18
Copy link
Contributor

@cpanato cpanato Sep 22, 2020

Choose a reason for hiding this comment

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

this looks like a new feature I would bump to 2.13.0

Copy link
Collaborator Author

@jenting jenting Sep 22, 2020

Choose a reason for hiding this comment

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

It's a bug fixing, lots of issues have been reported recently and I think we missed the annotation to kind: Schedule at PR #109. @cpanato

Copy link

Choose a reason for hiding this comment

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

This is just a bug fix. I hope it could be merged as soon as possible.

Choose a reason for hiding this comment

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

also would like to see it merged 👍 🙏

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@cpanato could you please take a look at this PR, thanks 🙇

@jenting jenting requested a review from cpanato September 24, 2020 01:04
cpanato
cpanato previously approved these changes Sep 25, 2020
JenTing Hsiao added 3 commits October 5, 2020 15:55
Signed-off-by: JenTing Hsiao <[email protected]>
Signed-off-by: JenTing Hsiao <[email protected]>
Copy link
Contributor

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

thanks for this!
lgtm

@jenting jenting requested a review from zubron October 6, 2020 08:57
Copy link
Contributor

@carlisia carlisia left a comment

Choose a reason for hiding this comment

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

👍 thank you!

@carlisia carlisia merged commit ad9670e into vmware-tanzu:main Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
6 participants