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 Could'nt read record from /sys/class #391

Merged
merged 11 commits into from
Oct 6, 2024

Conversation

CalmaIndustry
Copy link

to fix Could'nt read record from /sys/class ...
#351
need to run the container with privileged:true

helm/scaphandre/values.yaml Outdated Show resolved Hide resolved
@CalmaIndustry
Copy link
Author

Add new commit to create parameters groups and add documentation.

@JMLX42
Copy link

JMLX42 commented Sep 24, 2024

I am not a maintainer but I plan to test this on k3s and report back ASAP.

@JMLX42
Copy link

JMLX42 commented Sep 25, 2024

@bpetit I hope you can review this yourself.

AFAIK it works as expected and the documentation is OK.

@CalmaIndustry
Copy link
Author

project seems to be dead

@bpetit
Copy link
Contributor

bpetit commented Oct 4, 2024

Thanks for the PR, will review and merge soon.

(project not dead, but we rely on consultancy to live, and sponsorship doesn't work, so best effort so far ;) )

@bpetit bpetit changed the base branch from main to dev October 4, 2024 08:37
@JMLX42
Copy link

JMLX42 commented Oct 5, 2024

(project not dead, but we rely on consultancy to live, and sponsorship doesn't work, so best effort so far ;) )

@bpetit I just sent you an e-mail about that. Please make sure you read it!

@CalmaIndustry
Copy link
Author

@bpetit token seems to be expired ? Or missing ?

@bpetit bpetit merged commit e44bff0 into hubblo-org:dev Oct 6, 2024
6 of 7 checks passed
@bpetit
Copy link
Contributor

bpetit commented Oct 6, 2024

Something needs fixing in the pipeline, but this PR doesn't touch code so nevermind, I'll fix this afterwards.

Thanks for the PR ! This will be in the next release, please use the dev branch until it's published.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants