)]}'
{"id":"openstack%2Fpython-magnumclient~424398","triplet_id":"openstack%2Fpython-magnumclient~master~I4f5424403a5187d6845d59d005be2324ff40f354","project":"openstack/python-magnumclient","branch":"master","topic":"bug/1658845","hashtags":[],"change_id":"I4f5424403a5187d6845d59d005be2324ff40f354","subject":"Remove H903 error in sources.","status":"MERGED","created":"2017-01-24 01:16:45.000000000","updated":"2017-01-24 06:22:23.000000000","submitted":"2017-01-24 06:22:22.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9dff42d871d592577c81aadaf443dd4a9bda3aa8","_number":424398,"virtual_id_number":424398,"owner":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2017-01-24 06:22:22.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"}],"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":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2017-01-24 06:16:59.000000000","_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":2,"date":"2017-01-24 01:18:04.000000000","_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"}],"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":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2017-01-24 06:17:11.000000000","_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"}],"values":{"-2":"Do Not Backport","-1":"Not A Backport Candidate"," 0":"Backport Review Needed","+1":"Proposed Backport","+2":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"},{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-01-24 01:18:04.000000000","updated_by":{"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"},"reviewer":{"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"},"state":"REVIEWER"},{"updated":"2017-01-24 06:17:11.000000000","updated_by":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"reviewer":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"state":"REVIEWER"},{"updated":"2017-01-24 06:22:22.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"db476e5954b717de266d572270861d251e9c1383","author":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"date":"2017-01-24 01:16:45.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"65a4b846ae8a99e764ca7817861f8e4cae8d4ad0","author":{"_account_id":668,"name":"Adrian Otto","email":"aotto@aotto.com","username":"aotto"},"date":"2017-01-24 01:18:04.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"91f4d8b55127b521dea1bb38c0ca6aaf490def06","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-01-24 01:25:35.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-python-magnumclient-docs-ubuntu-xenial http://docs-draft.openstack.org/98/424398/1/check/gate-python-magnumclient-docs-ubuntu-xenial/af0c7f1//doc/build/html/ : SUCCESS in 3m 33s\n- gate-python-magnumclient-pep8-ubuntu-xenial http://logs.openstack.org/98/424398/1/check/gate-python-magnumclient-pep8-ubuntu-xenial/870b0ba/ : SUCCESS in 2m 26s\n- gate-python-magnumclient-python27-ubuntu-xenial http://logs.openstack.org/98/424398/1/check/gate-python-magnumclient-python27-ubuntu-xenial/c805b31/ : SUCCESS in 2m 03s\n- gate-python-magnumclient-python35 http://logs.openstack.org/98/424398/1/check/gate-python-magnumclient-python35/c5615bb/ : SUCCESS in 2m 00s\n- python-magnumclient-coverage-ubuntu-xenial http://logs.openstack.org/98/424398/1/check/python-magnumclient-coverage-ubuntu-xenial/5a4b268/ : SUCCESS in 2m 06s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"6d47fa706d9647c5770b7bfd2e8a6f509a99753b","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2017-01-24 02:58:54.000000000","message":"Patch Set 1: Code-Review-1\n\n@Hieu, a patch was there few days ago: https://review.openstack.org/#/c/410282/\n\nHow you got this error, is tox reported?\nIt\u0027s Ok to do this but can something be done to automate the reporting process(via tox or something else) so it never occurs.","accounts_in_message":[],"_revision_number":1},{"id":"dfba1979f5108a3e3e053d8db45e6426acc42537","author":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"date":"2017-01-24 03:49:04.000000000","message":"Patch Set 1:\n\n\u003e @Hieu, a patch was there few days ago: https://review.openstack.org/#/c/410282/\n \u003e \n \u003e How you got this error, is tox reported?\n \u003e It\u0027s Ok to do this but can something be done to automate the\n \u003e reporting process(via tox or something else) so it never occurs.\n\nyatin, this report was generated via tox with py35 based via flake8 v2.\n\nalso pep8 gate in Magnum ignore H903, I think we need to get rid of this error kind in development environment (cause it make my eye bleed)","accounts_in_message":[],"_revision_number":1},{"id":"bfdf321e3a8ee13193bbe0c4e5ac763e7cb4dca8","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2017-01-24 06:16:59.000000000","message":"Patch Set 1: Code-Review+2\n\n\u003e \u003e @Hieu, a patch was there few days ago: https://review.openstack.org/#/c/410282/\n \u003e \u003e\n \u003e \u003e How you got this error, is tox reported?\n \u003e \u003e It\u0027s Ok to do this but can something be done to automate the\n \u003e \u003e reporting process(via tox or something else) so it never occurs.\n \u003e \n \u003e yatin, this report was generated via tox with py35 based via flake8\n \u003e v2.\n \u003e \n \u003e also pep8 gate in Magnum ignore H903, I think we need to get rid of\n \u003e this error kind in development environment (cause it make my eye\n \u003e bleed)\n\nOk, tox is reporting in py35 environment, not seen this in py27 environment. We should enforce H903 for py27 so we can avoid such patches in future. \n@Hieu, others if someone knows how this can be done submit a seperate patch for this.","accounts_in_message":[],"_revision_number":1},{"id":"e35a33b2421c01aac62a9a50d966f39f2359f986","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2017-01-24 06:17:11.000000000","message":"Patch Set 1: Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"af4fa3a365ca29e4465862aee41c670b6000a3d3","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-01-24 06:17:21.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"90573955d90736ba2edc147b1c1d588635fb19a4","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-01-24 06:22:22.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-python-magnumclient-docs-ubuntu-xenial http://docs-draft.openstack.org/98/424398/1/gate/gate-python-magnumclient-docs-ubuntu-xenial/b8a749c//doc/build/html/ : SUCCESS in 3m 29s\n- gate-python-magnumclient-pep8-ubuntu-xenial http://logs.openstack.org/98/424398/1/gate/gate-python-magnumclient-pep8-ubuntu-xenial/6280403/ : SUCCESS in 3m 53s\n- gate-python-magnumclient-python27-ubuntu-xenial http://logs.openstack.org/98/424398/1/gate/gate-python-magnumclient-python27-ubuntu-xenial/db318bd/ : SUCCESS in 3m 03s\n- gate-python-magnumclient-python35 http://logs.openstack.org/98/424398/1/gate/gate-python-magnumclient-python35/84f99ac/ : SUCCESS in 3m 14s","accounts_in_message":[],"_revision_number":1},{"id":"349660ef3bb876e629c29755ed7d2e5bd6c807dc","date":"2017-01-24 06:22:23.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"de91fd3e0ad2937aac4082136396ad147162acc0","revisions":{"de91fd3e0ad2937aac4082136396ad147162acc0":{"kind":"REWORK","_number":1,"created":"2017-01-24 01:16:45.000000000","uploader":{"_account_id":19741,"name":"Hieu LE","email":"hieulq2@viettel.com.vn","username":"hieulq"},"ref":"refs/changes/98/424398/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/python-magnumclient","ref":"refs/changes/98/424398/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/98/424398/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/98/424398/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/python-magnumclient refs/changes/98/424398/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/python-magnumclient refs/changes/98/424398/1"}}},"commit":{"parents":[{"commit":"af7e5f605a1d84def0555a26aea8a186ca9d1e92","subject":"Merge \"Add OSC command for cluster_templates list\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/af7e5f605a1d84def0555a26aea8a186ca9d1e92"}]}],"author":{"name":"Hieu LE","email":"hieulq@vn.fujitsu.com","date":"2017-01-24 01:17:11.000000000","tz":420},"committer":{"name":"Hieu LE","email":"hieulq@vn.fujitsu.com","date":"2017-01-24 01:17:11.000000000","tz":420},"subject":"Remove H903 error in sources.","message":"Remove H903 error in sources.\n\nSome codes in python magnumclient use Windows style\nline endings, this violates H903 error pep8.\n\nChange-Id: I4f5424403a5187d6845d59d005be2324ff40f354\nCloses-Bug: #1658845\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/de91fd3e0ad2937aac4082136396ad147162acc0"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/python-magnumclient/commit/de91fd3e0ad2937aac4082136396ad147162acc0"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
