Skip to content

Conversation

@kotsaloscv
Copy link
Contributor

No description provided.

@kotsaloscv
Copy link
Contributor Author

cscs-ci run alps-beverin-bencher;MY_UENV=prgenv-gnu/25.07-6.3.3:v9;CRON_RUN=true

@kotsaloscv
Copy link
Contributor Author

cscs-ci run alps-daint-bencher;MY_UENV=prgenv-gnu/25.06:v2;CRON_RUN=true

@jgphpc
Copy link
Collaborator

jgphpc commented Nov 18, 2025

cscs-ci run alps-beverin-bencher;MY_UENV=prgenv-gnu/25.07-6.3.3:v9;CRON_RUN=true

@jgphpc jgphpc requested a review from ekouts November 18, 2025 09:12
@jgphpc
Copy link
Collaborator

jgphpc commented Nov 18, 2025

@jgphpc jgphpc marked this pull request as ready for review November 18, 2025 11:34
@kotsaloscv
Copy link
Contributor Author

cscs-ci run alps-beverin-bencher;MY_UENV=prgenv-gnu/25.07-6.3.3:v9;CRON_RUN=true

@kotsaloscv
Copy link
Contributor Author

cscs-ci run alps-daint-bencher;MY_UENV=prgenv-gnu/25.06:v2;CRON_RUN=true

@kotsaloscv
Copy link
Contributor Author

cscs-ci run alps-beverin-bencher;MY_UENV=prgenv-gnu/25.07-6.3.3:v8;CRON_RUN=true

@kotsaloscv
Copy link
Contributor Author

cscs-ci run alps-beverin-bencher;MY_UENV=prgenv-gnu/25.07-6.3.3:v8;CRON_RUN=true

@jgphpc jgphpc changed the title testing bencher Fixes for bencher CI Nov 19, 2025
@jgphpc jgphpc requested a review from Copilot November 19, 2025 07:03
Copilot finished reviewing on behalf of jgphpc November 19, 2025 07:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes issues in the bencher CI pipeline by adjusting timeouts, updating filesystem configurations, and modifying output handling for the ReFrame installation path.

  • Increased SLURM timelimit from 20 minutes to 1 hour for bencher API token jobs
  • Modified ReFrame path extraction to use pattern matching with grep/awk
  • Cleaned up obsolete filesystem entries and added new system configuration for beverin

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
ci/scripts/alps.sh Added debug marker comment to ReFrame path output
ci/alps_uenv.yml Increased SLURM timeout and modified path extraction logic using grep/awk
checks/system/systems_data/daint_data.json Removed obsolete filesystem entries but introduced incomplete object
checks/system/systems_data/beverin_data.json Added new system configuration for beverin with filesystem and package definitions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ekouts ekouts merged commit 2ee4884 into eth-cscs:main Nov 19, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants