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

Ignore istio version suffix used in the downstream builds #446

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

FilipB
Copy link
Collaborator

@FilipB FilipB commented Oct 23, 2024

What type of PR is this?

  • Enhancement / New Feature
  • Bug Fix
  • Refactor
  • Optimization
  • Test
  • Documentation Update

What this PR does / why we need it:

The test is expecting to see the same version parsed from versions.yaml and a version string returned by pilot-discovery version. This is not the same in downstream builds as we use the _ossm suffix.

The test is expecting to see the same version parsed from versions.yaml
and a version string returned by `pilot-discovery version`.
This is not the same in downstream builds as we use the `_ossm` suffix.

Signed-off-by: Filip Brychta <[email protected]>
@FilipB FilipB requested a review from a team as a code owner October 23, 2024 12:29
Copy link

codecov bot commented Oct 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.73%. Comparing base (06bb204) to head (3689c63).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #446      +/-   ##
==========================================
- Coverage   75.93%   75.73%   -0.20%     
==========================================
  Files          35       35              
  Lines        2061     2061              
==========================================
- Hits         1565     1561       -4     
- Misses        425      428       +3     
- Partials       71       72       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@pbajjuri20 pbajjuri20 left a comment

Choose a reason for hiding this comment

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

LGTM..!

eg: In Version:"1.23.0-snapshot", it will capture 1.23.0 and stop before the underscore.

@FilipB
Copy link
Collaborator Author

FilipB commented Oct 25, 2024

/retest

3 similar comments
@FilipB
Copy link
Collaborator Author

FilipB commented Oct 25, 2024

/retest

@FilipB
Copy link
Collaborator Author

FilipB commented Oct 25, 2024

/retest

@FilipB
Copy link
Collaborator Author

FilipB commented Oct 25, 2024

/retest

@istio-testing istio-testing merged commit 16c3e03 into istio-ecosystem:main Oct 25, 2024
16 of 17 checks passed
openshift-service-mesh-bot pushed a commit to openshift-service-mesh-bot/sail-operator that referenced this pull request Oct 25, 2024
* upstream/main:
  Ignore istio version suffix used in the downstream builds (istio-ecosystem#446)
  Pin the version of OLM we use for e2e tests (istio-ecosystem#450)
openshift-service-mesh-bot pushed a commit to openshift-service-mesh-bot/sail-operator that referenced this pull request Oct 25, 2024
* upstream/main:
  Ignore istio version suffix used in the downstream builds (istio-ecosystem#446)
  Pin the version of OLM we use for e2e tests (istio-ecosystem#450)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants