)]}'
{"id":"openstack%2Fnova~959420","triplet_id":"openstack%2Fnova~master~I544c62af23c98da8cab198d3be1c951645c19ce9","project":"openstack/nova","branch":"master","topic":"config-for-disabling-hypervisor-details-uptime-call","attention_set":{},"removed_from_attention_set":{"15334":{"account":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"last_update":"2025-09-04 10:56:34.000000000","reason":"Change was abandoned"},"26250":{"account":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"last_update":"2025-09-04 09:55:00.000000000","reason":"\u003cGERRIT_ACCOUNT_26250\u003e replied on the change","reason_account":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"}},"7166":{"account":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"last_update":"2025-09-04 10:56:34.000000000","reason":"Change was abandoned"},"33279":{"account":{"_account_id":33279,"name":"jakob","email":"jakob.karge@sap.com","username":"grandchild"},"last_update":"2025-09-04 10:19:27.000000000","reason":"Reviewer/Cc was removed"}},"hashtags":[],"change_id":"I544c62af23c98da8cab198d3be1c951645c19ce9","subject":"Conditionally disable hypervisor uptime in details API","status":"ABANDONED","created":"2025-09-04 09:40:46.000000000","updated":"2025-09-04 10:56:34.000000000","total_comment_count":5,"unresolved_comment_count":1,"has_review_started":true,"meta_rev_id":"e89b43785d70ccc61ee335d7b0a9588d85678753","_number":959420,"virtual_id_number":959420,"owner":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"actions":{},"labels":{"Verified":{"all":[{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"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":{"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"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":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},{"value":0,"permitted_voting_range":{"min":0,"max":2},"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"}],"values":{" 0":"Default Priority","+1":"Contributor Review Promise","+2":"Core Review Promise"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"}],"CC":[{"_account_id":33279,"name":"jakob","email":"jakob.karge@sap.com","username":"grandchild"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-09-04 09:40:46.000000000","updated_by":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"reviewer":{"_account_id":33279,"name":"jakob","email":"jakob.karge@sap.com","username":"grandchild"},"state":"REVIEWER"},{"updated":"2025-09-04 09:44:34.000000000","updated_by":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"reviewer":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"state":"REVIEWER"},{"updated":"2025-09-04 10:00:26.000000000","updated_by":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"reviewer":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"state":"REVIEWER"},{"updated":"2025-09-04 10:19:27.000000000","updated_by":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"reviewer":{"_account_id":33279,"name":"jakob","email":"jakob.karge@sap.com","username":"grandchild"},"state":"CC"}],"messages":[{"id":"924b82bd50fa2d74eb3c646fa59037c093a17572","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"date":"2025-09-04 09:40:46.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6f75dcf52279849d4e47a6c7c8d369de999e9ba1","author":{"_account_id":7166,"name":"Sylvain Bauza","email":"sbauza@redhat.com","username":"sbauza"},"date":"2025-09-04 09:44:34.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"4cd4ecf49768730aea0114ac097604180c10b9c8","author":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"date":"2025-09-04 09:55:00.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"768a58862240ad541deaa4eec9da04b2eca6fe81","author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"date":"2025-09-04 10:00:26.000000000","message":"Patch Set 1: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"65b863e67bd6f253792693006cc0c0adf5423dd9","author":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"date":"2025-09-04 10:06:07.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"3345ea3cd986b53398fa60596a96cd8716dc98a4","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"date":"2025-09-04 10:19:27.000000000","message":"Uploaded patch set 2: Commit message was updated.\n\nOutdated Votes:\n* Code-Review-1 (copy condition: \"changekind:TRIVIAL_REBASE OR is:MIN\")\n","accounts_in_message":[],"_revision_number":2},{"id":"529042a4381ffbadb3ae3cb3071a826f452fca9f","author":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"date":"2025-09-04 10:20:08.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"e89b43785d70ccc61ee335d7b0a9588d85678753","tag":"autogenerated:gerrit:abandon","author":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"date":"2025-09-04 10:56:34.000000000","message":"Abandoned\n\nThere are already plans to fix the slowness via stats collection and querying uptime from the ComputeNode objects in the DB.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"aa65e952bf2e9ad0706d6be9ea5bb9fd661d1deb","revisions":{"a3c6771d0d79230d1423ee782a28ddbd9c00de2c":{"kind":"REWORK","_number":1,"created":"2025-09-04 09:40:46.000000000","uploader":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"ref":"refs/changes/20/959420/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/20/959420/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/20/959420/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/20/959420/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/20/959420/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/20/959420/1"}}},"commit":{"parents":[{"commit":"9f156aa954fb57eeaf407cc13a0f782b8af1a5d3","subject":"Merge \"Fix \u0027nova-manage image_property set\u0027 command\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9f156aa954fb57eeaf407cc13a0f782b8af1a5d3"}]}],"author":{"name":"Jakob Karge","email":"jakob.karge@sap.com","date":"2023-06-02 13:43:30.000000000","tz":120},"committer":{"name":"Johannes Kulik","email":"johannes.kulik@sap.com","date":"2025-09-04 09:38:28.000000000","tz":120},"subject":"Conditionally disable hypervisor uptime in details API","message":"Conditionally disable hypervisor uptime in details API\n\nThe \"uptime\" field in the hypervisor-details API is\n- costly\n- not supported for every hypervisor (notably not ironic \u0026 vmware)\n- noisy in error tracing tools\n\nIntroduce a \"disable_hypervisor_uptime_detail\" setting (default off)\nto always return None without asking the compute services.\n\nChange-Id: I544c62af23c98da8cab198d3be1c951645c19ce9\nSigned-off-by: Jakob Karge \u003cjakob.karge@sap.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a3c6771d0d79230d1423ee782a28ddbd9c00de2c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/a3c6771d0d79230d1423ee782a28ddbd9c00de2c"}]},"branch":"refs/heads/master"},"aa65e952bf2e9ad0706d6be9ea5bb9fd661d1deb":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2025-09-04 10:19:27.000000000","uploader":{"_account_id":26250,"name":"Johannes Kulik","email":"johannes.kulik@sap.com","username":"jkulik"},"ref":"refs/changes/20/959420/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/nova","ref":"refs/changes/20/959420/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/nova refs/changes/20/959420/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/nova refs/changes/20/959420/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/nova refs/changes/20/959420/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/nova refs/changes/20/959420/2"}}},"commit":{"parents":[{"commit":"9f156aa954fb57eeaf407cc13a0f782b8af1a5d3","subject":"Merge \"Fix \u0027nova-manage image_property set\u0027 command\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/9f156aa954fb57eeaf407cc13a0f782b8af1a5d3"}]}],"author":{"name":"Jakob Karge","email":"jakob.karge@sap.com","date":"2023-06-02 13:43:30.000000000","tz":120},"committer":{"name":"Johannes Kulik","email":"johannes.kulik@sap.com","date":"2025-09-04 10:17:29.000000000","tz":120},"subject":"Conditionally disable hypervisor uptime in details API","message":"Conditionally disable hypervisor uptime in details API\n\nThe \"uptime\" field in the hypervisor-details API is\n- costly: it sequentially and synchronously calls every hypervisor\u0027s\n  driver via RPC for `get_host_uptime()`, gravely slowing down\n  hypervisor listing in large deployments\n- not supported for every hypervisor (notably not ironic \u0026 vmware)\n- noisy in error tracing tools for drivers not supporting\n  `get_host_uptime()`\n\nIntroduce a \"disable_hypervisor_uptime_detail\" setting (default off)\nto always return None without asking the compute services.\n\nChange-Id: I544c62af23c98da8cab198d3be1c951645c19ce9\nSigned-off-by: Jakob Karge \u003cjakob.karge@sap.com\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/aa65e952bf2e9ad0706d6be9ea5bb9fd661d1deb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/nova/commit/aa65e952bf2e9ad0706d6be9ea5bb9fd661d1deb"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Review-Priority","description":"Review Priority","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{}}}]}
