#13 [lookaside_upload_sig] Fix log message to reflect reality
Closed 2 years ago by arrfab. Opened 2 years ago by chantra.
chantra/centos-git-common fix_log  into  master

file modified
+1 -1
@@ -91,7 +91,7 @@ 

  f_log "Source to upload : ${file} "

  f_log "Hash parameter : ${hash_parameter}"

  f_log "Package name: $pkgname"

- f_log "sha1sum: ${checksum}"

+ f_log "${hash_parameter}sum: ${checksum}"

  f_log " ====== Trying to upload ======="

  echo ""

  

The log message was reporting sha1sum regardless of what actualy shaX
was used.

Tested by uploading a new src to the lookaside

../centos-git-common/lookaside_upload_sig -f createrepo_c-0.20.1.tar.gz -n createrepo_c
Base URL set to default: https://git.centos.org
[+] CentOS Lookaside upload tool -> Checking if file already uploaded
[+] CentOS Lookaside upload tool -> Initialing new upload to lookaside
[+] CentOS Lookaside upload tool -> URL : https://git.centos.org
[+] CentOS Lookaside upload tool -> Source to upload : createrepo_c-0.20.1.tar.gz
[+] CentOS Lookaside upload tool -> Hash parameter : sha512
[+] CentOS Lookaside upload tool -> Package name: createrepo_c
[+] CentOS Lookaside upload tool -> sha512sum: 54a2cc7c7cd3f3b9a0c23cd8c136ae1331e7fa7cc995189088e7e6f2276c78b2b84e21c2a2b93f4528b5e9e4018dd6525262c8aaba3bc8a1412a51dfafd101f7
[+] CentOS Lookaside upload tool ->  ====== Trying to upload =======

File createrepo_c-0.20.1.tar.gz size 614968 CHECKSUM 54a2cc7c7cd3f3b9a0c23cd8c136ae1331e7fa7cc995189088e7e6f2276c78b2b84e21c2a2b93f4528b5e9e4018dd6525262c8aaba3bc8a1412a51dfafd101f7 stored OK
[+] CentOS Lookaside upload tool -> Validating that source was correctly uploaded ....
[+] CentOS Lookaside upload tool -> [SUCCESS] Source should be available at https://git.centos.org/sources/createrepo_c/createrepo_c-0.20.1.tar.gz/sha512/54a2cc7c7cd3f3b9a0c23cd8c136ae1331e7fa7cc995189088e7e6f2276c78b2b84e21c2a2b93f4528b5e9e4018dd6525262c8aaba3bc8a1412a51dfafd101f7/createrepo_c-0.20.1.tar.gz

realizing that it was fixed and we just noticed your PR (fixed by #14 )

Pull-Request has been closed by arrfab

2 years ago
Metadata