)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"ffc04bb8db9d9ebe5b8f0214e19b4a95ec7f9292","unresolved":false,"context_lines":[{"line_number":16,"context_line":"Deprecate these for removal in a future release."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"FIXME: when we can require oslo.config\u003e\u003d3.2.0, we can uncomment the"},{"line_number":19,"context_line":"       deprecated_reason oslo.config parameters."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Change-Id: I69c4bdf47a7b34b8dc8cfa58f3d0e0557b5ce072"},{"line_number":22,"context_line":"Signed-off-by: Mark McLoughlin \u003cmarkmc@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa69d971_457c1272","line":19,"updated":"2016-01-08 18:24:51.000000000","message":"Since I08c6bd226a2e9e38d81fa3d251e939a976a3e090 we now have oslo.config\u003e\u003d3.2.0 in requirements.txt so I\u0027m going to rebase to use that.","commit_id":"37c870cce28cdfbc59ea65ee391a6ed12d8ab07b"}],"nova/virt/libvirt/driver.py":[{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"4979fc28137a6841797a872912a511512dd547b4","unresolved":false,"context_lines":[{"line_number":167,"context_line":"               deprecated_reason\u003d\u0027Migration flags may not only be modified \u0027"},{"line_number":168,"context_line":"                                 \u0027using the live_migration_paused, \u0027"},{"line_number":169,"context_line":"                                 \u0027live_migration_tunneled, and \u0027"},{"line_number":170,"context_line":"                                 \u0027live_migration_extra_flags config options\u0027),"},{"line_number":171,"context_line":"    cfg.StrOpt(\u0027block_migration_flag\u0027,"},{"line_number":172,"context_line":"               default\u003d\u0027VIR_MIGRATE_UNDEFINE_SOURCE, VIR_MIGRATE_PEER2PEER, \u0027"},{"line_number":173,"context_line":"                       \u0027VIR_MIGRATE_LIVE, VIR_MIGRATE_TUNNELLED, \u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa69d971_25a1acf8","line":170,"range":{"start_line":170,"start_character":34,"end_line":170,"end_character":60},"updated":"2016-01-11 12:08:26.000000000","message":"live_migration_tunneled is the only flags we allow now. Likewise for next parametere","commit_id":"d4a8eb30a5235c5170e25377767f664697301e3b"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"00d851a6bb3fa3c858e5bf89bb2ab3923e9d9813","unresolved":false,"context_lines":[{"line_number":167,"context_line":"               deprecated_reason\u003d\u0027Migration flags may not only be modified \u0027"},{"line_number":168,"context_line":"                                 \u0027using the live_migration_paused, \u0027"},{"line_number":169,"context_line":"                                 \u0027live_migration_tunneled, and \u0027"},{"line_number":170,"context_line":"                                 \u0027live_migration_extra_flags config options\u0027),"},{"line_number":171,"context_line":"    cfg.StrOpt(\u0027block_migration_flag\u0027,"},{"line_number":172,"context_line":"               default\u003d\u0027VIR_MIGRATE_UNDEFINE_SOURCE, VIR_MIGRATE_PEER2PEER, \u0027"},{"line_number":173,"context_line":"                       \u0027VIR_MIGRATE_LIVE, VIR_MIGRATE_TUNNELLED, \u0027"}],"source_content_type":"text/x-python","patch_set":5,"id":"da6ed579_2c17fbee","line":170,"range":{"start_line":170,"start_character":34,"end_line":170,"end_character":60},"in_reply_to":"fa69d971_25a1acf8","updated":"2016-01-11 22:30:18.000000000","message":"Good catch","commit_id":"d4a8eb30a5235c5170e25377767f664697301e3b"},{"author":{"_account_id":7461,"name":"Paul Murray","email":"pmurray@hpe.com","username":"ptm"},"change_message_id":"95e6d897a192cf6e804a05f8aa0c31bcc5180265","unresolved":false,"context_lines":[{"line_number":164,"context_line":"                       \u0027VIR_MIGRATE_LIVE, VIR_MIGRATE_TUNNELLED\u0027,"},{"line_number":165,"context_line":"               help\u003d\u0027Migration flags to be set for live migration\u0027,"},{"line_number":166,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":167,"context_line":"               deprecated_reason\u003d\u0027Migration flags may not only be modified \u0027"},{"line_number":168,"context_line":"                                 \u0027using the live_migration_tunnelled config \u0027"},{"line_number":169,"context_line":"                                 \u0027option\u0027),"},{"line_number":170,"context_line":"    cfg.StrOpt(\u0027block_migration_flag\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"ba63e191_aacf0b7e","line":167,"updated":"2016-01-19 17:51:26.000000000","message":"nit: I find this sentence hard to understand. Could be made more clear.","commit_id":"c96021c8f8432031c55be2e324ee281067de0f03"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"59810b48c3498a2e791938645dba933345bfdb58","unresolved":false,"context_lines":[{"line_number":164,"context_line":"                       \u0027VIR_MIGRATE_LIVE, VIR_MIGRATE_TUNNELLED\u0027,"},{"line_number":165,"context_line":"               help\u003d\u0027Migration flags to be set for live migration\u0027,"},{"line_number":166,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":167,"context_line":"               deprecated_reason\u003d\u0027Migration flags may not only be modified \u0027"},{"line_number":168,"context_line":"                                 \u0027using the live_migration_tunnelled config \u0027"},{"line_number":169,"context_line":"                                 \u0027option\u0027),"},{"line_number":170,"context_line":"    cfg.StrOpt(\u0027block_migration_flag\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"7a5de9d1_db6e0696","line":167,"in_reply_to":"7a5de9d1_29991320","updated":"2016-02-02 15:36:41.000000000","message":"sounds great, ship it.","commit_id":"c96021c8f8432031c55be2e324ee281067de0f03"},{"author":{"_account_id":7461,"name":"Paul Murray","email":"pmurray@hpe.com","username":"ptm"},"change_message_id":"a6247dcc862002e6d780655ac8e53f927d5b97fb","unresolved":false,"context_lines":[{"line_number":164,"context_line":"                       \u0027VIR_MIGRATE_LIVE, VIR_MIGRATE_TUNNELLED\u0027,"},{"line_number":165,"context_line":"               help\u003d\u0027Migration flags to be set for live migration\u0027,"},{"line_number":166,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":167,"context_line":"               deprecated_reason\u003d\u0027Migration flags may not only be modified \u0027"},{"line_number":168,"context_line":"                                 \u0027using the live_migration_tunnelled config \u0027"},{"line_number":169,"context_line":"                                 \u0027option\u0027),"},{"line_number":170,"context_line":"    cfg.StrOpt(\u0027block_migration_flag\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"7a5de9d1_29991320","line":167,"in_reply_to":"7a5de9d1_5c453ab1","updated":"2016-02-02 15:33:49.000000000","message":"How about: \"The correct live migration flags can be inferred from the new live_migration_tunnelled option. live_migration_flag will be removed to avoid potential misconfiguration.\"","commit_id":"c96021c8f8432031c55be2e324ee281067de0f03"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"494ae9b7edb4da8b624e4ac35d8a9daae804ac63","unresolved":false,"context_lines":[{"line_number":164,"context_line":"                       \u0027VIR_MIGRATE_LIVE, VIR_MIGRATE_TUNNELLED\u0027,"},{"line_number":165,"context_line":"               help\u003d\u0027Migration flags to be set for live migration\u0027,"},{"line_number":166,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":167,"context_line":"               deprecated_reason\u003d\u0027Migration flags may not only be modified \u0027"},{"line_number":168,"context_line":"                                 \u0027using the live_migration_tunnelled config \u0027"},{"line_number":169,"context_line":"                                 \u0027option\u0027),"},{"line_number":170,"context_line":"    cfg.StrOpt(\u0027block_migration_flag\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"7a5de9d1_5c453ab1","line":167,"in_reply_to":"ba63e191_aacf0b7e","updated":"2016-02-02 12:56:37.000000000","message":"yeh, I\u0027m not sure I fully understand the deprecation reason here. As an op I\u0027d fined that pretty confusing.\n\nIf we can get better wording I\u0027m +2.","commit_id":"c96021c8f8432031c55be2e324ee281067de0f03"},{"author":{"_account_id":7461,"name":"Paul Murray","email":"pmurray@hpe.com","username":"ptm"},"change_message_id":"95e6d897a192cf6e804a05f8aa0c31bcc5180265","unresolved":false,"context_lines":[{"line_number":173,"context_line":"                       \u0027VIR_MIGRATE_NON_SHARED_INC\u0027,"},{"line_number":174,"context_line":"               help\u003d\u0027Migration flags to be set for block migration\u0027,"},{"line_number":175,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":176,"context_line":"               deprecated_reason\u003d\u0027Migration flags may not only be modified \u0027"},{"line_number":177,"context_line":"                                 \u0027using the live_migration_tunnelled config \u0027"},{"line_number":178,"context_line":"                                 \u0027option\u0027),"},{"line_number":179,"context_line":"    cfg.BoolOpt(\u0027live_migration_tunnelled\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"ba63e191_4aa52fb3","line":176,"updated":"2016-01-19 17:51:26.000000000","message":"Same as above.","commit_id":"c96021c8f8432031c55be2e324ee281067de0f03"},{"author":{"_account_id":7461,"name":"Paul Murray","email":"pmurray@hpe.com","username":"ptm"},"change_message_id":"a6247dcc862002e6d780655ac8e53f927d5b97fb","unresolved":false,"context_lines":[{"line_number":173,"context_line":"                       \u0027VIR_MIGRATE_NON_SHARED_INC\u0027,"},{"line_number":174,"context_line":"               help\u003d\u0027Migration flags to be set for block migration\u0027,"},{"line_number":175,"context_line":"               deprecated_for_removal\u003dTrue,"},{"line_number":176,"context_line":"               deprecated_reason\u003d\u0027Migration flags may not only be modified \u0027"},{"line_number":177,"context_line":"                                 \u0027using the live_migration_tunnelled config \u0027"},{"line_number":178,"context_line":"                                 \u0027option\u0027),"},{"line_number":179,"context_line":"    cfg.BoolOpt(\u0027live_migration_tunnelled\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"7a5de9d1_c96b9ff0","line":176,"in_reply_to":"ba63e191_4aa52fb3","updated":"2016-02-02 15:33:49.000000000","message":"similar to above","commit_id":"c96021c8f8432031c55be2e324ee281067de0f03"}]}
