Skip to content
This repository has been archived by the owner on Mar 9, 2022. It is now read-only.

Skip node e2e test in travis. #541

Merged
merged 1 commit into from Jan 12, 2018

Conversation

Random-Liu
Copy link
Member

Node e2e test always timeout in travis, because for free account travis has 50min timeout (see here).

Since we've already run node e2e as presubmit and CI in Kubernetes test infra, we don't need to run it in travis now.

I don't remove the code for now, because we may want to reuse it, e.g. in containerd repo in the future.

Signed-off-by: Lantao Liu lantaol@google.com

@abhi
Copy link
Member

abhi commented Jan 12, 2018

will this be misleading ? Coz on the checks it shows as succeeded. We can remove it and revert it based on history.

@Random-Liu
Copy link
Member Author

@abhi I'm fine with removing it. Will do it then.

Signed-off-by: Lantao Liu <lantaol@google.com>
@Random-Liu
Copy link
Member Author

@abhi Done.

Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

/LGTM

Sorry to see it go as it's nice to run these in as many environments as possible. Maybe open up an issue to to split up the node e2e bucket on travis into two parallel test runs?

Copy link
Member

@abhi abhi left a comment

Choose a reason for hiding this comment

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

LGTM
+1 on @mikebrow suggestion

@Random-Liu
Copy link
Member Author

Sorry to see it go as it's nice to run these in as many environments as possible. Maybe open up an issue to to split up the node e2e bucket on travis into two parallel test runs?

Will do.

@Random-Liu Random-Liu merged commit d23853e into containerd:master Jan 12, 2018
@Random-Liu Random-Liu deleted the skip-node-e2e branch January 12, 2018 18:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants