Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 <jiaxun.yang@flygoat.com>
@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

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

Successfully merging this pull request may close these issues.

2 participants