Skip to content

Conversation

@lexming
Copy link
Contributor

@lexming lexming commented Jan 27, 2021

(created using eb --new-pr)

Major revision of the easyconfigs of SimpleITK for v2.0.2:

Depends on #12054

@boegelbot
Copy link
Collaborator

@lexming: Tests failed in GitHub Actions, see https://github.com/easybuilders/easybuild-easyconfigs/actions/runs/516146143
Output from first failing test suite run:

Found 11331 easyconfigs...
ERROR: test__parse_easyconfig_SimpleITK-2.0.2-foss-2020a-Python-3.8.2.eb (test.easyconfigs.easyconfigs.EasyConfigTest)
Failed to determine merge base (ec: 1, output: ''), falling back to specifying target branch develop
Test for parsing of easyconfig SimpleITK-2.0.2-foss-2020a-Python-3.8.2.eb
Failed to determine merge base (ec: 1, output: ''), falling back to specifying target branch develop
----------------------------------------------------------------------

Traceback (most recent call last):
List of added easyconfig files in this PR: SimpleITK-2.0.2-foss-2020a-Python-3.8.2.eb
  File "test/easyconfigs/easyconfigs.py", line 1058, in innertest
SimpleITK-2.0.2-fosscuda-2020a-Python-3.8.2.eb
    template_easyconfig_test(self, spec_path)
(skipped conflicts test)
  File "test/easyconfigs/easyconfigs.py", line 854, in template_easyconfig_test
(skipped dep graph test)
    ecs = process_easyconfig(spec)
  File "/opt/hostedtoolcache/Python/2.7.18/x64/lib/python2.7/site-packages/easybuild/framework/easyconfig/easyconfig.py", line 1977, in process_easyconfig
    raise EasyBuildError("Failed to process easyconfig %s: %s", spec, err.msg)
EasyBuildError: "Failed to process easyconfig /home/runner/work/easybuild-easyconfigs/easybuild-easyconfigs/easybuild/easyconfigs/s/SimpleITK/SimpleITK-2.0.2-foss-2020a-Python-3.8.2.eb: Use of 1 unknown easyconfig parameters detected in SimpleITK-2.0.2-foss-2020a-Python-3.8.2.eb: start_dir_python\nIf these are just local variables please rename them to start with 'local_', or try using --fix-deprecated-easyconfigs to do this automatically.\nFor more information, see https://easybuild.readthedocs.io/en/latest/Easyconfig-files-local-variables.html ."

======================================================================
ERROR: test__parse_easyconfig_SimpleITK-2.0.2-fosscuda-2020a-Python-3.8.2.eb (test.easyconfigs.easyconfigs.EasyConfigTest)
Test for parsing of easyconfig SimpleITK-2.0.2-fosscuda-2020a-Python-3.8.2.eb
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test/easyconfigs/easyconfigs.py", line 1058, in innertest
    template_easyconfig_test(self, spec_path)
  File "test/easyconfigs/easyconfigs.py", line 854, in template_easyconfig_test
    ecs = process_easyconfig(spec)
  File "/opt/hostedtoolcache/Python/2.7.18/x64/lib/python2.7/site-packages/easybuild/framework/easyconfig/easyconfig.py", line 1977, in process_easyconfig
    raise EasyBuildError("Failed to process easyconfig %s: %s", spec, err.msg)
EasyBuildError: "Failed to process easyconfig /home/runner/work/easybuild-easyconfigs/easybuild-easyconfigs/easybuild/easyconfigs/s/SimpleITK/SimpleITK-2.0.2-fosscuda-2020a-Python-3.8.2.eb: Use of 1 unknown easyconfig parameters detected in SimpleITK-2.0.2-fosscuda-2020a-Python-3.8.2.eb: start_dir_python\nIf these are just local variables please rename them to start with 'local_', or try using --fix-deprecated-easyconfigs to do this automatically.\nFor more information, see https://easybuild.readthedocs.io/en/latest/Easyconfig-files-local-variables.html ."

======================================================================
FAIL: test_changed_files_pull_request (test.easyconfigs.easyconfigs.EasyConfigTest)
Specific checks only done for the (easyconfig) files that were changed in a pull request.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "test/easyconfigs/easyconfigs.py", line 822, in test_changed_files_pull_request
    self.assertTrue(False, error_msg)
AssertionError: Failed to find parsed easyconfig for SimpleITK-2.0.2-foss-2020a-Python-3.8.2.eb (and could not isolate it in easyconfigs archive either)

----------------------------------------------------------------------
Ran 11340 tests in 819.573s

FAILED (failures=1, errors=2)
ERROR: Not all tests were successful.

bleep, bloop, I'm just a bot (boegelbot v20200716.01)
Please talk to my owner @boegel if you notice you me acting stupid),
or submit a pull request to https://github.com/boegel/boegelbot fix the problem.

@verdurin
Copy link
Member

@lexming Saw a failure during the test step:

== 2021-04-19 12:27:00,450 easyblock.py:3138 INFO Running method test_step part of step test
== 2021-04-19 12:27:00,450 environment.py:91 INFO Environment variable CTEST_OUTPUT_ON_FAILURE set to True (previously undefined)
== 2021-04-19 12:27:00,450 run.py:206 DEBUG run_cmd: running cmd  make True  (in /dev/shm/SimpleITK/2.0.2/foss-2020a-Python-3.8.2/easybuild_obj)
== 2021-04-19 12:27:00,450 run.py:225 INFO running cmd:  make True
== 2021-04-19 12:27:00,474 build_log.py:169 ERROR EasyBuild crashed with an error (at easybuild/base/exceptions.py:124 in __init__): cmd " make True " exited with exit code 2 and output:
make: *** No rule to make target `True'.  Stop.
 (at easybuild/tools/run.py:537 in parse_cmd_output)

Also, did you have any thoughts on the style complaint about the variable?

@verdurin
Copy link
Member

Test report by @verdurin
FAILED
Build succeeded for 5 out of 7 (2 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/74fec3e74406473c48485af2cc7846d2 for a full test report.

@verdurin
Copy link
Member

Test report by @verdurin
Using easyblocks from PR(s) easybuilders/easybuild-easyblocks#2318
SUCCESS
Build succeeded for 2 out of 2 (2 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/25577881580ec2b50d4f7f0d5fc1e203 for a full test report.

@lexming
Copy link
Contributor Author

lexming commented Apr 20, 2021

@verdurin since this PR depends on easybuilders/easybuild-easyblocks#2318, I will check the style complaints once the easyblock is merged

@lexming
Copy link
Contributor Author

lexming commented Jan 17, 2023

@lexming lexming closed this Jan 17, 2023
@lexming lexming deleted the 20210127212207_new_pr_SimpleITK202 branch January 17, 2023 13:33
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.

3 participants