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 yunikorn-history-server charts #132

Conversation

ljubon
Copy link
Collaborator

@ljubon ljubon commented Jul 15, 2024

TODO:

  • When GitHub app is created for YHS add the values to the change in this PR

Goal of this PR is to allow YHS to publish it's chart to official G-Research repository

YHS release PR to publish change: G-Research/unicorn-history-server#101

pavlovic-ivan
pavlovic-ivan previously approved these changes Jul 15, 2024
Copy link
Contributor

@pavlovic-ivan pavlovic-ivan left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @ljubon! 🚀 Let's wait for that key fingerprint

@ljubon ljubon force-pushed the 779-publish-yhs-repo-chart-to-g-researchcharts branch from b02d6de to 9e9af0e Compare July 15, 2024 14:26
@ljubon
Copy link
Collaborator Author

ljubon commented Jul 15, 2024

LGTM. Thanks @ljubon! 🚀 Let's wait for that key fingerprint

Thanks to @dave-gantenbein we have new app created i just updated SHA
@pavlovic-ivan please approve/review when possible 🫡

Copy link
Contributor

@pavlovic-ivan pavlovic-ivan left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @ljubon! 🚀

@pavlovic-ivan pavlovic-ivan merged commit 390e99c into G-Research:master Jul 15, 2024
1 check passed
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.

2 participants