K3s travis #7
Merged
K3s travis #7
Conversation
This file contains 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
…try.centos.org This fixes k3s being unable to download it from registry.centos.org, which is due to either the fact that it requires the docker:// v1 protocol, or it being an insecure registry. It seems to be the same image, but newer by 1 month & a small diff in size. Both by CentOS themselves. Same args. We cannot use the RHEL8 postgres image because it is publicly searchable but not publicly accessible. Note that an attempt was tried to use docker://registry.centos.org as the image name, but it randomly stopped working (STATUS: InvalidImageName)
Test basic Pulp functionality, including writing to disk Using pulp_file's sync tests. Also, don't set CONTENT_HOST in the pulp-server ConfigMap. If set incorrectly, it causes pulp_file test scripts to fail. We haven't implemented querying it via the K8s APIs, or having users specify it yet. Note that the RequestsDependencyWarning messages from httpie are annoying output, but pip3 is not on Ubuntu on Travis. It appears that Travis relies on the virtual env for Python3 fixes #5061 https://pulp.plan.io/issues/5061
Add different ConfigMaps for known use cases like Travis Partially implements: ref #5137 https://pulp.plan.io/issues/5137
Partially implements: ref #5136 https://pulp.plan.io/issues/5136
dkliban
approved these changes
Jul 23, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
I developed an alternative solution for:
https://pulp.plan.io/issues/5061
Develop a script to launch 4 pulp containers in just Docker/Podman
In order to implement this, I had to partially implement:
https://pulp.plan.io/issues/5136
pulp-operator needs a stable & configurable accessible port
https://pulp.plan.io/issues/5137
pulp-operator needs configurable and expandable storage size
And some other changes like the postgresql image change.
I also cleaned up some code.
The scripts that content plugin devs would run (in Travis as early phases like
install, which Travis collapses the output of) for testing their plugins in containers are:They would also depend on a minority of other files in the repo existing. I think only the
deploydirectory.Note that pulp-operator's image is not automatically built and used. Doing so is task #5174 Currently, the developer has to build and push (per README.md).
Also, as soon as this is merged, I need to do a separate PR to have deploy/operator.yml use the
latesttag rather than thek3s-travistag. Or I need to build the image and push it with the latest tag after review, and do the commit as part of this PR. (This limitation will be addressed later as part of #5174 & #5175.)