)]}'
{"id":"openstack%2Fpaunch~675494","triplet_id":"openstack%2Fpaunch~master~I4bd7d7e238f4d6405c92ac864618a7c34830fd01","project":"openstack/paunch","branch":"master","topic":"bug/1839559","hashtags":[],"change_id":"I4bd7d7e238f4d6405c92ac864618a7c34830fd01","subject":"runner: do not run exec if container not running","status":"ABANDONED","created":"2019-08-08 22:54:35.000000000","updated":"2019-08-19 22:40:22.000000000","total_comment_count":18,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"0d1a7eb35fe2e259d97cc7e42dbc112066db6c00","_number":675494,"virtual_id_number":675494,"owner":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"all":[{"value":-1,"date":"2019-08-12 21:56:48.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},{"value":-1,"date":"2019-08-12 21:04:08.000000000","_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},{"date":"2019-08-13 13:14:14.000000000","_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"date":"2019-08-12 22:22:34.000000000","_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"}],"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":{"rejected":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"all":[{"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":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},{"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"]},{"value":-2,"date":"2019-08-13 07:34:28.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"}],"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":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},{"_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":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-08-09 04:26:29.000000000","updated_by":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"reviewer":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"state":"REVIEWER"},{"updated":"2019-08-09 04:29:23.000000000","updated_by":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"reviewer":{"_account_id":20778,"name":"Damien Ciabrini","email":"dciabrin@redhat.com","username":"dciabrin"},"state":"REVIEWER"},{"updated":"2019-08-09 09:39:49.000000000","updated_by":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"reviewer":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"state":"REVIEWER"},{"updated":"2019-08-09 16:59:29.000000000","updated_by":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"reviewer":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"state":"REVIEWER"},{"updated":"2019-08-12 21:04:08.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-08-12 21:56:48.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":"2019-08-12 22:22:34.000000000","updated_by":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"reviewer":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"state":"REVIEWER"},{"updated":"2019-08-13 07:34:28.000000000","updated_by":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"reviewer":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"state":"REVIEWER"},{"updated":"2019-08-13 13:14:14.000000000","updated_by":{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},"reviewer":{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},"state":"REVIEWER"}],"messages":[{"id":"d75fabd90db6b3e771c182d01d82c8d30d46aa23","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2019-08-08 22:54:35.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"fdbe6c598f186282d478952c5c00fb45243dcca5","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2019-08-08 22:58:15.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"bec12cd4ac1bf160871cda366ef52e8506ba2e49","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2019-08-08 23:01:00.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"031754aa1c1f18bc7c6f1fbb0b62bd5f44d2384a","author":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"date":"2019-08-08 23:51:28.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001 http://logs.rdoproject.org/94/675494/2/openstack-check/tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001/aa75fa4/ : FAILURE in 49m 45s","accounts_in_message":[],"_revision_number":2},{"id":"ac510c80af23efd0400d310c17d151c05eed96dc","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-09 01:03:53.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tripleo-ci-centos-7-containers-multinode https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-containers-multinode/40323bc/ : FAILURE in 1h 07m 03s\n- tripleo-ci-centos-7-undercloud-containers https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-undercloud-containers/8a17979/ : FAILURE in 1h 03m 52s\n- tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates/88e8f63/ : FAILURE in 56m 46s\n- tripleo-ci-centos-7-scenario000-multinode-oooq-container-upgrades https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-scenario000-multinode-oooq-container-upgrades/27e7a27/ : FAILURE in 54m 30s (non-voting)\n- tripleo-ci-centos-7-standalone https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-standalone/b3218fc/ : FAILURE in 53m 36s\n- tripleo-ci-fedora-28-standalone https://logs.opendev.org/94/675494/2/check/tripleo-ci-fedora-28-standalone/46ef332/ : FAILURE in 55m 52s (non-voting)\n- tripleo-ci-centos-7-standalone-upgrade https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-standalone-upgrade/48d661e/ : FAILURE in 1h 38m 55s (non-voting)\n- openstack-tox-py35 https://logs.opendev.org/94/675494/2/check/openstack-tox-py35/05c590f/ : SUCCESS in 3m 39s\n- openstack-tox-py36 https://logs.opendev.org/94/675494/2/check/openstack-tox-py36/ac967ed/ : SUCCESS in 4m 14s\n- openstack-tox-py37 https://logs.opendev.org/94/675494/2/check/openstack-tox-py37/3f2490d/ : SUCCESS in 5m 35s\n- openstack-tox-docs https://logs.opendev.org/94/675494/2/check/openstack-tox-docs/5a04279/html/ : SUCCESS in 3m 53s\n- openstack-tox-pep8 https://logs.opendev.org/94/675494/2/check/openstack-tox-pep8/ecbd5c7/ : SUCCESS in 4m 15s\n- openstack-tox-py27 https://logs.opendev.org/94/675494/2/check/openstack-tox-py27/544901a/ : SUCCESS in 3m 09s\n- tripleo-ci-centos-7-scenario007-multinode-oooq-container https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-scenario007-multinode-oooq-container/4c1143c/ : FAILURE in 55m 31s (non-voting)\n- tripleo-ci-centos-7-scenario009-multinode-oooq-container https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-scenario009-multinode-oooq-container/6f44932/ : FAILURE in 1h 02m 54s (non-voting)\n- tripleo-ci-centos-7-scenario001-standalone https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-scenario001-standalone/d4efe20/ : FAILURE in 1h 06m 44s\n- tripleo-ci-centos-7-scenario002-standalone https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-scenario002-standalone/4e77127/ : FAILURE in 55m 39s\n- tripleo-ci-centos-7-scenario003-standalone https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-scenario003-standalone/96a2971/ : FAILURE in 55m 31s\n- tripleo-ci-centos-7-scenario004-standalone https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-scenario004-standalone/6391ad4/ : SUCCESS in 1h 45m 00s\n- tripleo-ci-centos-7-scenario012-standalone https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-scenario012-standalone/a51c548/ : FAILURE in 1h 10m 28s (non-voting)\n- tripleo-ci-centos-7-containerized-undercloud-upgrades https://logs.opendev.org/94/675494/2/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/6525cf7/ : FAILURE in 1h 42m 53s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"99796b4b364883408667ee4b61aa46a3605c9e47","author":{"_account_id":20172,"name":"Michele Baldessari","email":"michele@acksyn.org","username":"michele"},"date":"2019-08-09 04:26:29.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"982c184a307ddd36fc888590f94832842c96b78e","author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"date":"2019-08-09 08:54:19.000000000","message":"Patch Set 2: Code-Review-1\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"261666aa4882bf6924eaa03d9e0f6d851dee83ee","author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"date":"2019-08-09 09:12:45.000000000","message":"Patch Set 2: -Code-Review\n\nOk, I read the bug and I see that you want to start raising exceptions. The best approach would be to add an exceptions.py and use exceptions from there.","accounts_in_message":[],"_revision_number":2},{"id":"ef1391853fd964af7654cad407191afaad2fc18c","author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"date":"2019-08-09 09:19:05.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"381e8cb944b03cec8d2607a5e0e1ecf48e497d5b","author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"date":"2019-08-09 09:33:04.000000000","message":"Patch Set 2:\n\n(4 comments)","accounts_in_message":[],"_revision_number":2},{"id":"c2d25fe2c76ab890f5898a970798ed181c5417b1","author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"date":"2019-08-09 09:39:49.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"0747ddcd513a1cb487e8e5ae734f4c55a6f25b8b","author":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"date":"2019-08-09 13:06:32.000000000","message":"Patch Set 2:\n\n(1 comment)\n\nHolding off on +1 until validation_passed debate is settled.  My feeling is that continuing regardless of whether or not a container exists defeats the purpose of the validation.","accounts_in_message":[],"_revision_number":2},{"id":"6e3d65a48cd7359af026bedcab5b83bea6286542","author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"date":"2019-08-09 13:20:21.000000000","message":"Patch Set 2: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"45d767cdfc23454648510db4f843ba4828ad9c69","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2019-08-09 13:21:03.000000000","message":"Patch Set 2:\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"0db1074a8f4e5b7588fb648ed6baad35481c7096","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2019-08-09 13:51:24.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"27384331f317fa3e9832b3a5f72396579496b65d","author":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"date":"2019-08-09 13:53:18.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"136f5d2e47a0c094a1cb1c09c5acc8323d2f6d0e","author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"date":"2019-08-09 13:58:00.000000000","message":"Patch Set 2:\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"b87164dfadc3655b2352e7b8a4c35285eceeaf65","author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"date":"2019-08-09 13:58:21.000000000","message":"Patch Set 3: Code-Review-1","accounts_in_message":[],"_revision_number":3},{"id":"421ceb32a49fd8d2ff06b839d8a51d713685889e","author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"date":"2019-08-09 14:00:32.000000000","message":"Patch Set 3:\n\n(2 comments)","accounts_in_message":[],"_revision_number":3},{"id":"29aed1b91770a7f01e6f65da1aaf85b32e1971a8","author":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"date":"2019-08-09 14:41:37.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001 http://logs.rdoproject.org/94/675494/3/openstack-check/tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001/e49e0ad/ : FAILURE in 49m 24s","accounts_in_message":[],"_revision_number":3},{"id":"ff5b0105539c167c9299f2a9c5270dd947f6e303","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2019-08-09 15:08:41.000000000","message":"Patch Set 3:\n\nI\u0027ve proposed an alternative solution: https://review.opendev.org/675637","accounts_in_message":[],"_revision_number":3},{"id":"c715d10fd84815a8623bc46324519bdb7e29591b","author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"date":"2019-08-09 16:59:29.000000000","message":"Patch Set 3: Code-Review-1\n\nI like the alternative solution more. While the outcome is the same, this implementation may be more robust. That said, I\u0027m not sure it matters. If we need to take concepts from this review we can in a future iteration.","accounts_in_message":[],"_revision_number":3},{"id":"304f92865741f734b560e0005e2b857bb9b0b804","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-09 23:54:43.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tripleo-ci-centos-7-containers-multinode https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-containers-multinode/e2ea89b/ : FAILURE in 1h 05m 10s\n- tripleo-ci-centos-7-undercloud-containers https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-undercloud-containers/f116739/ : FAILURE in 1h 03m 37s\n- tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates/538098a/ : FAILURE in 58m 54s\n- tripleo-ci-centos-7-scenario000-multinode-oooq-container-upgrades https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-scenario000-multinode-oooq-container-upgrades/0c63ef6/ : FAILURE in 1h 04m 41s (non-voting)\n- tripleo-ci-centos-7-standalone https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-standalone/cb93a90/ : FAILURE in 53m 54s\n- tripleo-ci-fedora-28-standalone https://logs.opendev.org/94/675494/3/check/tripleo-ci-fedora-28-standalone/49abde3/ : FAILURE in 54m 57s (non-voting)\n- tripleo-ci-centos-7-standalone-upgrade https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-standalone-upgrade/f107f07/ : FAILURE in 1h 45m 11s (non-voting)\n- openstack-tox-py35 https://logs.opendev.org/94/675494/3/check/openstack-tox-py35/7a179f0/ : SUCCESS in 7m 01s\n- openstack-tox-py36 https://logs.opendev.org/94/675494/3/check/openstack-tox-py36/574a430/ : SUCCESS in 5m 09s\n- openstack-tox-py37 https://logs.opendev.org/94/675494/3/check/openstack-tox-py37/f4caafa/ : SUCCESS in 5m 24s\n- openstack-tox-docs https://logs.opendev.org/94/675494/3/check/openstack-tox-docs/ebc8639/html/ : SUCCESS in 3m 58s\n- openstack-tox-pep8 https://logs.opendev.org/94/675494/3/check/openstack-tox-pep8/656deba/ : SUCCESS in 6m 14s\n- openstack-tox-py27 https://logs.opendev.org/94/675494/3/check/openstack-tox-py27/8196c2f/ : SUCCESS in 6m 28s\n- tripleo-ci-centos-7-scenario007-multinode-oooq-container https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-scenario007-multinode-oooq-container/10a6680/ : FAILURE in 54m 52s (non-voting)\n- tripleo-ci-centos-7-scenario009-multinode-oooq-container https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-scenario009-multinode-oooq-container/2828c48/ : FAILURE in 1h 05m 32s (non-voting)\n- tripleo-ci-centos-7-scenario001-standalone https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-scenario001-standalone/1bcaf31/ : FAILURE in 1h 23m 23s\n- tripleo-ci-centos-7-scenario002-standalone https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-scenario002-standalone/3c0ccc0/ : FAILURE in 52m 28s\n- tripleo-ci-centos-7-scenario003-standalone https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-scenario003-standalone/e60747d/ : FAILURE in 58m 58s\n- tripleo-ci-centos-7-scenario004-standalone https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-scenario004-standalone/d5b5f68/ : SUCCESS in 1h 47m 18s\n- tripleo-ci-centos-7-scenario012-standalone https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-scenario012-standalone/f5ce868/ : SUCCESS in 2h 00m 02s (non-voting)\n- tripleo-ci-centos-7-containerized-undercloud-upgrades https://logs.opendev.org/94/675494/3/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/cd6af59/ : FAILURE in 1h 37m 40s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"7a2f5f9090b498eebec03782b455ce586a13be7d","author":{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},"date":"2019-08-12 18:44:19.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"326ad53161bde0c910dd368ac9a28c949b78e035","author":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"date":"2019-08-12 19:07:05.000000000","message":"Patch Set 4: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001 http://logs.rdoproject.org/94/675494/4/openstack-check/tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001/88408f8/ : FAILURE in 22m 00s","accounts_in_message":[],"_revision_number":4},{"id":"de32155b9758452b2267ec2c6ccaa1b95980bcdc","author":{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},"date":"2019-08-12 20:44:56.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"2156973e00f59ed8ecac3242a38a62f81b61fbb9","author":{"_account_id":23181,"name":"RDO Third Party CI","email":"dmsimard+rdothirdparty@redhat.com","username":"rdothirdparty","tags":["SERVICE_USER"]},"date":"2019-08-12 21:04:08.000000000","message":"Patch Set 5: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001 http://logs.rdoproject.org/94/675494/5/openstack-check/tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset001/ef285e4/ : FAILURE in 18m 28s","accounts_in_message":[],"_revision_number":5},{"id":"50305c21a74eafa3714a297a6c3e9fc001c28569","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-08-12 21:56:48.000000000","message":"Patch Set 5: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- tripleo-ci-centos-7-containers-multinode https://zuul.opendev.org/t/openstack/build/None : SKIPPED\n- tripleo-ci-centos-7-undercloud-containers https://zuul.opendev.org/t/openstack/build/bb41fbc549884836b2c777a92ed5af61 : FAILURE in 23m 50s\n- tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates https://zuul.opendev.org/t/openstack/build/None : SKIPPED\n- tripleo-ci-centos-7-scenario000-multinode-oooq-container-upgrades https://zuul.opendev.org/t/openstack/build/None : SKIPPED (non-voting)\n- tripleo-ci-centos-7-standalone https://zuul.opendev.org/t/openstack/build/0c07082587204e6a90691de2ecdeb9c9 : FAILURE in 24m 14s\n- tripleo-ci-fedora-28-standalone https://zuul.opendev.org/t/openstack/build/2d260ec677d64ca1bfe414b9058155f3 : FAILURE in 27m 34s (non-voting)\n- tripleo-ci-centos-7-standalone-upgrade https://zuul.opendev.org/t/openstack/build/None : SKIPPED (non-voting)\n- openstack-tox-py35 https://zuul.opendev.org/t/openstack/build/c3cecf8228974667b8ba76d10777c659 : FAILURE in 3m 37s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/733329702d73441eae854be5d608a623 : FAILURE in 3m 06s\n- openstack-tox-py37 https://zuul.opendev.org/t/openstack/build/17cb74629c934ab58ad52661e58345ac : FAILURE in 3m 30s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/bcf2e114dd9f4b3e8aa28be3472dad9e : SUCCESS in 3m 11s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/492d12a3c9fc46989d72169b7bb26b7b : FAILURE in 3m 42s\n- openstack-tox-py27 https://zuul.opendev.org/t/openstack/build/77b81fe38f1d4aaea7645b00043ebd6c : FAILURE in 3m 26s\n- tripleo-ci-centos-7-scenario007-multinode-oooq-container https://zuul.opendev.org/t/openstack/build/None : SKIPPED (non-voting)\n- tripleo-ci-centos-7-scenario009-multinode-oooq-container https://zuul.opendev.org/t/openstack/build/None : SKIPPED (non-voting)\n- tripleo-ci-centos-7-scenario001-standalone https://zuul.opendev.org/t/openstack/build/None : SKIPPED\n- tripleo-ci-centos-7-scenario002-standalone https://zuul.opendev.org/t/openstack/build/None : SKIPPED\n- tripleo-ci-centos-7-scenario003-standalone https://zuul.opendev.org/t/openstack/build/None : SKIPPED\n- tripleo-ci-centos-7-scenario004-standalone https://zuul.opendev.org/t/openstack/build/None : SKIPPED\n- tripleo-ci-centos-7-scenario012-standalone https://zuul.opendev.org/t/openstack/build/None : SKIPPED (non-voting)\n- tripleo-ci-centos-7-containerized-undercloud-upgrades https://zuul.opendev.org/t/openstack/build/None : SKIPPED (non-voting)","accounts_in_message":[],"_revision_number":5},{"id":"6386f67bc937180e939d43ca46e1e5d71a7c1eac","author":{"_account_id":27427,"name":"David Peacock","email":"dpeacock@redhat.com","username":"davidjpeacock"},"date":"2019-08-12 22:22:34.000000000","message":"Patch Set 5:\n\nI like the flow and idea here Luke.  Lot\u0027s of failures so let\u0027s debug together tomorrow.","accounts_in_message":[],"_revision_number":5},{"id":"4fd5442da9b5dfbfd7ee54331ed1a0eb8e159de0","author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"date":"2019-08-13 07:34:28.000000000","message":"Patch Set 5: Code-Review-2\n\nthis way only increases the times we call podman CLI commands x2 -\u003e x3 per *each* exec action. And we have a plenty, both deploy time and runtime (healthchecks). I tend to think we should continue with https://review.opendev.org/675637 instead","accounts_in_message":[],"_revision_number":5},{"id":"a7c8e0ea4c462597fe1a7d259420920ce2a26627","author":{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},"date":"2019-08-13 13:14:14.000000000","message":"Patch Set 5:\n\nOne reason to keep this patch is that running `podman ps` is a lot faster and less expensive then running `podman inspect`. With that being said, I will have a look at the other patch.","accounts_in_message":[],"_revision_number":5},{"id":"51f19bee12e37011b05dc69a9181dce7e7a54e22","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2019-08-19 22:40:22.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":5}],"current_revision_number":5,"current_revision":"ff281087a6856b966e22636980a160beb9ebedb4","revisions":{"2594c78351031915ccf37a2c237690de250230af":{"kind":"REWORK","_number":1,"created":"2019-08-08 22:54:35.000000000","uploader":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"ref":"refs/changes/94/675494/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/paunch","ref":"refs/changes/94/675494/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/paunch refs/changes/94/675494/1"}}},"commit":{"parents":[{"commit":"aa0e32b004c8f053444f9214af6a0e0019a03908","subject":"Merge \"Handle defined containers that are stopped.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/aa0e32b004c8f053444f9214af6a0e0019a03908"}]}],"author":{"name":"Emilien Macchi","email":"emilien@redhat.com","date":"2019-08-08 22:42:14.000000000","tz":-240},"committer":{"name":"Emilien Macchi","email":"emilien@redhat.com","date":"2019-08-08 22:42:14.000000000","tz":-240},"subject":"WIP - runner: do not run exec if container not running","message":"WIP - runner: do not run exec if container not running\n\nWhen running \"podman exec\", paunch was only trying to get the container\nname from a \"podman ps -a\" and then run the exec.\nIt can be problematic if the container isn\u0027t running for some reasons.\n\nWe should first make sure the container is actually started and then\nrun the exec.\n\nThis patch does:\n\n1) Introduce container_running() which use podman ps stdout to figure if\n   whether or not the container is actually running.\n\nTODO: write unit tests e.g. test_container_not_running\n\n2) In the running, if the action is \"exec\", we expect the container to\n   be running so we call container_running and if the result is\n   negative, we raise.\n\n3) To investigate, but I don\u0027t think the \u0027validations_passed\u0027 variable\n   introduced in Ibc458d3badb807a0deef5a799db6682d7128dbc1 really makes\n   sense. It doesn\u0027t validate anything and moreover we were skipping\n   containers which failed that \"validation\" (!!!).\n   In this patch, I commented the variables, suggesting we should change\n   them and also converted the \"continue\" into a \"raise\", and increased\n   the log level from debug to error if validation didn\u0027t pass.\n   We should rework that piece.\n\nCloses-Bug: #1839559\nrhbz#1739224\nCo-Authored-By: Luke Short \u003cekultails@gmail.com\u003e\nCo-Authored-By: David Peacock \u003cdpeacock@redhat.com\u003e\n\nChange-Id: I4bd7d7e238f4d6405c92ac864618a7c34830fd01\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/2594c78351031915ccf37a2c237690de250230af"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/2594c78351031915ccf37a2c237690de250230af"}]},"branch":"refs/heads/master"},"8fc93dd537e9007e61eb852a0e585d4a7fa2e491":{"kind":"REWORK","_number":2,"created":"2019-08-08 23:01:00.000000000","uploader":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"ref":"refs/changes/94/675494/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/paunch","ref":"refs/changes/94/675494/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/paunch refs/changes/94/675494/2"}}},"commit":{"parents":[{"commit":"aa0e32b004c8f053444f9214af6a0e0019a03908","subject":"Merge \"Handle defined containers that are stopped.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/aa0e32b004c8f053444f9214af6a0e0019a03908"}]}],"author":{"name":"Emilien Macchi","email":"emilien@redhat.com","date":"2019-08-08 22:42:14.000000000","tz":-240},"committer":{"name":"Emilien Macchi","email":"emilien@redhat.com","date":"2019-08-08 23:00:30.000000000","tz":-240},"subject":"WIP - runner: do not run exec if container not running","message":"WIP - runner: do not run exec if container not running\n\nWhen running \"podman exec\", paunch was only trying to get the container\nname from a \"podman ps -a\" and then run the exec.\nIt can be problematic if the container isn\u0027t running for some reasons.\n\nWe should first make sure the container is actually started and then\nrun the exec.\n\nThis patch does:\n\n1) Introduce container_running() which use podman ps stdout to figure if\n   whether or not the container is actually running.\n\nTODO: write unit tests e.g. test_container_not_running\n\n2) In the running, if the action is \"exec\", we expect the container to\n   be running so we call container_running and if the result is\n   negative, we raise.\n\n3) To investigate, but I don\u0027t think the \u0027validations_passed\u0027 variable\n   introduced in Ibc458d3badb807a0deef5a799db6682d7128dbc1 really makes\n   sense. It doesn\u0027t validate anything and moreover we were skipping\n   containers which failed that \"validation\" (!!!).\n   In this patch, I commented the variables, suggesting we should change\n   them and also converted the \"continue\" into a \"raise\", and increased\n   the log level from debug to error if validation didn\u0027t pass.\n   We should rework that piece.\n\nCloses-Bug: #1839559\nrhbz#1739224\nCo-Authored-By: Luke Short \u003cekultails@gmail.com\u003e\nCo-Authored-By: David Peacock \u003cdpeacock@redhat.com\u003e\n\nChange-Id: I4bd7d7e238f4d6405c92ac864618a7c34830fd01\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/8fc93dd537e9007e61eb852a0e585d4a7fa2e491"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/8fc93dd537e9007e61eb852a0e585d4a7fa2e491"}]},"branch":"refs/heads/master"},"b893958cd80282411d57ab44c52a25d2e716d9ce":{"kind":"REWORK","_number":3,"created":"2019-08-09 13:51:24.000000000","uploader":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"ref":"refs/changes/94/675494/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/paunch","ref":"refs/changes/94/675494/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/paunch refs/changes/94/675494/3"}}},"commit":{"parents":[{"commit":"aa0e32b004c8f053444f9214af6a0e0019a03908","subject":"Merge \"Handle defined containers that are stopped.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/aa0e32b004c8f053444f9214af6a0e0019a03908"}]}],"author":{"name":"Emilien Macchi","email":"emilien@redhat.com","date":"2019-08-08 22:42:14.000000000","tz":-240},"committer":{"name":"Emilien Macchi","email":"emilien@redhat.com","date":"2019-08-09 13:50:54.000000000","tz":-240},"subject":"runner: do not run exec if container not running","message":"runner: do not run exec if container not running\n\nWhen running \"podman exec\", paunch was only trying to get the container\nname from a \"podman ps -a\" and then run the exec.\nIt can be problematic if the container isn\u0027t running for some reasons.\n\nWe should first make sure the container is actually started and then\nrun the exec.\n\nThis patch does:\n\n1) Introduce container_running() which use podman ps stdout to figure if\n   whether or not the container is actually running.\n\n2) In the running, if the action is \"exec\", we expect the container to\n   be running so we call container_running and if the result is\n   negative, we raise.\n\nCloses-Bug: #1839559\nrhbz#1739224\nCo-Authored-By: Luke Short \u003cekultails@gmail.com\u003e\nCo-Authored-By: David Peacock \u003cdpeacock@redhat.com\u003e\n\nChange-Id: I4bd7d7e238f4d6405c92ac864618a7c34830fd01\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/b893958cd80282411d57ab44c52a25d2e716d9ce"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/b893958cd80282411d57ab44c52a25d2e716d9ce"}]},"branch":"refs/heads/master"},"5e8cc1ca07962a3e7b08f8cabc09099e82fe76a3":{"kind":"REWORK","_number":4,"created":"2019-08-12 18:44:19.000000000","uploader":{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},"ref":"refs/changes/94/675494/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/paunch","ref":"refs/changes/94/675494/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/paunch refs/changes/94/675494/4"}}},"commit":{"parents":[{"commit":"aa0e32b004c8f053444f9214af6a0e0019a03908","subject":"Merge \"Handle defined containers that are stopped.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/aa0e32b004c8f053444f9214af6a0e0019a03908"}]}],"author":{"name":"Emilien Macchi","email":"emilien@redhat.com","date":"2019-08-08 22:42:14.000000000","tz":-240},"committer":{"name":"Luke Short","email":"ekultails@gmail.com","date":"2019-08-12 18:44:06.000000000","tz":-240},"subject":"runner: do not run exec if container not running","message":"runner: do not run exec if container not running\n\nWhen running \"podman exec\", paunch was only trying to get the container\nname from a \"podman ps -a\" and then run the exec.\nIt can be problematic if the container isn\u0027t running for some reasons.\n\nWe should first make sure the container is actually started and then\nrun the exec.\n\nThis patch does:\n\n1) Introduce container_running() which use podman ps stdout to figure if\n   whether or not the container is actually running.\n\n2) In the running, if the action is \"exec\", we expect the container to\n   be running so we call container_running and if the result is\n   negative, we raise.\n\nCloses-Bug: #1839559\nrhbz#1739224\nCo-Authored-By: Luke Short \u003cekultails@gmail.com\u003e\nCo-Authored-By: David Peacock \u003cdpeacock@redhat.com\u003e\n\nChange-Id: I4bd7d7e238f4d6405c92ac864618a7c34830fd01\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/5e8cc1ca07962a3e7b08f8cabc09099e82fe76a3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/5e8cc1ca07962a3e7b08f8cabc09099e82fe76a3"}]},"branch":"refs/heads/master"},"ff281087a6856b966e22636980a160beb9ebedb4":{"kind":"REWORK","_number":5,"created":"2019-08-12 20:44:56.000000000","uploader":{"_account_id":25877,"name":"Luke Short","email":"ekultails@gmail.com","username":"ekultails"},"ref":"refs/changes/94/675494/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/paunch","ref":"refs/changes/94/675494/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/paunch refs/changes/94/675494/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/paunch refs/changes/94/675494/5"}}},"commit":{"parents":[{"commit":"aa0e32b004c8f053444f9214af6a0e0019a03908","subject":"Merge \"Handle defined containers that are stopped.\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/aa0e32b004c8f053444f9214af6a0e0019a03908"}]}],"author":{"name":"Emilien Macchi","email":"emilien@redhat.com","date":"2019-08-08 22:42:14.000000000","tz":-240},"committer":{"name":"Luke Short","email":"ekultails@gmail.com","date":"2019-08-12 20:32:42.000000000","tz":-240},"subject":"runner: do not run exec if container not running","message":"runner: do not run exec if container not running\n\nWhen running \"podman exec\", paunch was only trying to get the container\nname from a \"podman ps -a\" and then run the exec.\nIt can be problematic if the container isn\u0027t running for some reasons.\n\nWe should first make sure the container is actually started and then\nrun the exec.\n\nThis patch does:\n\n1) Introduce container_running() which use podman ps stdout to figure if\n   whether or not the container is actually running.\n\n2) In the running, if the action is \"exec\", we expect the container to\n   be running so we call container_running and if the result is\n   negative, we raise.\n\nCloses-Bug: #1839559\nrhbz#1739224\nCo-Authored-By: Luke Short \u003cekultails@gmail.com\u003e\nCo-Authored-By: David Peacock \u003cdpeacock@redhat.com\u003e\n\nChange-Id: I4bd7d7e238f4d6405c92ac864618a7c34830fd01\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/ff281087a6856b966e22636980a160beb9ebedb4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/paunch/commit/ff281087a6856b966e22636980a160beb9ebedb4"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}