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

[COST-4390] OCP changing GB to GiB #4938

Merged
merged 20 commits into from
Mar 12, 2024
Merged

[COST-4390] OCP changing GB to GiB #4938

merged 20 commits into from
Mar 12, 2024

Conversation

bacciotti
Copy link
Contributor

Jira Ticket

COST-4390

Description

This change will change "GB" units for "GiB" units on OCP.

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

Copy link

codecov bot commented Feb 26, 2024

Codecov Report

Merging #4938 (c02f2f7) into main (c58f3e0) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #4938   +/-   ##
=====================================
  Coverage   94.0%   94.0%           
=====================================
  Files        375     375           
  Lines      31129   31129           
  Branches    3698    3698           
=====================================
  Hits       29266   29266           
- Misses      1189    1191    +2     
+ Partials     674     672    -2     

@bacciotti
Copy link
Contributor Author

/retest

@myersCody myersCody changed the title [COST-4390] Changing GB to GiB. [COST-4390] OCP changing GB to GiB. Feb 26, 2024
@bacciotti
Copy link
Contributor Author

/retest

1 similar comment
@bacciotti
Copy link
Contributor Author

/retest

@bacciotti bacciotti added the smoke-tests pr_check will build the image and run minimal required smokes label Feb 27, 2024
@bacciotti
Copy link
Contributor Author

/retest

6 similar comments
@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

ci/functions.sh Outdated
@@ -127,7 +127,7 @@ function run_smoke_tests_stage() {
echo "Running E2E tests with IQE:"
echo "IQE_MARKER_EXPRESSION: '$IQE_MARKER_EXPRESSION'"
echo "IQE_FILTER_EXPRESSION: '$IQE_FILTER_EXPRESSION'"

export IQE_IMAGE_TAG="cost-management-1891-e26f8349"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious why we are adding this IQE_IMAGE_TAG?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@djnakabaale this is a temporary solution to run this PR against a modified iqe plugin version :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bacciotti would you be interested in documenting how you created the tag and how you can use this env var to run the smokes off a modified version? I think it could be helpful for the team moving forward.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure I would! I can do that!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FTI I've created a document explaining this. Please ping me on slack, then I can send you.

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

4 similar comments
@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

@bacciotti
Copy link
Contributor Author

/retest

Removing image qe tag.
@bacciotti bacciotti requested a review from lcouzens March 7, 2024 11:59
@bacciotti bacciotti enabled auto-merge (squash) March 7, 2024 11:59
@lcouzens
Copy link
Contributor

lcouzens commented Mar 7, 2024

linty lint is sad

@samdoran samdoran changed the title [COST-4390] OCP changing GB to GiB. [COST-4390] OCP changing GB to GiB Mar 7, 2024
samdoran
samdoran previously approved these changes Mar 7, 2024
@bacciotti bacciotti disabled auto-merge March 8, 2024 15:41
lcouzens
lcouzens previously approved these changes Mar 8, 2024
@esebesto
Copy link
Contributor

Tested locally - clean pass with full smokes

@bacciotti bacciotti enabled auto-merge (squash) March 12, 2024 09:57
@bacciotti bacciotti merged commit 8a7245b into main Mar 12, 2024
10 of 11 checks passed
@bacciotti bacciotti deleted the COST4390-ocp-units branch March 12, 2024 10:00
@myersCody myersCody added the customer-impact This PR impacts the customer label Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-impact This PR impacts the customer ocp-smoke-tests pr_check will build the image and run ocp + ocp on [clouds] smoke tests smokes-required
Projects
None yet
6 participants