easybuild-framework icon indicating copy to clipboard operation
easybuild-framework copied to clipboard

Cray module swap for craype-<cpu-target> (REVIEW)

Open pforai opened this issue 10 years ago • 14 comments

Some refactoring and enhancement for auto swapping the craype- module instead of relying on the modules tool to just bail on conflict.

~~Same logic will be used in follow-up commits for PrgEnv-<gnu/intel/cray> modules. As discussed with @boegel this will probably land before the call to super() in the prepare() method.~~

pforai avatar May 21 '15 20:05 pforai

Refer to this link for build results (access rights to CI server needed): https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/1725/ EasyBuild framework unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/1725/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

hpcugentbot avatar May 21 '15 20:05 hpcugentbot

@pforai: what's the status on this? I need to close down for 2.2 in the next couple of days, it seems like this will need more love and should be targeted to 2.3 (early Sept'15)?

boegel avatar Jul 03 '15 10:07 boegel

@pforai: I won't have the time required to give this a thorough look in time for EB v2.2, so I'm moving the milestone to v2.3 on this one;

let's rediscuss this in August?

boegel avatar Jul 07 '15 17:07 boegel

EasyBuild framework unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2900/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

hpcugentbot avatar Mar 25 '16 12:03 hpcugentbot

EasyBuild framework unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2901/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

hpcugentbot avatar Mar 25 '16 12:03 hpcugentbot

Rebased on newer develop. /cc @gppezzi can you give this a review?

pforai avatar Mar 25 '16 12:03 pforai

EasyBuild framework unit test suite PASSed (see https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2902/console for more details).

This pull request is now ready for review/testing.

Please try and find someone who can tackle this; contact @boegel if you're not sure what to do.

hpcugentbot avatar Mar 25 '16 13:03 hpcugentbot

EasyBuild framework unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2904/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

hpcugentbot avatar Mar 25 '16 15:03 hpcugentbot

EasyBuild framework unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2905/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

hpcugentbot avatar Mar 25 '16 15:03 hpcugentbot

EasyBuild framework unit test suite FAILed.

See https://jenkins1.ugent.be/job/easybuild-framework-pr-builder/2907/console for more details.

Please fix the reported issues by pushing additional commits to the branch corresponding with this pull request; contact @boegel if you're not sure what to do.

hpcugentbot avatar Mar 25 '16 16:03 hpcugentbot

@pforai: is this still relevant?

cc @gppezzi

boegel avatar Jul 04 '16 18:07 boegel

@pforai, @gppezzi ping on this? Is it worth reviving this?

boegel avatar Jan 14 '17 13:01 boegel

I think it would make sense to revive as it is itsn't too hard to do and offers some convenience.

pforai avatar Jan 16 '17 13:01 pforai

@pforai OK, let's start with syncing this with current develop: https://github.com/pforai/easybuild-framework/pull/41

boegel avatar Jan 16 '17 14:01 boegel