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

Fix prow execution #50

Merged
merged 1 commit into from
May 24, 2023
Merged

Conversation

electrocucaracha
Copy link
Member

@electrocucaracha electrocucaracha commented May 24, 2023

The gce_run.sh bash script is executed from the e2e folder, which causes some issues during the prow execution. This change modifies that behavior to run it from its own folder. This also deprecates the usage of E2E_instance.txt file.

Signed-off-by: Victor Morales <v.morales@samsung.com>
skip_report: false
decorate: true
cluster: default
extra_refs:
Copy link
Collaborator

Choose a reason for hiding this comment

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

why are we removing this?

Copy link
Member Author

Choose a reason for hiding this comment

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

That action is redundant, we're cloning the repo during the execution of gce_run.sh script.

@@ -17,15 +17,10 @@ presubmits:
- name: e2e
annotations:
labels:
run_if_changed: 'E2E_instance.txt'
run_if_changed: '^e2e/'
Copy link
Collaborator

Choose a reason for hiding this comment

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

curious as to what the interpretation of ^e2e/ is?

Copy link
Member Author

Choose a reason for hiding this comment

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

My understanding is that this job will be triggered on any change under the e2e folder

@vjayaramrh
Copy link
Collaborator

/lgtm
/approve

@vjayaramrh
Copy link
Collaborator

/lgtm

@vjayaramrh
Copy link
Collaborator

@aravind254 would appreciate a review and lgtm when you get a chance, Thanks

@vjayaramrh
Copy link
Collaborator

/retest

@gvbalaji
Copy link

/lgtm

@radoslawc
Copy link
Collaborator

/approve

@nephio-prow
Copy link
Contributor

nephio-prow bot commented May 24, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: electrocucaracha, radoslawc, vjayaramrh

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@nephio-prow nephio-prow bot added the approved label May 24, 2023
@nephio-prow nephio-prow bot merged commit e78ee88 into nephio-project:main May 24, 2023
@electrocucaracha electrocucaracha deleted the fix_prow branch May 24, 2023 23:21
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