Skip to content
This repository was archived by the owner on Mar 25, 2024. It is now read-only.

Adding e2e Testing #53

Merged
merged 2 commits into from
Jul 16, 2019
Merged

Adding e2e Testing #53

merged 2 commits into from
Jul 16, 2019

Conversation

luthermonson
Copy link
Contributor

Adding e2e testing using k3s to the build process. See ./e2e/README.md for more details on how to run and develop with the e2e tests. Additionally added some bug fixes for cleanup and retries which were found while running these tests.

@dramich
Copy link
Contributor

dramich commented Jul 16, 2019

lgtm knowing this won't be ran locally and would require further work later to make that work. #54

@luthermonson
Copy link
Contributor Author

closes #38

@@ -15,7 +15,7 @@ fi
echo Running validation

echo Running: golangci-lint
golangci-lint run
golangci-lint run --deadline 10m

echo Running: go fmt
Copy link

Choose a reason for hiding this comment

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

Should these be swapped? Run gofmt first since golangci may take much longer?

Copy link

@daxmc99 daxmc99 left a comment

Choose a reason for hiding this comment

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

LGTM

@dramich dramich merged commit b14c391 into rancher:master Jul 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants