Skip to content

Commit 2b3964f

Browse files
committed
Be sure to have dcv installed at runtime
The dcv_install recipe is not called at runtime, because it is included in the default.rb, but it is called only at AMI creation time. At runtime the entry point is the <scheduler>_config, that calls base_config and base_install, so the dcv_install was not included. From now we are calling the dcv_install at runtime case too, the recipe is idempotent so it is not a problem. Signed-off-by: Enrico Usai <[email protected]>
1 parent 41b4468 commit 2b3964f

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

recipes/dcv_config.rb

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,10 @@ def allow_gpu_acceleration
5959
end
6060

6161
if node['platform'] == 'centos' && node['platform_version'].to_i == 7 && node['cfncluster']['cfn_node_type'] == "MasterServer"
62+
63+
# be sure to have DCV packages installed
64+
include_recipe "aws-parallelcluster::dcv_install"
65+
6266
node.default['cfncluster']['dcv']['is_graphic_instance'] = graphic_instance?
6367

6468
if node.default['cfncluster']['dcv']['is_graphic_instance']

0 commit comments

Comments
 (0)