)]}'
{"id":"openstack%2Fopenstacksdk~755692","triplet_id":"openstack%2Fopenstacksdk~master~I8783f09e91a24fdae492666bb20eca9e3d7e2cc1","project":"openstack/openstacksdk","branch":"master","topic":"bp/address-groups-in-sg-rules","hashtags":[],"change_id":"I8783f09e91a24fdae492666bb20eca9e3d7e2cc1","subject":"Revert \"Add neutron address group CRUD\"","status":"ABANDONED","created":"2020-10-02 05:28:03.000000000","updated":"2020-10-08 05:03:58.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f45158f1cd486ce21cfcd1236e95688a2699e8cf","_number":755692,"virtual_id_number":755692,"owner":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},{"tag":"autogenerated:zuul:check","value":1,"date":"2020-10-02 06:50:03.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":28159,"name":"Hang Yang","email":"hangyang@yahooinc.com","username":"hangyang"},{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},{"date":"2020-10-02 09:35:08.000000000","_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":28159,"name":"Hang Yang","email":"hangyang@yahooinc.com","username":"hangyang"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":-1,"date":"2020-10-03 13:37:18.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"}],"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":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"date":"2020-10-02 08:54:30.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"date":"2020-10-02 15:01:57.000000000","_account_id":28159,"name":"Hang Yang","email":"hangyang@yahooinc.com","username":"hangyang"},{"date":"2020-10-02 05:28:03.000000000","_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Allow-Post-Review":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":28159,"name":"Hang Yang","email":"hangyang@yahooinc.com","username":"hangyang"},{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"}],"values":{" 0":"Abstain","+1":"Approved for Zuul Post Review Pipeline"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},{"_account_id":28159,"name":"Hang Yang","email":"hangyang@yahooinc.com","username":"hangyang"},{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-10-02 05:28:03.000000000","updated_by":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"reviewer":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"state":"REVIEWER"},{"updated":"2020-10-02 06:50:03.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"},{"updated":"2020-10-02 08:54:30.000000000","updated_by":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"reviewer":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"state":"REVIEWER"},{"updated":"2020-10-02 15:01:57.000000000","updated_by":{"_account_id":28159,"name":"Hang Yang","email":"hangyang@yahooinc.com","username":"hangyang"},"reviewer":{"_account_id":28159,"name":"Hang Yang","email":"hangyang@yahooinc.com","username":"hangyang"},"state":"REVIEWER"},{"updated":"2020-10-03 13:37:18.000000000","updated_by":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"reviewer":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"state":"REVIEWER"}],"messages":[{"id":"09b0a06311dc1f970c581f3648aab5649180c85e","author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"date":"2020-10-02 05:28:03.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"47eda25997613cc3ca8c22ed53ae3fb86b972313","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-10-02 06:50:03.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/11455222f0ce4eb1b73d7b8e266de829 : SUCCESS in 5m 58s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/0f7e7a1f10a6425496a198bfbcffc4cb : SUCCESS in 3m 36s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/51ab78209789423497978f23bc29c510 : SUCCESS in 3m 57s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/c16a635622774745ad69d4b2f8c3d6b6 : SUCCESS in 5m 59s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/b28857c2f5be41c6b787d4f0feea8fc4 : SUCCESS in 5m 42s (non-voting)\n- openstacksdk-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/403f7ea226e74d3ea3474a3886232ae1 : SUCCESS in 56m 32s\n- openstacksdk-tox-py38-tips https://zuul.opendev.org/t/openstack/build/3575557a08054c75b113f07ed3824ef7 : SUCCESS in 4m 52s\n- os-client-config-tox-py37-tips https://zuul.opendev.org/t/openstack/build/f399dd81fd8f42f69ccaf1c187963379 : SUCCESS in 4m 59s\n- osc-tox-py36-tips https://zuul.opendev.org/t/openstack/build/73a93fd72c8e44c78d93a8fc3987d72f : SUCCESS in 4m 47s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6e9b8a0d1f6443bf9ef2409cba6ba2b5 : SUCCESS in 6m 38s\n- opendev-buildset-registry https://zuul.opendev.org/t/openstack/build/9261b3f41ac44678bce815e13662a3e4 : SUCCESS in 1h 03m 01s\n- nodepool-build-image-siblings https://zuul.opendev.org/t/openstack/build/6e6f9712b30e43e5880acd63b3c4e8de : SUCCESS in 17m 33s\n- nodepool-functional-container-openstack-siblings https://zuul.opendev.org/t/openstack/build/2d6d7807a72c41b0872ffe7731dd9dd4 : SUCCESS in 40m 19s\n- openstacksdk-functional-devstack https://zuul.opendev.org/t/openstack/build/b9ba82b0d07242a7b8f187bf80fcff52 : SUCCESS in 56m 58s\n- openstacksdk-functional-devstack-networking https://zuul.opendev.org/t/openstack/build/543499fd62674e6d9fda86de032d8510 : SUCCESS in 1h 19m 32s\n- openstacksdk-functional-devstack-senlin https://zuul.opendev.org/t/openstack/build/858ec883892e4dfbbdf9dfd014526d5c : SUCCESS in 54m 07s\n- openstacksdk-functional-devstack-magnum https://zuul.opendev.org/t/openstack/build/13d9ef7755064d05aad51b33eea38771 : POST_FAILURE in 29m 24s (non-voting)\n- openstacksdk-functional-devstack-masakari https://zuul.opendev.org/t/openstack/build/48720af562d4498f9c72b1b8a58d1e75 : FAILURE in 50m 24s (non-voting)\n- openstacksdk-functional-devstack-ironic https://zuul.opendev.org/t/openstack/build/33ce9f42fd3246f985aed5edb9046d3a : POST_FAILURE in 28m 43s (non-voting)\n- osc-functional-devstack-tips https://zuul.opendev.org/t/openstack/build/c69daf3a069e4b26a596566c8ddec24a : FAILURE in 52m 42s (non-voting)\n- ironic-inspector-tempest-openstacksdk-src https://zuul.opendev.org/t/openstack/build/82666930f94647bc811ec5eea113845c : SUCCESS in 54m 36s (non-voting)\n- bifrost-integration-tinyipa-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/dd256f51ddb34a199c6ff1ea10d7251a : SUCCESS in 22m 09s (non-voting)\n- metalsmith-integration-openstacksdk-src https://zuul.opendev.org/t/openstack/build/8c03306606c84dc08aa4028eb6acfd3c : SUCCESS in 1h 02m 39s (non-voting)\n- ansible-collections-openstack-functional-devstack https://zuul.opendev.org/t/openstack/build/bcf53f8ad1a44b61a84706212add8586 : SUCCESS in 44m 49s (non-voting)\n- openstacksdk-ansible-stable-2.8-functional-devstack https://zuul.opendev.org/t/openstack/build/002fc9f6ac3e4deeb8597aba2aa10d6e : FAILURE in 35m 50s (non-voting)\n- openstacksdk-ansible-stable-2.9-functional-devstack https://zuul.opendev.org/t/openstack/build/6c3b48ef08534a78804ae36b39d871f7 : FAILURE in 39m 47s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"a42311c9b2d4d6a20439fd6d5e39b28afc310a37","author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"date":"2020-10-02 08:54:30.000000000","message":"Patch Set 1:\n\nCan you please provide some more details on how only adding new functions affects tests? Or are you talking about new functional test added (in this case let\u0027s perhaps update/disable temporarily the test itself)?","accounts_in_message":[],"_revision_number":1},{"id":"14e0e9a415af28611ef7d53c5d770706e971a8fe","author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"date":"2020-10-02 09:35:08.000000000","message":"Patch Set 1:\n\nFor an example of the error in testing see https://aa8dd8dcd025e9c032ca-2413b0a350e1f4a4b77815cdd4c6afd0.ssl.cf2.rackcdn.com/755405/1/check/openstacksdk-functional-devstack/657dd57/job-output.txt\n\nBut I\u0027m also concerned about consumers getting errors when trying to use these functions against an older cloud. Or do we limit those kind of things only on the OSC side?","accounts_in_message":[],"_revision_number":1},{"id":"7eb61fe81eac9e84ce61ac3195d8718932bcc6d2","author":{"_account_id":28159,"name":"Hang Yang","email":"hangyang@yahooinc.com","username":"hangyang"},"date":"2020-10-02 14:26:59.000000000","message":"Patch Set 1:\n\nInstead of revert it, is there a way to make the address group functional test only run for the master version of neutron? I\u0027m wondering how do we add the client-side changes that depend on the latest server-side update?","accounts_in_message":[],"_revision_number":1},{"id":"1ac5a816aa4fcc49139adad2472fe373cc61301f","author":{"_account_id":28159,"name":"Hang Yang","email":"hangyang@yahooinc.com","username":"hangyang"},"date":"2020-10-02 15:01:57.000000000","message":"Patch Set 1:\n\n@Jens, I added this patch https://review.opendev.org/755818 and please let me know if that can fix the issue.","accounts_in_message":[],"_revision_number":1},{"id":"c8da6d4e07d76dd973b7c499cb52b1ea8f941b41","author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"date":"2020-10-03 13:37:18.000000000","message":"Patch Set 1: Code-Review-1\n\nThere is already proposed fix for that issue in https://review.opendev.org/#/c/755818/ so I don\u0027t think we need to revert it.","accounts_in_message":[],"_revision_number":1},{"id":"eb9ac1879e911146f92d9cd07a7e5789baa1ab8c","author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"date":"2020-10-08 05:03:58.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"3060816a83552fc778d63fac3653369cb37805b3","revisions":{"3060816a83552fc778d63fac3653369cb37805b3":{"kind":"REWORK","_number":1,"created":"2020-10-02 05:28:03.000000000","uploader":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"ref":"refs/changes/92/755692/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstacksdk","ref":"refs/changes/92/755692/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstacksdk refs/changes/92/755692/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstacksdk refs/changes/92/755692/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstacksdk refs/changes/92/755692/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstacksdk refs/changes/92/755692/1"}}},"commit":{"parents":[{"commit":"778f0f9451bab23224f58e3264f4aa0ee8e47bda","subject":"Add neutron address group CRUD","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstacksdk/commit/778f0f9451bab23224f58e3264f4aa0ee8e47bda"}]}],"author":{"name":"Jens Harbott (frickler)","email":"j.harbott@x-ion.de","date":"2020-10-02 05:28:03.000000000","tz":0},"committer":{"name":"Jens Harbott (frickler)","email":"j.harbott@x-ion.de","date":"2020-10-02 05:28:03.000000000","tz":0},"subject":"Revert \"Add neutron address group CRUD\"","message":"Revert \"Add neutron address group CRUD\"\n\nThis breaks testing on stable/ussuri or older where the functionality is missing in neutron.\n\nThis reverts commit 778f0f9451bab23224f58e3264f4aa0ee8e47bda.\n\nChange-Id: I8783f09e91a24fdae492666bb20eca9e3d7e2cc1\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstacksdk/commit/3060816a83552fc778d63fac3653369cb37805b3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstacksdk/commit/3060816a83552fc778d63fac3653369cb37805b3"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}