Skip to content
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

fix printing environment variables in GitLab CI #2423

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

SimeonEhrig
Copy link
Member

@SimeonEhrig SimeonEhrig commented Nov 14, 2024

Fixed several bugs in the output of environment variables that must be set if the container build is to be reproduced locally (yellow text at the begin of the job log).

  • wrong variables was printed (e.g. PWD, which causes strange behavior if you overwrite the variable manually)
  • some variables was not printed because of wrong regex
  • CI_RUNNER_TAGS was missing in general, which is required if you build HIP builds with enabled tests

Also improved the output. Group the variables by prefix and sort the output.

@SimeonEhrig SimeonEhrig added this to the 2.0.0 milestone Nov 14, 2024
@SimeonEhrig SimeonEhrig force-pushed the fixPrintEvnScript branch 3 times, most recently from 3b25f39 to 64fdafd Compare November 14, 2024 14:53
@SimeonEhrig SimeonEhrig marked this pull request as ready for review November 14, 2024 15:53
@psychocoderHPC psychocoderHPC merged commit af5ed36 into alpaka-group:develop Nov 18, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants