Code review comment for lp:~asac/lava-dispatcher/drop-redundant-sudo-lamc

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

Andy Doan <email address hidden> writes:

> I think the thought was that it would be nice to eventually run the dispatcher as a non-root user. And granting sudoer access to l-a-m-c for a given user would allow that to work. However, there are several other places we require root access using a variety of commands.
>
> So I think this is fine. I'd still like to here Michael's thoughts though.

Don't really care tbh -- getting to the point where we can run the
dispatcher without root is going to be so much work that one sudo here
or there isn't going to make much difference, so I'm happy for asac to
merge this.

Cheers,
mwh

« Back to merge proposal