)]}'
{"id":"openstack%2Fkolla~670255","triplet_id":"openstack%2Fkolla~master~I7a7e06124e05df5da26f356d3d60b9b53c16547d","project":"openstack/kolla","branch":"master","topic":"neutron-migrate-fix","hashtags":[],"change_id":"I7a7e06124e05df5da26f356d3d60b9b53c16547d","subject":"Revert \"[neutron] Do not migrate all subprojects\"","status":"ABANDONED","created":"2019-07-11 09:16:55.000000000","updated":"2021-12-16 10:57:56.000000000","total_comment_count":11,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"bd3842fe439c64d07eabe4a7aa0d94d308923b37","_number":670255,"virtual_id_number":670255,"owner":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"_account_id":17130,"name":"melissaml","email":"ma.lei@99cloud.net","username":"malei"},{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"value":1,"date":"2019-07-12 10:08:14.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},{"date":"2019-07-12 09:24:35.000000000","_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":17130,"name":"melissaml","email":"ma.lei@99cloud.net","username":"malei"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"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":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"_account_id":17130,"name":"melissaml","email":"ma.lei@99cloud.net","username":"malei"},{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"_account_id":17130,"name":"melissaml","email":"ma.lei@99cloud.net","username":"malei"},{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]}],"values":{"-1":"Do Not Backport"," 0":"Backport Review Needed","+1":"Should Backport"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"date":"2019-07-12 08:40:52.000000000","_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"_account_id":17130,"name":"melissaml","email":"ma.lei@99cloud.net","username":"malei"},{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"_account_id":17130,"name":"melissaml","email":"ma.lei@99cloud.net","username":"malei"},{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-07-11 09:16:55.000000000","updated_by":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"reviewer":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"state":"REVIEWER"},{"updated":"2019-07-11 09:16:55.000000000","updated_by":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"reviewer":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"state":"REVIEWER"},{"updated":"2019-07-11 09:16:55.000000000","updated_by":{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},"reviewer":{"_account_id":27781,"name":"zhulingjie","email":"easyzlj@gmail.com","username":"zhulingjie"},"state":"REVIEWER"},{"updated":"2019-07-11 10:50:53.000000000","updated_by":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"reviewer":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"state":"REVIEWER"},{"updated":"2019-07-11 12:03:19.000000000","updated_by":{"_account_id":17130,"name":"melissaml","email":"ma.lei@99cloud.net","username":"malei"},"reviewer":{"_account_id":17130,"name":"melissaml","email":"ma.lei@99cloud.net","username":"malei"},"state":"REVIEWER"},{"updated":"2019-07-11 13:59:22.000000000","updated_by":{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},"reviewer":{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},"state":"REVIEWER"},{"updated":"2019-07-12 08:19:06.000000000","updated_by":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"reviewer":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"state":"REVIEWER"},{"updated":"2019-07-12 09:24:35.000000000","updated_by":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"reviewer":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2019-07-12 10:08:14.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":"7dc097606d65ba41f25d807d716d92abb0820fa7","author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"date":"2019-07-11 09:16:55.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"cda357b6437f6c90586014da2313a4ea207db00f","author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"date":"2019-07-11 09:17:28.000000000","message":"Patch Set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"2d054d74fa375460bae4ca9a6d5e9a0570853fa4","author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"date":"2019-07-11 09:24:45.000000000","message":"Patch Set 2:\n\n(2 comments)\n\nThese are the schema migrations missing after the change http://paste.openstack.org/show/754292/","accounts_in_message":[],"_revision_number":2},{"id":"21ff4a8b2ac08086ebe7197e8abe34ab1344ba19","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-07-11 09:26:38.000000000","message":"Patch Set 2: Code-Review+2\n\nfine to me as a revert\n\nneeds backporting to stein and rocky","accounts_in_message":[],"_revision_number":2},{"id":"7d92c33b98bb69b50db6c23079a00b950bd5e1a4","author":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"date":"2019-07-11 10:00:01.000000000","message":"Patch Set 2:\n\nBuild succeeded (check pipeline).\n\n- tripleo-build-containers-fedora-28-master http://logs.rdoproject.org/55/670255/2/openstack-check/tripleo-build-containers-fedora-28-master/acbabca/ : FAILURE in 6m 38s (non-voting)\n- tripleo-build-containers-fedora-28-stein http://logs.rdoproject.org/55/670255/2/openstack-check/tripleo-build-containers-fedora-28-stein/d5cd2ae/ : SUCCESS in 41m 26s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"c95c02042045270d0498b0efed62bbb168200efd","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-11 10:36:36.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- kolla-build-centos-binary http://logs.openstack.org/55/670255/2/check/kolla-build-centos-binary/9d622a0/ : SUCCESS in 32m 22s (non-voting)\n- kolla-build-centos-source http://logs.openstack.org/55/670255/2/check/kolla-build-centos-source/81744bc/ : SUCCESS in 41m 46s\n- kolla-ansible-centos-source http://logs.openstack.org/55/670255/2/check/kolla-ansible-centos-source/9db460e/ : SUCCESS in 1h 13m 38s\n- kolla-ansible-centos-binary http://logs.openstack.org/55/670255/2/check/kolla-ansible-centos-binary/6b10cc9/ : SUCCESS in 1h 07m 12s (non-voting)\n- kolla-ansible-centos-source-upgrade http://logs.openstack.org/55/670255/2/check/kolla-ansible-centos-source-upgrade/c62ec0b/ : SUCCESS in 1h 17m 06s\n- tripleo-build-containers-centos-7 http://logs.openstack.org/55/670255/2/check/tripleo-build-containers-centos-7/a0f3b36/ : SUCCESS in 51m 55s\n- openstack-tox-lower-constraints http://logs.openstack.org/55/670255/2/check/openstack-tox-lower-constraints/7431430/ : SUCCESS in 3m 14s\n- openstack-tox-pep8 http://logs.openstack.org/55/670255/2/check/openstack-tox-pep8/d1b4fbc/ : SUCCESS in 2m 53s\n- openstack-tox-py27 http://logs.openstack.org/55/670255/2/check/openstack-tox-py27/9fb887e/ : SUCCESS in 3m 31s\n- openstack-tox-py36 http://logs.openstack.org/55/670255/2/check/openstack-tox-py36/b2b036b/ : SUCCESS in 3m 11s\n- openstack-tox-py37 http://logs.openstack.org/55/670255/2/check/openstack-tox-py37/3436cba/ : SUCCESS in 4m 08s\n- openstack-tox-docs http://logs.openstack.org/55/670255/2/check/openstack-tox-docs/be27301/html/ : SUCCESS in 3m 10s\n- kolla-build-ubuntu-binary http://logs.openstack.org/55/670255/2/check/kolla-build-ubuntu-binary/39c3496/ : SUCCESS in 35m 17s (non-voting)\n- kolla-build-ubuntu-source http://logs.openstack.org/55/670255/2/check/kolla-build-ubuntu-source/35b9c1e/ : SUCCESS in 32m 12s\n- kolla-ansible-ubuntu-source http://logs.openstack.org/55/670255/2/check/kolla-ansible-ubuntu-source/70b44db/ : SUCCESS in 44m 10s\n- kolla-ansible-ubuntu-binary http://logs.openstack.org/55/670255/2/check/kolla-ansible-ubuntu-binary/923b3ec/ : SUCCESS in 53m 16s (non-voting)\n- kolla-ansible-ubuntu-source-upgrade http://logs.openstack.org/55/670255/2/check/kolla-ansible-ubuntu-source-upgrade/4eea21f/ : SUCCESS in 55m 49s","accounts_in_message":[],"_revision_number":2},{"id":"1e97d679e2f78b587f9d168044a1423b9fbf5645","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2019-07-11 10:50:53.000000000","message":"Patch Set 2: Code-Review+1\n\nWould like to get a reply from the author before merging.","accounts_in_message":[],"_revision_number":2},{"id":"79827519198ed0453b04d99e283db11c192edd1e","author":{"_account_id":17130,"name":"melissaml","email":"ma.lei@99cloud.net","username":"malei"},"date":"2019-07-11 12:03:19.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"f7834d3761975cd300b52db93911e66bb368c6ef","author":{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},"date":"2019-07-11 13:59:22.000000000","message":"Patch Set 2:\n\nI am the original author of the patch. The reason we submitted the patch was we were seeing issues with the upgrade path failing on a pre-existing Neutron installation, due to it attempting to upgrade a Neutron feature (LBaaS I believe) that was not enabled in the environment, but nonetheless had been enabled at some point, and so the upgrade was failing because it was trying to upgrade it from a very old state. Admittedly the \"real\" fix here is ... not sure? Wipe out the old tables entirely? Bite the bullet and do a stepwise major release upgrade? Anyways, that is the context here.\n\nAgree that if we\u0027re going to just use `upgrade head`, we should remove the other `upgrade` calls, as they are redundant and can be confusing to the reader.","accounts_in_message":[],"_revision_number":2},{"id":"bb7d91572fe4e18aad53a63ee82bb90c2400ccad","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2019-07-11 14:16:34.000000000","message":"Patch Set 2:\n\n\u003e I am the original author of the patch. The reason we submitted the\n \u003e patch was we were seeing issues with the upgrade path failing on a\n \u003e pre-existing Neutron installation, due to it attempting to upgrade\n \u003e a Neutron feature (LBaaS I believe) that was not enabled in the\n \u003e environment, but nonetheless had been enabled at some point, and so\n \u003e the upgrade was failing because it was trying to upgrade it from a\n \u003e very old state. Admittedly the \"real\" fix here is ... not sure?\n \u003e Wipe out the old tables entirely? Bite the bullet and do a stepwise\n \u003e major release upgrade? Anyways, that is the context here.\n \u003e \n \u003e Agree that if we\u0027re going to just use `upgrade head`, we should\n \u003e remove the other `upgrade` calls, as they are redundant and can be\n \u003e confusing to the reader.\n\nThanks for providing context. It is surprising that migrating old tables would cause a problem - I thought migrations should just get you from any version to the latest. Perhaps it\u0027s due to missing out on data migrations through expand/contract. It does feel like that is more of a corner case than having other subprojects enabled though.","accounts_in_message":[],"_revision_number":2},{"id":"f638b6183460e571edbe6e9a8b086faee690bd83","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2019-07-11 14:21:04.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"f0bb56ecd3460828b0a38d22c6ea097a887aad2f","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2019-07-11 14:22:31.000000000","message":"Patch Set 2:\n\nFWIW the upgrade guide doesn\u0027t even mention --subproject, so we\u0027re a bit off-piste here.\n\nhttps://docs.openstack.org/neutron/stein/contributor/internals/upgrade.html","accounts_in_message":[],"_revision_number":2},{"id":"685ac25ae3bf49fac2efd9404e62a7aabd6d9dcd","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2019-07-11 14:27:43.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"9d6d905fb43c67d2890abede206b4c8999c8ed0b","author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"date":"2019-07-12 08:16:22.000000000","message":"Patch Set 3: Patch Set 2 was rebased","accounts_in_message":[],"_revision_number":3},{"id":"a0a88290d7713e35931c6a70284a5d9c4baaa931","author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"date":"2019-07-12 08:17:04.000000000","message":"Patch Set 3:\n\nMade a rebase to use latest code as Mark saw the differences in previous PS","accounts_in_message":[],"_revision_number":3},{"id":"935dffb5e9ee7dc47d55a773c0d66f5bb9700c78","author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"date":"2019-07-12 08:19:06.000000000","message":"Patch Set 3: Code-Review+1\n\ndecreasing to +1 because discussion is ongoing","accounts_in_message":[],"_revision_number":3},{"id":"84352792a66f9cf9461cbdaee514741493efc78f","author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"date":"2019-07-12 08:21:18.000000000","message":"Patch Set 3:\n\n@Jason, could you open a bug with all the information you have about the issue while upgrading?\nRegarding the fix, we will need to backport to stable branches the revert as it breaks other functionalities we were not aware of, but please feel free to propose other changes to improve the way we make use of this.","accounts_in_message":[],"_revision_number":3},{"id":"83e7a429bab51b531af8878a8374e5a90e9e3c6f","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2019-07-12 08:30:35.000000000","message":"Patch Set 3:\n\n\u003e @Jason, could you open a bug with all the information you have\n \u003e about the issue while upgrading?\n \u003e Regarding the fix, we will need to backport to stable branches the\n \u003e revert as it breaks other functionalities we were not aware of, but\n \u003e please feel free to propose other changes to improve the way we\n \u003e make use of this.\n\nIMO we should just fix this in one patch rather than keeping to a \u0027clean\u0027 revert.","accounts_in_message":[],"_revision_number":3},{"id":"fd7fac4c05c47b2ebf9ddbf1774f37b3f6d1483d","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2019-07-12 08:33:42.000000000","message":"Patch Set 3:\n\n(4 comments)\n\nEduardo are you going to continue working on this patch or would you like someone else to pick it up?","accounts_in_message":[],"_revision_number":3},{"id":"5117c5c7142645df35113fe222909f49c6677643","author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"date":"2019-07-12 08:40:27.000000000","message":"Patch Set 3:\n\n(2 comments)","accounts_in_message":[],"_revision_number":3},{"id":"65884dd4dfbe1789ed34c0b94c4ee62857a4660b","author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"date":"2019-07-12 08:40:52.000000000","message":"Patch Set 4: Commit message was updated.","accounts_in_message":[],"_revision_number":4},{"id":"d94a6138e4b087bf02564524755bbb000114431a","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2019-07-12 09:10:36.000000000","message":"Patch Set 3:\n\n(1 comment)","accounts_in_message":[],"_revision_number":3},{"id":"ec35664915e940b68301f8e68575133974e10c47","author":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"date":"2019-07-12 09:24:35.000000000","message":"Patch Set 4:\n\nBuild succeeded (check pipeline).\n\n- tripleo-build-containers-fedora-28-master http://logs.rdoproject.org/55/670255/4/openstack-check/tripleo-build-containers-fedora-28-master/a90098c/ : FAILURE in 5m 37s (non-voting)\n- tripleo-build-containers-fedora-28-stein http://logs.rdoproject.org/55/670255/4/openstack-check/tripleo-build-containers-fedora-28-stein/7ea7cc0/ : SUCCESS in 40m 01s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"816b03a16fb14183520e3e8931bc1d574bd78364","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-12 10:08:14.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- kolla-build-centos-binary http://logs.openstack.org/55/670255/4/check/kolla-build-centos-binary/2576a14/ : SUCCESS in 42m 41s (non-voting)\n- kolla-build-centos-source http://logs.openstack.org/55/670255/4/check/kolla-build-centos-source/289f19d/ : SUCCESS in 33m 07s\n- kolla-ansible-centos-source http://logs.openstack.org/55/670255/4/check/kolla-ansible-centos-source/48c8a25/ : SUCCESS in 58m 52s\n- kolla-ansible-centos-binary http://logs.openstack.org/55/670255/4/check/kolla-ansible-centos-binary/ee87a43/ : SUCCESS in 1h 10m 19s (non-voting)\n- kolla-ansible-centos-source-upgrade http://logs.openstack.org/55/670255/4/check/kolla-ansible-centos-source-upgrade/7bae368/ : SUCCESS in 1h 25m 06s\n- tripleo-build-containers-centos-7 http://logs.openstack.org/55/670255/4/check/tripleo-build-containers-centos-7/70ba704/ : SUCCESS in 45m 21s\n- openstack-tox-lower-constraints http://logs.openstack.org/55/670255/4/check/openstack-tox-lower-constraints/5f2a606/ : SUCCESS in 3m 17s\n- openstack-tox-pep8 http://logs.openstack.org/55/670255/4/check/openstack-tox-pep8/45f1f4d/ : SUCCESS in 3m 57s\n- openstack-tox-py27 http://logs.openstack.org/55/670255/4/check/openstack-tox-py27/0914124/ : SUCCESS in 3m 26s\n- openstack-tox-py36 http://logs.openstack.org/55/670255/4/check/openstack-tox-py36/fd5b8ea/ : SUCCESS in 3m 22s\n- openstack-tox-py37 http://logs.openstack.org/55/670255/4/check/openstack-tox-py37/44120b2/ : SUCCESS in 3m 34s\n- openstack-tox-docs http://logs.openstack.org/55/670255/4/check/openstack-tox-docs/de487c8/html/ : SUCCESS in 2m 30s\n- kolla-build-ubuntu-binary http://logs.openstack.org/55/670255/4/check/kolla-build-ubuntu-binary/234bb6e/ : SUCCESS in 28m 49s (non-voting)\n- kolla-build-ubuntu-source http://logs.openstack.org/55/670255/4/check/kolla-build-ubuntu-source/e6e74b6/ : SUCCESS in 34m 06s\n- kolla-ansible-ubuntu-source http://logs.openstack.org/55/670255/4/check/kolla-ansible-ubuntu-source/5ba75be/ : SUCCESS in 51m 05s\n- kolla-ansible-ubuntu-binary http://logs.openstack.org/55/670255/4/check/kolla-ansible-ubuntu-binary/c3f6f04/ : SUCCESS in 47m 15s (non-voting)\n- kolla-ansible-ubuntu-source-upgrade http://logs.openstack.org/55/670255/4/check/kolla-ansible-ubuntu-source-upgrade/5131442/ : SUCCESS in 1h 01m 13s","accounts_in_message":[],"_revision_number":4},{"id":"bd3842fe439c64d07eabe4a7aa0d94d308923b37","tag":"autogenerated:gerrit:abandon","author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"date":"2021-12-16 10:57:56.000000000","message":"Abandoned\n\npatch has been inactive for over two years - please restore if you plan to work on it","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"b4c1f80d7f3bb303a814d63aa97ea080d5f46bb9","revisions":{"3be979f299b4449cad5cffd92d0ee822139eb6f4":{"kind":"REWORK","_number":1,"created":"2019-07-11 09:16:55.000000000","uploader":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"ref":"refs/changes/55/670255/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/kolla","ref":"refs/changes/55/670255/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/kolla refs/changes/55/670255/1"}}},"commit":{"parents":[{"commit":"f36d7aacf28bc75a4b0ece73b45aa09591cd262c","subject":"[neutron] Do not migrate all subprojects","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/f36d7aacf28bc75a4b0ece73b45aa09591cd262c"}]}],"author":{"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","date":"2019-07-11 09:16:55.000000000","tz":0},"committer":{"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","date":"2019-07-11 09:16:55.000000000","tz":0},"subject":"Revert \"[neutron] Do not migrate all subprojects\"","message":"Revert \"[neutron] Do not migrate all subprojects\"\n\nThis reverts commit f36d7aacf28bc75a4b0ece73b45aa09591cd262c.\n\nWe don\u0027t have the logic in ansible nor in the extend_start.sh to do migrations of every subproject, is a potential issue to user already using those subprojects and upgrading, as well as it breaks new deployments.\n\nSubprojects which haven\u0027t been migrated bgpvpn, l2gw, odl, neutron-dynamic-routing, fwaas, vpnaas and neutron-nsx\n\nChange-Id: I7a7e06124e05df5da26f356d3d60b9b53c16547d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/3be979f299b4449cad5cffd92d0ee822139eb6f4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/3be979f299b4449cad5cffd92d0ee822139eb6f4"}]},"branch":"refs/heads/master"},"e3669b47bd56ebac4dabf9723e6f487c79fb93ba":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2019-07-11 09:17:28.000000000","uploader":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"ref":"refs/changes/55/670255/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/kolla","ref":"refs/changes/55/670255/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/kolla refs/changes/55/670255/2"}}},"commit":{"parents":[{"commit":"f36d7aacf28bc75a4b0ece73b45aa09591cd262c","subject":"[neutron] Do not migrate all subprojects","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/f36d7aacf28bc75a4b0ece73b45aa09591cd262c"}]}],"author":{"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","date":"2019-07-11 09:16:55.000000000","tz":0},"committer":{"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","date":"2019-07-11 09:17:25.000000000","tz":0},"subject":"Revert \"[neutron] Do not migrate all subprojects\"","message":"Revert \"[neutron] Do not migrate all subprojects\"\n\nThis reverts commit f36d7aacf28bc75a4b0ece73b45aa09591cd262c.\n\nWe don\u0027t have the logic in ansible nor in the extend_start.sh to do\nmigrations of every subproject, is a potential issue to user already\nusing those subprojects and upgrading, as well as it breaks new deployments.\n\nSubprojects which haven\u0027t been migrated bgpvpn, l2gw, odl,\nneutron-dynamic-routing, fwaas, vpnaas and neutron-nsx\n\nChange-Id: I7a7e06124e05df5da26f356d3d60b9b53c16547d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/e3669b47bd56ebac4dabf9723e6f487c79fb93ba"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/e3669b47bd56ebac4dabf9723e6f487c79fb93ba"}]},"branch":"refs/heads/master"},"04d3d4dfd8645de4ac8b363c3961e2bf008c0c99":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2019-07-12 08:16:22.000000000","uploader":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"ref":"refs/changes/55/670255/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/kolla","ref":"refs/changes/55/670255/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/kolla refs/changes/55/670255/3"}}},"commit":{"parents":[{"commit":"008aa957f27b26caff3be8b18110e71cdaf70772","subject":"Add Python 3 Train unit tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/008aa957f27b26caff3be8b18110e71cdaf70772"}]}],"author":{"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","date":"2019-07-11 09:16:55.000000000","tz":0},"committer":{"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","date":"2019-07-12 08:16:22.000000000","tz":0},"subject":"Revert \"[neutron] Do not migrate all subprojects\"","message":"Revert \"[neutron] Do not migrate all subprojects\"\n\nThis reverts commit f36d7aacf28bc75a4b0ece73b45aa09591cd262c.\n\nWe don\u0027t have the logic in ansible nor in the extend_start.sh to do\nmigrations of every subproject, is a potential issue to user already\nusing those subprojects and upgrading, as well as it breaks new deployments.\n\nSubprojects which haven\u0027t been migrated bgpvpn, l2gw, odl,\nneutron-dynamic-routing, fwaas, vpnaas and neutron-nsx\n\nChange-Id: I7a7e06124e05df5da26f356d3d60b9b53c16547d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/04d3d4dfd8645de4ac8b363c3961e2bf008c0c99"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/04d3d4dfd8645de4ac8b363c3961e2bf008c0c99"}]},"branch":"refs/heads/master"},"b4c1f80d7f3bb303a814d63aa97ea080d5f46bb9":{"kind":"NO_CODE_CHANGE","_number":4,"created":"2019-07-12 08:40:52.000000000","uploader":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"ref":"refs/changes/55/670255/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/kolla","ref":"refs/changes/55/670255/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/kolla refs/changes/55/670255/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/kolla refs/changes/55/670255/4"}}},"commit":{"parents":[{"commit":"008aa957f27b26caff3be8b18110e71cdaf70772","subject":"Add Python 3 Train unit tests","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/008aa957f27b26caff3be8b18110e71cdaf70772"}]}],"author":{"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","date":"2019-07-11 09:16:55.000000000","tz":0},"committer":{"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","date":"2019-07-12 08:40:47.000000000","tz":0},"subject":"Revert \"[neutron] Do not migrate all subprojects\"","message":"Revert \"[neutron] Do not migrate all subprojects\"\n\nThis reverts commit f36d7aacf28bc75a4b0ece73b45aa09591cd262c.\n\nWe don\u0027t have the logic in ansible nor in the extend_start.sh to do\nmigrations of every subproject, is a potential issue to user already\nusing those subprojects and upgrading, as well as it breaks new deployments.\n\nSubprojects which haven\u0027t been migrated bgpvpn, l2gw, odl,\nneutron-dynamic-routing, fwaas, vpnaas and neutron-nsx\n\nCloses-Bug: #1835735\nChange-Id: I7a7e06124e05df5da26f356d3d60b9b53c16547d\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/b4c1f80d7f3bb303a814d63aa97ea080d5f46bb9"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/b4c1f80d7f3bb303a814d63aa97ea080d5f46bb9"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}