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

fix(rhosak): ent-4982 storage GiB as GB hours to months #943

Merged
merged 2 commits into from
Jun 2, 2022

Conversation

cdcabrera
Copy link
Member

@cdcabrera cdcabrera commented Jun 2, 2022

What's included

  • fix(rhosak): ent-4982 storage GiB as GB hours to months
    • config, rhosak, use updated monthly metricId, disable sort
    • locale, add monthly metricId, shift prior monthly changes
    • services, constants, apply gib monthly metricId

Notes

  • Doesn't appear that Storage-gibibyte-months is part of the sorting spec under InstanceReportSort, we're disabling it as part of this PR, it can be reactivated when the spec is updated, or otherwise
  • @diegomaranhao @mirekdlugosz @ntkathole

How to test

Proxy run check

  1. update the NPM packages with $ yarn
  2. make sure Docker is running, plus on network, then
  3. $ yarn start:proxy
  4. confirm strings display correctly, and metric id pulls from tally and instances API

Example

Screen Shot 2022-06-02 at 3 56 37 PM
Screen Shot 2022-06-02 at 3 56 44 PM

Updates issue/story

ent-4982

* config, rhosak, use updated monthly metricId
* locale, add monthly metricId, shift prior monthly changes
* services, constants, apply gib monthly metricId
@cdcabrera cdcabrera added the 202208 project phase label Jun 2, 2022
* missing from spec, disabling for now
@codecov-commenter
Copy link

codecov-commenter commented Jun 2, 2022

Codecov Report

Merging #943 (574438f) into ci (91f8a03) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##               ci     #943   +/-   ##
=======================================
  Coverage   94.05%   94.05%           
=======================================
  Files         129      129           
  Lines        4020     4020           
  Branches     1571     1571           
=======================================
  Hits         3781     3781           
  Misses        221      221           
  Partials       18       18           
Impacted Files Coverage Δ
src/config/product.rhosak.js 87.17% <ø> (ø)
src/services/rhsm/rhsmConstants.js 100.00% <ø> (ø)
src/services/rhsm/rhsmServices.js 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 91f8a03...574438f. Read the comment docs.

@cdcabrera cdcabrera merged commit a10c265 into RedHatInsights:ci Jun 2, 2022
cdcabrera added a commit that referenced this pull request Jun 3, 2022
* config, rhosak, use updated monthly metricId, disable sort
* locale, add monthly metricId, shift prior monthly changes
* services, constants, apply gib monthly metricId
cdcabrera added a commit that referenced this pull request Jun 3, 2022
* config, rhosak, use updated monthly metricId, disable sort
* locale, add monthly metricId, shift prior monthly changes
* services, constants, apply gib monthly metricId
cdcabrera added a commit that referenced this pull request Jun 14, 2022
* config, rhosak, use updated monthly metricId, disable sort
* locale, add monthly metricId, shift prior monthly changes
* services, constants, apply gib monthly metricId
@cdcabrera cdcabrera mentioned this pull request Jun 14, 2022
cdcabrera added a commit that referenced this pull request Jun 24, 2022
* config, rhosak, use updated monthly metricId, disable sort
* locale, add monthly metricId, shift prior monthly changes
* services, constants, apply gib monthly metricId
cdcabrera added a commit that referenced this pull request Jul 11, 2022
* config, rhosak, use updated monthly metricId, disable sort
* locale, add monthly metricId, shift prior monthly changes
* services, constants, apply gib monthly metricId
cdcabrera added a commit that referenced this pull request Jul 11, 2022
* config, rhosak, use updated monthly metricId, disable sort
* locale, add monthly metricId, shift prior monthly changes
* services, constants, apply gib monthly metricId
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
202208 project phase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants