search

actions / runner-images

GitHub Actions runner images
MIT License
9.22k stars 2.86k forks source link

Ubuntu 22.04, GNU C++ 13.1.0 removed from image 20240514.2.1 #9866

Closed marcalff closed 2 weeks ago

marcalff commented 2 weeks ago

Description

This is a regression, GCC 13.1.0 was removed.

See commit:

Platforms affected

Runner images affected

Image version and build link

Image: ubuntu-22.04 Version: 20240514.2.0

Is it regression?

Build succeeds with:

  Image: ubuntu-22.04
  Version: 20240422.1.0

Expected behavior

GNU C++ 13.1.0 available

Actual behavior

GNU C++ 13.1.0 was available before, and was removed.

Repro steps

CI build break in opentelemetry-cpp

https://github.com/open-telemetry/opentelemetry-cpp/actions/runs/9108384703/job/25039040061?pr=2669#logs

CMake Error at /usr/local/share/cmake-3.29/Modules/CMakeDetermineCCompiler.cmake:49 (message):
  Could not find compiler set in environment variable CC:

  /usr/bin/gcc-13.
erik-bershel commented 2 weeks ago

Hey @marcalff!

It's an expected behaviour. We removed external PPA with GCC-9 and GCC-13 from the image. See there: https://github.com/actions/runner-images/issues/9679 You have two options:

It will take about three minutes:

          sudo add-apt-repository ppa:ubuntu-toolchain-r/test -y
          sudo apt-get update -y
          sudo apt-get install gcc-13
marcalff commented 2 weeks ago

Hi @erik-bershel

Thanks for the quick reply.

I am willing to accept there is a good reason for this, but calling this "expected behavior" is another story.

Now looking to upgrade our CI to use Ubuntu-24.04 then.

hartwork commented 1 week ago

@actions I've been quite happy with most of GitHub Actions so far but dropping GCC 13 from ubuntu-22.04 images is breaking API contracts, breaking existing CI across the board — were you really not aware of the consequences? None of these runs were suppose to fail in the first place…

…and now I'm forced to deal with that change in time or every new push and pull request will be red — ouch!

Please think again next time you're breaking CI for everyone, thank you.

CC @rpodgorny @kaixiong @hannob