-
Notifications
You must be signed in to change notification settings - Fork 79
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: the latter volume dirs overwrite the prior one #1214
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I assume that configMaps have same problem but not tested yet |
d70c7a0
to
34c2628
Compare
recheck |
Thanks for your feedback! Please sign the CLA by just posting a comment mentioned above and modifying your comment message and PR title accounting to the constraints in the CI. |
Pull Request Test Coverage Report for Build 9869423388Details
💛 - Coveralls |
I have read the CLA Document and I hereby sign the CLA |
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.
lgtm
@vietanhtwdk Thanks a lot for your contribution! This PR looks good to me, and it has been merged. Welcome to become one of the contributors of Kusion 👍 Hope you enjoy this journey : ) |
What type of PR is this?
/kind bug
What this PR does / why we need it:
Fix volumes deployment
Which issue(s) this PR fixes:
Fixes #1213
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., design docs, usage docs, etc.: