)]}'
{"kolla/common/config.py":[{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"cbcd7f4ad55de3b69a8d5cb6252c3189a4778685","unresolved":false,"context_lines":[{"line_number":959,"context_line":"        \u0027uid\u0027: 42475,"},{"line_number":960,"context_line":"        \u0027gid\u0027: 42475,"},{"line_number":961,"context_line":"    },"},{"line_number":962,"context_line":"    \u0027openvswitch-user\u0027: {"},{"line_number":963,"context_line":"        \u0027uid\u0027: 42476,"},{"line_number":964,"context_line":"        \u0027gid\u0027: 42476,"},{"line_number":965,"context_line":"    },"},{"line_number":966,"context_line":"    \u0027hugetlbfs-user\u0027: {"},{"line_number":967,"context_line":"        \u0027uid\u0027: 42477, # unused user, but we need the group for vhost socket"},{"line_number":968,"context_line":"        \u0027gid\u0027: 42477,"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_301ad16e","line":965,"range":{"start_line":962,"start_character":0,"end_line":965,"end_character":6},"updated":"2017-12-07 10:10:09.000000000","message":"unused too, we run ovs as root","commit_id":"da7bbfe7c2403ebe59af7c3bab00994c9aeab370"},{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"6664f43753acaadeddc2070db84da4987190de32","unresolved":false,"context_lines":[{"line_number":959,"context_line":"        \u0027uid\u0027: 42475,"},{"line_number":960,"context_line":"        \u0027gid\u0027: 42475,"},{"line_number":961,"context_line":"    },"},{"line_number":962,"context_line":"    \u0027openvswitch-user\u0027: {"},{"line_number":963,"context_line":"        \u0027uid\u0027: 42476,"},{"line_number":964,"context_line":"        \u0027gid\u0027: 42476,"},{"line_number":965,"context_line":"    },"},{"line_number":966,"context_line":"    \u0027hugetlbfs-user\u0027: {"},{"line_number":967,"context_line":"        \u0027uid\u0027: 42477, # unused user, but we need the group for vhost socket"},{"line_number":968,"context_line":"        \u0027gid\u0027: 42477,"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_70de196e","line":965,"range":{"start_line":962,"start_character":0,"end_line":965,"end_character":6},"in_reply_to":"df87a7cf_301ad16e","updated":"2017-12-07 10:16:35.000000000","message":"Yes, currently it is run as root:root. But it is changed in ovs2.8 version, below are the file which does it.\n\nhttps://github.com/openvswitch/ovs/blob/e3e738a3d0580a9a7178adfc9300a193b8df4ae5/rhel/usr_share_openvswitch_scripts_systemd_sysconfig.template#L26\nhttps://github.com/openvswitch/ovs/blob/master/rhel/openvswitch-fedora.spec.in#L394","commit_id":"da7bbfe7c2403ebe59af7c3bab00994c9aeab370"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"876004a476c04180ba42ab6a73a7afb3adf355d1","unresolved":false,"context_lines":[{"line_number":959,"context_line":"        \u0027uid\u0027: 42475,"},{"line_number":960,"context_line":"        \u0027gid\u0027: 42475,"},{"line_number":961,"context_line":"    },"},{"line_number":962,"context_line":"    \u0027openvswitch-user\u0027: {"},{"line_number":963,"context_line":"        \u0027uid\u0027: 42476,"},{"line_number":964,"context_line":"        \u0027gid\u0027: 42476,"},{"line_number":965,"context_line":"    },"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f96bb07_e10bc028","line":962,"updated":"2018-01-19 08:55:01.000000000","message":"this is unused too, please add a comment","commit_id":"d2945cc5fe2a9081b8e966f276b2052044b93765"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"4c92c8afe40af01551acf7e499412d51f341c87c","unresolved":false,"context_lines":[{"line_number":959,"context_line":"        \u0027uid\u0027: 42475,"},{"line_number":960,"context_line":"        \u0027gid\u0027: 42475,"},{"line_number":961,"context_line":"    },"},{"line_number":962,"context_line":"    \u0027openvswitch-user\u0027: {"},{"line_number":963,"context_line":"        \u0027uid\u0027: 42476,"},{"line_number":964,"context_line":"        \u0027gid\u0027: 42476,"},{"line_number":965,"context_line":"    },"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f96bb07_c7b3344d","line":962,"in_reply_to":"7f96bb07_644b3e5f","updated":"2018-01-19 09:39:02.000000000","message":"I mean, the ovs user is not defined in any dockerfile, please add a comment as in the other user:\n\n# unused user\n\nIn our case, ovs containers are running as root","commit_id":"d2945cc5fe2a9081b8e966f276b2052044b93765"},{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"3b1d4e8d23b48c567c40af1fa64f6dcab3ed1bff","unresolved":false,"context_lines":[{"line_number":959,"context_line":"        \u0027uid\u0027: 42475,"},{"line_number":960,"context_line":"        \u0027gid\u0027: 42475,"},{"line_number":961,"context_line":"    },"},{"line_number":962,"context_line":"    \u0027openvswitch-user\u0027: {"},{"line_number":963,"context_line":"        \u0027uid\u0027: 42476,"},{"line_number":964,"context_line":"        \u0027gid\u0027: 42476,"},{"line_number":965,"context_line":"    },"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f96bb07_87920c4b","line":962,"in_reply_to":"7f96bb07_c7b3344d","updated":"2018-01-19 09:52:55.000000000","message":"Sure, thanks.","commit_id":"d2945cc5fe2a9081b8e966f276b2052044b93765"},{"author":{"_account_id":18575,"name":"Saravanan KR","email":"krsacme@gmail.com","username":"saravanankr"},"change_message_id":"366647af41b71140981cefa7d1f0414e102b514d","unresolved":false,"context_lines":[{"line_number":959,"context_line":"        \u0027uid\u0027: 42475,"},{"line_number":960,"context_line":"        \u0027gid\u0027: 42475,"},{"line_number":961,"context_line":"    },"},{"line_number":962,"context_line":"    \u0027openvswitch-user\u0027: {"},{"line_number":963,"context_line":"        \u0027uid\u0027: 42476,"},{"line_number":964,"context_line":"        \u0027gid\u0027: 42476,"},{"line_number":965,"context_line":"    },"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f96bb07_644b3e5f","line":962,"in_reply_to":"7f96bb07_e10bc028","updated":"2018-01-19 09:28:41.000000000","message":"By default ovs will use openvswitch:openvswitch permissions. But when DPDK is enabled in the package, the permissions will be openvswitch:hugetlbfs.\n\nSo in principle openvswitch user and group will be used. And also hugetlbfs group will be used. \n\nI am confused on which comment you are talking off?","commit_id":"d2945cc5fe2a9081b8e966f276b2052044b93765"}]}