)]}'
{"cinder/values.yaml":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"adc43e0f3e610c6ebf2387b0b8e9f4108e331806","unresolved":false,"context_lines":[{"line_number":1249,"context_line":"    ingress:"},{"line_number":1250,"context_line":"      - {}"},{"line_number":1251,"context_line":"    egress:"},{"line_number":1252,"context_line":"      - {}"},{"line_number":1253,"context_line":"      - to:"},{"line_number":1254,"context_line":"        - podSelector:"},{"line_number":1255,"context_line":"            matchLabels:"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"3f79a3b5_b89e584f","line":1252,"range":{"start_line":1252,"start_character":0,"end_line":1252,"end_character":10},"updated":"2018-11-12 16:34:00.000000000","message":"wouldn\u0027t this just open all ports on the egress?","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"},{"author":{"_account_id":28592,"name":"Mike Pham","email":"tp6510@att.com","username":"tp6510"},"change_message_id":"326c29caca069616063f56e2f3a0b0b7f0a5288d","unresolved":false,"context_lines":[{"line_number":1249,"context_line":"    ingress:"},{"line_number":1250,"context_line":"      - {}"},{"line_number":1251,"context_line":"    egress:"},{"line_number":1252,"context_line":"      - {}"},{"line_number":1253,"context_line":"      - to:"},{"line_number":1254,"context_line":"        - podSelector:"},{"line_number":1255,"context_line":"            matchLabels:"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"3f79a3b5_b3aaf9cc","line":1252,"range":{"start_line":1252,"start_character":0,"end_line":1252,"end_character":10},"in_reply_to":"3f79a3b5_b89e584f","updated":"2018-12-13 17:30:48.000000000","message":"It need to tighten up later when Calico network policy is enable. Cinder need to talk to Ceph.","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"adc43e0f3e610c6ebf2387b0b8e9f4108e331806","unresolved":false,"context_lines":[{"line_number":1267,"context_line":"            matchLabels:"},{"line_number":1268,"context_line":"              application: cinder"},{"line_number":1269,"context_line":"      - to:"},{"line_number":1270,"context_line":"        - namespaceSelector:"},{"line_number":1271,"context_line":"            matchLabels:"},{"line_number":1272,"context_line":"              name: default"},{"line_number":1273,"context_line":"        - namespaceSelector:"},{"line_number":1274,"context_line":"            matchLabels:"},{"line_number":1275,"context_line":"              name: kube-public"},{"line_number":1276,"context_line":"        - namespaceSelector:"},{"line_number":1277,"context_line":"            matchLabels:"},{"line_number":1278,"context_line":"              name: kube-system"},{"line_number":1279,"context_line":"        - namespaceSelector:"},{"line_number":1280,"context_line":"            matchLabels:"},{"line_number":1281,"context_line":"              name: ceph"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"3f79a3b5_18050c68","line":1278,"range":{"start_line":1270,"start_character":0,"end_line":1278,"end_character":31},"updated":"2018-11-12 16:34:00.000000000","message":"shouldn\u0027t these be part of the values.yaml so we don\u0027t need to explicitly specify kube-* connect in the egress?","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"adc43e0f3e610c6ebf2387b0b8e9f4108e331806","unresolved":false,"context_lines":[{"line_number":1287,"context_line":"          protocol: TCP"},{"line_number":1288,"context_line":"        - port: 80"},{"line_number":1289,"context_line":"          protocol: TCP"},{"line_number":1290,"context_line":"        - port: 5000"},{"line_number":1291,"context_line":"          protocol: TCP"},{"line_number":1292,"context_line":""},{"line_number":1293,"context_line":"manifests:"},{"line_number":1294,"context_line":"  configmap_bin: true"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"3f79a3b5_3808087d","line":1291,"range":{"start_line":1290,"start_character":8,"end_line":1291,"end_character":23},"updated":"2018-11-12 16:34:00.000000000","message":"is this port for keystone?","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"}],"heat/values.yaml":[{"author":{"_account_id":22636,"name":"Cliff Parsons","email":"cliffhparsons@aol.com","username":"cliffparsons"},"change_message_id":"b38c27aa63fdf001a72f67a6ea30dfa04c77dc79","unresolved":false,"context_lines":[{"line_number":942,"context_line":"        internal: 5000"},{"line_number":943,"context_line":"      http:"},{"line_number":944,"context_line":"        default: 80"},{"line_number":945,"context_line":"  default:"},{"line_number":946,"context_line":"    namespace: default"},{"line_number":947,"context_line":"  kube_system:"},{"line_number":948,"context_line":"    namespace: kube-system"},{"line_number":949,"context_line":"  kube_public:"},{"line_number":950,"context_line":"    namespace: kube-public"},{"line_number":951,"context_line":""},{"line_number":952,"context_line":"pod:"},{"line_number":953,"context_line":"  user:"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"3f79a3b5_a74b5bb1","line":950,"range":{"start_line":945,"start_character":0,"end_line":950,"end_character":26},"updated":"2018-12-13 22:33:16.000000000","message":"Are these really endpoints? If I was just reading through this values.yaml document, it would be unclear to me as to why these values are here. Can there be some indicator that these are for egress network policy enforcement? Seems like these values should be in a separate section.","commit_id":"c4f6ff91a168ac67cd7657df4b6cc9d160f9fb0d"},{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"f3a140827b5952777996742635ebd87325f13a2a","unresolved":false,"context_lines":[{"line_number":942,"context_line":"        internal: 5000"},{"line_number":943,"context_line":"      http:"},{"line_number":944,"context_line":"        default: 80"},{"line_number":945,"context_line":"  default:"},{"line_number":946,"context_line":"    namespace: default"},{"line_number":947,"context_line":"  kube_system:"},{"line_number":948,"context_line":"    namespace: kube-system"},{"line_number":949,"context_line":"  kube_public:"},{"line_number":950,"context_line":"    namespace: kube-public"},{"line_number":951,"context_line":""},{"line_number":952,"context_line":"pod:"},{"line_number":953,"context_line":"  user:"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"3f79a3b5_c7a45750","line":950,"range":{"start_line":945,"start_character":0,"end_line":950,"end_character":26},"in_reply_to":"3f79a3b5_a74b5bb1","updated":"2018-12-13 22:41:41.000000000","message":"I don\u0027t hate the idea of these being here, personally.  I think these must be injected into the generated policy to to allow for things like DNS lookups and apiserver access, and making them overrideable in values.yaml allows operators to customize based on different k8s setups.  Definitely not \"normal\" endpoints, but endpoint-ish? :)  I agree that a comment on their purpose for network policy would be a good add.","commit_id":"c4f6ff91a168ac67cd7657df4b6cc9d160f9fb0d"},{"author":{"_account_id":28592,"name":"Mike Pham","email":"tp6510@att.com","username":"tp6510"},"change_message_id":"b38fe39f04a1a9ab75d5bdc72101bb944577f964","unresolved":false,"context_lines":[{"line_number":942,"context_line":"        internal: 5000"},{"line_number":943,"context_line":"      http:"},{"line_number":944,"context_line":"        default: 80"},{"line_number":945,"context_line":"  default:"},{"line_number":946,"context_line":"    namespace: default"},{"line_number":947,"context_line":"  kube_system:"},{"line_number":948,"context_line":"    namespace: kube-system"},{"line_number":949,"context_line":"  kube_public:"},{"line_number":950,"context_line":"    namespace: kube-public"},{"line_number":951,"context_line":""},{"line_number":952,"context_line":"pod:"},{"line_number":953,"context_line":"  user:"}],"source_content_type":"text/x-yaml","patch_set":54,"id":"3f79a3b5_230e5130","line":950,"range":{"start_line":945,"start_character":0,"end_line":950,"end_character":26},"in_reply_to":"3f79a3b5_c7a45750","updated":"2018-12-14 14:27:15.000000000","message":"Yes, i agree with Matt.","commit_id":"c4f6ff91a168ac67cd7657df4b6cc9d160f9fb0d"}],"horizon/values.yaml":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"adc43e0f3e610c6ebf2387b0b8e9f4108e331806","unresolved":false,"context_lines":[{"line_number":2076,"context_line":"              name: kube-system"},{"line_number":2077,"context_line":"    egress:"},{"line_number":2078,"context_line":"      - to:"},{"line_number":2079,"context_line":"        - namespaceSelector:"},{"line_number":2080,"context_line":"            matchLabels:"},{"line_number":2081,"context_line":"              name: default"},{"line_number":2082,"context_line":"        - namespaceSelector:"},{"line_number":2083,"context_line":"            matchLabels:"},{"line_number":2084,"context_line":"              name: kube-public"},{"line_number":2085,"context_line":"        - namespaceSelector:"},{"line_number":2086,"context_line":"            matchLabels:"},{"line_number":2087,"context_line":"              name: kube-system"},{"line_number":2088,"context_line":"      - to:"},{"line_number":2089,"context_line":"        ports:"},{"line_number":2090,"context_line":"        - port: 443"},{"line_number":2091,"context_line":"          protocol: TCP"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"3f79a3b5_f81db0be","line":2088,"range":{"start_line":2079,"start_character":0,"end_line":2088,"end_character":11},"updated":"2018-11-12 16:34:00.000000000","message":"same - can these be generated from values.yaml","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"}],"keystone/values.yaml":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"adc43e0f3e610c6ebf2387b0b8e9f4108e331806","unresolved":false,"context_lines":[{"line_number":409,"context_line":"        - podSelector:"},{"line_number":410,"context_line":"            matchLabels:"},{"line_number":411,"context_line":"              application: ceph"},{"line_number":412,"context_line":"      - to:"},{"line_number":413,"context_line":"        - namespaceSelector:"},{"line_number":414,"context_line":"            matchLabels:"},{"line_number":415,"context_line":"              name: default"},{"line_number":416,"context_line":"        - namespaceSelector:"},{"line_number":417,"context_line":"            matchLabels:"},{"line_number":418,"context_line":"              name: kube-public"},{"line_number":419,"context_line":"        - namespaceSelector:"},{"line_number":420,"context_line":"            matchLabels:"},{"line_number":421,"context_line":"              name: kube-system"},{"line_number":422,"context_line":"        - namespaceSelector:"},{"line_number":423,"context_line":"            matchLabels:"},{"line_number":424,"context_line":"              name: ceph"},{"line_number":425,"context_line":"      - to:"},{"line_number":426,"context_line":"        ports:"},{"line_number":427,"context_line":"        - port: 443"},{"line_number":428,"context_line":"          protocol: TCP"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"3f79a3b5_98183cce","line":425,"range":{"start_line":412,"start_character":0,"end_line":425,"end_character":11},"updated":"2018-11-12 16:34:00.000000000","message":"ditto","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"}],"tools/deployment/developer/ceph/150-libvirt.sh":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"adc43e0f3e610c6ebf2387b0b8e9f4108e331806","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    ingress:"},{"line_number":28,"context_line":"      - {}"},{"line_number":29,"context_line":"    egress:"},{"line_number":30,"context_line":"      - {}"},{"line_number":31,"context_line":"      - to:"},{"line_number":32,"context_line":"        - podSelector:"},{"line_number":33,"context_line":"            matchLabels:"}],"source_content_type":"text/x-sh","patch_set":26,"id":"3f79a3b5_7860c034","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":10},"updated":"2018-11-12 16:34:00.000000000","message":"ditto.","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"adc43e0f3e610c6ebf2387b0b8e9f4108e331806","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        - podSelector:"},{"line_number":42,"context_line":"            matchLabels:"},{"line_number":43,"context_line":"              application: cinder"},{"line_number":44,"context_line":"      - to:"},{"line_number":45,"context_line":"        - namespaceSelector:"},{"line_number":46,"context_line":"            matchLabels:"},{"line_number":47,"context_line":"              name: default"},{"line_number":48,"context_line":"        - namespaceSelector:"},{"line_number":49,"context_line":"            matchLabels:"},{"line_number":50,"context_line":"              name: kube-public"},{"line_number":51,"context_line":"        - namespaceSelector:"},{"line_number":52,"context_line":"            matchLabels:"},{"line_number":53,"context_line":"              name: kube-system"},{"line_number":54,"context_line":"        - namespaceSelector:"},{"line_number":55,"context_line":"            matchLabels:"},{"line_number":56,"context_line":"              name: ceph"},{"line_number":57,"context_line":"      - to:"},{"line_number":58,"context_line":"        ports:"},{"line_number":59,"context_line":"        - port: 443"},{"line_number":60,"context_line":"          protocol: TCP"}],"source_content_type":"text/x-sh","patch_set":26,"id":"3f79a3b5_3856c85e","line":57,"range":{"start_line":44,"start_character":0,"end_line":57,"end_character":11},"updated":"2018-11-12 16:34:00.000000000","message":"ditto","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"ea3708d0c1748556e069d779a06c84caa3037ce2","unresolved":false,"context_lines":[{"line_number":33,"context_line":"            matchLabels:"},{"line_number":34,"context_line":"              application: neutron"},{"line_number":35,"context_line":"      - to:"},{"line_number":36,"context_line":"        - namespaceSelector:"},{"line_number":37,"context_line":"            matchLabels:"},{"line_number":38,"context_line":"              name: default"},{"line_number":39,"context_line":"        - namespaceSelector:"},{"line_number":40,"context_line":"            matchLabels:"},{"line_number":41,"context_line":"              name: kube-public"},{"line_number":42,"context_line":"        - namespaceSelector:"},{"line_number":43,"context_line":"            matchLabels:"},{"line_number":44,"context_line":"              name: kube-system"},{"line_number":45,"context_line":"        - namespaceSelector:"},{"line_number":46,"context_line":"            matchLabels:"},{"line_number":47,"context_line":"              name: ceph"}],"source_content_type":"text/x-sh","patch_set":35,"id":"3f79a3b5_98c32e8d","line":44,"range":{"start_line":36,"start_character":2,"end_line":44,"end_character":31},"updated":"2018-11-15 04:09:57.000000000","message":"should these be part of endpoints?","commit_id":"74e0a406eadb8ec02c5ae218928e3f479f802e14"}],"tools/deployment/developer/common/030-ingress.sh":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"ea3708d0c1748556e069d779a06c84caa3037ce2","unresolved":false,"context_lines":[{"line_number":90,"context_line":"        - port: 8004"},{"line_number":91,"context_line":"          protocol: TCP"},{"line_number":92,"context_line":"      - to:"},{"line_number":93,"context_line":"        - podSelector:"},{"line_number":94,"context_line":"            matchLabels:"},{"line_number":95,"context_line":"              application: ingress"},{"line_number":96,"context_line":"        - podSelector:"},{"line_number":97,"context_line":"            matchLabels:"},{"line_number":98,"context_line":"              application: glance"},{"line_number":99,"context_line":"        - podSelector:"},{"line_number":100,"context_line":"            matchLabels:"},{"line_number":101,"context_line":"              application: neutron"},{"line_number":102,"context_line":"        - podSelector:"},{"line_number":103,"context_line":"            matchLabels:"},{"line_number":104,"context_line":"              application: nova"},{"line_number":105,"context_line":"        - podSelector:"},{"line_number":106,"context_line":"            matchLabels:"},{"line_number":107,"context_line":"              application: keystone"},{"line_number":108,"context_line":"        - podSelector:"},{"line_number":109,"context_line":"            matchLabels:"},{"line_number":110,"context_line":"              application: ironic"},{"line_number":111,"context_line":"        - podSelector:"},{"line_number":112,"context_line":"            matchLabels:"},{"line_number":113,"context_line":"              application: placement"},{"line_number":114,"context_line":"        - podSelector:"},{"line_number":115,"context_line":"            matchLabels:"},{"line_number":116,"context_line":"              application: cinder"},{"line_number":117,"context_line":"        - podSelector:"},{"line_number":118,"context_line":"            matchLabels:"},{"line_number":119,"context_line":"              application: libvirt"},{"line_number":120,"context_line":"        - podSelector:"},{"line_number":121,"context_line":"            matchLabels:"},{"line_number":122,"context_line":"              application: openvswitch"},{"line_number":123,"context_line":"EOF"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"#NOTE: Deploy command"}],"source_content_type":"text/x-sh","patch_set":35,"id":"3f79a3b5_78bad20c","line":122,"range":{"start_line":93,"start_character":0,"end_line":122,"end_character":38},"updated":"2018-11-15 04:09:57.000000000","message":"for the egress of the ingress controller, it might be okay to open it up for everything - it is guaranteed that this will need to reach everything.","commit_id":"74e0a406eadb8ec02c5ae218928e3f479f802e14"}],"tools/deployment/developer/common/060-rabbitmq.sh":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"adc43e0f3e610c6ebf2387b0b8e9f4108e331806","unresolved":false,"context_lines":[{"line_number":81,"context_line":"          port: 20000"},{"line_number":82,"context_line":"    egress:"},{"line_number":83,"context_line":"      - to:"},{"line_number":84,"context_line":"        - namespaceSelector:"},{"line_number":85,"context_line":"            matchLabels:"},{"line_number":86,"context_line":"              name: default"},{"line_number":87,"context_line":"        - namespaceSelector:"},{"line_number":88,"context_line":"            matchLabels:"},{"line_number":89,"context_line":"              name: kube-public"},{"line_number":90,"context_line":"        - namespaceSelector:"},{"line_number":91,"context_line":"            matchLabels:"},{"line_number":92,"context_line":"              name: kube-system"},{"line_number":93,"context_line":"      - to:"},{"line_number":94,"context_line":"        ports:"},{"line_number":95,"context_line":"        - port: 443"}],"source_content_type":"text/x-sh","patch_set":26,"id":"3f79a3b5_585dc479","line":92,"range":{"start_line":84,"start_character":0,"end_line":92,"end_character":31},"updated":"2018-11-12 16:34:00.000000000","message":"ditto","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"}],"tools/deployment/developer/common/140-openvswitch.sh":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"adc43e0f3e610c6ebf2387b0b8e9f4108e331806","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    ingress:"},{"line_number":28,"context_line":"      - {}"},{"line_number":29,"context_line":"    egress:"},{"line_number":30,"context_line":"      - {}"},{"line_number":31,"context_line":"      - to:"},{"line_number":32,"context_line":"        - podSelector:"},{"line_number":33,"context_line":"            matchLabels:"}],"source_content_type":"text/x-sh","patch_set":26,"id":"3f79a3b5_b868981e","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":10},"updated":"2018-11-12 16:34:00.000000000","message":"ditto","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"},{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"75ae5b0138cf75405a4902f22f39a4bbada62cad","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  openvswitch:"},{"line_number":27,"context_line":"    ingress:"},{"line_number":28,"context_line":"      - {}"},{"line_number":29,"context_line":"    egress:"},{"line_number":30,"context_line":"      - {}"},{"line_number":31,"context_line":"EOF"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-sh","patch_set":54,"id":"3f79a3b5_e794f3e8","line":29,"updated":"2018-12-13 22:37:59.000000000","message":"Why do we need to override the ingress \u0026 egress network policy for testing?  I\u0027m a little uncomfortable with that - seems like our default policy should work by default, and if we need to override them to make the gates work, maybe we have the wrong defaults.  But I may also be missing something, let me know.","commit_id":"c4f6ff91a168ac67cd7657df4b6cc9d160f9fb0d"},{"author":{"_account_id":28592,"name":"Mike Pham","email":"tp6510@att.com","username":"tp6510"},"change_message_id":"b38fe39f04a1a9ab75d5bdc72101bb944577f964","unresolved":false,"context_lines":[{"line_number":26,"context_line":"  openvswitch:"},{"line_number":27,"context_line":"    ingress:"},{"line_number":28,"context_line":"      - {}"},{"line_number":29,"context_line":"    egress:"},{"line_number":30,"context_line":"      - {}"},{"line_number":31,"context_line":"EOF"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-sh","patch_set":54,"id":"3f79a3b5_b1106a5d","line":29,"in_reply_to":"3f79a3b5_e794f3e8","updated":"2018-12-14 14:27:15.000000000","message":"Yes, the default network policy is working. You are right, we don\u0027t need to override it here.","commit_id":"c4f6ff91a168ac67cd7657df4b6cc9d160f9fb0d"}],"tools/deployment/developer/nfs/150-libvirt.sh":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"ea3708d0c1748556e069d779a06c84caa3037ce2","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        - podSelector:"},{"line_number":39,"context_line":"            matchLabels:"},{"line_number":40,"context_line":"              application: openvswitch"},{"line_number":41,"context_line":"        - podSelector:"},{"line_number":42,"context_line":"            matchLabels:"},{"line_number":43,"context_line":"              application: cinder"},{"line_number":44,"context_line":"      - to:"},{"line_number":45,"context_line":"        - namespaceSelector:"},{"line_number":46,"context_line":"            matchLabels:"}],"source_content_type":"text/x-sh","patch_set":35,"id":"3f79a3b5_d82e865a","line":43,"range":{"start_line":41,"start_character":0,"end_line":43,"end_character":33},"updated":"2018-11-15 04:09:57.000000000","message":"does libvirt need to hit cinder?","commit_id":"74e0a406eadb8ec02c5ae218928e3f479f802e14"}],"zuul.d/project.yaml":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"adc43e0f3e610c6ebf2387b0b8e9f4108e331806","unresolved":false,"context_lines":[{"line_number":25,"context_line":"      jobs:"},{"line_number":26,"context_line":"        - openstack-helm-lint"},{"line_number":27,"context_line":"        - openstack-helm-dev-deploy-ceph-swift-ovs:"},{"line_number":28,"context_line":"            voting: false"},{"line_number":29,"context_line":"        - openstack-helm-dev-deploy-ceph-radosgw-ovs:"},{"line_number":30,"context_line":"            voting: false"},{"line_number":31,"context_line":"        - openstack-helm-dev-deploy-ceph-rbd-ovs:"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"3f79a3b5_d8679451","line":28,"range":{"start_line":28,"start_character":12,"end_line":28,"end_character":25},"updated":"2018-11-12 16:34:00.000000000","message":"we need to have these voting, else the only gate job we have is openstack-helm-dev-deploy-nfs-ovs","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"},{"author":{"_account_id":28592,"name":"Mike Pham","email":"tp6510@att.com","username":"tp6510"},"change_message_id":"326c29caca069616063f56e2f3a0b0b7f0a5288d","unresolved":false,"context_lines":[{"line_number":25,"context_line":"      jobs:"},{"line_number":26,"context_line":"        - openstack-helm-lint"},{"line_number":27,"context_line":"        - openstack-helm-dev-deploy-ceph-swift-ovs:"},{"line_number":28,"context_line":"            voting: false"},{"line_number":29,"context_line":"        - openstack-helm-dev-deploy-ceph-radosgw-ovs:"},{"line_number":30,"context_line":"            voting: false"},{"line_number":31,"context_line":"        - openstack-helm-dev-deploy-ceph-rbd-ovs:"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"3f79a3b5_9341fd39","line":28,"range":{"start_line":28,"start_character":12,"end_line":28,"end_character":25},"in_reply_to":"3f79a3b5_d8679451","updated":"2018-12-13 17:30:48.000000000","message":"Done","commit_id":"d13dab759e1376648ea2127074705916c3a5f3ba"}]}