Skip to content

fix broken source URL for AOCC 4.0.0#17794

Merged
Micket merged 1 commit intoeasybuilders:developfrom
kserradell:aocc_link
Apr 28, 2023
Merged

fix broken source URL for AOCC 4.0.0#17794
Micket merged 1 commit intoeasybuilders:developfrom
kserradell:aocc_link

Conversation

@kserradell
Copy link
Copy Markdown

@kserradell kserradell commented Apr 25, 2023

It seems the current link does not work and has been replaced.

== 2023-04-25 09:38:03,924 filetools.py:842 WARNING IOError occurred while trying to download http://developer.amd.com/wordpress/media/files/aocc-compiler-4.0.0.tar to /gxfs/apps/easybuild/sources/a/AOCC/aocc-compiler-4.0.0.tar: The read operation timed out

'https://download.amd.com/developer/eula/aocc-compiler/' works well. The file and checksum are the same.

@verdurin
Copy link
Copy Markdown
Member

verdurin commented Apr 28, 2023

Test report by @verdurin
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
easybuild-c7.novalocal - Linux CentOS Linux 7.9.2009, x86_64, Intel Xeon Processor (Skylake, IBRS), Python 3.6.8
See https://gist.github.com/verdurin/3cc853e07b8b8f77fe7b92fd8330a2aa for a full test report.

@verdurin
Copy link
Copy Markdown
Member

verdurin commented Apr 28, 2023

Test report by @verdurin
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
easybuild-c7.novalocal - Linux CentOS Linux 7.9.2009, x86_64, Intel Xeon Processor (Skylake, IBRS), Python 3.6.8
See https://gist.github.com/verdurin/741e8606f18d6a73a83654f36e2a826b for a full test report.

@verdurin
Copy link
Copy Markdown
Member

verdurin commented Apr 28, 2023

@boegelbot please test @ generoso
EB_ARGS=" --accept-eula-for=AOCC "

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Apr 28, 2023

@verdurin: Request for testing this PR well received on login1

PR test command 'EB_PR=17794 EB_ARGS=" --accept-eula-for=AOCC " EB_CONTAINER= /opt/software/slurm/bin/sbatch --job-name test_PR_17794 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 10722

Test results coming soon (I hope)...

Details

- notification for comment with ID 1527796871 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Copy Markdown
Collaborator

boegelbot commented Apr 28, 2023

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns1 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/boegelbot/95fdcebc8d08c9be66ebc776b1ff41ba for a full test report.

@Micket Micket added the bug fix label Apr 28, 2023
@Micket Micket added this to the next release (4.7.2) milestone Apr 28, 2023
Micket
Micket approved these changes Apr 28, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Apr 28, 2023

Test report by @Micket
SUCCESS
Build succeeded for 18 out of 18 (1 easyconfigs in total)
vera-c1 - Linux Rocky Linux 8.6, x86_64, Intel Xeon Processor (Skylake), Python 3.6.8
See https://gist.github.com/Micket/e2c4159cb1316e99bbe88e65a05e7163 for a full test report.

@Micket
Copy link
Copy Markdown
Contributor

Micket commented Apr 28, 2023

Going in, thanks @kserradell!

@Micket Micket merged commit 254f5dd into easybuilders:develop Apr 28, 2023
@kserradell
Copy link
Copy Markdown
Author

kserradell commented Apr 30, 2023

Thanks @Micket!

@boegel boegel changed the title Replacing AOCC broken link fix broken source URL for AOCC 4.0.0 May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Micket Micket Micket approved these changes

Assignees

No one assigned

Labels

bug fix

Projects

None yet

Milestone

4.7.2

Development

Successfully merging this pull request may close these issues.

5 participants

@kserradell @verdurin @boegelbot @Micket @kserradell-meteomatics