Skip to content

Update URLs for test data for WRF to https#2335

Merged
boegel merged 1 commit intoeasybuilders:developfrom
dithwick:wps_fix_test_urls
Feb 17, 2021
Merged

Update URLs for test data for WRF to https#2335
boegel merged 1 commit intoeasybuilders:developfrom
dithwick:wps_fix_test_urls

Conversation

@dithwick
Copy link
Copy Markdown
Contributor

This change just updates the urls for the test data to https as they've now moved.

@boegel boegel changed the title Updated urls for test data to https. Updated urls for test data for WRF to https Feb 17, 2021
@boegel boegel added this to the 4.3.3 (next release) milestone Feb 17, 2021
@boegel boegel added the bug fix label Feb 17, 2021
@boegel
Copy link
Copy Markdown
Member

boegel commented Feb 17, 2021

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS WPS-4.0.2-foss-2018b-dmpar.eb
  • SUCCESS WPS-4.1-intel-2019b-dmpar.eb

Build succeeded for 2 out of 2 (2 easyconfigs in total)
node3108.skitty.os - Linux centos linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/6f9779d6dcc0c6908b3f508932c22bbe for a full test report.

Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel changed the title Updated urls for test data for WRF to https Update URLs for test data for WRF to https Feb 17, 2021
@boegel boegel merged commit 9f18b50 into easybuilders:develop Feb 17, 2021
@dithwick dithwick deleted the wps_fix_test_urls branch February 17, 2021 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants