This repository was archived by the owner on Nov 9, 2022. It is now read-only.
This repository is currently being migrated. It's locked while the migration is in progress.
e2e: Run operator locally with go run#178
Open
darkowlzz wants to merge 1 commit intostorageos:masterfrom
Open
Conversation
This runs the e2e operator-sdk tests (non-OLM) with go run, instead of running as a container. Container image is built only for OLM bases test setup. Also, uses the new kind load command to upload the container image from host to a KinD environment.
darkowlzz
force-pushed
the
e2e-up-local
branch
from
6f84771 to
2c932eb
Compare
Contributor
Author
|
There may be an issue when running the operator locally which results in the node-controller failing to update the node labels. |
Contributor
|
LGTM but failing test? |
domodwyer
approved these changes
Oct 7, 2019
Contributor
Author
|
When the operator is run this way, it's operating from outside the KinD environment. The operator controllers cannot connect to the storageos server running inside the KinD environment. This works fine for openshift test job as openshift is deployed directly on the host. A workaround for this would be to create a proxy to storageos server inside KinD from the host. We already do something similar in the e2e test for node controller tests. We can move the proxy creation before running any test. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This runs the e2e operator-sdk tests (non-OLM) with go run, instead of
running in a container.
Container image is built only for OLM based test setup.
Also, uses the new kind load command to upload the container image from
host to a KinD environment.