scylladb / scylla-machine-image

Apache License 2.0
18 stars 25 forks source link

packer-shell: replace semicolon separators with double ampersand #465

Closed benipeled closed 11 months ago

benipeled commented 11 months ago

As part of investigating another issue (missing linux-azure-lts package) I found out that the update-kernel shell commands are set with semicolon separators which means all commands will run regardless of the results of previous commands,

This commits change it to be separated by double ampersand to make sure commands will be run only if the previous command is done successfully

benipeled commented 11 months ago

Verified with 3 different https://jenkins.scylladb.com/job/scylla-master/job/releng-testing/job/azure-image builds and 2 different https://jenkins.scylladb.com/job/scylla-master/job/releng-testing/job/next-machine-image/ builds (which for some reason didn't fail on the other issue of missing linux-azure-lts)