Skip to content

Conversation

@AMecea
Copy link
Contributor

@AMecea AMecea commented Dec 6, 2018

fixes #172

This PR also fixes a bug that removes the PVC of pod-0 when the cluster is scaled down to 0.

@AMecea AMecea changed the title [wip] E2e test PVCs E2e test PVCs Dec 7, 2018
@AMecea AMecea requested a review from calind December 7, 2018 12:49
@AMecea AMecea merged commit 8cce817 into master Dec 14, 2018
@AMecea AMecea deleted the e2e-test-pvc branch January 23, 2019 10:22
chapsuk pushed a commit to chapsuk/mysql-operator that referenced this pull request Oct 16, 2023
…e#178)

* Ensure that we allocate a new map to avoiding sharing memory, and copy over for safety for etcd case.

Signed-off-by: Peter Farr <Peter@PrismaPhonic.com>

* Copy resources for vttablet pod since it contains pointers.

Signed-off-by: Peter Farr <Peter@PrismaPhonic.com>

* We also use resources type for mysqld container.

Signed-off-by: Peter Farr <Peter@PrismaPhonic.com>

* Safeguard everywhere else we use resources.

Signed-off-by: Peter Farr <Peter@PrismaPhonic.com>

* Remove return so we fall through appropriately.

Signed-off-by: Peter Farr <Peter@PrismaPhonic.com>
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.

Add e2e tests for PVC cleanup

3 participants