)]}'
{"id":"openstack%2Fceilometer~14921","triplet_id":"openstack%2Fceilometer~master~Icd7de1b2cbce12396a82a3f34acb6c39c1120992","project":"openstack/ceilometer","branch":"master","hashtags":[],"change_id":"Icd7de1b2cbce12396a82a3f34acb6c39c1120992","subject":"Add new cpu_util meter recording CPU utilization %.","status":"MERGED","created":"2012-11-02 15:24:18.000000000","updated":"2012-11-02 15:24:18.000000000","submitted":"2012-11-02 15:24:18.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"3ea5c9cc94013c8710a23a339ad1b3481cf294ff","_number":14921,"virtual_id_number":14921,"owner":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":2,"date":"2012-11-02 15:24:18.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":595,"name":"Nick Barcet","email":"nicolas@barcet.com","username":"nijaba"},{"value":0,"_account_id":4277,"name":"John Tran","email":"jhtran@att.com","username":"jtran"},{"value":0,"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":0,"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},{"value":0,"_account_id":5679,"name":"Yaniv Kaul","email":"ykaul@redhat.com","username":"ykaul"}],"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":595,"name":"Nick Barcet","email":"nicolas@barcet.com","username":"nijaba"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":2,"date":"2012-11-02 15:24:18.000000000","_account_id":595,"name":"Nick Barcet","email":"nicolas@barcet.com","username":"nijaba"},{"value":2,"date":"2012-11-02 15:24:18.000000000","_account_id":4277,"name":"John Tran","email":"jhtran@att.com","username":"jtran"},{"value":0,"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":0,"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},{"value":0,"_account_id":5679,"name":"Yaniv Kaul","email":"ykaul@redhat.com","username":"ykaul"}],"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":4277,"name":"John Tran","email":"jhtran@att.com","username":"jtran"},"all":[{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"date":"2012-11-02 15:24:18.000000000","_account_id":595,"name":"Nick Barcet","email":"nicolas@barcet.com","username":"nijaba"},{"value":1,"date":"2012-11-02 15:24:18.000000000","_account_id":4277,"name":"John Tran","email":"jhtran@att.com","username":"jtran"},{"value":0,"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"value":0,"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"value":0,"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},{"value":0,"_account_id":5679,"name":"Yaniv Kaul","email":"ykaul@redhat.com","username":"ykaul"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":595,"name":"Nick Barcet","email":"nicolas@barcet.com","username":"nijaba"},{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},{"_account_id":4277,"name":"John Tran","email":"jhtran@att.com","username":"jtran"},{"_account_id":5679,"name":"Yaniv Kaul","email":"ykaul@redhat.com","username":"ykaul"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2012-11-02 15:24:18.000000000","updated_by":{"_account_id":595,"name":"Nick Barcet","email":"nicolas@barcet.com","username":"nijaba"},"reviewer":{"_account_id":595,"name":"Nick Barcet","email":"nicolas@barcet.com","username":"nijaba"},"state":"REVIEWER"},{"updated":"2012-11-02 15:24:18.000000000","updated_by":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"reviewer":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"state":"REVIEWER"},{"updated":"2012-11-02 15:24:18.000000000","updated_by":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"reviewer":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"state":"REVIEWER"},{"updated":"2012-11-02 15:24:18.000000000","updated_by":{"_account_id":4277,"name":"John Tran","email":"jhtran@att.com","username":"jtran"},"reviewer":{"_account_id":4277,"name":"John Tran","email":"jhtran@att.com","username":"jtran"},"state":"REVIEWER"},{"updated":"2012-11-02 15:24:18.000000000","updated_by":{"_account_id":5679,"name":"Yaniv Kaul","email":"ykaul@redhat.com","username":"ykaul"},"reviewer":{"_account_id":5679,"name":"Yaniv Kaul","email":"ykaul@redhat.com","username":"ykaul"},"state":"REVIEWER"},{"updated":"2012-11-02 15:24:18.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"29e38986d65bab07da24c9fd885b748bcb752d1a","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 2: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/14921/2/check/gate-ceilometer-merge/571 : SUCCESS\n- http://logs.openstack.org/14921/2/check/gate-ceilometer-pep8/568 : SUCCESS\n- http://logs.openstack.org/14921/2/check/gate-ceilometer-python26/568 : SUCCESS\n- http://logs.openstack.org/14921/2/check/gate-ceilometer-python27/570 : SUCCESS\n","accounts_in_message":[],"_revision_number":2},{"id":"9d937f2672d81734a629958285d69afad0c09173","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2012-11-02 15:24:18.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"0451317c0ba852017b490666d2ba3f3773144ede","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/14921/3/check/gate-ceilometer-merge/572 : SUCCESS\n- http://logs.openstack.org/14921/3/check/gate-ceilometer-pep8/569 : SUCCESS\n- http://logs.openstack.org/14921/3/check/gate-ceilometer-python26/569 : SUCCESS\n- http://logs.openstack.org/14921/3/check/gate-ceilometer-python27/571 : SUCCESS\n","accounts_in_message":[],"_revision_number":3},{"id":"6ff267508eef9c102d3b908bcf7f54476c8c415d","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3:\n\n@Doug I understand that this may be needed for capacity planning, but wouldn\u0027t that be more smart to compute it via the API rather than storing it twice in the db?","accounts_in_message":[],"_revision_number":3},{"id":"db4f7c561d1be447e3cf0665042d9cd2222951b8","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3:\n\nI\u0027m ok to merge this as long as this is really needed by someone and that there\u0027s a comment with a FIXME above stating that should be moved elsewhere.\n\nIf it can wait, I\u0027d prefer to have this handled by the CW publisher and the API.","accounts_in_message":[],"_revision_number":3},{"id":"e3039cc37814a8869b246ac616e0f4eb8742827f","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\nI\u0027m having a hard time about this. I really don\u0027t like the idea to compute stuff and build new counters based on this.\n\nTo me, this is something that should be provided by the API, otherwise we\u0027re going to meter and *store* dozens of different version of the same source value. I\u0027d be happier if we\u0027d store one format and compute the other through the API.","accounts_in_message":[],"_revision_number":3},{"id":"bc3d6e5a824921e51d5004213254db5528802fea","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3:\n\nI agree with the secondary publisher approach rather than a second metric. That\u0027d mean your publisher would convert what the pollster gets to what it needs (i.e. %)\n\nFor the total number of cores, if this is needed I think the best way is to add this in the metadata fields on the current counter. :)","accounts_in_message":[],"_revision_number":3},{"id":"0925baf013fddac40c13c064ed9b734df572b715","author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3:\n\nNo, unless I miss something, I don\u0027t think that his interest Ceilometer users at all as it is. Only CW users, that\u0027s why it should be a problem in the CW publisher, not in the pollster.","accounts_in_message":[],"_revision_number":3},{"id":"ae9d58f083b51f0a07850c3c882ebb44d65033ef","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2012-11-02 15:24:18.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"914c2583574b2f065f42d1e86e1826d6f0d94f6f","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3:\n\n@jd_\n\nOK, so the application I had in mind was publishing the CPU utilization % to CloudWatch (in addition to the celio metering store).\n\nIn this case, for reasons of timeliness IMO it would make much more sense to drive the CW PutMetricData call directly from a publisher associated with the libvirt pollster, as opposed to deriving it as a secondary metric from data already stored in the ceilometer metering store.\n\nAlso in order to estimate the CPU util for a guest, we\u0027d need to start storing additional info in the metering store, specifically the total number of cores on the host (in addition to the number of core associated with the guest).\n\nDoes that reasoning change your disposition to this patch any?\n\nCheers,\nEoghan","accounts_in_message":[],"_revision_number":3},{"id":"a93c3302427461dcf20d3a90a2197fe751845446","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3:\n\n@jd_\n\nSo if the CPU utilization metric was *only* of interest to CloudWatch, then I guess doing the CPU time-\u003eutil % conversion in the relevant CW Transformer would be reasonable.\n\nHowever, in this case, IIUC this measure is of interest both to pure ceilometer users and to a future CW consumer. \n\nIn cases where more than one publisher has interest in a measure, it seems to make most sense to compute it once and once only.\n\nIn the case of CW, that one place should be as close as possible to the source of the measure, for reasons of timeliness.\n\nWhich all seems to point to the pollster as the most reasonable location for this calculation.\n\nCheers,\nEoghan","accounts_in_message":[],"_revision_number":3},{"id":"1621826185a6de1bd2c262cbe22b5f7482c5904c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 4: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/14921/4/check/gate-ceilometer-merge/602 : SUCCESS\n- http://logs.openstack.org/14921/4/check/gate-ceilometer-pep8/599 : SUCCESS\n- http://logs.openstack.org/14921/4/check/gate-ceilometer-python26/599 : SUCCESS\n- http://logs.openstack.org/14921/4/check/gate-ceilometer-python27/601 : SUCCESS\n","accounts_in_message":[],"_revision_number":4},{"id":"9ee43a3cbb72f84b37222c806149e5a6c7cca770","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2012-11-02 15:24:18.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"0d6dca432d71dcc8b3c17c2bb2c8f8932833ca39","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 5: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/14921/5/check/gate-ceilometer-merge/616 : SUCCESS\n- http://logs.openstack.org/14921/5/check/gate-ceilometer-pep8/613 : SUCCESS\n- http://logs.openstack.org/14921/5/check/gate-ceilometer-python26/613 : SUCCESS\n- http://logs.openstack.org/14921/5/check/gate-ceilometer-python27/615 : SUCCESS\n","accounts_in_message":[],"_revision_number":5},{"id":"529a9305ea70422a2a6f5280517d219548c983ad","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 5:\n\nLatest patch set accounts for the CPU time being reset during the polling interval (i.e. avoids reporting a negative utilization in that case).\n\nHowever, if the polling interval is long enough for the instance to have been restarted several times, or the reset cumulative time has caught up with the previous time, then the utilization will be under-reported.","accounts_in_message":[],"_revision_number":5},{"id":"5dbbfc4d8b6cc384a63fa01b4e0faff9796ffb50","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2012-11-02 15:24:18.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"d308df065ec5229330593cbdb58de3ef74ee5b2b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 6: No score\n\nStarting gate jobs.\nhttps://jenkins.openstack.org/zuul/status","accounts_in_message":[],"_revision_number":6},{"id":"9efdb0e27fbb91c973a1ce8af09479256d03ccdb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 6: Works for me\n\nBuild successful\n\n- http://logs.openstack.org/14921/6/check/gate-ceilometer-merge/637 : SUCCESS\n- http://logs.openstack.org/14921/6/check/gate-ceilometer-pep8/633 : SUCCESS\n- http://logs.openstack.org/14921/6/check/gate-ceilometer-python26/633 : SUCCESS\n- http://logs.openstack.org/14921/6/check/gate-ceilometer-python27/635 : SUCCESS\n","accounts_in_message":[],"_revision_number":6},{"id":"ae1d255f9d5c512afbaa925ac6e7a0bc6ac4155c","author":{"_account_id":595,"name":"Nick Barcet","email":"nicolas@barcet.com","username":"nijaba"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 6: Looks good to me (core reviewer)\n\nI am giving +2 on this on the basis that we should never prevent anyone from adding counters.  This, however, reinforces the need to be able to set which counters we care about in a given deployment on an individual basis.","accounts_in_message":[],"_revision_number":6},{"id":"600bc9306611dbae75609e7fe72f33f5b4008996","author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"date":"2012-11-02 15:24:18.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"1db7245ed557b57f45805c3e17b187fd2ac066dc","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3:\n\nOur product manager is interesting in seeing these values for capacity planning during our beta. We could use a custom pollster to collect them, but it seems like they might be useful to other people, too (for the same reason, outside of CW triggers). We\u0027ve also discussed collecting % utilization of the hypervisor host to be used in the nova scheduler. That\u0027s similar data in yet another form.\n\nI\u0027m not sure if any of that helps with the decision of whether to include the new meter or not. Maybe we need to have a discussion about what to do with these \"redundant\" measures?","accounts_in_message":[],"_revision_number":3},{"id":"0bdc6521913bf51f3e208b92ab94c93502831a2f","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3: (3 inline comments)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"2c54f563080301f4143dd3cff168dd65673fe1fc","author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3:\n\nRegarding whether to compute this from the  agent or from the API, I\u0027m really not sure. I assume with the efforts to merge similar stats from heat we may have other counters we want to publish but not store. Do we wait to create those until we have a way to configure where the data should be sent? Either way makes sense to me.","accounts_in_message":[],"_revision_number":3},{"id":"3640d4f2e35c9f104a358537b12a10028ea009d9","author":{"_account_id":4277,"name":"John Tran","email":"jhtran@att.com","username":"jtran"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 5:\n\nI would love to approve this but to be honest all the maths is way over my head","accounts_in_message":[],"_revision_number":5},{"id":"d6abfb65d2bf5a71c8641dff442356e5a169e0f7","author":{"_account_id":4277,"name":"John Tran","email":"jhtran@att.com","username":"jtran"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 6: Looks good to me (core reviewer); Approved\n\nNick: Agreed","accounts_in_message":[],"_revision_number":6},{"id":"bd14e584cdeb517154cf86243cdb0ea101c42759","author":{"_account_id":5679,"name":"Yaniv Kaul","email":"ykaul@redhat.com","username":"ykaul"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 3: (1 inline comment)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"3b26304158e3cee1ea39a307321ea2c9298dbe1b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-02 15:24:18.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":6},{"id":"3ea5c9cc94013c8710a23a339ad1b3481cf294ff","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2012-11-02 15:24:18.000000000","message":"Patch Set 6: Verified\n\nBuild successful\n\n- http://logs.openstack.org/14921/6/gate/gate-ceilometer-merge/645 : SUCCESS\n- http://logs.openstack.org/14921/6/gate/gate-ceilometer-pep8/641 : SUCCESS\n- http://logs.openstack.org/14921/6/gate/gate-ceilometer-python26/641 : SUCCESS\n- http://logs.openstack.org/14921/6/gate/gate-ceilometer-python27/643 : SUCCESS\n","accounts_in_message":[],"_revision_number":6}],"current_revision_number":6,"current_revision":"15eeeb9fc16996ccf667f60752fead99597c983f","revisions":{"de5a2ff4d2f5cc8a757b341e4bd991fd2fa39732":{"kind":"REWORK","_number":1,"created":"2012-11-02 15:24:18.000000000","uploader":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"ref":"refs/changes/21/14921/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/21/14921/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/1"}}},"commit":{"parents":[{"commit":"acf8fb88d56b776cf0b83041f5e87e1b6ee88fd2","subject":"api: add source detail retrieval","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/acf8fb88d56b776cf0b83041f5e87e1b6ee88fd2"}]}],"author":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-26 21:33:36.000000000","tz":0},"committer":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-26 21:46:23.000000000","tz":60},"subject":"Add new cpu_util meter recording CPU utilization %.","message":"Add new cpu_util meter recording CPU utilization %.\n\nThe existing cpu meter records cumlative time, which is not\nthe most useful measure in all cases.\n\nWe now also estimate the CPU utilization % based on sampling\nthe time over subsequent polling cycles, and correcting for the\nfraction of host cores available to the instance. The usefulness\nof the util estimate depends in part on the polling period,\nwhich is assumed will be short (of the order of 1s) when this\nmeasurement is used in anger (for example to drive alarming).\n\nIt appears the CPUPollster is currently untested, presumably\nbecause of some difficulty in faking out meaningful responses\nfrom libvirt. I\u0027ll address this lack of test coverage in a\nsubsequent patch.\n\nChange-Id: Icd7de1b2cbce12396a82a3f34acb6c39c1120992\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/de5a2ff4d2f5cc8a757b341e4bd991fd2fa39732"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/de5a2ff4d2f5cc8a757b341e4bd991fd2fa39732"}]},"branch":"refs/heads/master"},"9ed8b49d42ff90d1ea1b982e00500c2ed72455ea":{"kind":"REWORK","_number":2,"created":"2012-11-02 15:24:18.000000000","uploader":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"ref":"refs/changes/21/14921/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/21/14921/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/2"}}},"commit":{"parents":[{"commit":"acf8fb88d56b776cf0b83041f5e87e1b6ee88fd2","subject":"api: add source detail retrieval","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/acf8fb88d56b776cf0b83041f5e87e1b6ee88fd2"}]}],"author":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-26 21:33:36.000000000","tz":0},"committer":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-26 21:51:52.000000000","tz":60},"subject":"Add new cpu_util meter recording CPU utilization %.","message":"Add new cpu_util meter recording CPU utilization %.\n\nThe existing cpu meter records cumlative time, which is not\nthe most useful measure in all cases.\n\nWe now also estimate the CPU utilization % based on sampling\nthe time over subsequent polling cycles, and correcting for the\nfraction of host cores available to the instance. The usefulness\nof the util estimate depends in part on the polling period,\nwhich is assumed will be short (of the order of 1s) when this\nmeasurement is used in anger (for example to drive alarming).\n\nIt appears the CPUPollster is currently untested, presumably\nbecause of some difficulty in faking out meaningful responses\nfrom libvirt. I\u0027ll address this lack of test coverage in a\nsubsequent patch.\n\nChange-Id: Icd7de1b2cbce12396a82a3f34acb6c39c1120992\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/9ed8b49d42ff90d1ea1b982e00500c2ed72455ea"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/9ed8b49d42ff90d1ea1b982e00500c2ed72455ea"}]},"branch":"refs/heads/master"},"f8ffad35e8856bb6fdf703d91a4839a39cba49af":{"kind":"REWORK","_number":3,"created":"2012-11-02 15:24:18.000000000","uploader":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"ref":"refs/changes/21/14921/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/21/14921/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/3"}}},"commit":{"parents":[{"commit":"acf8fb88d56b776cf0b83041f5e87e1b6ee88fd2","subject":"api: add source detail retrieval","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/acf8fb88d56b776cf0b83041f5e87e1b6ee88fd2"}]}],"author":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-26 21:33:36.000000000","tz":0},"committer":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-26 22:18:26.000000000","tz":60},"subject":"Add new cpu_util meter recording CPU utilization %.","message":"Add new cpu_util meter recording CPU utilization %.\n\nThe existing cpu meter records cumlative time, which is not\nthe most useful measure in all cases.\n\nWe now also estimate the CPU utilization % based on sampling\nthe time over subsequent polling cycles, and correcting for the\nfraction of host cores available to the instance. The usefulness\nof the util estimate depends in part on the polling period,\nwhich is assumed will be short (of the order of 1s) when this\nmeasurement is used in anger (for example to drive alarming).\n\nIt appears the CPUPollster is currently untested, presumably\nbecause of some difficulty in faking out meaningful responses\nfrom libvirt. I\u0027ll address this lack of test coverage in a\nsubsequent patch.\n\nChange-Id: Icd7de1b2cbce12396a82a3f34acb6c39c1120992\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/f8ffad35e8856bb6fdf703d91a4839a39cba49af"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/f8ffad35e8856bb6fdf703d91a4839a39cba49af"}]},"branch":"refs/heads/master"},"5532f73eab049b2675fae691cd779826d8853000":{"kind":"REWORK","_number":4,"created":"2012-11-02 15:24:18.000000000","uploader":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"ref":"refs/changes/21/14921/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/21/14921/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/4"}}},"commit":{"parents":[{"commit":"b937aa509ba1036bfc0df6965235c63cca3202cb","subject":"Merge \"add unit test for CPUPollster\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/b937aa509ba1036bfc0df6965235c63cca3202cb"}]}],"author":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-26 21:33:36.000000000","tz":0},"committer":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-30 17:07:16.000000000","tz":0},"subject":"Add new cpu_util meter recording CPU utilization %.","message":"Add new cpu_util meter recording CPU utilization %.\n\nThe existing cpu meter records cumlative time, which is not\nthe most useful measure in all cases.\n\nWe now also estimate the CPU utilization % based on sampling\nthe time over subsequent polling cycles, and correcting for the\nfraction of host cores available to the instance. The usefulness\nof the util estimate depends in part on the polling period,\nwhich is assumed will be short (of the order of 1s) when this\nmeasurement is used in anger (for example to drive alarming).\n\nChange-Id: Icd7de1b2cbce12396a82a3f34acb6c39c1120992\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/5532f73eab049b2675fae691cd779826d8853000"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/5532f73eab049b2675fae691cd779826d8853000"}]},"branch":"refs/heads/master"},"60e91907710f0bf740342caa45000b585e571f0a":{"kind":"REWORK","_number":5,"created":"2012-11-02 15:24:18.000000000","uploader":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"ref":"refs/changes/21/14921/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/21/14921/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/5"}}},"commit":{"parents":[{"commit":"2aec7f16f9b541673f8114d2402c24a35965d536","subject":"set tox to ignore global packages","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/2aec7f16f9b541673f8114d2402c24a35965d536"}]}],"author":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-26 21:33:36.000000000","tz":0},"committer":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-31 14:48:30.000000000","tz":0},"subject":"Add new cpu_util meter recording CPU utilization %.","message":"Add new cpu_util meter recording CPU utilization %.\n\nThe existing cpu meter records cumlative time, which is not\nthe most useful measure in all cases.\n\nWe now also estimate the CPU utilization % based on sampling\nthe time over subsequent polling cycles, and correcting for the\nfraction of host cores available to the instance. The usefulness\nof the util estimate depends in part on the polling period,\nwhich is assumed will be short (of the order of 1s) when this\nmeasurement is used in anger (for example to drive alarming).\n\nChange-Id: Icd7de1b2cbce12396a82a3f34acb6c39c1120992\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/60e91907710f0bf740342caa45000b585e571f0a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/60e91907710f0bf740342caa45000b585e571f0a"}]},"branch":"refs/heads/master"},"15eeeb9fc16996ccf667f60752fead99597c983f":{"kind":"REWORK","_number":6,"created":"2012-11-02 15:24:18.000000000","uploader":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"ref":"refs/changes/21/14921/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/ceilometer","ref":"refs/changes/21/14921/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/ceilometer refs/changes/21/14921/6"}}},"commit":{"parents":[{"commit":"2aec7f16f9b541673f8114d2402c24a35965d536","subject":"set tox to ignore global packages","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/2aec7f16f9b541673f8114d2402c24a35965d536"}]}],"author":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-10-26 21:33:36.000000000","tz":0},"committer":{"name":"Eoghan Glynn","email":"eglynn@redhat.com","date":"2012-11-01 17:14:47.000000000","tz":0},"subject":"Add new cpu_util meter recording CPU utilization %.","message":"Add new cpu_util meter recording CPU utilization %.\n\nThe existing cpu meter records cumlative time, which is not\nthe most useful measure in all cases.\n\nWe now also estimate the CPU utilization % based on sampling\nthe time over subsequent polling cycles, and correcting for the\nfraction of host cores available to the instance. The usefulness\nof the util estimate depends in part on the polling period,\nwhich is assumed will be short (of the order of 1s) when this\nmeasurement is used in anger (for example to drive alarming).\n\nChange-Id: Icd7de1b2cbce12396a82a3f34acb6c39c1120992\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/15eeeb9fc16996ccf667f60752fead99597c983f"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/ceilometer/commit/15eeeb9fc16996ccf667f60752fead99597c983f"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}