)]}'
{"id":"openstack%2Fopenstack-helm-infra~666731","triplet_id":"openstack%2Fopenstack-helm-infra~master~Icb389372761be94dda05bbd7843eaaa2ae240f81","project":"openstack/openstack-helm-infra","branch":"master","hashtags":[],"change_id":"Icb389372761be94dda05bbd7843eaaa2ae240f81","subject":"Update wait for pod script","status":"ABANDONED","created":"2019-06-20 21:07:19.000000000","updated":"2019-06-21 20:57:04.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f7b1b37f5d2654bb7640e5557dec092d8a44a453","_number":666731,"virtual_id_number":666731,"owner":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"date":"2019-06-21 20:57:00.000000000","_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"value":1,"date":"2019-06-20 23:30:58.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":8749,"name":"Evgeniy L","email":"eli@mirantis.com","username":"evgeniyl"}],"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":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"value":-1,"date":"2019-06-21 16:00:31.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"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":8749,"name":"Evgeniy L","email":"eli@mirantis.com","username":"evgeniyl"}],"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":[{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},{"date":"2019-06-21 17:24:33.000000000","_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},{"date":"2019-06-21 14:48:45.000000000","_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"date":"2019-06-21 14:50:48.000000000","_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"date":"2019-06-21 14:50:35.000000000","_account_id":8749,"name":"Evgeniy L","email":"eli@mirantis.com","username":"evgeniyl"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":8749,"name":"Evgeniy L","email":"eli@mirantis.com","username":"evgeniyl"},{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-06-20 23:30:58.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-06-21 14:48:45.000000000","updated_by":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"reviewer":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"state":"REVIEWER"},{"updated":"2019-06-21 14:50:35.000000000","updated_by":{"_account_id":8749,"name":"Evgeniy L","email":"eli@mirantis.com","username":"evgeniyl"},"reviewer":{"_account_id":8749,"name":"Evgeniy L","email":"eli@mirantis.com","username":"evgeniyl"},"state":"REVIEWER"},{"updated":"2019-06-21 14:50:48.000000000","updated_by":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"reviewer":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"state":"REVIEWER"},{"updated":"2019-06-21 16:00:31.000000000","updated_by":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"reviewer":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"state":"REVIEWER"},{"updated":"2019-06-21 17:24:33.000000000","updated_by":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"reviewer":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"state":"REVIEWER"}],"messages":[{"id":"71acf8fb1a74b8837432739e1550f2fe64726983","author":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"date":"2019-06-20 21:07:19.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"4043370eb03a7c33b20e9085507207195b572353","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-20 21:57:55.000000000","message":"Patch Set 1: 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- openstack-tox-docs http://logs.openstack.org/31/666731/1/check/openstack-tox-docs/d5f041e/html/ : SUCCESS in 7m 13s\n- openstack-helm-lint http://logs.openstack.org/31/666731/1/check/openstack-helm-lint/9699ae7/ : SUCCESS in 2m 22s\n- openstack-helm-infra-aio-logging http://logs.openstack.org/31/666731/1/check/openstack-helm-infra-aio-logging/c7f9fd9/ : FAILURE in 29m 58s\n- openstack-helm-infra-aio-monitoring http://logs.openstack.org/31/666731/1/check/openstack-helm-infra-aio-monitoring/ad2a143/ : FAILURE in 27m 55s\n- openstack-helm-infra-aio-network-policy http://logs.openstack.org/31/666731/1/check/openstack-helm-infra-aio-network-policy/d93d560/ : FAILURE in 28m 19s (non-voting)\n- openstack-helm-infra-openstack-support http://logs.openstack.org/31/666731/1/check/openstack-helm-infra-openstack-support/e173b66/ : FAILURE in 27m 13s\n- openstack-helm-infra-kubernetes-keystone-auth http://logs.openstack.org/31/666731/1/check/openstack-helm-infra-kubernetes-keystone-auth/aafa352/ : FAILURE in 48m 31s\n- openstack-helm-infra-airship-divingbell http://logs.openstack.org/31/666731/1/check/openstack-helm-infra-airship-divingbell/94a3d56/ : SUCCESS in 11m 56s (non-voting)\n- openstack-helm-infra-aio-podsecuritypolicy http://logs.openstack.org/31/666731/1/check/openstack-helm-infra-aio-podsecuritypolicy/c12368a/ : FAILURE in 29m 14s (non-voting)\n- openstack-helm-infra-apparmor http://logs.openstack.org/31/666731/1/check/openstack-helm-infra-apparmor/a9d6215/ : FAILURE in 30m 20s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"e86f80f68c9ecfcdeaa86f02b36a3ea6f70233c6","author":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"date":"2019-06-20 22:26:49.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"015c6dcf1b918e86cf1b658a7e6712581a016c0c","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-06-20 23:30:58.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs http://logs.openstack.org/31/666731/2/check/openstack-tox-docs/3ffdbab/html/ : SUCCESS in 3m 59s\n- openstack-helm-lint http://logs.openstack.org/31/666731/2/check/openstack-helm-lint/fba8589/ : SUCCESS in 1m 25s\n- openstack-helm-infra-aio-logging http://logs.openstack.org/31/666731/2/check/openstack-helm-infra-aio-logging/6bc04d6/ : SUCCESS in 45m 54s\n- openstack-helm-infra-aio-monitoring http://logs.openstack.org/31/666731/2/check/openstack-helm-infra-aio-monitoring/bde8bd5/ : SUCCESS in 33m 05s\n- openstack-helm-infra-aio-network-policy http://logs.openstack.org/31/666731/2/check/openstack-helm-infra-aio-network-policy/bf57003/ : FAILURE in 28m 59s (non-voting)\n- openstack-helm-infra-openstack-support http://logs.openstack.org/31/666731/2/check/openstack-helm-infra-openstack-support/0fb8188/ : SUCCESS in 48m 16s\n- openstack-helm-infra-kubernetes-keystone-auth http://logs.openstack.org/31/666731/2/check/openstack-helm-infra-kubernetes-keystone-auth/9ad33f4/ : SUCCESS in 51m 58s\n- openstack-helm-infra-airship-divingbell http://logs.openstack.org/31/666731/2/check/openstack-helm-infra-airship-divingbell/3515f51/ : SUCCESS in 10m 45s (non-voting)\n- openstack-helm-infra-aio-podsecuritypolicy http://logs.openstack.org/31/666731/2/check/openstack-helm-infra-aio-podsecuritypolicy/f1eca9c/ : FAILURE in 14m 09s (non-voting)\n- openstack-helm-infra-apparmor http://logs.openstack.org/31/666731/2/check/openstack-helm-infra-apparmor/f7a3a4b/ : SUCCESS in 25m 51s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"e894dbd5b3550e72dd7a3cdbd2ac08324c626e79","author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"date":"2019-06-21 16:00:31.000000000","message":"Patch Set 2: Code-Review-1\n\nDoesnt permitting \u0027init\u0027 as a state nullify the functionality of this script?","accounts_in_message":[],"_revision_number":2},{"id":"9856a7f2999418b23ed6777138f903a0f46ed447","author":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"date":"2019-06-21 17:03:29.000000000","message":"Patch Set 2:\n\n\u003e Doesnt permitting \u0027init\u0027 as a state nullify the functionality of\n \u003e this script?\nAgree with you Pete , but this script is so fast to come out if we delete a pod and its not waiting for new pod to come into running state.\nwe need to rerun the script to catch new pod .\nplease let me know if there is a  best way to handle this .\n\nsteps to reproduce:\n1. delete a pod like ceph-osd in ceph namespace\n2. run this script to wait for all pods in ceph namespace .","accounts_in_message":[],"_revision_number":2},{"id":"e6af7bda6c978e57d048b441cd89833e37899fb1","author":{"_account_id":29974,"name":"Stephen Taylor","email":"stephen.taylor.1@att.com","username":"st053q"},"date":"2019-06-21 17:24:33.000000000","message":"Patch Set 2:\n\n\u003e \u003e Doesnt permitting \u0027init\u0027 as a state nullify the functionality of\n \u003e \u003e this script?\n \u003e Agree with you Pete , but this script is so fast to come out if we\n \u003e delete a pod and its not waiting for new pod to come into running\n \u003e state.\n \u003e we need to rerun the script to catch new pod .\n \u003e please let me know if there is a  best way to handle this .\n \u003e \n \u003e steps to reproduce:\n \u003e 1. delete a pod like ceph-osd in ceph namespace\n \u003e 2. run this script to wait for all pods in ceph namespace .\n\nWe have the same problem in the ceph-osd orchestrator with a ceph-osd pod restart causing peering which we want to wait for before moving on. If we check too quickly it appears to have finished because it hasn\u0027t started yet. We solved this problem there by initiating a 2-phase wait. The first phase waits until peering is seen, then the second phase waits until it isn\u0027t seen anymore. Can a similar approach be used here?","accounts_in_message":[],"_revision_number":2},{"id":"098bf7a334873ed20b38f472abe5c34677d42887","author":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"date":"2019-06-21 20:57:00.000000000","message":"Patch Set 2:\n\n\u003e \u003e \u003e Doesnt permitting \u0027init\u0027 as a state nullify the functionality\n \u003e of\n \u003e \u003e \u003e this script?\n \u003e \u003e Agree with you Pete , but this script is so fast to come out if\n \u003e we\n \u003e \u003e delete a pod and its not waiting for new pod to come into running\n \u003e \u003e state.\n \u003e \u003e we need to rerun the script to catch new pod .\n \u003e \u003e please let me know if there is a  best way to handle this .\n \u003e \u003e\n \u003e \u003e steps to reproduce:\n \u003e \u003e 1. delete a pod like ceph-osd in ceph namespace\n \u003e \u003e 2. run this script to wait for all pods in ceph namespace .\n \u003e \n \u003e We have the same problem in the ceph-osd orchestrator with a\n \u003e ceph-osd pod restart causing peering which we want to wait for\n \u003e before moving on. If we check too quickly it appears to have\n \u003e finished because it hasn\u0027t started yet. We solved this problem\n \u003e there by initiating a 2-phase wait. The first phase waits until\n \u003e peering is seen, then the second phase waits until it isn\u0027t seen\n \u003e anymore. Can a similar approach be used here?\n\nagree with pete here that handling init containers is not scope of this script.","accounts_in_message":[],"_revision_number":2},{"id":"f7b1b37f5d2654bb7640e5557dec092d8a44a453","author":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"date":"2019-06-21 20:57:04.000000000","message":"Abandoned","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"a0d2b631776bcd9170182b2f96205e554ab84b9f","revisions":{"521d84b67831116b124647e6f2125e1a02d3e81c":{"kind":"REWORK","_number":1,"created":"2019-06-20 21:07:19.000000000","uploader":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"ref":"refs/changes/31/666731/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/31/666731/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/31/666731/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/31/666731/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/31/666731/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/31/666731/1"}}},"commit":{"parents":[{"commit":"eefcf33f61f7dde89e119977e0c56e86b4479f41","subject":"Merge \"[Ceph]: Remove duplicate values\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/eefcf33f61f7dde89e119977e0c56e86b4479f41"}]}],"author":{"name":"Chinasubbareddy Mallavarapu","email":"cr3938@att.com","date":"2019-06-20 21:04:55.000000000","tz":-300},"committer":{"name":"Chinasubbareddy Mallavarapu","email":"cr3938@att.com","date":"2019-06-20 21:04:55.000000000","tz":-300},"subject":"Update wait for pod script","message":"Update wait for pod script\n\nThis is to  enhance the script to wait for init pods  and pods in\nterminating or initilization phases as slow environments may take\nsome time in both the cases.\n\nChange-Id: Icb389372761be94dda05bbd7843eaaa2ae240f81\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/521d84b67831116b124647e6f2125e1a02d3e81c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/521d84b67831116b124647e6f2125e1a02d3e81c"}]},"branch":"refs/heads/master"},"a0d2b631776bcd9170182b2f96205e554ab84b9f":{"kind":"REWORK","_number":2,"created":"2019-06-20 22:26:49.000000000","uploader":{"_account_id":28372,"name":"chinasubbareddy mallavarapu","email":"cr3938@att.com","username":"chinasubbareddy"},"ref":"refs/changes/31/666731/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/31/666731/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/31/666731/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/31/666731/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/31/666731/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/31/666731/2"}}},"commit":{"parents":[{"commit":"eefcf33f61f7dde89e119977e0c56e86b4479f41","subject":"Merge \"[Ceph]: Remove duplicate values\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/eefcf33f61f7dde89e119977e0c56e86b4479f41"}]}],"author":{"name":"Chinasubbareddy Mallavarapu","email":"cr3938@att.com","date":"2019-06-20 21:04:55.000000000","tz":-300},"committer":{"name":"Chinasubbareddy Mallavarapu","email":"cr3938@att.com","date":"2019-06-20 22:26:45.000000000","tz":-300},"subject":"Update wait for pod script","message":"Update wait for pod script\n\nThis is to  enhance the script to wait for init pods  and pods in\nterminating or initilization phases as slow environments may take\nsome time in both the cases.\n\nChange-Id: Icb389372761be94dda05bbd7843eaaa2ae240f81\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/a0d2b631776bcd9170182b2f96205e554ab84b9f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/a0d2b631776bcd9170182b2f96205e554ab84b9f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}