Skip to content

Commit

Permalink
Merge pull request #16871 from agrare/child_manager_dependent_destroy
Browse files Browse the repository at this point in the history
Don't dependent => destroy child managers
(cherry picked from commit 38225e1)

https://bugzilla.redhat.com/show_bug.cgi?id=1538240
  • Loading branch information
Fryguy authored and simaishi committed Jan 24, 2018
1 parent d95396d commit bc913ae
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 6 deletions.
3 changes: 1 addition & 2 deletions app/models/mixins/cinder_manager_mixin.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,7 @@ module CinderManagerMixin
has_one :cinder_manager,
:foreign_key => :parent_ems_id,
:class_name => "ManageIQ::Providers::StorageManager::CinderManager",
:autosave => true,
:dependent => :destroy
:autosave => true

delegate :cloud_volumes,
:cloud_volume_snapshots,
Expand Down
3 changes: 1 addition & 2 deletions app/models/mixins/has_network_manager_mixin.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@ module HasNetworkManagerMixin
has_one :network_manager,
:foreign_key => :parent_ems_id,
:class_name => "ManageIQ::Providers::NetworkManager",
:autosave => true,
:dependent => :destroy
:autosave => true

delegate :floating_ips,
:security_groups,
Expand Down
3 changes: 1 addition & 2 deletions app/models/mixins/swift_manager_mixin.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@ module SwiftManagerMixin
has_one :swift_manager,
:foreign_key => :parent_ems_id,
:class_name => "ManageIQ::Providers::StorageManager::SwiftManager",
:autosave => true,
:dependent => :destroy
:autosave => true

delegate :cloud_object_store_containers,
:cloud_object_store_objects,
Expand Down

0 comments on commit bc913ae

Please sign in to comment.