Re: [PATCH v2] base-passwd: Add kvm group

[Jacob Kroon <jacob.kroon@gmail.com>]

Forgot to mention, the number chosen for the kvm group, 47, I just 
picked one of the available slots, no further reasoning behind that choice.

/Jacob

On 10/22/19 8:32 AM, Jacob Kroon wrote:
> Hi Joshua, Mark,
> 
>>> In general, I don't think we should be "patching in" users and groups
>>> like this to base-passwd. If upstream doesn't have them, they need to
>>> be added through the normal useradd.bbclass mechanism so that users of
>>> useradd-staticids can choose what ID they get mapped to.
>>>
>>> Is it possible to revert this and add the kvm group via the 
>>> useradd.bbclass?
>>>
>>>>
>>>>    udevd[<pid>]: specified group 'kvm' unknown
>>
>> As for this particular patch, why is adding a specific KVM group 
>> required?
>> Unless something is hardcoded in KVM to use group '47', a regular 
>> system type
>> useradd call should be done instead.  This would avoid the problem 
>> that Joshua
>> is experiencing.
>>
> 
> The groups "shutdown" and "input" were already patched in in the same 
> way, so I figured that was the way to do it. But it sounds like you're 
> pointing out that we can drop those three patches, and instead use 
> useradd.bbclass in base-passwd recipe ? Sounds like a nice cleanup.
> 
> Cheers,
> Jacob