Skip to content

Make environment variable consistent#663

Merged
istio-testing merged 1 commit intoistio-ecosystem:mainfrom
yannuil:e2e-test-script-fix
Feb 18, 2025
Merged

Make environment variable consistent#663
istio-testing merged 1 commit intoistio-ecosystem:mainfrom
yannuil:e2e-test-script-fix

Conversation

@yannuil
Copy link
Copy Markdown
Contributor

@yannuil yannuil commented Feb 18, 2025

What type of PR is this?

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

What this PR does / why we need it:

VERSIONS_YAML_FILE is used to referenced only the filename any other places in codebase. This PR makes the usage consistent.

Signed-off-by: Yann Liu <yannliu@redhat.com>
@yannuil yannuil requested a review from a team as a code owner February 18, 2025 09:20
@istio-testing
Copy link
Copy Markdown
Collaborator

Hi @yannuil. Thanks for your PR.

I'm waiting for a istio-ecosystem or istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@yannuil
Copy link
Copy Markdown
Contributor Author

yannuil commented Feb 18, 2025

/cherry-pick release-1.0

@istio-testing
Copy link
Copy Markdown
Collaborator

@yannuil: only istio-ecosystem org members may request cherry picks. If you are already part of the org, make sure to change your membership to public. Otherwise you can still do the cherry-pick manually.

Details

In response to this:

/cherry-pick release-1.0

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.57%. Comparing base (13a1029) to head (8d8cddb).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #663      +/-   ##
==========================================
+ Coverage   74.41%   74.57%   +0.15%     
==========================================
  Files          42       42              
  Lines        2584     2584              
==========================================
+ Hits         1923     1927       +4     
+ Misses        568      565       -3     
+ Partials       93       92       -1     

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

@fjglira
Copy link
Copy Markdown
Contributor

fjglira commented Feb 18, 2025

/cherry-pick release-1.0

@istio-testing
Copy link
Copy Markdown
Collaborator

@fjglira: once the present PR merges, I will cherry-pick it on top of release-1.0 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-1.0

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@fjglira
Copy link
Copy Markdown
Contributor

fjglira commented Feb 18, 2025

/ok-to-test

@istio-testing istio-testing merged commit e6c2390 into istio-ecosystem:main Feb 18, 2025
@istio-testing
Copy link
Copy Markdown
Collaborator

@fjglira: new pull request created: #664

Details

In response to this:

/cherry-pick release-1.0

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@yannuil yannuil deleted the e2e-test-script-fix branch February 18, 2025 14:17
FilipB added a commit to FilipB/sail-operator that referenced this pull request Feb 24, 2025
* upstream-eco/main:
  Ensure e2e tests use the correct versions yaml file (istio-ecosystem#667)
  Make environment  variable consistent (istio-ecosystem#663)
  Add contributor meeting information to the contributor.md file (istio-ecosystem#623)
  Add version alias support  (istio-ecosystem#566)
zmiklank pushed a commit to zmiklank/sail-operator that referenced this pull request Mar 16, 2026
Co-authored-by: openshift-service-mesh-bot <null>
dgn pushed a commit to dgn/sail-operator that referenced this pull request Mar 17, 2026
Signed-off-by: Yann Liu <yannliu@redhat.com>
Signed-off-by: Daniel Grimm <dgrimm@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants