Skip to content

also check on post-install patches in easyconfigs test suite#23331

Merged
branfosj merged 2 commits intoeasybuilders:5.1.xfrom
boegel:ci_postinstallpatches
Jul 5, 2025
Merged

also check on post-install patches in easyconfigs test suite#23331
branfosj merged 2 commits intoeasybuilders:5.1.xfrom
boegel:ci_postinstallpatches

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Jul 5, 2025

@boegel boegel added this to the 5.1.1 milestone Jul 5, 2025
@boegel boegel changed the base branch from develop to 5.1.x July 5, 2025 09:05
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Jul 5, 2025

Tests are failing as expected due to missing patch file, which is being fixed in #23330

FAIL: test__parse_easyconfig_tensorboard-2.15.1-gfbf-2023b.eb (test.easyconfigs.easyconfigs.EasyConfigTest.test__parse_easyconfig_tensorboard-2.15.1-gfbf-2023b.eb)
Test for easyconfig tensorboard-2.15.1-gfbf-2023b.eb
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/easybuild-easyconfigs/easybuild-easyconfigs/test/easyconfigs/easyconfigs.py", line 1927, in innertest
    template_easyconfig_test(self, spec_path)
  File "/home/runner/work/easybuild-easyconfigs/easybuild-easyconfigs/test/easyconfigs/easyconfigs.py", line 1779, in template_easyconfig_test
    self.fail('Verification for %s failed:\n' % os.path.basename(spec) + '\n'.join(failing_checks))
AssertionError: Verification for tensorboard-2.15.1-gfbf-2023b.eb failed:
patch file tensorboard-2.10_jupyterhub-support.patch is missing

I'll re-trigger the tests here when #23330 is merged

@branfosj branfosj enabled auto-merge July 5, 2025 09:22
@branfosj branfosj merged commit 5ceef41 into easybuilders:5.1.x Jul 5, 2025
8 checks passed
@boegel boegel deleted the ci_postinstallpatches branch July 5, 2025 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants