-
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
update bot configuration for EESSI 2023.06 #262
Conversation
Instance
|
…nstead of linux/aarch64/graviton*
3c005ba
to
cf85694
Compare
I think this should go into a different repository. Maybe one that captures information about the setup of our infrastructure. |
It used to be in a bot repo, but since the configuration is specific to the repo for which it's being used, we moved it here. If we move it somewhere else, don't we re-introduce that problem, and create a disconnect between the config and the repo for which it is used? That said, there's also stuff missing here (the |
Partially true (if the bot would only build for a specific repository).
Possibly. Eventually one might want to split the configuration into settings which a specific to a bot instance (and store that in some
Those are probably target repo specific settings and could be moved there easily. |
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.
Instead of blocking this, we can leave it (bot-eessi-aws-citc.cfg
) in the software-layer repository.
Possibly one could add the repos
directory which includes CVMFS config files and settings for different target repositories. Could also be done in a follow up PR.
…M/11-foss/2023a {2023.06}[foss/2023a] OpenFOAM V11
Updated bot configuration by @trz42 .
WIP because needs a change foraarch64/*
targets, where we should switch to usingaarch64/neoverse_n1
rather thanaarch64/graviton2
andaarch64/neoverse_v1
instead ofaarch64/graviton3
, see archspec/archspec-json#41 + archspec/archspec-json#57 .We should also get rid of2023.04
, since we won't be using that anymore (see #257 + #258)