Remove removal step from bot/build.sh#31
Merged
ocaisa merged 10 commits intoEESSI:mainfrom Jul 8, 2025
Merged
Conversation
Contributor
Author
|
bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws arch:x86_64/amd/zen2 |
|
New job on instance
|
Contributor
Author
|
Rebuilding EESSI-extend from an easystack did not work properly, as it's already loaded. But #2 added a new mechanism that will rebuild EESSI-extend whenever its easyconfig in the git repo is changed. So let's try that instead. bot: build repo:eessi.io-2023.06-software instance:eessi-bot-mc-aws arch:x86_64/amd/zen2 |
|
New job on instance
|
Contributor
Author
|
That worked, so let's try it on the "problematic" nodes (which did not have support for fakeroot/rebuilds): bot: build repo:eessi.io-2023.06-software instance:eessi-bot-deucalion architecture:aarch64/a64fx |
|
New job on instance
|
|
New job on instance
|
Contributor
Author
|
Those builds are probably failing because the attr sources are currently unavailable, had the same issue on the AWS cluster.... |
Contributor
Author
|
Let's try bot: build repo:eessi.io-2023.06-software instance:eessi-bot-deucalion architecture:aarch64/a64fx |
|
New job on instance
|
|
New job on instance
|
Contributor
Author
|
Okay, that worked fine. This PR should be ready, but the dummy easystacks need to be removed before merging this. I'll do that after this has been approved, so we can still test additional changes if required. |
ocaisa
requested changes
Jul 8, 2025
Member
ocaisa
left a comment
ocaisa
left a comment
There was a problem hiding this comment.
PR looks good, will approve once the easystack and EESSI-extend changes are removed
ocaisa
approved these changes
Jul 8, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Also removes the
EESSI-remove.shscript. This basically reverts the changes from EESSI/software-layer#488, which were implemented as a workaround to rebuild software with fuse-overlayfs. With unionfs we no longer have these issues, so we can get rid of this step. By doing this, we no longer depend on fakeroot, which was causing issue on our grace and a64fx nodes.In order to test this, I'll add a dummy easystack and trigger some test builds, and then remove the dummy easystacks again.