Skip to content

[CI][3/3] Finalize Ubuntu 20->22 migration #7671

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Apr 7, 2023
Merged

[CI][3/3] Finalize Ubuntu 20->22 migration #7671

merged 4 commits into from
Apr 7, 2023

Conversation

pvchupin
Copy link
Contributor

@pvchupin pvchupin commented Dec 7, 2022 •
edited
Loading

Follow up on #7362

  • Erase Ubuntu 20 containers build
  • Move everywhere to Ubuntu 22 containers by default
  • Update clang version that we use in build to Ubuntu 22 version (jammy)
  • Update documentation

@pvchupin pvchupin requested review from a team as code owners December 7, 2022 00:43
@pvchupin pvchupin mentioned this pull request Dec 7, 2022
Merged
@pvchupin
Copy link
Contributor Author

pvchupin commented Dec 8, 2022

Oh... looks like we have the same docker problem on CUDA machine. We need to upgrade runner host to Ubuntu 22.

[CI][3/3] Finalize Ubuntu 20->22 migration
876c50f 
Follow up on intel#7362

* Erase Ubuntu 20 containers build
* Move everywhere to Ubuntu 22 containers by default
* Update clang version that we use in build to Ubuntu 22 version (jammy)
* Update documentation
@pvchupin pvchupin temporarily deployed to aws December 22, 2022 01:22 — with GitHub Actions Inactive
@pvchupin pvchupin temporarily deployed to aws December 22, 2022 02:08 — with GitHub Actions Inactive
@pvchupin pvchupin mentioned this pull request Dec 22, 2022
Merged
@pvchupin
Copy link
Contributor Author

I forgot about CUDA CTS run that depends on the CUDA machine. :(
Currently there are 3 problems here:

  • CUDA run requires Ubuntu 22 host, or we need CTS ported to AWS run
  • Build with clang-15 brought up libstdc++ 12 as a dependency and there is the same issue as observed at [CMake] Disable STL-internal assertions in device code #7236
  • New issue on layout_handler.cpp on clang-15 build. We should update dumped symbols it seems.

@bader bader self-assigned this Dec 23, 2022
@ldrumm ldrumm mentioned this pull request Jan 18, 2023
Open
@bader bader temporarily deployed to aws January 27, 2023 04:10 — with GitHub Actions Inactive
@bader bader temporarily deployed to aws January 27, 2023 04:47 — with GitHub Actions Inactive
@bader bader mentioned this pull request Apr 3, 2023
Merged
@bader bader temporarily deployed to aws April 6, 2023 22:19 — with GitHub Actions Inactive
@bader bader temporarily deployed to aws April 6, 2023 22:52 — with GitHub Actions Inactive
@bader bader merged commit 5504c74 into intel:sycl Apr 7, 2023
@bader
Copy link
Contributor

bader commented Apr 7, 2023

  • CUDA run requires Ubuntu 22 host

Unfortunately, this workaround didn't work. Instead of reverting the whole commit, I disabled CTS testing on CUDA in post-commit with 39d1c65.

@bader bader temporarily deployed to aws April 19, 2023 23:03 — with GitHub Actions Inactive
@bader bader temporarily deployed to aws April 20, 2023 01:31 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bader bader bader approved these changes

Assignees

@bader bader

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pvchupin @bader