)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"074115a507a97d7dbaae500b5699f89a67077a25","unresolved":true,"context_lines":[{"line_number":16,"context_line":"up and the scenario fails."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"With this patch, the load lasts 2x (the ceilometer polling"},{"line_number":19,"context_line":"interval + prometheus scrape interval),"},{"line_number":20,"context_line":"which guarantees that at least on polling period"},{"line_number":21,"context_line":"will be with 100% load, which will result in a high CPU"},{"line_number":22,"context_line":"utilization registered by ceilometer."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"bbe80235_9647182c","line":19,"range":{"start_line":19,"start_character":8,"end_line":19,"end_character":37},"updated":"2024-08-08 08:48:12.000000000","message":"The actual change does not do this.","commit_id":"4ec1f85148b244f0ecc8bc1548c58810b1f90357"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"3b85aee7a1d6e0ef52f964cc32161dfd6e68e6a4","unresolved":true,"context_lines":[{"line_number":16,"context_line":"up and the scenario fails."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"With this patch, the load lasts 2x (the ceilometer polling"},{"line_number":19,"context_line":"interval + prometheus scrape interval),"},{"line_number":20,"context_line":"which guarantees that at least on polling period"},{"line_number":21,"context_line":"will be with 100% load, which will result in a high CPU"},{"line_number":22,"context_line":"utilization registered by ceilometer."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"2da15670_89653ea2","line":19,"range":{"start_line":19,"start_character":8,"end_line":19,"end_character":37},"in_reply_to":"bbe80235_9647182c","updated":"2024-08-08 10:10:03.000000000","message":"Oops, my highlight was left bit short. So it seems that the code itself only does the scrape * 2 not the polling interval+scrape*2 or did I miss something?","commit_id":"4ec1f85148b244f0ecc8bc1548c58810b1f90357"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"1b708896d850f27aad6552a91c8e832c82220b99","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3866dabc_f765d561","updated":"2024-08-02 07:20:04.000000000","message":"Looks like something is wrong with liberasure-devel package: \"No match for argument: liberasurecode-devel\"","commit_id":"478cb567b397e75a303d1f7ef5f9e83d2d2264b8"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"69b6def00c5492cc8ebc39e831eb78a436276f28","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"dfad472e_3e877df3","updated":"2024-08-06 13:35:54.000000000","message":"recheck","commit_id":"478cb567b397e75a303d1f7ef5f9e83d2d2264b8"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"fd7d19ad71300c94743c50c39fdbd66aa0690fb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"40e5cfaf_d6b74eef","updated":"2024-09-30 09:41:09.000000000","message":"lgtm","commit_id":"4ec1f85148b244f0ecc8bc1548c58810b1f90357"},{"author":{"_account_id":32968,"name":"Juan Larriba","email":"jlarriba@redhat.com","username":"jlarriba"},"change_message_id":"4b641b9f4d80aa4df17954dfd6e0c349ddde9ee8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"03d0c8a0_a5ec491c","updated":"2024-09-05 15:53:46.000000000","message":"simple and atomic change, lgtm","commit_id":"4ec1f85148b244f0ecc8bc1548c58810b1f90357"}],"telemetry_tempest_plugin/scenario/test_telemetry_integration_prometheus.py":[{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"074115a507a97d7dbaae500b5699f89a67077a25","unresolved":true,"context_lines":[{"line_number":135,"context_line":"            \"STACK_NAME\": self.stack_name,"},{"line_number":136,"context_line":"            \"RESOURCE_PREFIX\": resource_prefix,"},{"line_number":137,"context_line":"            \"PROMETHEUS_RATE_DURATION\": str(prometheus_rate_duration),"},{"line_number":138,"context_line":"            \"LOAD_LENGTH\": str(prometheus_rate_duration * 2),"},{"line_number":139,"context_line":"        })"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"aa471864_ae9b108c","line":138,"range":{"start_line":138,"start_character":31,"end_line":138,"end_character":59},"updated":"2024-08-08 08:48:12.000000000","message":"This odes not correlate with the commit message.","commit_id":"4ec1f85148b244f0ecc8bc1548c58810b1f90357"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"fd7d19ad71300c94743c50c39fdbd66aa0690fb4","unresolved":true,"context_lines":[{"line_number":135,"context_line":"            \"STACK_NAME\": self.stack_name,"},{"line_number":136,"context_line":"            \"RESOURCE_PREFIX\": resource_prefix,"},{"line_number":137,"context_line":"            \"PROMETHEUS_RATE_DURATION\": str(prometheus_rate_duration),"},{"line_number":138,"context_line":"            \"LOAD_LENGTH\": str(prometheus_rate_duration * 2),"},{"line_number":139,"context_line":"        })"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"b49e4649_834fe38b","line":138,"range":{"start_line":138,"start_character":31,"end_line":138,"end_character":59},"in_reply_to":"446bc680_57dc3ebd","updated":"2024-09-30 09:41:09.000000000","message":"So I indeed missed something, sorry.","commit_id":"4ec1f85148b244f0ecc8bc1548c58810b1f90357"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"3fa6042e9a60e379d59719b5e355742b81240f3f","unresolved":true,"context_lines":[{"line_number":135,"context_line":"            \"STACK_NAME\": self.stack_name,"},{"line_number":136,"context_line":"            \"RESOURCE_PREFIX\": resource_prefix,"},{"line_number":137,"context_line":"            \"PROMETHEUS_RATE_DURATION\": str(prometheus_rate_duration),"},{"line_number":138,"context_line":"            \"LOAD_LENGTH\": str(prometheus_rate_duration * 2),"},{"line_number":139,"context_line":"        })"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"446bc680_57dc3ebd","line":138,"range":{"start_line":138,"start_character":31,"end_line":138,"end_character":59},"in_reply_to":"56844f43_beedf750","updated":"2024-09-04 16:05:52.000000000","message":"@jokke@usr.fi I answered your comment. WDYT?","commit_id":"4ec1f85148b244f0ecc8bc1548c58810b1f90357"},{"author":{"_account_id":34975,"name":"Jaromír Wysoglad","email":"jwysogla@redhat.com","username":"jwysogla"},"change_message_id":"d660e005c18e455fcd8cc03f5f76656ad813baa8","unresolved":true,"context_lines":[{"line_number":135,"context_line":"            \"STACK_NAME\": self.stack_name,"},{"line_number":136,"context_line":"            \"RESOURCE_PREFIX\": resource_prefix,"},{"line_number":137,"context_line":"            \"PROMETHEUS_RATE_DURATION\": str(prometheus_rate_duration),"},{"line_number":138,"context_line":"            \"LOAD_LENGTH\": str(prometheus_rate_duration * 2),"},{"line_number":139,"context_line":"        })"},{"line_number":140,"context_line":""},{"line_number":141,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"56844f43_beedf750","line":138,"range":{"start_line":138,"start_character":31,"end_line":138,"end_character":59},"in_reply_to":"aa471864_ae9b108c","updated":"2024-08-09 07:05:28.000000000","message":"Take a look at L115-L117 to see what the variable actually is. I reused this variable, because it\u0027s already a sum of the ceilometer polling and prometheus scrape intervals. Do you think it would be better to just create a completely new varable without the reuse of \"prometheus_rate_duration\"? like:\n\nload_length \u003d 2* (conf.CONF.telemetry.ceilometer_polling_interval + conf.CONF.telemetry.prometheus_scrape_interval)","commit_id":"4ec1f85148b244f0ecc8bc1548c58810b1f90357"}]}