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

Update controller-util, fixes cascade deletion issue #221

Merged
merged 2 commits into from
Feb 1, 2019

Conversation

AMecea
Copy link
Contributor

@AMecea AMecea commented Feb 1, 2019

This PR updates controller-util to version v0.1.8 which contains a fix for cascade deletion.

Copy link
Member

@calind calind left a comment

Choose a reason for hiding this comment

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

LGTM

@AMecea AMecea force-pushed the update-controller-util branch from 5f06d22 to 5ebafe1 Compare February 1, 2019 11:05
@AMecea AMecea merged commit 99618ea into master Feb 1, 2019
@AMecea AMecea deleted the update-controller-util branch February 11, 2019 14:55
@AMecea AMecea restored the update-controller-util branch February 11, 2019 14:55
@AMecea AMecea deleted the update-controller-util branch February 11, 2019 14:56
@AMecea AMecea added this to the 0.2.4 milestone Feb 13, 2019
chapsuk pushed a commit to chapsuk/mysql-operator that referenced this pull request Oct 16, 2023
In order for dynamic local storage provisioners to influence pod
scheduling for future PV creation, ensure the vtbackup data PVC exists
before the Pod.

This matches the order in reconcile_tablets.go.

Decouple the orphan Pod detection for pods of all phases (Pending,
Running, Succeeded, Failed, Unknown). Ensure that temporary node
failures marking pods as Failed do not inadvertently delete PVCs until
reconciling a termination of the Pod.

Fixes: bitpoke#221

Signed-off-by: Sean Treadway <[email protected]>
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