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 compilation on ubuntu-noble #55

Merged
merged 3 commits into from
Jan 29, 2025
Merged

Conversation

abg
Copy link
Member

@abg abg commented Jan 22, 2025

This introduces two primary changes:

  • vendor pkg-config; This seems to be required for MySQL to correctly detect OpenSSL on ubuntu-noble now.

    This adds a new bosh blob, pkg-config v0.29.2

  • vendor libaio; libaio version was bumped and the old packaging method of compiling against a "stub" libaio but used the copy on the stemcell no longer works.

Fixes #54

Thanks for opening a PR. Please make sure you've read and followed the Contributing guide, including signing the Contributor License Agreement.

Feature or Bug Description

What does this PR change?

This PR introduces changes to packaging scripts for MySQL components to compile under the ubuntu-noble stemcell.

Additionally introduces a new build dependency: pkg-config.

Motivation

Tell us about the problem you are facing, with context, that this PR solves.

Solving the compilation issues noted in #54.

Related Issue

If this PR was first opened as an issue, please provide the link to that issue here.

See #54

@abg
Copy link
Member Author

abg commented Jan 22, 2025

Marking as a draft PR pending some validation of this work on an internal pipeline.

abg added 2 commits January 24, 2025 15:39
- vendor pkg-config; This seems to be required for MySQL to correctly
  detect OpenSSL on ubuntu-noble now.

  This adds a new bosh blob, pkg-config v0.29.2
- vendor libaio; libaio version was bumped and the old packaging method
  of compiling against a "stub" libaio but used the copy on the stemcell
  no longer works.

Fixes #54
e2e-tests use the default manifests at the top-level of the repo and
their default stemcell.

Add a new STEMCELL_OS parameter that, if set, will override the stemcell
os name to test against an alternate stemcell.

This allows testing against both jammy and noble by changing a test
parameter.
@abg abg force-pushed the feature/support-ubuntu-noble-stemcells branch 2 times, most recently from 32b77d8 to 52e9938 Compare January 25, 2025 22:32
@abg abg marked this pull request as ready for review January 27, 2025 15:48
This may be short lived, but fixes intended use of the bootstrap errand
which coordinates with this agent to start MySQL instances in the
correct order after a cluster has gone offline.
@abg abg force-pushed the feature/support-ubuntu-noble-stemcells branch from 52e9938 to ba02c08 Compare January 29, 2025 17:07
@abg
Copy link
Member Author

abg commented Jan 29, 2025

Reviewed with @ryanwittrup earlier this week, made a few small adjustments and have seen e2e-tests go green. Merging.

@abg abg merged commit e7d3a1f into main Jan 29, 2025
1 check passed
@abg abg deleted the feature/support-ubuntu-noble-stemcells branch January 29, 2025 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

pxc-release v1.0.33 fails to compile on Ubuntu Noble stemcell v0.26
1 participant