)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"45b89a6224d7c7b93e44af9d2f36fa5adb4ba119","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     guang-yee \u003cguang.yee@hpe.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-07 10:11:37 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Change global requirements forcing cliff!\u003d1.17.0"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"cliff 1.17.0 does not work when an empty list is provided"},{"line_number":10,"context_line":"(Bug report on cliff: #1539770)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_8bb28292","line":7,"updated":"2016-03-10 00:26:22.000000000","message":"I understand this is a backport but I think in this case a backport is incorrect.\n\nAt the very least the commit messages needs to be updated to reflect what thew change actually does,","commit_id":"85ba20f92994355ada3599dee886f1a566edcc12"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"3a7af1a1e5f071135097c8142b7e62ce43e6bcb4","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     guang-yee \u003cguang.yee@hpe.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-07 10:11:37 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Change global requirements forcing cliff!\u003d1.17.0"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"cliff 1.17.0 does not work when an empty list is provided"},{"line_number":10,"context_line":"(Bug report on cliff: #1539770)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_c0d2695b","line":7,"in_reply_to":"5aef4532_8bb28292","updated":"2016-03-10 23:11:23.000000000","message":"Not sure if I understand. This is because we can run into cliff bug 1539770 in the stable branch (see line 9-10). Can you please suggest what should the commit message looks like?","commit_id":"85ba20f92994355ada3599dee886f1a566edcc12"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"b4e9685c71619873e2d5a84cfdca5900f084a24a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     guang-yee \u003cguang.yee@hpe.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-03-07 10:11:37 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Change global requirements forcing cliff!\u003d1.17.0"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"cliff 1.17.0 does not work when an empty list is provided"},{"line_number":10,"context_line":"(Bug report on cliff: #1539770)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_e3f4cff1","line":7,"in_reply_to":"5aef4532_c0d2695b","updated":"2016-03-10 23:40:01.000000000","message":"So My point is that the change you\u0027re backporting only added a ban on 1.17.0 which is what your commit message says BUT you patch does 3 things\n 1. raises the minimum to \u003e\u003d 1.15.0\n 2. bans 1.16.0\n 3. bans 1.17.0\n\nSo you I think what you want to achieve is ban 1.17.0 (due to the bug in question) but due to the nature of global-requirements that isn\u0027t a backport in the true sense.\n\nSo I suggest just doing a new change and referencing the bug and and thew master change.","commit_id":"85ba20f92994355ada3599dee886f1a566edcc12"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"45b89a6224d7c7b93e44af9d2f36fa5adb4ba119","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Conflicts:"},{"line_number":13,"context_line":"\tglobal-requirements.txt"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ie77a9622d16bd02b3607fc1c9f8da20dc1ffb856"},{"line_number":16,"context_line":"(cherry picked from commit 0ad86c4d711e4145ad3a37951949ee30bf2c7b81)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_2bb9ae79","line":14,"updated":"2016-03-10 00:26:22.000000000","message":"Probably need a\n\n Related-Bug:\n\nHere.","commit_id":"85ba20f92994355ada3599dee886f1a566edcc12"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"3a7af1a1e5f071135097c8142b7e62ce43e6bcb4","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Conflicts:"},{"line_number":13,"context_line":"\tglobal-requirements.txt"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ie77a9622d16bd02b3607fc1c9f8da20dc1ffb856"},{"line_number":16,"context_line":"(cherry picked from commit 0ad86c4d711e4145ad3a37951949ee30bf2c7b81)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_e042050e","line":14,"in_reply_to":"5aef4532_2bb9ae79","updated":"2016-03-10 23:11:23.000000000","message":"Is \"Related-Bug\" applicable to the bugs filed against openstack/requirements only? I don\u0027t think the original commit have a bug associated with it. I can create a new one for openstack/requirements if needed. Please let me know.","commit_id":"85ba20f92994355ada3599dee886f1a566edcc12"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"b4e9685c71619873e2d5a84cfdca5900f084a24a","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Conflicts:"},{"line_number":13,"context_line":"\tglobal-requirements.txt"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ie77a9622d16bd02b3607fc1c9f8da20dc1ffb856"},{"line_number":16,"context_line":"(cherry picked from commit 0ad86c4d711e4145ad3a37951949ee30bf2c7b81)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5aef4532_407f7914","line":14,"in_reply_to":"5aef4532_e042050e","updated":"2016-03-10 23:40:01.000000000","message":"Yes related-Bug is fine for the existing bug","commit_id":"85ba20f92994355ada3599dee886f1a566edcc12"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"f21b5482de5d2d52f1222e1594396703bcc366af","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Change global requirements forcing cliff!\u003d1.16.0 and 1.17.0"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Both cliff 1.16.0 and 1.17.0 does not work when an empty list is provided"},{"line_number":10,"context_line":"(Bug report on cliff: #1539770). We need to exclude both versions. Notice"},{"line_number":11,"context_line":"that there are no 1.18.0 and 1.19.0 releases, and the problem was addressed"},{"line_number":12,"context_line":"in 2.0.0 release, we should be fine with the exclusion."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3afc51ec_6b20789c","line":10,"range":{"start_line":10,"start_character":14,"end_line":10,"end_character":30},"updated":"2016-03-11 18:50:31.000000000","message":"bug 1539770 is the reason we cannot use cliff 1.17.0.\nThis is not the reason of excluding 1.16.0.\nbug 1543958 is the reason of excluding 1.16.0.\nYou should mention bug 1543958 too.","commit_id":"3a9850d130948d0bde66ab362e021f8122c12e60"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"f21b5482de5d2d52f1222e1594396703bcc366af","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Both cliff 1.16.0 and 1.17.0 does not work when an empty list is provided"},{"line_number":10,"context_line":"(Bug report on cliff: #1539770). We need to exclude both versions. Notice"},{"line_number":11,"context_line":"that there are no 1.18.0 and 1.19.0 releases, and the problem was addressed"},{"line_number":12,"context_line":"in 2.0.0 release, we should be fine with the exclusion."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Conflicts:"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3afc51ec_6b6eb8ca","line":11,"range":{"start_line":11,"start_character":17,"end_line":11,"end_character":46},"updated":"2016-03-11 18:50:31.000000000","message":"I prefer to more generic description. how about 1.20.0?\n\n  There are no further releases in 1.x series after 1.17.0 and ....","commit_id":"3a9850d130948d0bde66ab362e021f8122c12e60"}],"global-requirements.txt":[{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"45b89a6224d7c7b93e44af9d2f36fa5adb4ba119","unresolved":false,"context_lines":[{"line_number":17,"context_line":"ceilometermiddleware\u003e\u003d0.1.0"},{"line_number":18,"context_line":"cachetools\u003e\u003d1.0.0 # MIT License"},{"line_number":19,"context_line":"cffi"},{"line_number":20,"context_line":"cliff!\u003d1.16.0,!\u003d1.17.0,\u003e\u003d1.15.0  # Apache-2.0"},{"line_number":21,"context_line":"cmd2\u003e\u003d0.6.7"},{"line_number":22,"context_line":"PuLP\u003e\u003d1.0.4"},{"line_number":23,"context_line":"contextlib2\u003e\u003d0.4.0 # PSF License"}],"source_content_type":"text/plain","patch_set":1,"id":"5aef4532_9ed642ef","line":20,"updated":"2016-03-10 00:26:22.000000000","message":"We don\u0027t raise the minimum on stable branches, and you commit messages talks about blocking 1.17.0 but you\u0027re clearly doing more than that.","commit_id":"85ba20f92994355ada3599dee886f1a566edcc12"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"3a7af1a1e5f071135097c8142b7e62ce43e6bcb4","unresolved":false,"context_lines":[{"line_number":17,"context_line":"ceilometermiddleware\u003e\u003d0.1.0"},{"line_number":18,"context_line":"cachetools\u003e\u003d1.0.0 # MIT License"},{"line_number":19,"context_line":"cffi"},{"line_number":20,"context_line":"cliff!\u003d1.16.0,!\u003d1.17.0,\u003e\u003d1.15.0  # Apache-2.0"},{"line_number":21,"context_line":"cmd2\u003e\u003d0.6.7"},{"line_number":22,"context_line":"PuLP\u003e\u003d1.0.4"},{"line_number":23,"context_line":"contextlib2\u003e\u003d0.4.0 # PSF License"}],"source_content_type":"text/plain","patch_set":1,"id":"5aef4532_602ef58c","line":20,"in_reply_to":"5aef4532_9ed642ef","updated":"2016-03-10 23:11:23.000000000","message":"Good to know. I\u0027ll back out the minimum requirement change.","commit_id":"85ba20f92994355ada3599dee886f1a566edcc12"}]}