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

Images used for verifying e2e tests should be built from source #1700

Closed
srinivashegde86 opened this issue Jul 25, 2018 · 5 comments
Closed
Assignees
Labels
area/test-and-release It flags unit/e2e/conformance/perf test issues for product features kind/bug Categorizes issue or PR as related to a bug.

Comments

@srinivashegde86
Copy link
Contributor

/area test-and-release
/kind bug
/kind dev

Expected Behavior

We should be building and pushing the image used by the e2e tests from source. So, whenever someone makes changes, we can upload the new image and test against that.

Actual Behavior

When engineers update one of the e2e tests that need an be build into a docker image like tha hello world or the autoscaler tests, they need to depend on one of the test/OWNERS to push the image to our e2e tests docker repo to start testing their changes. Else, prow will only test using the old image and fail continously.

Sample for source to url -> https://github.com/knative/docs/tree/master/serving/samples/source-to-url-go

@google-prow-robot google-prow-robot added area/test-and-release It flags unit/e2e/conformance/perf test issues for product features kind/bug Categorizes issue or PR as related to a bug. labels Jul 25, 2018
@srinivashegde86
Copy link
Contributor Author

I think we can accomplish this with ko.

ko publish github.com/knative/serving/test/e2e/test_images/autoscale publishes the to KO_DOCKER_REPO/github.com/knative/serving/test/e2e/test_images/autoscale

@srinivashegde86 srinivashegde86 self-assigned this Jul 26, 2018
@srinivashegde86
Copy link
Contributor Author

Tried using this, but is failing with permission error when uploading images when I run the e2e-tests.sh script locally.

@jessiezcc jessiezcc assigned steuhs and unassigned srinivashegde86 Jul 26, 2018
@adrcunha
Copy link
Contributor

adrcunha commented Aug 2, 2018

It should work with e2e-tests.sh if you publish to DOCKER_REPO_OVERRIDE, because it's guaranteed that the current user has (exclusive) write access to it.

@mattmoor
Copy link
Member

I believe we call ./test/upload-images.sh in our e2e tests, so what is missing?

@mattmoor mattmoor added this to the Ice Box milestone Nov 28, 2018
@adrcunha
Copy link
Contributor

Nothing, this is a dupe of #1720.

@dprotaso dprotaso removed this from the Ice Box milestone Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/test-and-release It flags unit/e2e/conformance/perf test issues for product features kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

6 participants