[LTP] [PATCH 1/1] lib: Skip tst_{disable, enable}_oom_protection() for non-root

[Li Wang]

Hi Xu, Petr,

On Wed, Dec 22, 2021 at 4:37 PM xuyang2018.jy@fujitsu.com <
xuyang2018.jy@fujitsu.com> wrote:

> Hi Petr
> > Hi Xu,
> >
> >>> This is not working as expected in Github CI. I'm still looking at the
> >>> problem.
> >>>
> https://github.com/wangli5665/ltp/runs/4602025797?check_suite_focus=true
> >> I tested local but it works well. I guess ci fails because of linux user
> >> namespace. Maybe we should require CAP_SYS_RESOURCE cap instead of using
> >> geteuid.
> > Good catch, verifying.
> You can refer to my ltp fork
> https://github.com/xuyang0410/ltp/commits/oom_kill_ci_fixes


+1

I was thinking of adding the permission but that's
might not be a good way for the namespace. Like your
method which only does check and return is better.

-- 
Regards,
Li Wang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linux.it/pipermail/ltp/attachments/20211222/8482a4c7/attachment.htm>