-
Notifications
You must be signed in to change notification settings - Fork 440
[no-relnote] Run e2e tests as reusable workflow #913
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
Conversation
b74e568 to
76b1b60
Compare
2d33574 to
3cf3118
Compare
deployments/container/native-only.mk
Outdated
| $(DEFAULT_PUSH_TARGET): | ||
| $(DOCKER) image inspect $(IMAGE) > /dev/null || $(DOCKER) pull $(IMAGE) | ||
| $(DOCKER) tag $(IMAGE) $(subst :$(IMAGE_TAG),:$(VERSION),$(IMAGE)) | ||
| endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: newline
| - "pull-request/[0-9]+" | ||
| - main | ||
| - release-* | ||
| workflow_call: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should there not also be a change to image.yaml?
03266c5 to
a251a5f
Compare
|
PR is ready for review |
.github/workflows/image.yaml
Outdated
| name: End-to-end Tests | ||
| uses: ./.github/workflows/e2e.yaml | ||
| secrets: inherit | ||
| needs: image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Newline:
| needs: image | |
| needs: image | |
elezar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor nit.
Signed-off-by: Carlos Eduardo Arango Gutierrez <eduardoa@nvidia.com>
a251a5f to
4ded119
Compare

No description provided.