Problem/Motivation

  • When a tenant is deleted, clean up all OpenStack resources. This includes all OpenStack Instances, Networks, Volumes...etc... as well as OpenStack project, roles and users.

Issue fork facade-3352245

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

baldwinlouie created an issue. See original summary.

baldwinlouie opened merge request !22

baldwinlouie’s picture

baldwinlouie commented
Status: Active » Needs review

@yas, please review this patch. This is the cleanup code that will run during tenant deletion. It cleans up all of a user's resources, and then deletes the actual project, role and user.

I also added a "shared" field to the projects to identify a project that is shared from another user. Shared projects do not get deleted.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Needs review » Needs work

@baldwinlouie

Thank you for the patch. I posted my minor comment. Thanks

baldwinlouie’s picture

baldwinlouie commented

@yas, I don't see your comments. Can you please post it again?

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Issue summary: View changes

@baldwinlouie

Sorry, I haven't checked out my comment. Can you check it? Thanks

baldwinlouie’s picture

baldwinlouie commented
Status: Needs work » Needs review

@yas, please review this patch.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Needs review » Reviewed & tested by the community

@baldwinlouie

Thank you for the update. It looks good to me now. I'll merge the patch to 1.0.x-dev, and close this issue as Fixed.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Title: Clean up OpenStack resources when a tenant is deleted » Add the function to clean up OpenStack resources when a tenant is deleted

  • yas committed 567b8401 on 1.0.x authored by baldwinlouie 
    Issue #3352245 by baldwinlouie, yas: Add the function to clean up...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.