openstack/puppet-horizon
Code Issues Proposed changes

Merge "Use array to define command line"

Browse Source
This commit is contained in:
Zuul
2025-08-25 19:32:28 +00:00
committed by Gerrit Code Review
parent 8021428ed1 a7de6c0a86
commit 4e56906713
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/init.pp
View File

@@ -776,7 +776,7 @@ class horizon (
if $compress_offline { if $compress_offline {
if $facts['os']['family'] == 'Debian' { if $facts['os']['family'] == 'Debian' {
exec { 'refresh_horizon_django_compress': exec { 'refresh_horizon_django_compress':
command => "${horizon::params::manage_py} compress --force", command => [$horizon::params::manage_py, 'compress', '--force'],
refreshonly => true, refreshonly => true,
tag => ['horizon-compress'], tag => ['horizon-compress'],
} }

2
spec/classes/horizon_init_spec.rb
View File

@@ -790,7 +790,7 @@ describe 'horizon' do
shared_examples_for 'horizon on Debian' do shared_examples_for 'horizon on Debian' do
it 'refreshes horizon django cache' do it 'refreshes horizon django cache' do
is_expected.to contain_exec('refresh_horizon_django_compress').with({ is_expected.to contain_exec('refresh_horizon_django_compress').with({
:command => '/usr/share/openstack-dashboard/manage.py compress --force', :command => ['/usr/share/openstack-dashboard/manage.py', 'compress', '--force'],
:refreshonly => true, :refreshonly => true,
}) })
is_expected.to contain_concat(platforms_params[:config_file]).that_notifies('Exec[refresh_horizon_django_compress]') is_expected.to contain_concat(platforms_params[:config_file]).that_notifies('Exec[refresh_horizon_django_compress]')