)]}'
{"id":"openstack%2Fpuppet-openstack-integration~776474","triplet_id":"openstack%2Fpuppet-openstack-integration~stable%2Fvictoria~I687f173d8d8a07c22d222f1ae90aca8c74d53d9c","project":"openstack/puppet-openstack-integration","branch":"stable/victoria","hashtags":[],"change_id":"I687f173d8d8a07c22d222f1ae90aca8c74d53d9c","subject":"Add retries to r10k command and sealrt command","status":"ABANDONED","created":"2021-02-18 14:52:25.000000000","updated":"2021-02-19 05:43:44.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"ace54aa838ee845d87023831b47e06ee98fb8905","_number":776474,"virtual_id_number":776474,"owner":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"tag":"autogenerated:zuul:check","value":1,"date":"2021-02-18 16:39:13.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","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":{"recommended":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"all":[{"value":1,"date":"2021-02-18 15:53:17.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","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":"","value":1,"default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-02-18 14:52:25.000000000","updated_by":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"reviewer":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"state":"REVIEWER"},{"updated":"2021-02-18 16:39:13.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":"4496caddb2f8e3c10f106cdf49dacc124052a787","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2021-02-18 14:52:25.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"eab23e32cd683d1674602ed4a6e1c8ed4401c9b1","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2021-02-18 14:53:13.000000000","message":"Uploaded patch set 2: New patch set was added with same tree, parent, and commit message as Patch Set 1.","accounts_in_message":[],"_revision_number":2},{"id":"884288ce11ad719eb9802815bc9b04abbaf6581f","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2021-02-18 14:57:13.000000000","message":"Uploaded patch set 3: Commit message was updated.","accounts_in_message":[],"_revision_number":3},{"id":"4b37fb539c9db12b9e4f493e1c65d41738a49050","author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"date":"2021-02-18 15:53:17.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"7707a310b1912339fce915e19a9ed61935309a49","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-02-18 16:39:13.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/de3f248573b94990b86da6016a9c11ea : SUCCESS in 4m 16s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/23af22c84fc242f2aa49deada4475b73 : SUCCESS in 3m 52s\n- puppet-openstack-integration-6-scenario001-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/31e8ad0b5db248c2a09613e0cce81d57 : FAILURE in 46m 45s (non-voting)\n- puppet-openstack-integration-6-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/5f13901b06694389af124df49260b7f7 : FAILURE in 38m 18s (non-voting)\n- puppet-openstack-integration-6-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/6f9f8f588b734cdc81b1548245440be5 : FAILURE in 34m 48s (non-voting)\n- puppet-openstack-integration-6-scenario004-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/8bce18322c6b406b967183f3359d06b3 : FAILURE in 28m 38s (non-voting)\n- puppet-openstack-integration-6-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/5181fdb4a41545a491951780a65c5037 : SUCCESS in 1h 25m 20s\n- puppet-openstack-integration-6-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/35843382a2134f1c862162a714ec1aaa : SUCCESS in 1h 23m 42s\n- puppet-openstack-integration-6-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/0defb4e3534d4c22842643698a278384 : SUCCESS in 1h 19m 04s\n- puppet-openstack-integration-6-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/d0d3e07c773847d7b1cbef0e059e461c : SUCCESS in 1h 02m 10s","accounts_in_message":[],"_revision_number":3},{"id":"ace54aa838ee845d87023831b47e06ee98fb8905","tag":"autogenerated:gerrit:abandon","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2021-02-19 05:43:44.000000000","message":"Abandoned\n\nbecause https://review.opendev.org/c/openstack/puppet-openstack-integration/+/776317 was merged","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"83a89fbadbc431071b901a2935124e54b280ce1c","revisions":{"26deb0827f19fe7418cef6f8d4e3f61d18be2deb":{"kind":"REWORK","_number":1,"created":"2021-02-18 14:52:25.000000000","uploader":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"ref":"refs/changes/74/776474/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-openstack-integration","ref":"refs/changes/74/776474/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/1"}}},"commit":{"parents":[{"commit":"4d3a00664f28e8f1c1ad291b8548b7fbd0908500","subject":"Set override-checkout to Victoria","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack-integration/commit/4d3a00664f28e8f1c1ad291b8548b7fbd0908500"}]}],"author":{"name":"yatinkarel","email":"ykarel@redhat.com","date":"2021-02-02 08:33:36.000000000","tz":330},"committer":{"name":"Takashi Kajinami","email":"tkajinam@redhat.com","date":"2021-02-18 14:52:00.000000000","tz":540},"subject":"Add retries to r10k command and sealrt command","message":"Add retries to r10k command and sealrt command\n\nThis is squashed commit of the following three pacthes in master,\nwhich introduces retries in r10k command and sealert command to avoid\nrandom failure possibly caused by temporal issues.\n\n---\nAdd retries to module installation with r10k\n\nSince the modules are git cloned when installed via r10k and\nclones may fail due to some network glich, adding retries should\nhelp in these cases.\n\nWe randomly hit failures during git clone in RDO promotion\njobs running on ci.centos.org, adding retries to r10k\ninstallation to handle those random failures.\n\nAdded function \u0027retry_cmd\u0027 which can also be used\nwith other commands.\n\nChange-Id: I5f3b94eaa5a65c76877363e9ca41684445c9cb12\n(cherry picked from commit 8d5b34c5522c17f0246d208d0104c86e3325250e)\n\n---\nRetry sealert command on failure\n\nWe randomly see issues while running sealert\nin RDO puppet promotion pipeline, sealert fails with:-\n\"SELinux is disabled or we can\u0027t open a policy file\"\nAs per logs selinux is enabled and selinuxfs is mounted\nso seems it\u0027s some filesystem issue.\n\nLocally was able to reproduce only with selinux disabled\nor umount /sys/fs/selinux. Adding retries in hope to\nfix these random failures.\n\nChange-Id: Icb07742d328827a41132cdd6a6c2bcfa3cde11ec\n(cherry picked from commit 425f454bdcb264eb80b305b67a5d4bcd4e04aadb)\n\n---\nImprove retry_cmd function to handle set -e\n\nIn run_tests.sh we use \u0027set -e\u0027 which makes\nscript fail as soon as error hit but we don\u0027t\nwant to fail in case of retries, this patch fixes it.\n\nChange-Id: Ia33ff3dbb88317171143a43e5b1c674038187254\n(cherry picked from commit ec57a35931babc9698a8399ca5c76773f069e377)\n\n---\n\nChange-Id: I687f173d8d8a07c22d222f1ae90aca8c74d53d9c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack-integration/commit/26deb0827f19fe7418cef6f8d4e3f61d18be2deb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack-integration/commit/26deb0827f19fe7418cef6f8d4e3f61d18be2deb"}]},"branch":"refs/heads/stable/victoria"},"7a0d52594f298fd4e6b86ebf4a13563c225ced70":{"kind":"NO_CHANGE","_number":2,"created":"2021-02-18 14:53:13.000000000","uploader":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"ref":"refs/changes/74/776474/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-openstack-integration","ref":"refs/changes/74/776474/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/2"}}},"commit":{"parents":[{"commit":"4d3a00664f28e8f1c1ad291b8548b7fbd0908500","subject":"Set override-checkout to Victoria","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack-integration/commit/4d3a00664f28e8f1c1ad291b8548b7fbd0908500"}]}],"author":{"name":"yatinkarel","email":"ykarel@redhat.com","date":"2021-02-02 08:33:36.000000000","tz":330},"committer":{"name":"Takashi Kajinami","email":"tkajinam@redhat.com","date":"2021-02-18 14:52:55.000000000","tz":540},"subject":"Add retries to r10k command and sealrt command","message":"Add retries to r10k command and sealrt command\n\nThis is squashed commit of the following three pacthes in master,\nwhich introduces retries in r10k command and sealert command to avoid\nrandom failure possibly caused by temporal issues.\n\n---\nAdd retries to module installation with r10k\n\nSince the modules are git cloned when installed via r10k and\nclones may fail due to some network glich, adding retries should\nhelp in these cases.\n\nWe randomly hit failures during git clone in RDO promotion\njobs running on ci.centos.org, adding retries to r10k\ninstallation to handle those random failures.\n\nAdded function \u0027retry_cmd\u0027 which can also be used\nwith other commands.\n\nChange-Id: I5f3b94eaa5a65c76877363e9ca41684445c9cb12\n(cherry picked from commit 8d5b34c5522c17f0246d208d0104c86e3325250e)\n\n---\nRetry sealert command on failure\n\nWe randomly see issues while running sealert\nin RDO puppet promotion pipeline, sealert fails with:-\n\"SELinux is disabled or we can\u0027t open a policy file\"\nAs per logs selinux is enabled and selinuxfs is mounted\nso seems it\u0027s some filesystem issue.\n\nLocally was able to reproduce only with selinux disabled\nor umount /sys/fs/selinux. Adding retries in hope to\nfix these random failures.\n\nChange-Id: Icb07742d328827a41132cdd6a6c2bcfa3cde11ec\n(cherry picked from commit 425f454bdcb264eb80b305b67a5d4bcd4e04aadb)\n\n---\nImprove retry_cmd function to handle set -e\n\nIn run_tests.sh we use \u0027set -e\u0027 which makes\nscript fail as soon as error hit but we don\u0027t\nwant to fail in case of retries, this patch fixes it.\n\nChange-Id: Ia33ff3dbb88317171143a43e5b1c674038187254\n(cherry picked from commit ec57a35931babc9698a8399ca5c76773f069e377)\n\n---\n\nChange-Id: I687f173d8d8a07c22d222f1ae90aca8c74d53d9c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack-integration/commit/7a0d52594f298fd4e6b86ebf4a13563c225ced70"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack-integration/commit/7a0d52594f298fd4e6b86ebf4a13563c225ced70"}]},"branch":"refs/heads/stable/victoria"},"83a89fbadbc431071b901a2935124e54b280ce1c":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2021-02-18 14:57:13.000000000","uploader":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"ref":"refs/changes/74/776474/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-openstack-integration","ref":"refs/changes/74/776474/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-openstack-integration refs/changes/74/776474/3"}}},"commit":{"parents":[{"commit":"4d3a00664f28e8f1c1ad291b8548b7fbd0908500","subject":"Set override-checkout to Victoria","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack-integration/commit/4d3a00664f28e8f1c1ad291b8548b7fbd0908500"}]}],"author":{"name":"yatinkarel","email":"ykarel@redhat.com","date":"2021-02-02 08:33:36.000000000","tz":330},"committer":{"name":"Takashi Kajinami","email":"tkajinam@redhat.com","date":"2021-02-18 14:56:48.000000000","tz":540},"subject":"Add retries to r10k command and sealrt command","message":"Add retries to r10k command and sealrt command\n\nThis is the squashed commit of the following three pacthes in master,\nwhich introduce retries in r10k command and sealert command to avoid\nrandom failure possibly caused by temporal issues.\n\n---\nAdd retries to module installation with r10k\n\nSince the modules are git cloned when installed via r10k and\nclones may fail due to some network glich, adding retries should\nhelp in these cases.\n\nWe randomly hit failures during git clone in RDO promotion\njobs running on ci.centos.org, adding retries to r10k\ninstallation to handle those random failures.\n\nAdded function \u0027retry_cmd\u0027 which can also be used\nwith other commands.\n\nChange-Id: I5f3b94eaa5a65c76877363e9ca41684445c9cb12\n(cherry picked from commit 8d5b34c5522c17f0246d208d0104c86e3325250e)\n\n---\nRetry sealert command on failure\n\nWe randomly see issues while running sealert\nin RDO puppet promotion pipeline, sealert fails with:-\n\"SELinux is disabled or we can\u0027t open a policy file\"\nAs per logs selinux is enabled and selinuxfs is mounted\nso seems it\u0027s some filesystem issue.\n\nLocally was able to reproduce only with selinux disabled\nor umount /sys/fs/selinux. Adding retries in hope to\nfix these random failures.\n\nChange-Id: Icb07742d328827a41132cdd6a6c2bcfa3cde11ec\n(cherry picked from commit 425f454bdcb264eb80b305b67a5d4bcd4e04aadb)\n\n---\nImprove retry_cmd function to handle set -e\n\nIn run_tests.sh we use \u0027set -e\u0027 which makes\nscript fail as soon as error hit but we don\u0027t\nwant to fail in case of retries, this patch fixes it.\n\nChange-Id: Ia33ff3dbb88317171143a43e5b1c674038187254\n(cherry picked from commit ec57a35931babc9698a8399ca5c76773f069e377)\n\n---\n\nChange-Id: I687f173d8d8a07c22d222f1ae90aca8c74d53d9c\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack-integration/commit/83a89fbadbc431071b901a2935124e54b280ce1c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstack-integration/commit/83a89fbadbc431071b901a2935124e54b280ce1c"}]},"branch":"refs/heads/stable/victoria"}},"requirements":[],"submit_records":[],"submit_requirements":[]}