)]}'
{"id":"openstack%2Fceilometer~790289","triplet_id":"openstack%2Fceilometer~master~I4735f20d75d329ccafc797fe285d9fef579f593e","project":"openstack/ceilometer","branch":"master","topic":"enable_timeout_set_dynamic_pollsters-upstream","hashtags":[],"change_id":"I4735f20d75d329ccafc797fe285d9fef579f593e","subject":"Introduce `timeout` in dynamic pollsters sub-system","status":"MERGED","created":"2021-05-07 17:37:59.000000000","updated":"2021-06-14 08:25:44.000000000","submitted":"2021-06-14 08:24:21.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"790289-enable_timeout_set_dynamic_pollsters-upstream","meta_rev_id":"c300a9bb6bda5d8ba938bc0884b302564d99bad9","_number":790289,"virtual_id_number":790289,"owner":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2021-06-14 08:24:14.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2021-06-06 15:16:31.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"}],"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":{"approved":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2021-06-14 07:14:04.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2021-05-07 18:56:39.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":"2021-06-06 15:16:31.000000000","updated_by":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"reviewer":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"state":"REVIEWER"}],"messages":[{"id":"b4e03730b049e0295781415c1477e634acb622a1","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"date":"2021-05-07 17:37:59.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"a9e4d301bb44a0a701a414af34eebd32e2a4ac0d","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-05-07 18:56:39.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/412db1f7e34d451a9b5c0718865f0e4b : SUCCESS in 6m 57s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/2eba047388a84e428565093fcf7a19b5 : SUCCESS in 4m 20s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/1b4630c5e8164240aad638751779bf93 : SUCCESS in 7m 18s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/9442d271a9024be291707c4782885c65 : SUCCESS in 6m 41s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/bbf4a5f23caa484b84f1ead31ca1a1e8 : SUCCESS in 7m 09s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e59ca396329842428c3185b6ebe414f8 : SUCCESS in 7m 50s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/f2c6d0f93aa441ffbb29dbad32516515 : SUCCESS in 9m 27s\n- grenade-ceilometer https://zuul.opendev.org/t/openstack/build/22c7e3c9d53247cf9b1cf4e04b87ca34 : FAILURE in 1h 12m 34s (non-voting)\n- telemetry-dsvm-integration https://zuul.opendev.org/t/openstack/build/b6d64c819181416a962618571fd53b1a : FAILURE in 51m 03s (non-voting)\n- telemetry-dsvm-integration-ipv6-only https://zuul.opendev.org/t/openstack/build/061609645f1f4accb1c054c653b535c8 : FAILURE in 59m 28s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"fb88091b4b5b3459a4c83faad1e2335ed7fbf212","author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"date":"2021-06-06 15:16:31.000000000","message":"Patch Set 1: Code-Review+2\n\nNeat!\nSorry it took so long to discover this change.","accounts_in_message":[],"_revision_number":1},{"id":"c170cadd81d8e86dbd1875f6a7c9a1badc408ae2","author":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"},"date":"2021-06-14 07:14:04.000000000","message":"Patch Set 1: Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"5320d35145b57874a842d4d8dd19efd4d433f04d","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-14 07:14:24.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"7b598bbca4d50f1429cd4c3ee32c2dc66df37b8b","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-14 08:24:14.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/9210db4c6e444cc0afca6ac355e442c8 : SUCCESS in 4m 35s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/b99ea8a9bf7446b2b85b2b76dc48d39a : SUCCESS in 6m 38s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/a61c67b90c3f46a5be8f6e73f4b6b025 : SUCCESS in 6m 18s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/838504e2aa1b4588a302caccea1ad8b4 : SUCCESS in 8m 01s\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/10c14f5fae944ea08a712955fda50f88 : SUCCESS in 7m 12s\n- grenade-ceilometer https://zuul.opendev.org/t/openstack/build/bbc1e3f939274b49a3ccc52007b56378 : FAILURE in 1h 09m 03s (non-voting)\n- telemetry-dsvm-integration https://zuul.opendev.org/t/openstack/build/b1495dc0dd604a54a579fcde7e601f8e : FAILURE in 55m 12s (non-voting)\n- telemetry-dsvm-integration-ipv6-only https://zuul.opendev.org/t/openstack/build/6523fef6a53f4ec29d49be5e6255ade5 : FAILURE in 55m 47s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"10907b6c06b2b3976283316b940f092bd0b83baa","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-14 08:24:21.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"c300a9bb6bda5d8ba938bc0884b302564d99bad9","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2021-06-14 08:25:44.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/8194623b09524872adadce5b139fd0f2 : SUCCESS in 58s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"30ada660f0d5e4bcbddb2b13aa6af834ab295444","revisions":{"30ada660f0d5e4bcbddb2b13aa6af834ab295444":{"kind":"REWORK","_number":1,"created":"2021-05-07 17:37:59.000000000","uploader":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"ref":"refs/changes/89/790289/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/89/790289/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/89/790289/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/89/790289/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/89/790289/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/89/790289/1"}}},"commit":{"parents":[{"commit":"122c55591fa90989e66fb803d9a5aac2db8a7211","subject":"Merge \"Use tox constraints instead of upper constraints\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/122c55591fa90989e66fb803d9a5aac2db8a7211"}]}],"author":{"name":"Rafael Weingärtner","email":"rafael@apache.org","date":"2021-05-07 17:36:26.000000000","tz":-180},"committer":{"name":"Rafael Weingärtner","email":"rafael@apache.org","date":"2021-05-07 17:36:32.000000000","tz":-180},"subject":"Introduce `timeout` in dynamic pollsters sub-system","message":"Introduce `timeout` in dynamic pollsters sub-system\n\nThis patch introduces a new option for the dynamic pollsters\u0027\nsub-system to define the request timeout for requests executed\nby the dynamic pollsters. The default timeout value will be 30\nseconds. If it (the timeout option) is set to `None`, this means\nthat the request never times out on the client-side. Therefore,\none might have problems if the server never closes the connection.\nThe pollsters are executed serially, one after the other; if the\nrequest hangs, all pollsters (including the non-dynamic ones) will\nstop executing.\n\nThis option is being introduced after a failure that we had when\nusing the dynamic pollsters; one of the APIs did not respond, and the\nserver did not close the connection of the request, which made\nCeilometer dynamic pollsters stop gathering data. To avoid such a\nsituation, we recommend the community review and merge this patch.\n\nChange-Id: I4735f20d75d329ccafc797fe285d9fef579f593e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/30ada660f0d5e4bcbddb2b13aa6af834ab295444"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/30ada660f0d5e4bcbddb2b13aa6af834ab295444"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"status":"CLOSED","labels":[{"label":"Verified","status":"OK","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"OK","applied_by":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"}},{"label":"Workflow","status":"OK","applied_by":{"_account_id":4264,"name":"Matthias Runge","email":"mrunge@redhat.com","username":"mrunge"}}]}],"submit_requirements":[{"name":"Verified","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Verified\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Workflow\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}},{"name":"Code-Review","status":"SATISFIED","is_legacy":true,"submittability_expression_result":{"expression":"label:Code-Review\u003dCustom-Rule","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dCustom-Rule"],"failing_atoms":[],"atom_explanations":{}}}]}
