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

Fix annotations in puppetdb-pvc and puppetserver-pvc #149

Merged
merged 5 commits into from
Jan 27, 2023

Conversation

akiryushin
Copy link
Contributor

No description provided.

@akiryushin akiryushin requested review from a team, Xtigyro and davidphay as code owners January 17, 2023 08:22
@CLAassistant
Copy link

CLAassistant commented Jan 17, 2023

CLA assistant check
All committers have signed the CLA.

@akiryushin akiryushin changed the title Fix annotations in puppetdb-pvc Fix annotations in puppetdb-pvc and puppetserver-pvc Jan 17, 2023
@davidphay
Copy link
Collaborator

Thanks for the PR! can you please sign the CLA & bump the Chart.yaml version, update the changelog.md & You might want to include your name in the Credits?

Xtigyro

This comment was marked as outdated.

@akiryushin akiryushin force-pushed the st_annotation_fix branch 3 times, most recently from d258dde to 31279b7 Compare January 17, 2023 12:00
@akiryushin
Copy link
Contributor Author

Done

@davidphay
Copy link
Collaborator

I add 2 suggestions, after that it will be ok for me.

@davidphay
Copy link
Collaborator

@akiryushin Hello, you have to rebase your branch, many commit has been added.

@akiryushin
Copy link
Contributor Author

@davidphay done

@davidphay
Copy link
Collaborator

@akiryushin in templates/puppetserver-pvc.yaml there still an issue.

@akiryushin
Copy link
Contributor Author

@davidphay sorry, didn't see. Check it

Copy link
Collaborator

@davidphay davidphay left a comment

Choose a reason for hiding this comment

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

Thanks @akiryushin !

CHANGELOG.md Outdated
@@ -5,6 +5,9 @@ numbering uses [semantic versioning](http://semver.org).

NOTE: The change log until version `v0.2.4` is auto-generated.

## [v7.1.1](https://github.com/puppetlabs/puppetserver-helm-chart/tree/v7.0.0) (2023-01-xx)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
## [v7.1.1](https://github.com/puppetlabs/puppetserver-helm-chart/tree/v7.0.0) (2023-01-xx)
## [v7.1.1](https://github.com/puppetlabs/puppetserver-helm-chart/tree/v7.2.2) (2023-01-18)

Please update the date here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

late?

templates/puppetserver-pvc.yaml Outdated Show resolved Hide resolved
@davidphay davidphay merged commit 2d814c1 into puppetlabs:master Jan 27, 2023
davidphay pushed a commit to davidphay/puppetserver-helm-chart that referenced this pull request Jan 27, 2023
* Fix annotations in puppetdb-pvc
* Fix puppetserver-pvc annotation
* Add akiryushin as Contributor
* Bump chart
* Fix puppetserver-pvc
@akiryushin akiryushin deleted the st_annotation_fix branch January 30, 2023 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants