)]}'
{"features.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"d6f429017fac73c3fa9f799448a2ea2fcc6acc82","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    base:"},{"line_number":113,"context_line":"      services: [cinder, c-api, c-vol, c-sch, c-bak]"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"  cinder-mn-grenade:"},{"line_number":116,"context_line":"    base:"},{"line_number":117,"context_line":"      rm-services: [c-vol]"},{"line_number":118,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3aaa91ec_6071330c","line":115,"updated":"2016-06-30 06:11:44.000000000","message":"Do we really want to do this with a feature, especially embedding branch names in it?\n\nThis seems really confusing just in here like this; a comment that this is a multi-node job and we\u0027re testing the upgraded api-side with the old volume side on the other node (my understanding of what\u0027s going on) is probably appropriate if we do this; because reading this you\u0027d say \"why would i not want c-vol for a cinder job\"?","commit_id":"e8216fd60f16fa4a316defba2cbf84629de18bbd"},{"author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"change_message_id":"839c4d65dc5f80235b3c90eaf90bcced856a759a","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    base:"},{"line_number":113,"context_line":"      services: [cinder, c-api, c-vol, c-sch, c-bak]"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"  cinder-mn-grenade:"},{"line_number":116,"context_line":"    base:"},{"line_number":117,"context_line":"      rm-services: [c-vol]"},{"line_number":118,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3aaa91ec_e3797fc9","line":115,"in_reply_to":"3aaa91ec_6071330c","updated":"2016-06-30 20:58:34.000000000","message":"Hmm, I guess Michal did it because he did not want c-vol on the primary and wanted c-vol with lvm-driver on sub node.","commit_id":"e8216fd60f16fa4a316defba2cbf84629de18bbd"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"ad2c678140905f94d19f941cd2a556860d81bb93","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    base:"},{"line_number":113,"context_line":"      services: [cinder, c-api, c-vol, c-sch, c-bak]"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"  cinder-mn-grenade:"},{"line_number":116,"context_line":"    base:"},{"line_number":117,"context_line":"      rm-services: [c-vol]"},{"line_number":118,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1aa78d24_49c00d7d","line":115,"in_reply_to":"3aaa91ec_e3797fc9","updated":"2016-07-04 19:28:21.000000000","message":"Exactly. I\u0027ll add a comment explaining what is going on here.","commit_id":"e8216fd60f16fa4a316defba2cbf84629de18bbd"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"f94ba383d17db0c639ef64f77d85d0cd3d2f8c69","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    features: [heat]"},{"line_number":48,"context_line":"  tlsproxy:"},{"line_number":49,"context_line":"    features: [tlsproxy]"},{"line_number":50,"context_line":"  cinder:"},{"line_number":51,"context_line":"    features: [cinder, cinder-mn-grenade]"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"branches:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1aa78d24_89e31533","line":50,"range":{"start_line":50,"start_character":2,"end_line":50,"end_character":8},"updated":"2016-07-04 19:28:13.000000000","message":"cinder? I think we want to export something like:\n\n export DEVSTACK_GATE_CINDER_MULTINODE_GRENADE\n\ninstead of confusing\n\n export DEVSTACK_GATE_CINDER\n\nAm I wrong?","commit_id":"855451d81a4e105e1f2f43d01b54c2b5b9e6ea33"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"53b209aefcfa9214d2318bd51adbc629167d6209","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    features: [heat]"},{"line_number":48,"context_line":"  tlsproxy:"},{"line_number":49,"context_line":"    features: [tlsproxy]"},{"line_number":50,"context_line":"  cinder:"},{"line_number":51,"context_line":"    features: [cinder, cinder-mn-grenade]"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"branches:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1aa78d24_f8d1b3ce","line":50,"range":{"start_line":50,"start_character":2,"end_line":50,"end_character":8},"in_reply_to":"1aa78d24_51b70786","updated":"2016-07-05 07:45:04.000000000","message":"I think in neutron case neutron-adv adds more services instead of removing one defined in cinder like here.","commit_id":"855451d81a4e105e1f2f43d01b54c2b5b9e6ea33"},{"author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"change_message_id":"4620597c8ea6e653b7c1dd66e28f6647af8f7c18","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    features: [heat]"},{"line_number":48,"context_line":"  tlsproxy:"},{"line_number":49,"context_line":"    features: [tlsproxy]"},{"line_number":50,"context_line":"  cinder:"},{"line_number":51,"context_line":"    features: [cinder, cinder-mn-grenade]"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"branches:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1aa78d24_51b70786","line":50,"range":{"start_line":50,"start_character":2,"end_line":50,"end_character":8},"in_reply_to":"1aa78d24_89e31533","updated":"2016-07-05 07:06:09.000000000","message":"I was trying to do what they doing with neutron and neutron-adv above.\n\n  neutron:\n    features: [neutron, neutron-adv]\n\nI think i was wrong here. Sorry about that.","commit_id":"855451d81a4e105e1f2f43d01b54c2b5b9e6ea33"},{"author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"change_message_id":"680de9a5a76fa95808b4763610ad87c66935eba1","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    base:"},{"line_number":113,"context_line":"      services: [cinder, c-api, c-vol, c-sch, c-bak]"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"  # This will be used to disable c-vol on primary node when running multinode grenade"},{"line_number":116,"context_line":"  # job that will test compatibility of new c-api, c-sch (primary) and old c-vol and c-bak (sub)."},{"line_number":117,"context_line":"  cinder-mn-grenade:"},{"line_number":118,"context_line":"    base:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_711cc313","line":115,"updated":"2016-07-05 07:04:53.000000000","message":"Ahh I see.. Dont we need to also three more sections removing c-api c-bkp, c-sch each and using new c-vol while retaining the others?\n\nTo understand this,  So we have one primary and one subnode.  So whenever it installs from master, it uses master code for scheduler, api (on primary) and old code(we may need to specify) on sub node.\n\nIs that the intention?","commit_id":"4f3bc0ef76bf42195b54a421c51579a076a7c33a"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"2700da8390b1c0f97b78f742504e5fb2c548fe95","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    base:"},{"line_number":113,"context_line":"      services: [cinder, c-api, c-vol, c-sch, c-bak]"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"  # This will be used to disable c-vol on primary node when running multinode grenade"},{"line_number":116,"context_line":"  # job that will test compatibility of new c-api, c-sch (primary) and old c-vol and c-bak (sub)."},{"line_number":117,"context_line":"  cinder-mn-grenade:"},{"line_number":118,"context_line":"    base:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_9b50690c","line":115,"in_reply_to":"1aa78d24_159f2a51","updated":"2016-07-05 07:51:52.000000000","message":"Actually here we can only define services for primary. Services on subnode are defined explicitly [1].\n\n[1] https://github.com/openstack-infra/devstack-gate/blob/4f3bc0ef76bf42195b54a421c51579a076a7c33a/devstack-vm-gate.sh#L216-L231","commit_id":"4f3bc0ef76bf42195b54a421c51579a076a7c33a"},{"author":{"_account_id":20140,"name":"Karthik Prabhu Vinod","email":"karthik.prabhu.vinod@intel.com","username":"kprabhuv"},"change_message_id":"710e4562ed4676b0eb321a973323a4df47404351","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    base:"},{"line_number":113,"context_line":"      services: [cinder, c-api, c-vol, c-sch, c-bak]"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"  # This will be used to disable c-vol on primary node when running multinode grenade"},{"line_number":116,"context_line":"  # job that will test compatibility of new c-api, c-sch (primary) and old c-vol and c-bak (sub)."},{"line_number":117,"context_line":"  cinder-mn-grenade:"},{"line_number":118,"context_line":"    base:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_159f2a51","line":115,"in_reply_to":"1aa78d24_711cc313","updated":"2016-07-05 07:21:34.000000000","message":"Shouldn\u0027t it something like this?\n\n\ncinder-mn-grenade:\n    base:\n      services: c-api, c-sch\n      rm-services: c-vol, c-bkp\n    liberty:\n      services: c-vol, c-bkp\n      rm-services: c-sch, c-api","commit_id":"4f3bc0ef76bf42195b54a421c51579a076a7c33a"}]}