)]}'
{"neutron/values.yaml":[{"author":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"change_message_id":"47ba0a15435ebdf49f0a491bdf2f91afee23dbe8","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    ks_service: docker.io/openstackhelm/heat:stein-ubuntu_bionic"},{"line_number":32,"context_line":"    ks_endpoints: docker.io/openstackhelm/heat:stein-ubuntu_bionic"},{"line_number":33,"context_line":"    neutron_server: docker.io/openstackhelm/neutron:stein-ubuntu_bionic"},{"line_number":34,"context_line":"    neutron_rpc_server: docker.io/openstackhelm/neutron:stein-ubuntu_bionic"},{"line_number":35,"context_line":"    neutron_dhcp: docker.io/openstackhelm/neutron:stein-ubuntu_bionic"},{"line_number":36,"context_line":"    neutron_metadata: docker.io/openstackhelm/neutron:stein-ubuntu_bionic"},{"line_number":37,"context_line":"    neutron_l3: docker.io/openstackhelm/neutron:stein-ubuntu_bionic"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f65232a_8f5be62c","line":34,"range":{"start_line":34,"start_character":0,"end_line":34,"end_character":75},"updated":"2020-10-21 15:49:02.000000000","message":"should we also swap this out with the nginx image?","commit_id":"d044a811e5e51df6313bfc2b1cef7172620c3ff6"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"0ade706831a60dd984d402ba129e4a60b4b20ed1","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    ks_service: docker.io/openstackhelm/heat:stein-ubuntu_bionic"},{"line_number":32,"context_line":"    ks_endpoints: docker.io/openstackhelm/heat:stein-ubuntu_bionic"},{"line_number":33,"context_line":"    neutron_server: docker.io/openstackhelm/neutron:stein-ubuntu_bionic"},{"line_number":34,"context_line":"    neutron_rpc_server: docker.io/openstackhelm/neutron:stein-ubuntu_bionic"},{"line_number":35,"context_line":"    neutron_dhcp: docker.io/openstackhelm/neutron:stein-ubuntu_bionic"},{"line_number":36,"context_line":"    neutron_metadata: docker.io/openstackhelm/neutron:stein-ubuntu_bionic"},{"line_number":37,"context_line":"    neutron_l3: docker.io/openstackhelm/neutron:stein-ubuntu_bionic"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f65232a_6f780af8","line":34,"range":{"start_line":34,"start_character":0,"end_line":34,"end_character":75},"in_reply_to":"3f65232a_8f5be62c","updated":"2020-10-21 16:03:47.000000000","message":"good catch and no - the nginx image is accounted for in the values override for tls. But this is now extraneous, and will be removed in the next patch set.","commit_id":"d044a811e5e51df6313bfc2b1cef7172620c3ff6"},{"author":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"change_message_id":"57b9e3202ec010a9ebe8b93cbe83b2484260dda3","unresolved":false,"context_lines":[{"line_number":517,"context_line":"      pod:"},{"line_number":518,"context_line":"        runAsUser: 42424"},{"line_number":519,"context_line":"      container:"},{"line_number":520,"context_line":"        neutron_server:"},{"line_number":521,"context_line":"          allowPrivilegeEscalation: false"},{"line_number":522,"context_line":"          readOnlyRootFilesystem: true"},{"line_number":523,"context_line":"    neutron_sriov_agent:"},{"line_number":524,"context_line":"      pod:"},{"line_number":525,"context_line":"        runAsUser: 42424"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f65232a_1eab8a1d","line":522,"range":{"start_line":520,"start_character":13,"end_line":522,"end_character":38},"updated":"2020-10-21 13:59:41.000000000","message":"does the nginx container require any default handling here?","commit_id":"d044a811e5e51df6313bfc2b1cef7172620c3ff6"},{"author":{"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},"change_message_id":"a0e521bb1f7644d7f834c7a1eacdd4b579c39d9f","unresolved":false,"context_lines":[{"line_number":517,"context_line":"      pod:"},{"line_number":518,"context_line":"        runAsUser: 42424"},{"line_number":519,"context_line":"      container:"},{"line_number":520,"context_line":"        neutron_server:"},{"line_number":521,"context_line":"          allowPrivilegeEscalation: false"},{"line_number":522,"context_line":"          readOnlyRootFilesystem: true"},{"line_number":523,"context_line":"    neutron_sriov_agent:"},{"line_number":524,"context_line":"      pod:"},{"line_number":525,"context_line":"        runAsUser: 42424"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f65232a_d16bc31c","line":522,"range":{"start_line":520,"start_character":13,"end_line":522,"end_character":38},"in_reply_to":"3f65232a_11ac1b8b","updated":"2020-10-21 14:42:37.000000000","message":"I tested with allowPrivilegeEscalation: false and readOnlyRootFilesystem: true set for nginx and it did not work. I did not flip the bools and test. You may have to test them.","commit_id":"d044a811e5e51df6313bfc2b1cef7172620c3ff6"},{"author":{"_account_id":18250,"name":"Roy Tang","email":"roy.s.tang@att.com","username":"rt7380"},"change_message_id":"e135315d1e686f8b05c6352553bc4f0e38276154","unresolved":false,"context_lines":[{"line_number":517,"context_line":"      pod:"},{"line_number":518,"context_line":"        runAsUser: 42424"},{"line_number":519,"context_line":"      container:"},{"line_number":520,"context_line":"        neutron_server:"},{"line_number":521,"context_line":"          allowPrivilegeEscalation: false"},{"line_number":522,"context_line":"          readOnlyRootFilesystem: true"},{"line_number":523,"context_line":"    neutron_sriov_agent:"},{"line_number":524,"context_line":"      pod:"},{"line_number":525,"context_line":"        runAsUser: 42424"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f65232a_1176db07","line":522,"range":{"start_line":520,"start_character":13,"end_line":522,"end_character":38},"in_reply_to":"3f65232a_11ac1b8b","updated":"2020-10-21 14:42:55.000000000","message":"can\u0027t argue with pushing things downstream as needed, no need to break things just to break things, please tell Sangeet I said so.","commit_id":"d044a811e5e51df6313bfc2b1cef7172620c3ff6"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"50aa3e519fb984b7304e479ba037044204b9c38a","unresolved":false,"context_lines":[{"line_number":517,"context_line":"      pod:"},{"line_number":518,"context_line":"        runAsUser: 42424"},{"line_number":519,"context_line":"      container:"},{"line_number":520,"context_line":"        neutron_server:"},{"line_number":521,"context_line":"          allowPrivilegeEscalation: false"},{"line_number":522,"context_line":"          readOnlyRootFilesystem: true"},{"line_number":523,"context_line":"    neutron_sriov_agent:"},{"line_number":524,"context_line":"      pod:"},{"line_number":525,"context_line":"        runAsUser: 42424"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f65232a_11ac1b8b","line":522,"range":{"start_line":520,"start_character":13,"end_line":522,"end_character":38},"in_reply_to":"3f65232a_1eab8a1d","updated":"2020-10-21 14:33:33.000000000","message":"According to Sangeet, no additional step is needed. Also this should be handled in the override even if special handling is required.","commit_id":"d044a811e5e51df6313bfc2b1cef7172620c3ff6"}]}