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

feat: add helm chart for kubeadmiral #325

Merged
merged 1 commit into from
Jun 17, 2024
Merged

Conversation

qclc
Copy link
Collaborator

@qclc qclc commented Jun 5, 2024

No description provided.

@qclc qclc force-pushed the feat-add-helm branch 2 times, most recently from 57ae9c0 to d88585b Compare June 5, 2024 06:33
@@ -0,0 +1,62 @@
---
Copy link
Member

@SOF3 SOF3 Jun 5, 2024

Choose a reason for hiding this comment

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

Are these files copied from the repo? Should we have a separate script to build the helm chart in CI and copy the CRD files to this directory in the CI instead of committing all files twice?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

There is a generated script file(hack/generate-groups.sh) for CRD, which has been updated here. It will be synchronized and updated to the Helm chart directory every time CRD is generated.

Copy link
Member

Choose a reason for hiding this comment

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

If you insist to commit the generated files twice, please add the new paths to .gitattributes to mark them as generated.

Copy link
Member

Choose a reason for hiding this comment

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

and typos.toml as well

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The .gitattributes and typos.toml has already been updated.

@qclc qclc force-pushed the feat-add-helm branch 3 times, most recently from 05a2433 to fced5da Compare June 14, 2024 07:11
Comment on lines +30 to +33
- /bin/sh
- -c
- |
bash <<'EOF'
Copy link
Member

Choose a reason for hiding this comment

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

Any particular reason we can't just [/bin/bash, -c, "set -ex... EOF"] here?

@mrlihanbo mrlihanbo merged commit 24911a2 into kubewharf:main Jun 17, 2024
5 of 6 checks 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.

3 participants