)]}'
{"id":"openstack%2Fironic-python-agent~779812","triplet_id":"openstack%2Fironic-python-agent~stable%2Fussuri~I5f967f666480a86422a082312b203492743354d5","project":"openstack/ironic-python-agent","branch":"stable/ussuri","topic":"tinyipa-failsafe","hashtags":[],"change_id":"I5f967f666480a86422a082312b203492743354d5","subject":"Re-enable ironic-python-agent-check-image-tinyipa-ussuri","status":"ABANDONED","created":"2021-03-10 15:26:09.000000000","updated":"2021-03-15 11:41:51.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"76970a040ca2c26958457c25792a5345e46e4bc1","_number":779812,"virtual_id_number":779812,"owner":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2021-03-10 17:28:28.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"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":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":-1,"date":"2021-03-10 15:57:16.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"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":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Backport-Candidate":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"}],"values":{"-1":"Do Not Backport"," 0":"Backport Review Needed","+1":"Should Backport"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-03-10 15:57:16.000000000","updated_by":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"reviewer":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"state":"REVIEWER"},{"updated":"2021-03-10 17:28:28.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":"fb4590176f2ee547a72f5112ffb0abaea59ac6df","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"date":"2021-03-10 15:26:09.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b2bf7d4117dea4d1c94f9698b7146e472426bb6e","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2021-03-10 15:57:16.000000000","message":"Patch Set 1: Code-Review-1\n\nWhat are the chances IPA code on a stable branch breaks the building of the tinyIPA image specifically, so that the other jobs don\u0027t catch that?","accounts_in_message":[],"_revision_number":1},{"id":"69370f776b0eae2c4d0fed8560793bc2cad94f18","author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"date":"2021-03-10 16:07:14.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1: Code-Review-1\n\u003e \n\u003e What are the chances IPA code on a stable branch breaks the building of the tinyIPA image specifically, so that the other jobs don\u0027t catch that?\n\nnot very high, probably none\nbut it could happen that we break tinyipa making changes to ipa-builder jobs (see the build iso issue recently)","accounts_in_message":[],"_revision_number":1},{"id":"42cd840ffe375eb0d9ece67158702fe6a4f4e6bc","author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"date":"2021-03-10 16:08:54.000000000","message":"Patch Set 1:\n\nAbsolutely agreed, but we\u0027re in IPA, not in IPA-builder. Any voting job is a potential instability.","accounts_in_message":[],"_revision_number":1},{"id":"46015ebfae04098203406037716f66e9fe46acb6","author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"date":"2021-03-10 16:55:25.000000000","message":"Patch Set 1:\n\n\u003e Patch Set 1:\n\u003e \n\u003e Absolutely agreed, but we\u0027re in IPA, not in IPA-builder. Any voting job is a potential instability.\n\nthe main issue in this case is that we publish ipa ramdisk in stable branches using ipa-builder master jobs (because of how zuul works with branchless projects), but ipa-builder pinned code because of incompatibility between tinycore versions and CI nodes (see bionic-tinycore10.x and focal-tinycore11.x).\nIf we need to publish a new image and ipa-builder master jobs break the execution in a stable branch we need to fail, otherwise we backport fixes but they\u0027re not published in the tinyipa images because the publishing job is broken but we don\u0027t catch it.\nIf you think we can ignore this, I will abandon this patch.\nIn any case we need to publish a new ipa ramdisk in ussuri because the current one is broken (it\u0027s tinycore 11.x which is not supported in ussuri as we use bionic nodes).","accounts_in_message":[],"_revision_number":1},{"id":"ea5590ddbd4bfc9961b89ecb9576ca37847c63b5","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-03-10 17:28:28.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/7b36472514b44e6eb6bc1b0f7d34a3a3 : SUCCESS in 4m 39s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/ab4d676fcc314124820444bf6ac0b4e9 : SUCCESS in 4m 37s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/6e2415e23990460f8f612035725f4c65 : SUCCESS in 6m 49s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/36a8390c02f64ffe9b75a87025f41894 : SUCCESS in 5m 31s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/da8e8edcb60044308faf2ddca3a52be4 : SUCCESS in 6m 37s\n- ipa-tempest-bios-ipmi-direct-tinyipa-src https://zuul.opendev.org/t/openstack/build/11c2a0d52f584941acc385aa53eb5eac : SUCCESS in 1h 12m 22s\n- ipa-tempest-bios-ipmi-iscsi-tinyipa-src https://zuul.opendev.org/t/openstack/build/206b011d51354596bda39a14b3633975 : SUCCESS in 1h 12m 09s\n- ironic-standalone-ipa-src https://zuul.opendev.org/t/openstack/build/773f394827b346e58888fda198b458a5 : SUCCESS in 1h 26m 59s\n- metalsmith-integration-ipa-src https://zuul.opendev.org/t/openstack/build/c93632afccf34acba14253dc02da4e7f : SUCCESS in 1h 14m 00s\n- ironic-python-agent-check-image-tinyipa-ussuri https://zuul.opendev.org/t/openstack/build/1b814aeef6424217a0a89d80af61c436 : SUCCESS in 16m 27s\n- ironic-python-agent-check-image-dib-centos8 https://zuul.opendev.org/t/openstack/build/faae7e738c0543ac890936608d9420b5 : SUCCESS in 11m 23s (non-voting)\n- ipa-tempest-ironic-inspector-src https://zuul.opendev.org/t/openstack/build/a7c06443cc62488d84e3ec3f5ce3800f : SUCCESS in 1h 55m 02s (non-voting)\n- ipa-tempest-wholedisk-bios-ipmi-direct-dib-src https://zuul.opendev.org/t/openstack/build/b199400c62cb4fccb84a5ac66f3f1f13 : FAILURE in 38m 53s (non-voting)\n- ipa-tempest-partition-bios-ipmi-iscsi-dib-src https://zuul.opendev.org/t/openstack/build/1a9bf81c436e4b67b1b517f8d800b38c : SUCCESS in 1h 14m 03s (non-voting)\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/4dddd83d387f4582899ae26023e350c3 : SUCCESS in 5m 20s (non-voting)\n- ipa-tox-bandit https://zuul.opendev.org/t/openstack/build/bd64d52f9c9a4281a5377d3beb0fd27e : SUCCESS in 4m 53s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"76970a040ca2c26958457c25792a5345e46e4bc1","tag":"autogenerated:gerrit:abandon","author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"date":"2021-03-15 11:41:51.000000000","message":"Abandoned\n\nclarified on irc, going to move the change to ipa-builder","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"0ec559c1b6309a7629bd552d58d3156f3fa5b5da","revisions":{"0ec559c1b6309a7629bd552d58d3156f3fa5b5da":{"kind":"REWORK","_number":1,"created":"2021-03-10 15:26:09.000000000","uploader":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"ref":"refs/changes/12/779812/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ironic-python-agent","ref":"refs/changes/12/779812/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/12/779812/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/12/779812/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ironic-python-agent refs/changes/12/779812/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ironic-python-agent refs/changes/12/779812/1"}}},"commit":{"parents":[{"commit":"b9c9b0be10ad960244f41b5b79229746ff34d15a","subject":"Use UPPER_CONSTRAINTS_FILE to deal with ipa-builder","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/b9c9b0be10ad960244f41b5b79229746ff34d15a"}]}],"author":{"name":"Riccardo Pittau","email":"elfosardo@gmail.com","date":"2021-03-10 15:25:06.000000000","tz":60},"committer":{"name":"Riccardo Pittau","email":"elfosardo@gmail.com","date":"2021-03-10 15:25:06.000000000","tz":60},"subject":"Re-enable ironic-python-agent-check-image-tinyipa-ussuri","message":"Re-enable ironic-python-agent-check-image-tinyipa-ussuri\n\nThe ironic-python-agent-check-image-tinyipa-ussuri is a failsafe against\nbroken tinyipa images in ussuri and we should keep it voting.\n\nChange-Id: I5f967f666480a86422a082312b203492743354d5\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/0ec559c1b6309a7629bd552d58d3156f3fa5b5da"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ironic-python-agent/commit/0ec559c1b6309a7629bd552d58d3156f3fa5b5da"}]},"branch":"refs/heads/stable/ussuri"}},"requirements":[],"submit_records":[],"submit_requirements":[]}