Skip to content

Commit

Permalink
Drop redundant cinder resource dependencies
Browse files Browse the repository at this point in the history
cinder-backup does not really need cinder-api.

Also now cinder_type resource automatically requires that all cinder
services are up.

Change-Id: I84ad7d0b6035d6234edd692a3096859bd0fb6648
  • Loading branch information
kajinamit committed Oct 25, 2024
1 parent a55614c commit abd01d8
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,5 @@
cinder_type { 'iscsi':
ensure => present,
properties => {'volume_backend_name' => 'lvm'},
require => Class['cinder::api'],
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,5 @@
cinder_type { $netapp_backend_name:
ensure => present,
properties => {'volume_backend_name' => $netapp_backend_name},
require => Class['cinder::api'],
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,5 @@
cinder_type { 'nfs':
ensure => present,
properties => {'volume_backend_name' => 'nfs'},
require => Class['cinder::api'],
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,5 @@
cinder_type { $solidfire_backend_name:
ensure => present,
properties => {'volume_backend_name' => $solidfire_backend_name },
require => Class['cinder::api'],
}
}
2 changes: 0 additions & 2 deletions packstack/puppet/modules/packstack/manifests/cinder/backup.pp
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,4 @@
backup_swift_url => "http://${cinder_backup_conf_ctrl_host}:8080/v1/AUTH_",
backup_swift_service_auth => true
}

Class['cinder::api'] ~> Service['cinder-backup']
}

0 comments on commit abd01d8

Please sign in to comment.