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

Move rocksdb native library resource registration to a feature #43782

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

zakkak
Copy link
Contributor

@zakkak zakkak commented Oct 9, 2024

Always registers the correct native library as the logic runs on the same architecture we are targeting (even when using containers).

Closes #43319

@zakkak
Copy link
Contributor Author

zakkak commented Oct 9, 2024

@rsvoboda I am trying to get access to an AArch64 machine to give this a go, but it takes much longer than expected. So if you have a setup you can easily use to confirm the PR resolves the issue please let me know. Thanks

@quarkus-bot

This comment has been minimized.

@rsvoboda
Copy link
Member

rsvoboda commented Oct 9, 2024

@zakkak I can look into it, provisioning machine from Beaker usually takes ~1 hour, so I will have to wait too ...

@gsmet
Copy link
Member

gsmet commented Oct 9, 2024

That's indeed very clever. I will probably use this pattern elsewhere as it makes a lot of sense.

@rsvoboda
Copy link
Member

rsvoboda commented Oct 9, 2024

I've checked the reproducer and the app is working well on RHEL8 aarch64

Always registers the correct native library as the logic runs on the
same architecture we are targeting (even when using containers).

Closes quarkusio#43319
@zakkak
Copy link
Contributor Author

zakkak commented Oct 9, 2024

I've checked the reproducer and the app is working well on RHEL8 aarch64

Great, thanks for the review and the testing @rsvoboda. I addressed both your comments.

@zakkak zakkak requested a review from rsvoboda October 9, 2024 18:14
@quarkus-bot
Copy link

quarkus-bot bot commented Oct 9, 2024

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 53d2635.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@rsvoboda rsvoboda merged commit b779ee9 into quarkusio:main Oct 9, 2024
20 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.16 - main milestone Oct 9, 2024
@rsvoboda
Copy link
Member

rsvoboda commented Oct 9, 2024

Should we backport this to 3.15 branch? @gsmet @zakkak

@gsmet
Copy link
Member

gsmet commented Oct 10, 2024

Probably at some point. I will add the label.

@zakkak zakkak deleted the 2024-10-09-fix-43319 branch October 10, 2024 08:35
zakkak added a commit to zakkak/quarkus that referenced this pull request Oct 11, 2024
Always registers the correct native library as the logic runs on the
same architecture we are targeting (even when using containers).

Related to quarkusio#43782
zakkak added a commit to zakkak/quarkus that referenced this pull request Oct 14, 2024
Always registers the correct native library as the logic runs on the
same architecture we are targeting (even when using containers).

Related to quarkusio#43782
zakkak added a commit to zakkak/quarkus that referenced this pull request Oct 14, 2024
Always registers the correct native library as the logic runs on the
same architecture we are targeting (even when using containers).

Related to quarkusio#43782
@gsmet gsmet modified the milestones: 3.16.0.CR1, 3.15.2 Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to run application with kafka-stream on aarch64 built using native builder image
3 participants