-
Notifications
You must be signed in to change notification settings - Fork 4
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
Prevent sysctl use in any lxc container #26
base: testing
Are you sure you want to change the base?
Prevent sysctl use in any lxc container #26
Conversation
!testme |
Your patch looks good to me (FWIW, I am not the maintainer of this app). I'd just suggest to also remove the |
260b6b1
to
4b8e905
Compare
Rebased on latest testing. |
!testme |
I missed this comment! Indeed... I pushed a commit that removes _common.sh, as it is now empty. |
283722a
to
1602d10
Compare
actually backtracking : I just removed the package check bypassing but I kept common.sh, until I hear from a reviewer, as it is in example_ynh. |
I often use ./ynh-dev and lxc containers to experiment, test, bugfix yunohost and applications. For this reason I propose to exclude sysctl exclusion on all lxc containers, not only in package_check CI.
This will allow me to restore backups taken from my instance in test instances, for example.
Thanks!