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 the broken CI due to the regression of 4.0.1 and some deprecated image and links #235

Merged
merged 18 commits into from
Jan 10, 2025

Conversation

BewareMyPower
Copy link
Contributor

@BewareMyPower BewareMyPower commented Dec 12, 2024

Fixes #234

  1. Downgrade the Pulsar image to 4.0.0
  2. Upgrade the macOS runner image from 12 to 14

Since the Boost download link is broken now, this PR also fixes the wheel build process by reusing the pre-built libraries to install pulsar-client-cpp on Linux and macOS. Specially, it links the libpulsarwithdeps.a instead of libpulsar.a and all 3rd-party libraries on macOS.

@BewareMyPower BewareMyPower self-assigned this Dec 12, 2024
@BewareMyPower BewareMyPower added this to the 3.6.0 milestone Dec 12, 2024
@BewareMyPower BewareMyPower marked this pull request as draft December 12, 2024 09:20
@BewareMyPower BewareMyPower marked this pull request as ready for review January 8, 2025 02:18
@BewareMyPower BewareMyPower marked this pull request as draft January 8, 2025 02:38
@BewareMyPower
Copy link
Contributor Author

Let me check if the Linux build can reuse pre-built RPM or APK libraries from the cpp release

@BewareMyPower BewareMyPower changed the title Fix the broken CI due to the regression of 4.0.1 and deprecated macOS runner image Fix the broken CI due to the regression of 4.0.1 and some deprecated image and links Jan 9, 2025
@BewareMyPower BewareMyPower marked this pull request as ready for review January 9, 2025 08:03
@BewareMyPower BewareMyPower marked this pull request as draft January 9, 2025 08:32
@BewareMyPower BewareMyPower marked this pull request as ready for review January 9, 2025 09:13
@BewareMyPower BewareMyPower merged commit fb4523b into apache:main Jan 10, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

test_deliver_after and test_deliver_at failed with latest Pulsar image
2 participants