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

UB sanitizer CI test: bump clang version from 15 to 17 #5181

Conversation

lucafedeli88
Copy link
Member

@lucafedeli88 lucafedeli88 commented Aug 28, 2024

With this PR, clang-17 instead of clang-15 is used for UB sanitizer test in CI .

Note: Ubuntu 24.04 seems to be in beta for GitHub-hosted runners (see https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#choosing-github-hosted-runners ), so in the future we won't need to use the Ubuntu 23.10 container.

@lucafedeli88 lucafedeli88 added the component: tests Tests and CI label Aug 28, 2024
@lucafedeli88 lucafedeli88 changed the title [WIP] UB sanitizer CI test: bump clang version from 15 to 17 UB sanitizer CI test: bump clang version from 15 to 17 Aug 28, 2024
@ax3l ax3l self-assigned this Aug 28, 2024
@ax3l ax3l self-requested a review August 28, 2024 20:17
Copy link
Member

@ax3l ax3l left a comment

Choose a reason for hiding this comment

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

LGTM, thx

@ax3l ax3l merged commit 761c662 into ECP-WarpX:development Aug 28, 2024
48 checks passed
RemiLehe pushed a commit that referenced this pull request Dec 3, 2024
I started seeing this error from the Clang sanitizer workflow in several
PRs:
```console
E: The repository 'http://archive.ubuntu.com/ubuntu mantic Release' does not have a Release file.
E: The repository 'http://archive.ubuntu.com/ubuntu mantic-updates Release' does not have a Release file.
E: The repository 'http://archive.ubuntu.com/ubuntu mantic-backports Release' does not have a Release file.
E: The repository 'http://security.ubuntu.com/ubuntu mantic-security Release' does not have a Release file.
```

The Ubuntu 23.10 container was introduced in #5181 when `ubuntu-24.04`
was pre-release as GitHub-hosted runner.

I think we can try to move to `ubuntu-24.04` now and see if this is
enough to fix the issue above.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: tests Tests and CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants