)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8125,"name":"Michael Davies","email":"michael@the-davies.net","username":"mrda"},"change_message_id":"d6f0886269ba3479dd22e43f1960bfb3042f42ce","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As a part of the process of upgrading from Nova baremetal"},{"line_number":10,"context_line":"driver to Ironic it\u0027s necessary to upgrade baremetal flavors\u0027"},{"line_number":11,"context_line":"metadata to reffer to Ironic deploy kernel and ramdisk."},{"line_number":12,"context_line":"This tool allows to do that by specifying the database connection"},{"line_number":13,"context_line":"details and a CSV file containing new flavor metadata in the"},{"line_number":14,"context_line":"flavor_id,kernel_id,ramdisk_id format."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_e51cb616","line":11,"updated":"2014-08-11 04:58:43.000000000","message":"s/reffer/refer","commit_id":"76ef52b15d0a35c0cf6f050570665c87c7624aef"},{"author":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"change_message_id":"26cc6446ee739099b6d0e24d3f32c8be54f67ce1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As a part of the process of upgrading from Nova baremetal"},{"line_number":10,"context_line":"driver to Ironic it\u0027s necessary to upgrade baremetal flavors\u0027"},{"line_number":11,"context_line":"metadata to reffer to Ironic deploy kernel and ramdisk."},{"line_number":12,"context_line":"This tool allows to do that by specifying the database connection"},{"line_number":13,"context_line":"details and a CSV file containing new flavor metadata in the"},{"line_number":14,"context_line":"flavor_id,kernel_id,ramdisk_id format."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa98f980_b8a09af5","line":11,"in_reply_to":"baada198_e51cb616","updated":"2014-09-01 14:01:16.000000000","message":"Done","commit_id":"76ef52b15d0a35c0cf6f050570665c87c7624aef"}],"ironic/migrate_nova/update_flavors.py":[{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"377f8f5f479cf9db095e8c226b8fc81192ef8d91","unresolved":false,"context_lines":[{"line_number":15,"context_line":"import csv"},{"line_number":16,"context_line":"import sys"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import prettytable"},{"line_number":19,"context_line":"import sqlalchemy as sa"},{"line_number":20,"context_line":"from sqlalchemy.orm import exc as sa_exc"},{"line_number":21,"context_line":"from sqlalchemy.orm import sessionmaker"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_dd4d089b","line":18,"updated":"2014-08-27 14:47:35.000000000","message":"I think you will need to add PrettyTable to the requirements.txt file (it has an entry in the global-requirements project).","commit_id":"76ef52b15d0a35c0cf6f050570665c87c7624aef"},{"author":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"change_message_id":"26cc6446ee739099b6d0e24d3f32c8be54f67ce1","unresolved":false,"context_lines":[{"line_number":15,"context_line":"import csv"},{"line_number":16,"context_line":"import sys"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import prettytable"},{"line_number":19,"context_line":"import sqlalchemy as sa"},{"line_number":20,"context_line":"from sqlalchemy.orm import exc as sa_exc"},{"line_number":21,"context_line":"from sqlalchemy.orm import sessionmaker"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa98f980_d80f1ee0","line":18,"in_reply_to":"1abeadc6_dd4d089b","updated":"2014-09-01 14:01:16.000000000","message":"Done","commit_id":"76ef52b15d0a35c0cf6f050570665c87c7624aef"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"377f8f5f479cf9db095e8c226b8fc81192ef8d91","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            sys.exit(2)"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"        kernel.value \u003d new_flavor_metadata[flavor_id][\u0027kernel\u0027]"},{"line_number":91,"context_line":"        ramdisk.value \u003d new_flavor_metadata[flavor_id][\u0027ramdisk\u0027]"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        session.commit()"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_f0202b9d","line":91,"updated":"2014-08-27 14:47:35.000000000","message":"Why not put the updates (and commit) within the try block as well?","commit_id":"76ef52b15d0a35c0cf6f050570665c87c7624aef"},{"author":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"change_message_id":"26cc6446ee739099b6d0e24d3f32c8be54f67ce1","unresolved":false,"context_lines":[{"line_number":88,"context_line":"            sys.exit(2)"},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"        kernel.value \u003d new_flavor_metadata[flavor_id][\u0027kernel\u0027]"},{"line_number":91,"context_line":"        ramdisk.value \u003d new_flavor_metadata[flavor_id][\u0027ramdisk\u0027]"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        session.commit()"},{"line_number":94,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa98f980_b8bfbacf","line":91,"in_reply_to":"1abeadc6_f0202b9d","updated":"2014-09-01 14:01:16.000000000","message":"Makes sense. Done","commit_id":"76ef52b15d0a35c0cf6f050570665c87c7624aef"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"377f8f5f479cf9db095e8c226b8fc81192ef8d91","unresolved":false,"context_lines":[{"line_number":106,"context_line":"                        help\u003d\u0027Path to the CSV file containing flavor specs. \u0027"},{"line_number":107,"context_line":"                             \u0027The file should be in the the following format: \u0027"},{"line_number":108,"context_line":"                             \u0027\"flavor_id, kernel_id,ramdisk_id\" where \u0027"},{"line_number":109,"context_line":"                             \u0027flavor_id is an id of a flavor, kernel_id and \u0027"},{"line_number":110,"context_line":"                             \u0027are the IDs of the new kernel and ramdisk that \u0027"},{"line_number":111,"context_line":"                             \u0027will be used by Ironic to deploy nodes.\u0027)"},{"line_number":112,"context_line":"    parser.add_argument(\u0027--nova-db\u0027, \u0027-d\u0027, type\u003dstr,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_5d21f8fa","line":109,"updated":"2014-08-27 14:47:35.000000000","message":"kernel_id and *ramdisk_id*","commit_id":"76ef52b15d0a35c0cf6f050570665c87c7624aef"},{"author":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"change_message_id":"26cc6446ee739099b6d0e24d3f32c8be54f67ce1","unresolved":false,"context_lines":[{"line_number":106,"context_line":"                        help\u003d\u0027Path to the CSV file containing flavor specs. \u0027"},{"line_number":107,"context_line":"                             \u0027The file should be in the the following format: \u0027"},{"line_number":108,"context_line":"                             \u0027\"flavor_id, kernel_id,ramdisk_id\" where \u0027"},{"line_number":109,"context_line":"                             \u0027flavor_id is an id of a flavor, kernel_id and \u0027"},{"line_number":110,"context_line":"                             \u0027are the IDs of the new kernel and ramdisk that \u0027"},{"line_number":111,"context_line":"                             \u0027will be used by Ironic to deploy nodes.\u0027)"},{"line_number":112,"context_line":"    parser.add_argument(\u0027--nova-db\u0027, \u0027-d\u0027, type\u003dstr,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa98f980_b817baa2","line":109,"in_reply_to":"1abeadc6_5d21f8fa","updated":"2014-09-01 14:01:16.000000000","message":"Done","commit_id":"76ef52b15d0a35c0cf6f050570665c87c7624aef"},{"author":{"_account_id":3099,"name":"David Shrewsbury","email":"dshrewsb@redhat.com","username":"dshrews"},"change_message_id":"377f8f5f479cf9db095e8c226b8fc81192ef8d91","unresolved":false,"context_lines":[{"line_number":109,"context_line":"                             \u0027flavor_id is an id of a flavor, kernel_id and \u0027"},{"line_number":110,"context_line":"                             \u0027are the IDs of the new kernel and ramdisk that \u0027"},{"line_number":111,"context_line":"                             \u0027will be used by Ironic to deploy nodes.\u0027)"},{"line_number":112,"context_line":"    parser.add_argument(\u0027--nova-db\u0027, \u0027-d\u0027, type\u003dstr,"},{"line_number":113,"context_line":"                        dest\u003d\u0027nova_conn_string\u0027, required\u003dTrue,"},{"line_number":114,"context_line":"                        help\u003d\u0027Connection string to Nova database.\\n\u0027"},{"line_number":115,"context_line":"                             \u0027Example: mysql://user:password@example.com/nova\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_7de5d4a6","line":112,"updated":"2014-08-27 14:47:35.000000000","message":"Should we make this --nova-bm-db to be consistent with the migrate_db.py script from 112402?","commit_id":"76ef52b15d0a35c0cf6f050570665c87c7624aef"},{"author":{"_account_id":6623,"name":"Roman Prykhodchenko","email":"me@romcheg.me","username":"prykhodchenko"},"change_message_id":"26cc6446ee739099b6d0e24d3f32c8be54f67ce1","unresolved":false,"context_lines":[{"line_number":109,"context_line":"                             \u0027flavor_id is an id of a flavor, kernel_id and \u0027"},{"line_number":110,"context_line":"                             \u0027are the IDs of the new kernel and ramdisk that \u0027"},{"line_number":111,"context_line":"                             \u0027will be used by Ironic to deploy nodes.\u0027)"},{"line_number":112,"context_line":"    parser.add_argument(\u0027--nova-db\u0027, \u0027-d\u0027, type\u003dstr,"},{"line_number":113,"context_line":"                        dest\u003d\u0027nova_conn_string\u0027, required\u003dTrue,"},{"line_number":114,"context_line":"                        help\u003d\u0027Connection string to Nova database.\\n\u0027"},{"line_number":115,"context_line":"                             \u0027Example: mysql://user:password@example.com/nova\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa98f980_9812f6b2","line":112,"in_reply_to":"1abeadc6_7de5d4a6","updated":"2014-09-01 14:01:16.000000000","message":"Then I will make it --nova-bm-db, -b to be completely consistent.","commit_id":"76ef52b15d0a35c0cf6f050570665c87c7624aef"}]}