-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
let bot run EESSI test suite during test phase (part of bot build procedure) #467
let bot run EESSI test suite during test phase (part of bot build procedure) #467
Conversation
Instance
|
…ger the bot build and test procedures and see if this PR actually works
bot: build repo:eessi.io-2023.06-software arch:x86_64/amd/zen2 |
Updates by the bot instance
|
New job on instance
|
And indeed, the two scripts don't have execute permission:
|
bot: build repo:eessi.io-2023.06-software arch:x86_64/amd/zen2 |
Updates by the bot instance
|
New job on instance
|
Sounds good but not quite there yet:
I don't really want to run the tests in a prefix environment. I'll need to see where this is checked and supress the error. The other thing I need to check is why this led to a 'Test step run succesfully'. It should have led to a "Failure to execute test step". |
bot: build repo:eessi.io-2023.06-software arch:x86_64/amd/zen2 |
Updates by the bot instance
|
New job on instance
|
Ok, runs as expected:
This results in a pattern match for this error:
And:
As expected. Next, I'll fix the requirement of this thing running in prefix, since that is not needed, so we shouldn't check for it. |
bot: build repo:eessi.io-2023.06-software arch:x86_64/amd/zen2 |
Updates by the bot instance
|
New job on instance
|
Ok, the detailed sentence is quite long, I'll need to see how I can hide that behind a 'details' thing like is done for the
Hm, did I actually initialize the EESSI environment here or not (yet)? |
Ah,
Doesn't set modulepath. That is arranged here:
But, I stripped the sourcing of the
|
Ok, there's actually a minimal change that I can do: Change |
…odulepath. It should be the same, and we no longer configure EasyBuild, since we don't use it when running the test suite
bot: build repo:eessi.io-2023.06-software arch:x86_64/amd/zen2 |
…sed for running test suite
…re run with generic software installations when intended
bot: build repo:eessi.io-2023.06-software arch:x86_64/generic |
Updates by the bot instance
|
New job on instance
|
New job on instance
|
New job on instance
|
New job on instance
|
bot: build repo:eessi.io-2023.06-software arch:x86_64/intel/haswell |
Updates by the bot instance
|
New job on instance
|
New job on instance
|
New job on instance
|
New job on instance
|
A second attempt, based on #366
This is probably a better starting point than #466
Let's see if this runs correctly...