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

Add deleting cells automation #846

Merged
merged 9 commits into from
Oct 10, 2024
Merged

Add deleting cells automation #846

merged 9 commits into from
Oct 10, 2024

Commits on Oct 4, 2024

  1. Add remove cell logic

    @mrkisaolamb @gibizer
    mrkisaolamb authored and gibizer committed Oct 4, 2024
    Configuration menu
    Copy the full SHA
    c3082cd View commit details
    Browse the repository at this point in the history

  2. Add job to clear cell from db

    @mrkisaolamb @gibizer
    mrkisaolamb authored and gibizer committed Oct 4, 2024
    Configuration menu
    Copy the full SHA
    d829bf9 View commit details
    Browse the repository at this point in the history

  3. Add kuttle test for cell deletion

    @mrkisaolamb @gibizer
    mrkisaolamb authored and gibizer committed Oct 4, 2024
    Configuration menu
    Copy the full SHA
    b896753 View commit details
    Browse the repository at this point in the history

  4. Correct cell deletion logic

    @mrkisaolamb @gibizer
    mrkisaolamb authored and gibizer committed Oct 4, 2024
    Configuration menu
    Copy the full SHA
    f4765e4 View commit details
    Browse the repository at this point in the history

  5. Move cell-delete test

    @mrkisaolamb @gibizer
    mrkisaolamb authored and gibizer committed Oct 4, 2024
    Configuration menu
    Copy the full SHA
    1f6c1f6 View commit details
    Browse the repository at this point in the history

  6. Delete additional cell delete job 

    We need to delete only nova finalizer
    @mrkisaolamb @gibizer
    mrkisaolamb authored and gibizer committed Oct 4, 2024
    Configuration menu
    Copy the full SHA
    2b92841 View commit details
    Browse the repository at this point in the history

  7. Reorganize kuttl tests for nova-cell-deletion

    @mrkisaolamb @gibizer
    mrkisaolamb authored and gibizer committed Oct 4, 2024
    Configuration menu
    Copy the full SHA
    83c08b0 View commit details
    Browse the repository at this point in the history

Commits on Oct 8, 2024

  1. Delete MariaDBAccount and Database 

    Both CR is created by the nova controller for the given cell, so when
that cell is deleted the CRs also needs to be deleted. We also need to
delete the secret connected to the MariaDBAccount as mariadb-operator
does not delete it today.

It also added graceful handling of NotFound during resource deletion and
refactored the cleanup logic to avoid bugs if the delete needs to be
retried.

This now depends on the
openstack-k8s-operators/mariadb-operator#268
instead of copying that change.
    @gibizer
    gibizer committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    290ffa2 View commit details
    Browse the repository at this point in the history

  2. [kuttl]Assert DB and Secrets are deleted for deleted cell

    @gibizer
    gibizer committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    37f25bd View commit details
    Browse the repository at this point in the history