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

Upgrade CI to min ubuntu 20.04 #839

Merged
merged 5 commits into from
Oct 18, 2022
Merged

Conversation

rukai
Copy link
Member

@rukai rukai commented Oct 5, 2022

Only upgrading to ubuntu 20.04 instead of directly to 22.04 to maintain compatibility with 20.04 in our binary builds.

I chose to create a .deb rather than directly mashing the files into /usr/lib so that:

  • local users have a way to uninstall the package.
  • caching just works:
    • we tell GA to cache the packages folder and then it will cache every package that gets built
      • built packages are really small, ~1MB for this one, so we dont need to worry about excessive cache usage.
      • packages are versioned so we dont have to worry about the cache accidentally pulling in an old version of cassandra-cpp-driver.

Alternatives:

1

Publish packages somewhere and wget them like we do currently with the datastax packages.
I didnt take this approach as we need somewhere to host them and it is a lot of extra work to package them every time a new release occurs that we want to use.

2

Annoy datastax into publishing packages for a modern ubuntu.
Not sure if they care enough about the cpp driver for that, and even if they did, we would still need to reannoy them in the future when a new ubuntu comes out.

3

Force installation of the 18.04 package onto a modern ubuntu.
I tested this and it doesnt work at all.
We can succesfully install the package with dpkg --force-install -i ....
However when we try to build the project it fails with linker errors.

References:

@rukai rukai force-pushed the escape_the_dying_planet branch 11 times, most recently from 531cdf8 to b395b34 Compare October 5, 2022 06:10
@rukai rukai marked this pull request as ready for review October 5, 2022 06:11
@benbromhead
Copy link
Member

I'm so mad at this dependency right now... this seems like the easiest path forward right now. In the mean time imma bug datastax

@rukai rukai merged commit cf87466 into shotover:main Oct 18, 2022
@rukai rukai mentioned this pull request Oct 18, 2022
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.

3 participants