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

Create travis file for eventing with power support #764

Merged
merged 1 commit into from
Mar 16, 2021

Conversation

nagesh4193
Copy link
Contributor

No description provided.

Copy link
Contributor

@gururajrkatti gururajrkatti left a comment

Choose a reason for hiding this comment

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

Can we not run End-to-end tests (e2e) ? Rest all LGTM

@nagesh4193
Copy link
Contributor Author

@gururajrkatti I have added a script for test and updated the travis file. Please review

Copy link
Contributor

@gururajrkatti gururajrkatti left a comment

Choose a reason for hiding this comment

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

Pl. refer to https://github.com/knative/eventing/blob/main/test/README.md and check if you are able to run e2e tests

@nagesh4193
Copy link
Contributor Author

@gururajrkatti I have updated the script with tag e2e. The tests are passing only for go test unit tests, but for running other tests, it is not working as the prerequisites does not yet support private registries. Can you please review it?

@nagesh4193
Copy link
Contributor Author

Guru, As discussed, this is the link knative/eventing#1862 where it gives the limitations from the community.

Copy link
Contributor

@gururajrkatti gururajrkatti left a comment

Choose a reason for hiding this comment

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

ok

@seth-priya seth-priya merged commit 96efecc into ppc64le:master Mar 16, 2021
@nagesh4193 nagesh4193 deleted the nages_eventing branch March 24, 2021 04:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants