puppet-slurm icon indicating copy to clipboard operation
puppet-slurm copied to clipboard

Archive failing in download.pp

Open trenta opened this issue 3 years ago • 2 comments

Hi,

Doing a fresh install for testing on RHEL 8 and having trouble with the archive part in https://github.com/ULHPC/puppet-slurm/blob/v1.4.3/manifests/download.pp

It downloads the file to /tmp and fails when trying to move it to /usr/local/src

Here are notice statements showing that the user and group variables are working

Notice: $archive returns slurm-20.11.8.tar.bz2
Notice: $path returns /usr/local/src/slurm-20.11.8.tar.bz2
Notice: $url returns https://download.schedmd.com/slurm//slurm-20.11.8.tar.bz2
Notice: $slurm::params::username returns slurm
Notice: $slurm::params::group returns slurm
Notice: $checksum_type returns sha1
Notice: $checksum_veriry returns (false or ! empty(dcf328865591b42b6c8f5586b3e396e6eb30dcd7))
Notice: $checksum returns dcf328865591b42b6c8f5586b3e396e6eb30dcd7

The puppet error is

Error: Could not set 'present' on ensure: wrong number of arguments (given 1, expected 0) (file: /etc/puppetlabs/code/environments/slurm_upgrade/modules/slurm/manifests/download.pp, line: 74)
Error: Could not set 'present' on ensure: wrong number of arguments (given 1, expected 0) (file: /etc/puppetlabs/code/environments/slurm_upgrade/modules/slurm/manifests/download.pp, line: 74)
Wrapped exception:
wrong number of arguments (given 1, expected 0)
Error: /Stage[main]/Slurm::Install/Slurm::Download[20.11.8]/Archive[slurm-20.11.8.tar.bz2]/ensure: change from 'absent' to 'present' failed: Could not set 'present' on ensure: wrong number of arguments (given 1, expected 0) (file: /etc/puppetlabs/code/environments/slurm_upgrade/modules/slurm/manifests/download.pp, line: 74)

Interestingly the pmix download with the same archive statement works fine.

trenta avatar Jul 15 '21 04:07 trenta

Also the docs indicate the default checksum type is md5 where sha1 is actually used.

trenta avatar Jul 15 '21 05:07 trenta

Also the docs and the comments in init.pp suggest the default src_checksum is ''. When I manually set that in the slurm class everything works fine.

trenta avatar Jul 15 '21 05:07 trenta