| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|\
| |
| |
| |
| | |
EricMountain-1A/fix_upstream_docker_registries_order
Preserve order of Docker registries
|
| |
| |
| |
| |
| |
| | |
List of additional/blocked/insecure Docker registries was being converted
to a set, thereby dropping the original order. Fixed to preserve order
of the registries (this allows prioritising registries).
|
|/
|
|
| |
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
|
| |
|
|
|
|
|
| |
They are not executable anymore, and tests are now meant to be run
through pytest.
|
|
|
|
| |
Signed-off-by: Monis Khan <mkhan@redhat.com>
|
| |
|
|\
| |
| | |
Adding support for multiple router shards.
|
| | |
|
| | |
|
| | |
|
| | |
|
|\ \
| | |
| | | |
Adding oc_project to lib_openshift.
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
That line is testing Python's list.count method, instead of yedit.
The assertion right above is a superset of it, as it checks for
equality to some expected value.
|
| | | |
|
|/ /
| |
| |
| | |
Detected by pylint. The fixture indeed doesn't require an argument.
|
|\ \
| | |
| | | |
node/sdn: make /var/lib/cni persistent to ensure IPAM allocations stick around across node restart
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
around across node restart
With the move to a CNI plugin, docker no longer handles IPAM, but CNI does through
openshift-sdn's usage of the 'host-local' CNI IPAM plugin. That plugin stores
IPAM allocations under /var/lib/cni/.
If the node container gets restarted, without presreving /var/lib/cni, the IPs
currently allocated to running pods get lost and on restart, openshift-sdn
may allocate those IPs to new pods causing duplicate allocations.
This never happened with docker because it has its own persistent IPAM store that
does not get removed when docker restarts. Also because (historically) when docker
restarted, all the containers died and the IP allocations were released by the
daemon.
Fix this by ensuring that IPAM allocations (which are tied to the life of the pod,
*not* the life of the openshift-node process) persist even if the openshift-node
process restarts.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1427789
|
| | |
| | |
| | |
| | | |
Just rely on PyYAML as a fallback and hope that's there
|
|/ / |
|
|\ \
| | |
| | | |
raise exceptions when walking through object path
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
if we're given path a.b.c and the existing object is:
a:
b:
- item1
raise an exception due to unexpected objects found while traversing the path (ie. b is a list, not a dict)
also, add_entry assumes new dicts for each sub element when creating elements besides the final assignment value.
doing something like a.b.c[0] = 12 where 'c' doesn't exist raises an exception
add test cases to cover:
access path that differs from existing object
create new objects with an embedded list in the path
create new object with a list at the end (define the end list in the passed in 'value' to avoid this exception)
|
|\ \ \
| | | |
| | | | |
Removing the openshift_master_facts dependency
|
| | | | |
|
|\ \ \ \
| |_|_|/
|/| | | |
Only set ownership to etcd for thirdparty datadir
|
| |/ /
| | |
| | |
| | | |
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1428532
|
|\ \ \
| | | |
| | | | |
bug 1420256. Initialize openshift_logging pvc_facts to empty
|
| | | | |
|
| | | | |
|
| |/ /
|/| | |
|
|\ \ \
| | | |
| | | | |
Fixing checkout for bindings with -binding suffix
|
| | | | |
|
|\ \ \ \
| |/ / /
|/| | | |
[wip] test fixes for openshift_certificates_expiry
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- create pytest fixtures for building certs at runtime
- update tests to use the fixtures
- add tests for load_and_handle_cert
- fix py2/py3 encode/decode issues raised by tests
- add get_extension_count method to fakeOpenSSLCertificate
- avoid using a temp file for passing ssl certificate to openssl
subprocess
- other test tweaks:
- exclude conftest.py and tests from coverage report
- reduce the fail_under to 26%, since the tests being included were
inflating our coverage
|
|/ / / |
|
|\ \ \
| |/ /
|/| | |
Fails Installation If Docker Version is Too Old On Atomic Host
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
If the docker version on a master or node is under 1.12 and the OS is an
Atomic Host variant the installation should fail.
Fixes: BZ1425583
|
|\ \ \
| | | |
| | | | |
Fixed error handling when oc adm ca create-server-cert fails. Fixed a logic error in secure.
|
| | | |
| | | |
| | | |
| | | | |
error in secure.
|