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

hack: setup and teardown scripts for e2e envs #590

Merged
merged 1 commit into from
Dec 3, 2021

Conversation

flavianmissi
Copy link
Collaborator

@flavianmissi flavianmissi commented Nov 25, 2021

includes 3 shell scripts:

  • hack/deploy.sh: deploys the operator and a quayregistry instance, and ensures everything is up and running. will exit 1 otherwise
  • hack/teardown.sh: removes everything created by deploy.sh
  • hack/storage.sh: installs the openshift container storage from the red hat catalog and creates a noobaa instance

note that some functions, like info, are duplicated in different files. since they are trivial, I didn't see much of a problem with it, at least for now.

Fixes PROJQUAY-2823

@flavianmissi flavianmissi changed the title hack: provide scripts to setup and teardown e2e environments hack: setup and teardown scripts for e2e envs Nov 25, 2021
dmage
dmage previously approved these changes Nov 25, 2021
hack/deploy.sh Outdated Show resolved Hide resolved
@flavianmissi
Copy link
Collaborator Author

e2e tests have started failing after some point in this PR (https://github.com/flavianmissi/quay-operator/runs/4332403123?check_suite_focus=true).
Please hold approaval until I have figured out what's up.

@flavianmissi
Copy link
Collaborator Author

Problem was that the line below, in hack/deploy.sh was missing the namespace, so it worked locally but not in the CI.

endpoint="$(oc get quayregistry skynet -o jsonpath='{.status.registryEndpoint}')"

After adding the namespace the new run is successful: https://github.com/flavianmissi/quay-operator/runs/4332862425?check_suite_focus=true
@dmage this is one is ready to go now, please have another look at your earliest convenience 🚀

hack/deploy.sh Show resolved Hide resolved
hack/deploy.sh Show resolved Hide resolved
Copy link
Contributor

@ricardomaraschini ricardomaraschini left a comment

Choose a reason for hiding this comment

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

/lgtm

@flavianmissi flavianmissi merged commit ca6d86d into quay:master Dec 3, 2021
@flavianmissi flavianmissi deleted the run-e2e-locally branch December 3, 2021 14:02
flavianmissi added a commit to flavianmissi/quay-operator that referenced this pull request Dec 6, 2021
* hack/deploy.sh: deploys the operator and a quayregistry instance, and ensures everything is up and running. will exit 1 otherwise
* hack/teardown.sh: removes everything created by deploy.sh
* hack/storage.sh: installs the openshift container storage from the red hat catalog and creates a noobaa instance

Fixes PROJQUAY-2823
flavianmissi added a commit that referenced this pull request Dec 6, 2021
* hack/deploy.sh: deploys the operator and a quayregistry instance, and ensures everything is up and running. will exit 1 otherwise
* hack/teardown.sh: removes everything created by deploy.sh
* hack/storage.sh: installs the openshift container storage from the red hat catalog and creates a noobaa instance

Fixes PROJQUAY-2823
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants