Skip to content

Conversation

ggeorgakoudis
Copy link
Contributor

  • Remove defunct ruby, lassen job files
  • Add build-and-test script
  • Add tioga job file

@ggeorgakoudis ggeorgakoudis requested a review from Copilot October 11, 2025 16:46
Copy link

@Copilot 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 updates the GitLab CI configuration to run on the Tioga machine instead of the previously used Ruby and Lassen machines. The changes remove defunct job configurations and introduce a new build-and-test script specifically designed for the Tioga environment.

  • Remove obsolete Ruby and Lassen job configurations
  • Add new build-and-test script with Conda environment setup and testing workflow
  • Enable Tioga CI jobs with proper resource allocation settings

Reviewed Changes

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

Show a summary per file
File Description
buildscripts/gitlab/build-and-test.sh New script that sets up Conda environments, builds the project, and runs OpenMP tests
.gitlab/subscribed-pipelines.yml Uncomments Tioga CI job definitions to enable them
.gitlab/jobs/tioga.yml Updates job configuration from Lassen to Tioga, removes environment setup commands
.gitlab/jobs/ruby.yml Removes entire Ruby job configuration file
.gitlab/custom-jobs-and-variables.yml Enables Tioga allocation variables with updated time limits
.gitlab-ci.yml Changes job command from package creation to build-and-test script

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ggeorgakoudis ggeorgakoudis merged commit 826a7c9 into main Oct 12, 2025
2 checks passed
@ggeorgakoudis ggeorgakoudis deleted the update-gitlab-ci branch October 12, 2025 03:26
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.

1 participant