)]}'
{"id":"openstack%2Fswift~673131","triplet_id":"openstack%2Fswift~stable%2Fpike~Ib654bfb76a44484489a4622edb38acc43a0da672","project":"openstack/swift","branch":"stable/pike","topic":"bug/1838123","hashtags":[],"change_id":"Ib654bfb76a44484489a4622edb38acc43a0da672","subject":"Fix the pep8 test will raise \"Unknown test found in profile: B109\"","status":"MERGED","created":"2019-07-27 08:44:04.000000000","updated":"2019-07-30 00:44:06.000000000","submitted":"2019-07-29 20:41:27.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"673131-1564432887020-66f2565c","meta_rev_id":"87d9bc5093c2817b8a87a82e003b8b4dbdbeadc3","_number":673131,"virtual_id_number":673131,"owner":{"_account_id":29062,"name":"CY Chiang","email":"cychiang@cht.com.tw","username":"cychiang"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"date":"2019-07-29 12:40:29.000000000","_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},{"value":2,"date":"2019-07-29 20:41:26.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"all":[{"value":0,"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2019-07-29 18:55:50.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"all":[{"value":0,"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2019-07-29 18:55:50.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-07-29 12:40:29.000000000","updated_by":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"reviewer":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"state":"REVIEWER"},{"updated":"2019-07-29 18:55:50.000000000","updated_by":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"reviewer":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"state":"REVIEWER"},{"updated":"2019-07-29 20:41:26.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"526dc9d762694a6dfd34ad819cbbaa4dcaa8bd4c","author":{"_account_id":29062,"name":"CY Chiang","email":"cychiang@cht.com.tw","username":"cychiang"},"date":"2019-07-27 08:44:04.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"72d196a2712b112d70ad297083da88d8d9419d0e","author":{"_account_id":29062,"name":"CY Chiang","email":"cychiang@cht.com.tw","username":"cychiang"},"date":"2019-07-27 08:51:02.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"ae2b2b1ba9469778b0efebef07e7227c244117f3","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-27 10:20:55.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/31/673131/2/check/build-openstack-sphinx-docs/8e67f6e/html/ : SUCCESS in 6m 16s\n- swift-tox-py27 http://logs.openstack.org/31/673131/2/check/swift-tox-py27/875e4a8/ : SUCCESS in 6m 33s\n- swift-tox-py35 http://logs.openstack.org/31/673131/2/check/swift-tox-py35/471a14b/ : SUCCESS in 4m 32s\n- swift-tox-func http://logs.openstack.org/31/673131/2/check/swift-tox-func/d8e6d65/ : SUCCESS in 7m 22s\n- swift-tox-func-post-as-copy http://logs.openstack.org/31/673131/2/check/swift-tox-func-post-as-copy/f2c2f90/ : SUCCESS in 7m 04s\n- swift-tox-func-encryption http://logs.openstack.org/31/673131/2/check/swift-tox-func-encryption/7cfeee2/ : SUCCESS in 7m 13s\n- swift-tox-func-ec http://logs.openstack.org/31/673131/2/check/swift-tox-func-ec/712d960/ : SUCCESS in 9m 48s\n- openstack-tox-pep8 http://logs.openstack.org/31/673131/2/check/openstack-tox-pep8/684fea0/ : SUCCESS in 5m 09s\n- swift-dsvm-functional http://logs.openstack.org/31/673131/2/check/swift-dsvm-functional/a1f7ecd/ : SUCCESS in 21m 06s\n- swift-dsvm-functional-identity-v2 http://logs.openstack.org/31/673131/2/check/swift-dsvm-functional-identity-v2/4d2c241/ : SUCCESS in 24m 35s\n- legacy-tempest-dsvm-neutron-full http://logs.openstack.org/31/673131/2/check/legacy-tempest-dsvm-neutron-full/fb6c186/ : SUCCESS in 1h 27m 06s\n- neutron-grenade http://logs.openstack.org/31/673131/2/check/neutron-grenade/7a30f39/ : SUCCESS in 54m 45s","accounts_in_message":[],"_revision_number":2},{"id":"3c80f369328a64c5d593c3d26d9400a7259bcfe3","author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"date":"2019-07-29 12:40:29.000000000","message":"Patch Set 2:\n\nHello Cy Chiang. Thank you for your interest in contributing with the Swift community. I noticed you have a couple of patches against the stable/pike. Pike was released almost 2 years ago and there\u0027s been 3 (almost 4) releases since then, which means we are not looking to keep merging changes to it, unless it is a serious security bug. Please feel free to submit changes against master and we will be happy to review and welcome your contributions.","accounts_in_message":[],"_revision_number":2},{"id":"36caec31cf28a3f763959582b49033b20ca9548d","author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"date":"2019-07-29 18:55:50.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\n(1 comment)\n\nLooks a whole lot like https://review.opendev.org/#/c/593048/ which was already backported to queens as https://review.opendev.org/#/c/598130/.\n\n`tox -e pep8` is decidedly busted on stable/pike; I feel like we\u0027ve got an interest in having it working in case some user-facing issue comes up that genuinely *should* have a backport this far back. As Thiago said, that would normally be limited to security vulnerabilities.\n\nIn general I agree that the advice for anyone trying to maintain stable/pike swift is and should be to upgrade to a more-recent release. But with the above concern (and considering how low-risk this change is; we don\u0027t even import oslo.config), I think I\u0027m OK with approving it.","accounts_in_message":[],"_revision_number":2},{"id":"40dfe3c29482b1fc891d0db90dbc519e13f94f0d","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-29 18:56:02.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"f5d2175b490ffadaa4b25bed77d672950246db31","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-29 20:41:26.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- build-openstack-sphinx-docs http://logs.openstack.org/31/673131/2/gate/build-openstack-sphinx-docs/c03e005/html/ : SUCCESS in 6m 48s\n- swift-tox-py27 http://logs.openstack.org/31/673131/2/gate/swift-tox-py27/e47604b/ : SUCCESS in 7m 33s\n- swift-tox-py35 http://logs.openstack.org/31/673131/2/gate/swift-tox-py35/0357875/ : SUCCESS in 4m 14s\n- swift-tox-func http://logs.openstack.org/31/673131/2/gate/swift-tox-func/1411dc9/ : SUCCESS in 7m 53s\n- swift-tox-func-post-as-copy http://logs.openstack.org/31/673131/2/gate/swift-tox-func-post-as-copy/990cb16/ : SUCCESS in 8m 36s\n- swift-tox-func-encryption http://logs.openstack.org/31/673131/2/gate/swift-tox-func-encryption/b4cc842/ : SUCCESS in 9m 36s\n- swift-tox-func-ec http://logs.openstack.org/31/673131/2/gate/swift-tox-func-ec/b87f33a/ : SUCCESS in 12m 55s\n- openstack-tox-pep8 http://logs.openstack.org/31/673131/2/gate/openstack-tox-pep8/069605d/ : SUCCESS in 4m 38s\n- swift-dsvm-functional http://logs.openstack.org/31/673131/2/gate/swift-dsvm-functional/c115d7f/ : SUCCESS in 24m 21s\n- swift-dsvm-functional-identity-v2 http://logs.openstack.org/31/673131/2/gate/swift-dsvm-functional-identity-v2/0da3b16/ : SUCCESS in 27m 50s\n- legacy-tempest-dsvm-neutron-full http://logs.openstack.org/31/673131/2/gate/legacy-tempest-dsvm-neutron-full/f69f452/ : SUCCESS in 1h 37m 19s\n- neutron-grenade http://logs.openstack.org/31/673131/2/gate/neutron-grenade/40b6593/ : SUCCESS in 1h 05m 14s","accounts_in_message":[],"_revision_number":2},{"id":"ebb00413017479107858cd911b53d1717d7bdae5","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-29 20:41:27.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":2},{"id":"0f6470c6161fab4e5e897d589f2dc3155c293367","author":{"_account_id":29062,"name":"CY Chiang","email":"cychiang@cht.com.tw","username":"cychiang"},"date":"2019-07-30 00:41:43.000000000","message":"Patch Set 2:\n\n\u003e (1 comment)\n \u003e \n \u003e Looks a whole lot like https://review.opendev.org/#/c/593048/ which\n \u003e was already backported to queens as https://review.opendev.org/#/c/598130/.\n \u003e \n \u003e `tox -e pep8` is decidedly busted on stable/pike; I feel like we\u0027ve\n \u003e got an interest in having it working in case some user-facing issue\n \u003e comes up that genuinely *should* have a backport this far back. As\n \u003e Thiago said, that would normally be limited to security\n \u003e vulnerabilities.\n \u003e \n \u003e In general I agree that the advice for anyone trying to maintain\n \u003e stable/pike swift is and should be to upgrade to a more-recent\n \u003e release. But with the above concern (and considering how low-risk\n \u003e this change is; we don\u0027t even import oslo.config), I think I\u0027m OK\n \u003e with approving it.\n\nHi Tim, Thank you advice.\nSo cloud I use the same bug id to update master bandit.yaml or need to open new bug to fix it?","accounts_in_message":[],"_revision_number":2},{"id":"fcffe6e74bb7a9969d4c0b078919c662d3a03956","author":{"_account_id":29062,"name":"CY Chiang","email":"cychiang@cht.com.tw","username":"cychiang"},"date":"2019-07-30 00:44:06.000000000","message":"Patch Set 2:\n\n\u003e Hello Cy Chiang. Thank you for your interest in contributing with\n \u003e the Swift community. I noticed you have a couple of patches against\n \u003e the stable/pike. Pike was released almost 2 years ago and there\u0027s\n \u003e been 3 (almost 4) releases since then, which means we are not\n \u003e looking to keep merging changes to it, unless it is a serious\n \u003e security bug. Please feel free to submit changes against master and\n \u003e we will be happy to review and welcome your contributions.\n\nHi Thiago, Thanks!\nMaybe I could test the doc in stable/rocky?","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"8e086c7b880ebb15dab72e1850e51dafccfd3de4","revisions":{"1663919971ed4db6cd6ce3bd37db35ab8aff2206":{"kind":"REWORK","_number":1,"created":"2019-07-27 08:44:04.000000000","uploader":{"_account_id":29062,"name":"CY Chiang","email":"cychiang@cht.com.tw","username":"cychiang"},"ref":"refs/changes/31/673131/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/31/673131/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/31/673131/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/31/673131/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/31/673131/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/31/673131/1"}}},"commit":{"parents":[{"commit":"ca9ecfa154bca731bc4c6e938fabfac434c6e678","subject":"Merge \"object-server can 409 in response to x-if-delete-at\" into stable/pike","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/ca9ecfa154bca731bc4c6e938fabfac434c6e678"}]}],"author":{"name":"cychiang","email":"cychiang@cht.com.tw","date":"2019-07-27 08:40:14.000000000","tz":480},"committer":{"name":"cychiang","email":"cychiang@cht.com.tw","date":"2019-07-27 08:44:01.000000000","tz":480},"subject":"Fix the pep8 test will raise \"Unknown test found in profile: B109\"","message":"Fix the pep8 test will raise \"Unknown test found in profile: B109\"\n\nAccording to the bandit current version document, the B109 plugin\nhas been removed. so the bandit.yaml needs to remove the B109 test.\n\nChange-Id: Ib654bfb76a44484489a4622edb38acc43a0da672\nCloses-Bug: #1838123\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1663919971ed4db6cd6ce3bd37db35ab8aff2206"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/1663919971ed4db6cd6ce3bd37db35ab8aff2206"}]},"branch":"refs/heads/stable/pike"},"8e086c7b880ebb15dab72e1850e51dafccfd3de4":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2019-07-27 08:51:02.000000000","uploader":{"_account_id":29062,"name":"CY Chiang","email":"cychiang@cht.com.tw","username":"cychiang"},"ref":"refs/changes/31/673131/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/swift","ref":"refs/changes/31/673131/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/swift refs/changes/31/673131/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/swift refs/changes/31/673131/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/swift refs/changes/31/673131/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/swift refs/changes/31/673131/2"}}},"commit":{"parents":[{"commit":"ca9ecfa154bca731bc4c6e938fabfac434c6e678","subject":"Merge \"object-server can 409 in response to x-if-delete-at\" into stable/pike","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/ca9ecfa154bca731bc4c6e938fabfac434c6e678"}]}],"author":{"name":"cychiang","email":"cychiang@cht.com.tw","date":"2019-07-27 08:40:14.000000000","tz":480},"committer":{"name":"cychiang","email":"cychiang@cht.com.tw","date":"2019-07-27 08:49:57.000000000","tz":480},"subject":"Fix the pep8 test will raise \"Unknown test found in profile: B109\"","message":"Fix the pep8 test will raise \"Unknown test found in profile: B109\"\n\nAccording to the bandit current version document, the B109 plugin\nhas been removed. So the bandit.yaml needs to remove the B109 test.\nReference: https://github.com/PyCQA/bandit/commit/ with commit id\nis d93eed549242013ca10f6fc990dd4234b69958f7.\n\nChange-Id: Ib654bfb76a44484489a4622edb38acc43a0da672\nCloses-Bug: #1838123\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/8e086c7b880ebb15dab72e1850e51dafccfd3de4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/swift/commit/8e086c7b880ebb15dab72e1850e51dafccfd3de4"}]},"branch":"refs/heads/stable/pike"}},"requirements":[],"submit_records":[],"submit_requirements":[]}