)]}'
{"tools/deployment/component/common/ingress.sh":[{"author":{"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},"change_message_id":"7e88c5a5ea45e1c5938089f64b782644bdc1eecc","unresolved":false,"context_lines":[{"line_number":77,"context_line":"#NOTE: Display info"},{"line_number":78,"context_line":"helm status ingress-openstack"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"helm upgrade --install ingress-ceph ${HELM_CHART_ROOT_PATH}/ingress \\"},{"line_number":81,"context_line":"  --namespace\u003dceph \\"},{"line_number":82,"context_line":"  --values\u003d/tmp/ingress-component.yaml \\"},{"line_number":83,"context_line":"  ${OSH_EXTRA_HELM_ARGS} \\"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3fa7e38b_820bb229","line":80,"updated":"2019-10-21 10:03:53.000000000","message":"do we want to always install ingress-ceph even on jobs/deployments that dont use ceph?","commit_id":"d893eadfbceceacdf8c1ee2dbb0f25c4ea2a99c8"},{"author":{"_account_id":9963,"name":"Jiri Suchomel","email":"jiri.suchomel@suse.com","username":"jsuchome"},"change_message_id":"5058c43de8497cbd657123c32c10d54bb7603da8","unresolved":false,"context_lines":[{"line_number":77,"context_line":"#NOTE: Display info"},{"line_number":78,"context_line":"helm status ingress-openstack"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"helm upgrade --install ingress-ceph ${HELM_CHART_ROOT_PATH}/ingress \\"},{"line_number":81,"context_line":"  --namespace\u003dceph \\"},{"line_number":82,"context_line":"  --values\u003d/tmp/ingress-component.yaml \\"},{"line_number":83,"context_line":"  ${OSH_EXTRA_HELM_ARGS} \\"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3fa7e38b_8172bf48","line":80,"in_reply_to":"3fa7e38b_820bb229","updated":"2019-10-22 07:44:16.000000000","message":"Yeah, this is actually changed behavior.\n\nOriginally I thought I would add extra variable (but what should be the default? developer and multinode script used to setup ceph ingres, but common script did not do it...)\n\nUltimately I thought it does not hurt.","commit_id":"d893eadfbceceacdf8c1ee2dbb0f25c4ea2a99c8"},{"author":{"_account_id":12281,"name":"Itxaka","email":"igarcia@suse.com","username":"itxaka"},"change_message_id":"7e88c5a5ea45e1c5938089f64b782644bdc1eecc","unresolved":false,"context_lines":[{"line_number":78,"context_line":"helm status ingress-openstack"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"helm upgrade --install ingress-ceph ${HELM_CHART_ROOT_PATH}/ingress \\"},{"line_number":81,"context_line":"  --namespace\u003dceph \\"},{"line_number":82,"context_line":"  --values\u003d/tmp/ingress-component.yaml \\"},{"line_number":83,"context_line":"  ${OSH_EXTRA_HELM_ARGS} \\"},{"line_number":84,"context_line":"  ${OSH_EXTRA_HELM_ARGS_INGRESS} \\"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3fa7e38b_422cdaab","line":81,"updated":"2019-10-21 10:03:53.000000000","message":"is it possible that the namespace doesnt exist for jobs that dont use ceph? (some are using nfs I think)","commit_id":"d893eadfbceceacdf8c1ee2dbb0f25c4ea2a99c8"},{"author":{"_account_id":9963,"name":"Jiri Suchomel","email":"jiri.suchomel@suse.com","username":"jsuchome"},"change_message_id":"5058c43de8497cbd657123c32c10d54bb7603da8","unresolved":false,"context_lines":[{"line_number":78,"context_line":"helm status ingress-openstack"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"helm upgrade --install ingress-ceph ${HELM_CHART_ROOT_PATH}/ingress \\"},{"line_number":81,"context_line":"  --namespace\u003dceph \\"},{"line_number":82,"context_line":"  --values\u003d/tmp/ingress-component.yaml \\"},{"line_number":83,"context_line":"  ${OSH_EXTRA_HELM_ARGS} \\"},{"line_number":84,"context_line":"  ${OSH_EXTRA_HELM_ARGS_INGRESS} \\"}],"source_content_type":"text/x-sh","patch_set":1,"id":"3fa7e38b_01b56fcd","line":81,"in_reply_to":"3fa7e38b_422cdaab","updated":"2019-10-22 07:44:16.000000000","message":"I\u0027m not sure - do you know who\u0027s creating the default namespaces?\nCI does not seem to complain though...","commit_id":"d893eadfbceceacdf8c1ee2dbb0f25c4ea2a99c8"}]}