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

feat(radarbox): Implement rbfeeder_fixcputemp #166

Merged
merged 1 commit into from
Apr 17, 2024

Conversation

FlyGoat
Copy link
Contributor

@FlyGoat FlyGoat commented Apr 14, 2024

Use LD_PRELOAD hook to workaround radarbox cpu temperature segfault issue. Allow container to run on userspace podman where sysfs volume is impossible.

Use LD_PRELOAD hook to workaround radarbox cpu temperature segfault issue.
Allow container to run on userspace podman where sysfs volume is impossible.

Signed-off-by: Jiaxun Yang <[email protected]>
@Thom-x
Copy link
Owner

Thom-x commented Apr 15, 2024

@Deploy

@Thom-x
Copy link
Owner

Thom-x commented Apr 15, 2024

@FlyGoat thank you for the PR ! I let you test the image before merging ?

@FlyGoat
Copy link
Contributor Author

FlyGoat commented Apr 15, 2024

@Thom-x Yes it works fine on my side, saving me from the sysfs hack.

@FlyGoat
Copy link
Contributor Author

FlyGoat commented Apr 15, 2024

Ah sorry I think I misunderstood your comment.
Do you want me to test the image built from actions of your repo?

@Thom-x
Copy link
Owner

Thom-x commented Apr 15, 2024

Yes exactly, so we are sure the build is OK and it works.

@Thom-x Thom-x added the bump:patch Bump patch version tag label Apr 15, 2024
@FlyGoat
Copy link
Contributor Author

FlyGoat commented Apr 15, 2024

@Thom-x Ah sorry for the problems on ghcr stuff, do you mind triggering deploy for this PR again?

@Thom-x
Copy link
Owner

Thom-x commented Apr 15, 2024

No problem, let's do it

@Thom-x
Copy link
Owner

Thom-x commented Apr 15, 2024

@Deploy

@FlyGoat
Copy link
Contributor Author

FlyGoat commented Apr 17, 2024

Just tested on my system and it’s working as intended.
Thanks

@Thom-x Thom-x merged commit 4e0359d into Thom-x:master Apr 17, 2024
6 checks passed
Copy link

🚀 [bumpr] Bumped! New version:1.32.1 Changes:1.32.0...1.32.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump:patch Bump patch version tag
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants