Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EUWE] Do not calculate useless group metrics #15337

Merged
merged 1 commit into from
Jun 9, 2017

Conversation

isimluk
Copy link
Member

@isimluk isimluk commented Jun 8, 2017

Manual backport of #15260. To make the tests green.

https://bugzilla.redhat.com/show_bug.cgi?id=1455690
https://bugzilla.redhat.com/show_bug.cgi?id=1454456

  • cpu is useless as it counts usagemhz and vm_numvcpus together
  • cpu_cores is useless as it only reports cpu_cores
  • memory is useless as counts used and available together
  • net_io is useless as it only reports net_ui_rate_avg
  • disk_io is useless as it only reports disk_usage_rate_avg
  • storage is useless as it counts used and allocated together
  • fixed is useless as it counts all the fixed metrics together
    (that equals to 4 for each day, not useful)

(cherry picked from commit afb10c1)

Conflicts:
app/models/chargeable_field.rb
spec/models/chargeback_vm_spec.rb

https://bugzilla.redhat.com/show_bug.cgi?id=1455690
https://bugzilla.redhat.com/show_bug.cgi?id=1454456

 - cpu is useless as it counts usagemhz and vm_numvcpus together
 - cpu_cores is useless as it only reports cpu_cores
 - memory is useless as counts used and available together
 - net_io is useless as it only reports net_ui_rate_avg
 - disk_io is useless as it only reports disk_usage_rate_avg
 - storage is useless as it counts used and allocated together
 - fixed is useless as it counts all the fixed metrics together
   (that equals to 4 for each day, not useful)

(cherry picked from commit afb10c1)

Conflicts:
	app/models/chargeable_field.rb
	spec/models/chargeback_vm_spec.rb
@miq-bot
Copy link
Member

miq-bot commented Jun 8, 2017

Checked commit isimluk@a5c8a79 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
2 files checked, 0 offenses detected
Everything looks fine. 👍

@simaishi simaishi merged commit 3a507f7 into ManageIQ:euwe Jun 9, 2017
@simaishi simaishi added this to the Sprint 63 Ending Jun 19, 2017 milestone Jun 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants