Group by docker label in chargeback for container images

[zeari]

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1497663
completed by: ManageIQ/manageiq-ui-classic#2277

cc @simon3z
@miq-bot add_label chargeback, providers/containers

[miq-bot]

Checked commit zeari@c6a5f18 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
5 files checked, 5 offenses detected

app/models/chargeback.rb

• ⚠️ - Line 54, Col 32 - Lint/UnusedMethodArgument - Unused method argument - consumption. If it's necessary, use _ or _consumption as an argument name to indicate that it won't be used. You can also write as groupby_label_value(*) if you want the method to accept any arguments but don't care about them.
• ⚠️ - Line 54, Col 45 - Lint/UnusedMethodArgument - Unused method argument - groupby_label. If it's necessary, use _ or _groupby_label as an argument name to indicate that it won't be used. You can also write as groupby_label_value(*) if you want the method to accept any arguments but don't care about them.
• ❗ - Line 116, Col 26 - Style/ParenthesesAroundCondition - Don't use parentheses around the condition of an if.
• ❗ - Line 118, Col 29 - Style/ParenthesesAroundCondition - Don't use parentheses around the condition of an elsif.

app/models/chargeback_container_image.rb

• ❗ - Line 57, Col 21 - Style/RedundantSelf - Redundant self detected.