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

Use PULL_NUMBER in e2e tests scripts #487

Merged
merged 1 commit into from
Jun 7, 2022

Conversation

mpryc
Copy link
Collaborator

@mpryc mpryc commented Jun 7, 2022

CI has one shell env variable PULL_NUMBER which should be used while running e2e tests.

This change adds such check and ensures the PULL_NUMBER Is actual number before modyfing values.yaml file.

@redhat-cop/mdt

CI has one shell env variable PULL_NUMBER which should be used while
running e2e tests.
@weshayutin
Copy link
Contributor

/LGTM

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 7, 2022
@weshayutin
Copy link
Contributor

worked for me :) Thanks @mpryc

@weshayutin weshayutin merged commit 58a5f6a into dora-metrics:master Jun 7, 2022
mpryc added a commit to mpryc/pelorus that referenced this pull request Jun 10, 2022
CI has one shell env variable PULL_NUMBER which should be used while
running e2e tests.

This is same PR as dora-metrics#487, however another merge reverted it, so
we need to have this one.
weshayutin pushed a commit that referenced this pull request Jun 10, 2022
CI has one shell env variable PULL_NUMBER which should be used while
running e2e tests.

This is same PR as #487, however another merge reverted it, so
we need to have this one.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants