-
Notifications
You must be signed in to change notification settings - Fork 46
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
SWATCH-1233: Drop UOM from the swatch #3776
Conversation
e007f75
to
45dac2c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
swatch-core/src/main/java/org/candlepin/subscriptions/db/model/TallyMeasurementKey.java:77: + ", uom="
swatch-tally/deploy/clowdapp.yaml:708: metric_id as uom,
/retest |
45dac2c
to
6e49a08
Compare
Updated
This is used by the floorplan queries, so it should stay as it is as noted in the JIRA description. PR updated. |
/retest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good in regression testing. I also added an IQE MR to remove variables named UOM in any test and replaced with metric_id when necessary. This will help to avoid anyone being confused about UOM still being seen in tests in the future. https://gitlab.cee.redhat.com/insights-qe/iqe-rhsm-subscriptions-plugin/-/merge_requests/874
/retest |
UOM is no longer referenced in swatch openapi definitions Automation tests are no longer referencing UOM UOM is no longer referenced in SWatch code UOM is no longer stored as part of events. Double check that no queries are using UOM check bin/ scripts to make sure UOM is no longer referenced
6e49a08
to
5186639
Compare
/retest |
Jira issue: SWATCH-1233
Description
UOM is no longer referenced in swatch openapi definitions Automation tests are no longer referencing UOM
UOM is no longer referenced in SWatch code
UOM is no longer stored as part of events.
Double check that no queries are using UOM
check bin/ scripts to make sure UOM is no longer referenced
Testing
Only regression testing.