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

Missing cascade delete for host_storages #16440

Merged

Conversation

Ladas
Copy link
Contributor

@Ladas Ladas commented Nov 10, 2017

Missing cascade delete for host_storages, leading to orphaned
records.

Fixes BZ:
https://bugzilla.redhat.com/show_bug.cgi?id=1508910

Missing cascade delete for host_storages, leading to orphaned
records.

Fixes BZ:
https://bugzilla.redhat.com/show_bug.cgi?id=1508910
@Ladas Ladas force-pushed the missing_cascade_delete_for_host_storages branch from eb5c586 to 85c0224 Compare November 10, 2017 15:05
@Ladas
Copy link
Contributor Author

Ladas commented Nov 10, 2017

@miq-bot assign @agrare

@miq-bot
Copy link
Member

miq-bot commented Nov 10, 2017

Checked commit Ladas@85c0224 with ruby 2.3.3, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@Ladas Ladas closed this Nov 13, 2017
@Ladas Ladas reopened this Nov 13, 2017
@agrare agrare merged commit b872a1f into ManageIQ:master Nov 13, 2017
@agrare agrare added this to the Sprint 73 Ending Nov 13, 2017 milestone Nov 13, 2017
simaishi pushed a commit that referenced this pull request Nov 14, 2017
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 1394df1cdc73bf65ef91885d09eb2deae78a8adf
Author: Adam Grare <agrare@redhat.com>
Date:   Mon Nov 13 08:44:29 2017 -0500

    Merge pull request #16440 from Ladas/missing_cascade_delete_for_host_storages
    
    Missing cascade delete for host_storages
    (cherry picked from commit b872a1f3eb36c0f7f40467beab2578ee2953e647)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1513118

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants