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.

Sign up for GitHub

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

Jump to bottom

Deleting helm chart exits with error #35

Closed
asifdxtreme opened this issue Mar 21, 2019 · 2 comments
Closed

Deleting helm chart exits with error #35

asifdxtreme opened this issue Mar 21, 2019 · 2 comments

Comments

@asifdxtreme
Copy link
Contributor

Is this a BUG REPORT or FEATURE REQUEST?:

/kind bug

What happened:

While deleting helm chart we get an error and all crds still exist.

# helm delete sid
Error: deletion completed with 1 error(s): mutatingwebhookconfigurations.admissionregistration.k8s.io "sid-mutate-job" already exists

because of which for deploying it next time we need to delete all crd's and then deploy again

What you expected to happen:
Delete helm chart should exit properly
@TommyLike
Copy link
Contributor

IMO, this could be an issue of helm, beside this, the webhooks and crds are all not eliminated during helm delete operation, try this command below:

helm delete <name> --purge

@TommyLike
Copy link
Contributor

reopen this if we need considering deleting chart without purge option

kevin-wangzefeng pushed a commit to kevin-wangzefeng/volcano that referenced this issue Jun 28, 2019
@volcano-sh-bot
Merge pull request volcano-sh#35 from lminzhw/performance_improvement
a92572f 
Performance improvement
kevin-wangzefeng pushed a commit to kevin-wangzefeng/volcano that referenced this issue Jun 28, 2019
@volcano-sh-bot
Merge pull request volcano-sh#35 from lminzhw/performance_improvement
e09c367 
Performance improvement
yolgun added a commit to yolgun/volcano that referenced this issue Sep 16, 2022
@yolgun
fix waitForFirstConsumer support (volcano-sh#35)
ef7ea7a 
chery picked commits from

- volcano-sh#1984
- volcano-sh#1796

Co-authored-by: Yunus Olgun <yunuso@spotify.com>
yolgun added a commit to yolgun/volcano that referenced this issue Oct 24, 2022
@yolgun
Revert "fix waitForFirstConsumer support (volcano-sh#35)"
382ad2a 
This reverts commit ef7ea7a
yolgun added a commit to yolgun/volcano that referenced this issue Oct 24, 2022
@yolgun
Yolgun/revert wait for first consumer (volcano-sh#42)
380f81e 
* Revert "fix scheduler panic issue (volcano-sh#39)"

This reverts commit 176de8f.

* Revert "fix waitForFirstConsumer support (volcano-sh#35)"

This reverts commit ef7ea7a

Co-authored-by: Yunus Olgun <yunuso@spotify.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TommyLike @asifdxtreme