)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6579,"name":"Nir Magnezi","email":"nmagnezi@redhat.com","username":"nmagnezi"},"change_message_id":"649c2716d30c871d92cfe6e731697e73d9c824ea","unresolved":false,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2017-06-28 10:39:42 +0530"},{"line_number":4,"context_line":"Commit:     Santhosh Fernandes \u003csanthosh.fernandes@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-07-08 08:35:02 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"In large build situations, nova can be slow to build VMs, this means that the"},{"line_number":8,"context_line":"default 100 second timeout may expire before the final status has been updated"},{"line_number":9,"context_line":"in the neutron database. This patch will emit provisioning status to be sync"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3f1d235d_14736282","line":6,"updated":"2017-07-11 09:25:11.000000000","message":"Please add a topic and a spaced line.\nAlso, please check https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"4c878d70a01180f671f26c94a677016c211fcddd"},{"author":{"_account_id":23889,"name":"Santhosh Fernandes","email":"santhosh.fernandes@gmail.com","username":"santhosh-fernandes"},"change_message_id":"f04e87b725d5ee2d2c1349f22cc13866ca1b9a86","unresolved":false,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2017-06-28 10:39:42 +0530"},{"line_number":4,"context_line":"Commit:     Santhosh Fernandes \u003csanthosh.fernandes@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-07-08 08:35:02 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"In large build situations, nova can be slow to build VMs, this means that the"},{"line_number":8,"context_line":"default 100 second timeout may expire before the final status has been updated"},{"line_number":9,"context_line":"in the neutron database. This patch will emit provisioning status to be sync"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"3f1d235d_63ebfa11","line":6,"in_reply_to":"3f1d235d_14736282","updated":"2017-07-11 11:37:57.000000000","message":"Done","commit_id":"4c878d70a01180f671f26c94a677016c211fcddd"}],"etc/octavia.conf":[{"author":{"_account_id":6579,"name":"Nir Magnezi","email":"nmagnezi@redhat.com","username":"nmagnezi"},"change_message_id":"649c2716d30c871d92cfe6e731697e73d9c824ea","unresolved":false,"context_lines":[{"line_number":71,"context_line":"#                            queue_event_streamer,"},{"line_number":72,"context_line":"#                            noop_event_streamer"},{"line_number":73,"context_line":"# event_streamer_driver \u003d noop_event_streamer"},{"line_number":74,"context_line":"# sync_provisioning_status \u003d False"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"[keystone_authtoken]"},{"line_number":77,"context_line":"# This group of config options are imported from keystone middleware. Thus the"}],"source_content_type":"text/plain","patch_set":9,"id":"3f1d235d_f4d6a653","line":74,"updated":"2017-07-11 09:25:11.000000000","message":"Add the help string to this example file as well:\n\"Enable provisioning status to sync with neutron db\"","commit_id":"4c878d70a01180f671f26c94a677016c211fcddd"},{"author":{"_account_id":23889,"name":"Santhosh Fernandes","email":"santhosh.fernandes@gmail.com","username":"santhosh-fernandes"},"change_message_id":"f04e87b725d5ee2d2c1349f22cc13866ca1b9a86","unresolved":false,"context_lines":[{"line_number":71,"context_line":"#                            queue_event_streamer,"},{"line_number":72,"context_line":"#                            noop_event_streamer"},{"line_number":73,"context_line":"# event_streamer_driver \u003d noop_event_streamer"},{"line_number":74,"context_line":"# sync_provisioning_status \u003d False"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"[keystone_authtoken]"},{"line_number":77,"context_line":"# This group of config options are imported from keystone middleware. Thus the"}],"source_content_type":"text/plain","patch_set":9,"id":"3f1d235d_63865a68","line":74,"in_reply_to":"3f1d235d_f4d6a653","updated":"2017-07-11 11:37:57.000000000","message":"Done","commit_id":"4c878d70a01180f671f26c94a677016c211fcddd"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"3c43d6258eb0ce8e30482c4ebc0195c7a67fe6be","unresolved":false,"context_lines":[{"line_number":72,"context_line":"#                            noop_event_streamer"},{"line_number":73,"context_line":"# event_streamer_driver \u003d noop_event_streamer"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"# Enable provisioning status to sync with neutron db"},{"line_number":76,"context_line":"# sync_provisioning_status \u003d False"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"[keystone_authtoken]"}],"source_content_type":"text/plain","patch_set":11,"id":"1f1a1f67_fa56345c","line":75,"updated":"2017-07-18 05:43:00.000000000","message":"*nitpick: remove \"to\"","commit_id":"adaae2748e181ffed649a014b9dc5f9cc3a732c9"}],"octavia/common/config.py":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"3c43d6258eb0ce8e30482c4ebc0195c7a67fe6be","unresolved":false,"context_lines":[{"line_number":185,"context_line":"                      \u0027noop_event_streamer\u0027),"},{"line_number":186,"context_line":"               default\u003d\u0027noop_event_streamer\u0027),"},{"line_number":187,"context_line":"    cfg.BoolOpt(\u0027sync_provisioning_status\u0027, default\u003dFalse,"},{"line_number":188,"context_line":"                help\u003d_(\"Enable provisioning status to sync with neutron db\"))]"},{"line_number":189,"context_line":""},{"line_number":190,"context_line":"oslo_messaging_opts \u003d ["},{"line_number":191,"context_line":"    cfg.StrOpt(\u0027topic\u0027),"}],"source_content_type":"text/x-python","patch_set":11,"id":"1f1a1f67_9a70600a","line":188,"updated":"2017-07-18 05:43:00.000000000","message":"*nitpick: remove \"to\"","commit_id":"adaae2748e181ffed649a014b9dc5f9cc3a732c9"}],"octavia/controller/healthmanager/update_db.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"3c902a69d002bc0e8d8867a81810b14c65738b62","unresolved":false,"context_lines":[{"line_number":59,"context_line":"            repo.update(session, entity_id, operating_status\u003dnew_op_status)"},{"line_number":60,"context_line":"            self.emit("},{"line_number":61,"context_line":"                entity_type, entity_id,"},{"line_number":62,"context_line":"                {constants.OPERATING_STATUS: new_op_status})"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def _emit_provisioning_status(self, session, repo, entity_type,"},{"line_number":65,"context_line":"                                  entity_id):"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f201791_d49c286b","line":62,"updated":"2017-06-28 16:40:01.000000000","message":"Can\u0027t we just extend this to also send the provisioning status?","commit_id":"eec939c94cefd8d06d6ae284dd63cac663a92621"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"b4de0ad20ff6b09794c164c68573e63bdd0b8521","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        LOG.debug(\"%s %s provisioning_status %s. Updating db and sending\""},{"line_number":65,"context_line":"                  \" event.\", entity_type, entity_id,"},{"line_number":66,"context_line":"                  entity.provisioning_status)"},{"line_number":67,"context_line":"        self.emit("},{"line_number":68,"context_line":"            entity_type, entity_id,"},{"line_number":69,"context_line":"            {constants.PROVISIONING_STATUS: entity.provisioning_status})"},{"line_number":70,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"3f1d235d_ccaf88e1","line":67,"updated":"2017-07-03 18:16:57.000000000","message":"This will emit a message on every health heartbeat for every object right? That is a lot of messages. Maybe we should add a config setting that is off by default to allow the operator to decide if they want this many messages going to neutron.","commit_id":"50332a9a2c58f2955cc3fcb44e2884512a5679ba"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"3c43d6258eb0ce8e30482c4ebc0195c7a67fe6be","unresolved":false,"context_lines":[{"line_number":66,"context_line":"                      entity.provisioning_status)"},{"line_number":67,"context_line":"            self.emit("},{"line_number":68,"context_line":"                entity_type, entity_id,"},{"line_number":69,"context_line":"                {constants.PROVISIONING_STATUS: entity.provisioning_status})"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def update_health(self, health):"},{"line_number":72,"context_line":"        \"\"\"This function is to update db info based on amphora status"}],"source_content_type":"text/x-python","patch_set":11,"id":"1f1a1f67_7a07c483","line":69,"updated":"2017-07-18 05:43:00.000000000","message":"OK, I do have one question here -- could it just be attached to the same message? It\u0027s sending a dict across, with key-\u003evalue, so I feel like you could just include both? Then it wouldn\u0027t be a that much more data to be sending across the queue, and I\u0027m not sure it would even need to be a config option. Michael suggested a config option because of the increase in the *number* of messages, right?\nI probably need to look at the neutron-lbaas side and see if the processing would be problematic.","commit_id":"adaae2748e181ffed649a014b9dc5f9cc3a732c9"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"1a06d47c4e5e10cd45ed69656b381c765c34a6af","unresolved":false,"context_lines":[{"line_number":66,"context_line":"                      entity.provisioning_status)"},{"line_number":67,"context_line":"            self.emit("},{"line_number":68,"context_line":"                entity_type, entity_id,"},{"line_number":69,"context_line":"                {constants.PROVISIONING_STATUS: entity.provisioning_status})"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    def update_health(self, health):"},{"line_number":72,"context_line":"        \"\"\"This function is to update db info based on amphora status"}],"source_content_type":"text/x-python","patch_set":11,"id":"1f1a1f67_1580a345","line":69,"in_reply_to":"1f1a1f67_7a07c483","updated":"2017-07-18 06:02:56.000000000","message":"Or at least, this is more efficient: http://paste.openstack.org/show/615686/","commit_id":"adaae2748e181ffed649a014b9dc5f9cc3a732c9"}],"releasenotes/notes/provisioning_neutron_db_sync-c019d96a7b64fe20.yaml":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"3c43d6258eb0ce8e30482c4ebc0195c7a67fe6be","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"upgrade:"},{"line_number":3,"context_line":"  - The sync_provisioning_status is new option allows to enables provisioning status to sync"},{"line_number":4,"context_line":"    with neutron db. This will create a message per amphora per health heartbeat interval."},{"line_number":5,"context_line":"fixes:"},{"line_number":6,"context_line":"  - This patch updates the provision status when it is sending neutron-lbaas a"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"1f1a1f67_5a31a859","line":3,"updated":"2017-07-18 05:43:00.000000000","message":"\"Added option \u0027sync_provisioning_status\u0027 to enable synchronizing provisioning status of loadbalancers with the neutron-lbaas database. Enabling this option will queue one additional message per amphora every heartbeat interval.\"","commit_id":"adaae2748e181ffed649a014b9dc5f9cc3a732c9"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"3c43d6258eb0ce8e30482c4ebc0195c7a67fe6be","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - The sync_provisioning_status is new option allows to enables provisioning status to sync"},{"line_number":4,"context_line":"    with neutron db. This will create a message per amphora per health heartbeat interval."},{"line_number":5,"context_line":"fixes:"},{"line_number":6,"context_line":"  - This patch updates the provision status when it is sending neutron-lbaas a"},{"line_number":7,"context_line":"    operating status update."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"1f1a1f67_1a10d0a7","line":6,"updated":"2017-07-18 05:43:00.000000000","message":"\"Resolved an issue that could cause provisioning status to become out of sync between neutron-lbaas and octavia during high load.\"","commit_id":"adaae2748e181ffed649a014b9dc5f9cc3a732c9"}]}