{"action":"created","issue":{"url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/1254","repository_url":"https://api.github.com/repos/rcbops/rpc-openstack","labels_url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/1254/labels{/name}","comments_url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/1254/comments","events_url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/1254/events","html_url":"https://github.com/rcbops/rpc-openstack/issues/1254","id":166859030,"number":1254,"title":"galera_check.py queries_per_second metric mislabeled","user":{"login":"alextricity25","id":4964964,"avatar_url":"https://avatars.githubusercontent.com/u/4964964?v=3","gravatar_id":"","url":"https://api.github.com/users/alextricity25","html_url":"https://github.com/alextricity25","followers_url":"https://api.github.com/users/alextricity25/followers","following_url":"https://api.github.com/users/alextricity25/following{/other_user}","gists_url":"https://api.github.com/users/alextricity25/gists{/gist_id}","starred_url":"https://api.github.com/users/alextricity25/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alextricity25/subscriptions","organizations_url":"https://api.github.com/users/alextricity25/orgs","repos_url":"https://api.github.com/users/alextricity25/repos","events_url":"https://api.github.com/users/alextricity25/events{/privacy}","received_events_url":"https://api.github.com/users/alextricity25/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-07-21T16:12:14Z","updated_at":"2016-07-21T18:06:10Z","closed_at":null,"body":"#### Do NOT include any confidential information, this issue will be created on a public repo\r\n#### This section is completed by the issue reporter\r\n##### On which version(s) do you encounter this issue?\r\n###### Please provide the output to 'git describe --all' for  all deployments where you have confirmed it occurs.\r\nmaster\r\n##### What type of issue is this? Please tick the appropriate box.\r\n- [ ] bug - existing functionality that does not work for you as documented\r\n- [ ] feature - new functionality that you would like to see added\r\n\r\n##### What is the expected behavior?\r\nThe queries_per_second metric should be renamed to \"queries\".\r\n\r\n##### What is actually happening?\r\nThe metric is storing the value for the \"Queries\" mysql status variable, which is a counter and *not* a rate of change as the name suggests.\r\n\r\n##### What were you attempting to do when you encountered this error (give as much detail as possible)? \r\nN/A -- it's just an observation that was made by Darrin. Thanks @mancdaz.\r\n\r\n##### How can we reproduce this issue?\r\nObserve the queries_per_second metric in the RAX intelligence dashboard.It doesn't provide a rate of change value, but a counter of the number of statements executed[1].\r\n[1] http://dev.mysql.com/doc/refman/5.7/en/server-status-variables.html#statvar_Queries\r\n\r\n##### How can we verify your issue is addressed?\r\nThe queries_per_second metric name is changed.\r\n\r\n##### If you are using or know of a workaround please describe the necessary steps?\r\nN/A\r\n##### What is your evaluation on the importance of this issue, and why?\r\nprio-2, because the name of this metric can be mis-leading to devs who use this plugins and cause issues for them for mis-interpreting the metric.\r\n----------------------------------------------------------------------------------------------------------------------------\r\n#### This section is completed during the issue resolution process. If you are reporting an issue, please do not modify the section below.\r\n##### Which branches are impacted by this issue?\r\n###### Remove branches that are not impacted.\r\n###### For each pull request, update the branch line to reference it.\r\n- [ ] master\r\n- [ ] liberty-12.2\r\n- [ ] kilo\r\n\r\n##### Upgrade impact. Consider the impact on moving between major, minor and patch releases.\r\n- [ ] This change does not impact upgrades\r\n- [ ] This change addresses a potential upgrade impact\r\n- [ ] The upgrade impact is being tracked by another issue.\r\n\r\nThe issue tracking the upgrade impact is:\r\n\r\n##### Testing impact\r\n- [ ] An issue/pull-request has been create or exists to track the testing gap, and is referenced below\r\n- [ ] This change includes additional testing\r\n- [ ] This change is tested by the existing setup\r\n\r\nAdditional testing links:\r\n\r\n##### Release note\r\n- [ ] This change includes a release note.\r\n- [ ] This change does not require a release note.\r\n\r\n##### Doc Impact\r\n##### Does your change require documentation?\r\n- [ ] Yes, and I submitted an issue to [Docs issues](https://github.com/rackerlabs/docs-rpc/issues \"Docs issues\")\r\n- [ ] Yes, and I contributed a documentation patch to [docs-rpc](https://github.com/rackerlabs/docs-rpc \"docs-rpc\")\r\n- [ ] No documentation required for this change\r\n\r\nAdditional documentation links (please add links if you have ticked yes):\r\n"},"comment":{"url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/comments/234335361","html_url":"https://github.com/rcbops/rpc-openstack/issues/1254#issuecomment-234335361","issue_url":"https://api.github.com/repos/rcbops/rpc-openstack/issues/1254","id":234335361,"user":{"login":"mancdaz","id":562262,"avatar_url":"https://avatars.githubusercontent.com/u/562262?v=3","gravatar_id":"","url":"https://api.github.com/users/mancdaz","html_url":"https://github.com/mancdaz","followers_url":"https://api.github.com/users/mancdaz/followers","following_url":"https://api.github.com/users/mancdaz/following{/other_user}","gists_url":"https://api.github.com/users/mancdaz/gists{/gist_id}","starred_url":"https://api.github.com/users/mancdaz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mancdaz/subscriptions","organizations_url":"https://api.github.com/users/mancdaz/orgs","repos_url":"https://api.github.com/users/mancdaz/repos","events_url":"https://api.github.com/users/mancdaz/events{/privacy}","received_events_url":"https://api.github.com/users/mancdaz/received_events","type":"User","site_admin":false},"created_at":"2016-07-21T18:06:10Z","updated_at":"2016-07-21T18:06:10Z","body":"Note, while the plugin can't report qps or rate of change, the alarm language can be used to interpret current poll versus previous poll and determine a rate based on that."}}
{"actor":{"display_login":"mancdaz"}}