Mao Geng updated MESOS-5909:
    Attachment: MESOS-5909-fix.diff

Just hit this error when running `make check`. Figured out a fix (as [~karya] 
said, sort both lists then compare) to the os_tests.cpp. See attached. 

I am new to Mesos community. If I want to submit a patch, should I create a PR 
on https://github.com/apache/mesos (I saw someone did), or just follow the 
http://mesos.apache.org/documentation/latest/submitting-a-patch/ to create a 
review using post-reviews.py? 

cc [~karya] [~gilbert]

> Stout "OsTest.User" test can fail on some systems
> -------------------------------------------------
>                 Key: MESOS-5909
>                 URL: https://issues.apache.org/jira/browse/MESOS-5909
>             Project: Mesos
>          Issue Type: Bug
>          Components: stout
>            Reporter: Kapil Arya
>            Assignee: Gilbert Song
>              Labels: mesosphere
>         Attachments: MESOS-5909-fix.diff
> Libc call {{getgrouplist}} doesn't return the {{gid}} list in a sorted manner 
> (in my case, it's returning "471 100") ... whereas {{id -G}} return a sorted 
> list ("100 471" in my case) causing the validation inside the loop to fail.
> We should sort both lists before comparing the values.

This message was sent by Atlassian JIRA

Reply via email to