-
Notifications
You must be signed in to change notification settings - Fork 61
{2023.06}[icelake,cascadelake] EasyStacks batch 9: 036 - 038 #1059
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
{2023.06}[icelake,cascadelake] EasyStacks batch 9: 036 - 038 #1059
Conversation
Instance
|
Instance
|
Instance
|
Instance
|
Instance
|
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws arch:icelake |
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
New job on instance
|
New job on instance
|
This failed in the build step of matplotlib's extension Cycler:
edit: ah, that's because we use |
…-layer into icelake_cclake_batch9
Fixed both issues: I've split up easystack 036 and moved ReFrame to a new easystack using EB 4.9.2 and bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws arch:icelake |
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
New job on instance
|
New job on instance
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't actually approve my own PR - but I'll at least review the parts that Bob changed. As discussed on chat, matplotlib and ReFrame have been separated out and build with newer EB versions then used originally. This was necessary since the --from-pr
would pull in unwanted changes. Updating to a newer EB allows us to use --from-commit
instead. It means we use a different framework, but actually have a better guarantee of using the same EasyConfigs, so that's a plus in my book.
Lgtm!
@bedroge if the builds are successful, feel free to put an 'approve' review yourself so this can be merged :)
…layer into icelake_cclake_batch9
I've renamed the easystacks a bit to keep the original numbering. Easystack 036 has been split into 036a - 036d. Shouldn't require any rebuilds, since the contents is still the same. |
The builds failed, because the check missing installations script was run with EB 4.9.1 for matplotlib (originally built with 4.9.0), but 4.9.1 also added additional dependencies to the Pillow easyconfig. Since Pillow is a dependency of matplotlib, the script now fails, as it's expecting them to be installed. Let's try to solve it by adding those dependencies here, and build Pillow with the easyconfig from the EB 4.9.1 release. If this works, we need to rebuild for all other targets as well, to make sure that they also have a Pillow version with the additional dependencies/functionality. Also note that the dependencies would have been installed by #1063, so we will need to remove them from that easystack (well, could also leave them there, it will just skip them anyway). bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws arch:icelake |
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
Updates by the bot instance
|
New job on instance
|
New job on instance
|
Label |
Tarballs ingested. |
PR merged! Moved |
PR merged! Moved |
PR merged! Moved |
This should be synced with the main branch once #1057 is merged. Then, it can be built.