)]}'
{"id":"openstack%2Fkolla~704531","triplet_id":"openstack%2Fkolla~master~I4e5face2d8591db7f54738fe54702e63bc28eea8","project":"openstack/kolla","branch":"master","topic":"do-not-build-existing-images","hashtags":[],"change_id":"I4e5face2d8591db7f54738fe54702e63bc28eea8","subject":"Do not rebuild existing images, but allow the children to be built","status":"ABANDONED","created":"2020-01-28 10:49:24.000000000","updated":"2020-03-16 13:00:01.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"1a89020b1f89d0234b561b9bca34b7f2100d6c2e","_number":704531,"virtual_id_number":704531,"owner":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"date":"2020-01-28 16:22:29.000000000","_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},{"date":"2020-01-28 12:16:41.000000000","_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},{"tag":"autogenerated:zuul:check","value":1,"date":"2020-01-28 12:21:59.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"date":"2020-01-28 11:24:38.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":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},{"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":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"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":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},{"_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"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},{"_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"]}],"values":{"-1":"Do Not Backport"," 0":"Backport Review Needed","+1":"Should Backport"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},{"_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"]}],"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":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},{"_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"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-01-28 11:24:38.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":"2020-01-28 12:16:41.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":"2020-01-28 12:21:59.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":"be3f00ee885df1ac95744a2cfac245501f31340e","author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"date":"2020-01-28 10:49:24.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"04835c6b5567c51e85bd877c3f5d52d9e154b728","author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"date":"2020-01-28 10:54:10.000000000","message":"Patch Set 1:\n\nYou know that when you say what you want to build you are giving regexp?\n\nkolla-build openswitch \u003d\u003d .*openswitch.* so openswitch-base is also built.\n\nMaybe you want --skip-parents? Or exact list of images to build?","accounts_in_message":[],"_revision_number":1},{"id":"50ebcd1512a5815de86a0646199a6f4b25a8c4ad","author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"date":"2020-01-28 11:13:40.000000000","message":"Patch Set 1:\n\nWe do it this way:\n\nWe build our base images in stage 1 (e.g. with openvswitch-base). This will build base + openvswitch-base.\n\nIn stage 2 we build required base images (e.g. base + openvswitch-base when building openvswitch images) and now we start the build of the openvswitch images with \"^openvswitch\". We do not want to maintain a list of all openvswitch images, we want to build everything inside the openvswitch directory.","accounts_in_message":[],"_revision_number":1},{"id":"45b618d23d930575573f9af17111cc225014d328","author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"date":"2020-01-28 11:14:11.000000000","message":"Patch Set 1:\n\nIn stage 2 we pull required base images and build the openvswitch images...","accounts_in_message":[],"_revision_number":1},{"id":"d718662284f189bf2032d4623bd751f714ab271d","author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"date":"2020-01-28 11:15:59.000000000","message":"Patch Set 1:\n\n--skip-parents did not work for all images on Rocky. Maybe this was fixed in the meanwhile? Then this change is not necessary.","accounts_in_message":[],"_revision_number":1},{"id":"021c44c894c62575795c2b85fdc5af744c41871c","author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"date":"2020-01-28 11:21:13.000000000","message":"Patch Set 1:\n\n\u003e --skip-parents did not work for all images on Rocky. Maybe this was\n \u003e fixed in the meanwhile? Then this change is not necessary.\n\nNo idea does work as expected. It is so rarely used part of Kolla. I know about it as had to play with it during adding UNBUILDABLE_IMAGES support.","accounts_in_message":[],"_revision_number":1},{"id":"565acde4460b72608ed7640eedf70474c2091cf3","author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"date":"2020-01-28 11:22:32.000000000","message":"Patch Set 1:\n\nI tried it with \"kolla-build --skip-parents ^openvswitch\" on Rocky and this skipped the base image but it rebuilt the openvswitch-base image.","accounts_in_message":[],"_revision_number":1},{"id":"d6d0b981a63c919e0cb63eebefc635e1e3f5d267","author":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"date":"2020-01-28 11:24:38.000000000","message":"Patch Set 1:\n\nBuild succeeded (check pipeline).\n\n- tripleo-build-containers-rhel-8-master https://review.rdoproject.org/zuul/build/9df1be48cee54bcf892497f47acb3293 : SUCCESS in 27m 18s","accounts_in_message":[],"_revision_number":1},{"id":"2d72e1ad570ce2b8eabf88792af93150275279c3","author":{"_account_id":24072,"name":"Marcin Juszkiewicz","email":"mjuszkiewicz@redhat.com","username":"hrw"},"date":"2020-01-28 12:16:41.000000000","message":"Patch Set 1:\n\n(venv3) 12:48 (7s) hrw@puchatek:kolla$ ./tools/build.py openvswitch-base\nINFO:kolla.common.utils:\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nINFO:kolla.common.utils:Successfully built images\nINFO:kolla.common.utils:\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nINFO:kolla.common.utils:base\nINFO:kolla.common.utils:openvswitch-base\n\n(venv3) 13:14 (9s) hrw@puchatek:kolla$ ./tools/build.py ^openvswitch --skip-parents\nINFO:kolla.common.utils:\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nINFO:kolla.common.utils:Successfully built images\nINFO:kolla.common.utils:\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nINFO:kolla.common.utils:openvswitch-db-server\nINFO:kolla.common.utils:openvswitch-vswitchd\nINFO:kolla.common.utils:\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nINFO:kolla.common.utils:Images skipped due to build options\nINFO:kolla.common.utils:\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nINFO:kolla.common.utils:openvswitch-base\nINFO:kolla.common.utils:\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nINFO:kolla.common.utils:Images not buildable due to build options\nINFO:kolla.common.utils:\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\nINFO:kolla.common.utils:openvswitch-netcontrold\n\n\nLooks like it works now. Did not even tried to build openvswitch-base in second run.","accounts_in_message":[],"_revision_number":1},{"id":"2992a7dbeca2199d3ec6f04bc640252552e13395","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-28 12:21:59.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- kolla-build-centos-binary https://zuul.opendev.org/t/openstack/build/0f47578e556b4f2d8cb8e83829eed92d : SUCCESS in 35m 04s (non-voting)\n- kolla-build-centos-source https://zuul.opendev.org/t/openstack/build/84ad0885248c45bfa9b59d7989714ca6 : SUCCESS in 44m 54s\n- kolla-build-centos8-binary https://zuul.opendev.org/t/openstack/build/f84d145760c0498a94bf0107c6f3baa2 : SUCCESS in 6m 56s (non-voting)\n- kolla-build-centos8-source https://zuul.opendev.org/t/openstack/build/dc7c06064a024bae927e9780341c1918 : SUCCESS in 6m 01s\n- kolla-ansible-centos-source https://zuul.opendev.org/t/openstack/build/2999636ba424409cb73fd525b69f60cf : SUCCESS in 1h 18m 49s\n- kolla-ansible-centos-binary https://zuul.opendev.org/t/openstack/build/1ba9c834d0f047b28f5bb859bd835d1b : SUCCESS in 1h 14m 35s (non-voting)\n- kolla-ansible-centos-source-upgrade https://zuul.opendev.org/t/openstack/build/39e06101a24448dc82cdf9cb4c0587c5 : SUCCESS in 1h 31m 00s\n- tripleo-build-containers-centos-7 https://zuul.opendev.org/t/openstack/build/7a5362d50ad340439313a2a022ffca04 : SUCCESS in 47m 15s\n- kolla-build-debian-source https://zuul.opendev.org/t/openstack/build/fc0efb19ab39463e96ea1cb84378b704 : SUCCESS in 36m 28s (non-voting)\n- kolla-build-debian-binary https://zuul.opendev.org/t/openstack/build/318ed413a4b74ce9bffcb4e679236243 : SUCCESS in 21m 11s (non-voting)\n- kolla-ansible-debian-source https://zuul.opendev.org/t/openstack/build/e45b4538fc1e45909166da68e6b33c80 : SUCCESS in 54m 59s (non-voting)\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/f92f4619ae134a949580caadc5a638b8 : SUCCESS in 3m 49s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f7d94f3dc17c435394843eb6098f2e30 : SUCCESS in 4m 16s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/7e248f735a38416cb21a9fb1464b4a8b : SUCCESS in 5m 16s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/53a9b48c0547434a803a13ea6618e418 : SUCCESS in 4m 43s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/a1762d24093c4b9390753673b72ac40a : SUCCESS in 4m 04s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/0f00d776a98d4eb4863b80553d5d8fa7 : SUCCESS in 7m 08s\n- kolla-build-ubuntu-binary https://zuul.opendev.org/t/openstack/build/e278e07871e646a0b1da50727086d0d7 : SUCCESS in 34m 53s (non-voting)\n- kolla-build-ubuntu-source https://zuul.opendev.org/t/openstack/build/06c070749ab44a649f57efb89b203a89 : SUCCESS in 33m 59s\n- kolla-ansible-ubuntu-source https://zuul.opendev.org/t/openstack/build/790b881760a148859f311fe0088a5691 : SUCCESS in 56m 33s\n- kolla-ansible-ubuntu-binary https://zuul.opendev.org/t/openstack/build/6ad91fb2d01b4e639d45cecc8fafd704 : SUCCESS in 55m 32s (non-voting)\n- kolla-ansible-ubuntu-source-upgrade https://zuul.opendev.org/t/openstack/build/28ea11fb74a34eedbe2575c77fa7c12d : SUCCESS in 1h 01m 50s","accounts_in_message":[],"_revision_number":1},{"id":"5644bbcf346f1a118088f05eaa419c4f1af556a0","tag":"autogenerated:zuul:check-arm64","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-28 13:46:09.000000000","message":"Patch Set 1:\n\nBuild succeeded (ARM64 pipeline).\n\n- kolla-build-debian-source-aarch64 https://zuul.opendev.org/t/openstack/build/5325df100be240f6a9bb6510ecd42e27 : SUCCESS in 2h 48m 37s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"72aff241ed870c3d019e9c00e38d9db4612e92e9","author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"date":"2020-01-28 16:22:29.000000000","message":"Patch Set 1:\n\nWe have divided the build process into several stages. Base Images, Infrastructure Images, OpenStack Core Images and OpenStack Additional Images.\n\nThanks for the test. Then the review can be closed if it works on the master.","accounts_in_message":[],"_revision_number":1},{"id":"ad3c43e3501ee0e0f84ffb060c8948999ba074e1","author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"date":"2020-01-28 16:22:33.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":1},{"id":"cdf28f0e916932a861e1d8f0b7bc536501bb802f","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2020-03-16 11:31:02.000000000","message":"Patch Set 1:\n\n(1 comment)\n\nWorking on an alternative. Would like to know if it fixes your issue.","accounts_in_message":[],"_revision_number":1},{"id":"589a1688624643867a8f86e3668ceb9b1ae0c177","author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"date":"2020-03-16 13:00:01.000000000","message":"Patch Set 1:\n\n\u003e (1 comment)\n \u003e \n \u003e Working on an alternative. Would like to know if it fixes your\n \u003e issue.\n\nhttps://review.opendev.org/713221","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"9f739dfa3ad2c7302b7fd9bc3e1d91f262845c47","revisions":{"9f739dfa3ad2c7302b7fd9bc3e1d91f262845c47":{"kind":"REWORK","_number":1,"created":"2020-01-28 10:49:24.000000000","uploader":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"ref":"refs/changes/31/704531/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/kolla","ref":"refs/changes/31/704531/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/kolla refs/changes/31/704531/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/kolla refs/changes/31/704531/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/kolla refs/changes/31/704531/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/kolla refs/changes/31/704531/1"}}},"commit":{"parents":[{"commit":"fa2e1304362d3b81b9903b77d180aa2eb3f1af8b","subject":"build: list only buildable templates for --template-only","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/fa2e1304362d3b81b9903b77d180aa2eb3f1af8b"}]}],"author":{"name":"Christian Berendt","email":"berendt@betacloud-solutions.de","date":"2020-01-28 10:42:21.000000000","tz":60},"committer":{"name":"Christian Berendt","email":"berendt@betacloud-solutions.de","date":"2020-01-28 10:45:03.000000000","tz":60},"subject":"Do not rebuild existing images, but allow the children to be built","message":"Do not rebuild existing images, but allow the children to be built\n\nWhen using --skip-existing, only parent images are currently checked.\n\nE.g. if openvswitch is to be built it is checked if base exists. If base\nexists it will be skipped. If openvswitch-base already exists it will be\nbuilt anyway.\n\nWith this change, existing images are really not rebuilt. Builds by children\nof the skipped images are still triggered.\n\nChange-Id: I4e5face2d8591db7f54738fe54702e63bc28eea8\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/9f739dfa3ad2c7302b7fd9bc3e1d91f262845c47"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/kolla/commit/9f739dfa3ad2c7302b7fd9bc3e1d91f262845c47"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}