)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"cb13e055283a895e6ab7b39cfd64506592774ef8","unresolved":false,"context_lines":[{"line_number":11,"context_line":"not get updated. This is because in migration 21 sets the values for"},{"line_number":12,"context_line":"these quotas in the DB, which overrides cinder.conf. This patch fixes"},{"line_number":13,"context_line":"this issue by removing the data from the DB."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I479958d1a8830097ee0b21e4f4c47a9abf3a07ec"},{"line_number":16,"context_line":"Closes-Bug:1465393"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_a0174c50","line":14,"updated":"2015-09-14 21:47:09.000000000","message":"Need to add \u0027DocImpact\u0027 here.  Make a note that the values in the database are no longer used but instead values from cinder.conf will be used.","commit_id":"3c83acb95dbdb94ce59e98d2adfdd19028abcc91"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"bee96ffe5dfab603892b21c0f149ad48623eb3f1","unresolved":false,"context_lines":[{"line_number":11,"context_line":"not get updated. This is because in migration 21 sets the values for"},{"line_number":12,"context_line":"these quotas in the DB, which overrides cinder.conf. This patch fixes"},{"line_number":13,"context_line":"this issue by removing the data from the DB."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I479958d1a8830097ee0b21e4f4c47a9abf3a07ec"},{"line_number":16,"context_line":"Closes-Bug:1465393"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_abbaa16c","line":14,"in_reply_to":"ba15a1d1_a0174c50","updated":"2015-09-15 18:38:29.000000000","message":"Done","commit_id":"3c83acb95dbdb94ce59e98d2adfdd19028abcc91"},{"author":{"_account_id":8202,"name":"chenzongliang","email":"chenzongliang@huawei.com","username":"zongliang"},"change_message_id":"5ac7b49e45516e8a6c35cc1dbd92ad4552f867ca","unresolved":false,"context_lines":[{"line_number":12,"context_line":"these quotas in the DB, which overrides cinder.conf. This patch fixes"},{"line_number":13,"context_line":"this issue by removing the data from the DB."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I479958d1a8830097ee0b21e4f4c47a9abf3a07ec"},{"line_number":16,"context_line":"Closes-Bug:1465393"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_663b89ec","line":15,"updated":"2015-09-15 01:21:34.000000000","message":"I agree with Jay for the database changed","commit_id":"3c83acb95dbdb94ce59e98d2adfdd19028abcc91"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"bee96ffe5dfab603892b21c0f149ad48623eb3f1","unresolved":false,"context_lines":[{"line_number":12,"context_line":"these quotas in the DB, which overrides cinder.conf. This patch fixes"},{"line_number":13,"context_line":"this issue by removing the data from the DB."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I479958d1a8830097ee0b21e4f4c47a9abf3a07ec"},{"line_number":16,"context_line":"Closes-Bug:1465393"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ba15a1d1_0baced32","line":15,"in_reply_to":"ba15a1d1_663b89ec","updated":"2015-09-15 18:38:29.000000000","message":"Done","commit_id":"3c83acb95dbdb94ce59e98d2adfdd19028abcc91"}],"cinder/db/sqlalchemy/migrate_repo/versions/056_remove_default_quota_values_from_table.py":[{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"cb13e055283a895e6ab7b39cfd64506592774ef8","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from sqlalchemy import MetaData, Table"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"# Get default values via config.  The defaults will either"},{"line_number":21,"context_line":"# come from the default values set in the quota option"},{"line_number":22,"context_line":"# configuration or via cinder.conf if the user has configured"},{"line_number":23,"context_line":"# default values for quotas there."},{"line_number":24,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba15a1d1_7b09e52b","line":21,"updated":"2015-09-14 21:47:09.000000000","message":"Should specify that the default values were set in the DB.","commit_id":"3c83acb95dbdb94ce59e98d2adfdd19028abcc91"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"bee96ffe5dfab603892b21c0f149ad48623eb3f1","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from sqlalchemy import MetaData, Table"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"# Get default values via config.  The defaults will either"},{"line_number":21,"context_line":"# come from the default values set in the quota option"},{"line_number":22,"context_line":"# configuration or via cinder.conf if the user has configured"},{"line_number":23,"context_line":"# default values for quotas there."},{"line_number":24,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba15a1d1_8bb79d85","line":21,"in_reply_to":"ba15a1d1_7b09e52b","updated":"2015-09-15 18:38:29.000000000","message":"Done","commit_id":"3c83acb95dbdb94ce59e98d2adfdd19028abcc91"},{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"1767189bd697e666fe5f233bff038d36cb3e9017","unresolved":false,"context_lines":[{"line_number":26,"context_line":"CONF \u003d cfg.CONF"},{"line_number":27,"context_line":"CONF.import_opt(\u0027quota_volumes\u0027, \u0027cinder.quota\u0027)"},{"line_number":28,"context_line":"CONF.import_opt(\u0027quota_snapshots\u0027, \u0027cinder.quota\u0027)"},{"line_number":29,"context_line":"CONF.import_opt(\u0027quota_gigabytes\u0027, \u0027cinder.quota\u0027)"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"CLASS_NAME \u003d \u0027default\u0027"},{"line_number":32,"context_line":"CREATED_AT \u003d datetime.datetime.now()  # noqa"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba15a1d1_30b2434d","line":29,"updated":"2015-09-16 20:59:44.000000000","message":"the entire point of the bug is that we should NOT be requiring CONF file entries to set quotas on migration.   This just creates the same bug somewhere else after nuking settings that an existing deployment may already have.","commit_id":"2d926378e7db54971bf388f93cddb05c19fef7f3"},{"author":{"_account_id":16308,"name":"Sheel Rana","email":"ranasheel2000@gmail.com","username":"sheel.rana"},"change_message_id":"9b4a1b69c1364ee6faeaef90e9d055cc03213d9c","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    qci.execute({quota_classes.c.resource: \u0027snapshots\u0027})"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def downgrade(migrate_engine):"},{"line_number":50,"context_line":"    pass"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a5ae5dd_80287034","line":49,"range":{"start_line":49,"start_character":4,"end_line":49,"end_character":13},"updated":"2016-02-06 20:56:44.000000000","message":"Downgrade support is not present.\n\nI think test will also fail due to presence of \"downgrade\" method.\nSo, \"downgrade()\" should be completely removed.","commit_id":"2d926378e7db54971bf388f93cddb05c19fef7f3"}],"cinder/tests/unit/test_migrations.py":[{"author":{"_account_id":16917,"name":"Alex O\u0027Rourke","email":"alex.orourke@hpe.com","username":"aorourke"},"change_message_id":"b9e5b82784ae55efb67c431218bcc5cf85d7c395","unresolved":false,"context_lines":[{"line_number":930,"context_line":"        self.assertNotIn(\u0027volumes\u0027, quota_classes.c)"},{"line_number":931,"context_line":"        self.assertNotIn(\u0027gigabytes\u0027, quota_classes.c)"},{"line_number":932,"context_line":"        self.assertNotIn(\u0027snapshots\u0027, quota_classes.c)"},{"line_number":933,"context_line":"        "},{"line_number":934,"context_line":""},{"line_number":935,"context_line":"    def test_walk_versions(self):"},{"line_number":936,"context_line":"        self.walk_versions(True, False)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba15a1d1_95b5a8ce","line":933,"updated":"2015-09-14 21:00:46.000000000","message":"Quick fix of this minor error","commit_id":"3c83acb95dbdb94ce59e98d2adfdd19028abcc91"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"bee96ffe5dfab603892b21c0f149ad48623eb3f1","unresolved":false,"context_lines":[{"line_number":930,"context_line":"        self.assertNotIn(\u0027volumes\u0027, quota_classes.c)"},{"line_number":931,"context_line":"        self.assertNotIn(\u0027gigabytes\u0027, quota_classes.c)"},{"line_number":932,"context_line":"        self.assertNotIn(\u0027snapshots\u0027, quota_classes.c)"},{"line_number":933,"context_line":"        "},{"line_number":934,"context_line":""},{"line_number":935,"context_line":"    def test_walk_versions(self):"},{"line_number":936,"context_line":"        self.walk_versions(True, False)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba15a1d1_ebc099e2","line":933,"in_reply_to":"ba15a1d1_95b5a8ce","updated":"2015-09-15 18:38:29.000000000","message":"Done","commit_id":"3c83acb95dbdb94ce59e98d2adfdd19028abcc91"}]}