summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Augmenting documentation for router sharding.Kenny Woodson2017-03-071-0/+44
|
* openshift_facts: ensure system containers deps are installedGiuseppe Scrivano2017-03-072-0/+14
| | | | Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
* Merge pull request #3517 from ewolinetz/idempotency_role_bindingsEric Wolinetz2017-03-061-4/+20
|\ | | | | Adding changed_whens for role, rolebinding, and scc reconciliation ba…
| * Updating stdout check for changed_whenewolinetz2017-03-021-5/+5
| |
| * Resolving yammlint errorsewolinetz2017-03-011-8/+8
| |
| * Adding changed_whens for role, rolebinding, and scc reconciliation based on ↵ewolinetz2017-02-281-4/+20
| | | | | | | | output from oadm policy command
* | Enable coveralls.ioJason DeTiberus2017-03-062-1/+5
| |
* | Merge pull request #3575 from sdodson/BZ1423430Scott Dodson2017-03-061-1/+1
|\ \ | | | | | | Fix indentation of run_once
| * | Fix indentation of run_onceScott Dodson2017-03-061-1/+1
|/ / | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1423430
* | Merge pull request #3569 from detiber/consolidateToxJason DeTiberus2017-03-0616-279/+44
|\ \ | | | | | | Consolidate root/utils tests
| * | Update docs for test consolidation and remove the MakefileJason DeTiberus2017-03-063-169/+6
| | |
| * | Consolidate root/utils testsJason DeTiberus2017-03-0613-110/+38
| | | | | | | | | | | | - Consolidate tests between the root of the repo and utils
* | | Merge pull request #3570 from rhcarvalho/more-misc-cleanupJason DeTiberus2017-03-0616-324/+0
|\ \ \ | | | | | | | | Remove lots of dead code in tests
| * | | Remove dummy setup/teardown methodsRodolfo Carvalho2017-03-0613-104/+0
| | | |
| * | | Clean up test filesRodolfo Carvalho2017-03-0615-181/+0
| | | | | | | | | | | | | | | | | | | | They are not executable anymore, and tests are now meant to be run through pytest.
| * | | Remove commented-out test codeRodolfo Carvalho2017-03-061-39/+0
|/ / /
* | | Merge pull request #3535 from enj/enj/f/reserve_ns_moduleScott Dodson2017-03-068-578/+1051
|\ \ \ | | | | | | | | Add pre-upgrade check for reserved namespaces
| * | | Make generic OCObjectValidator from OCSDNValidatorMonis Khan2017-03-068-578/+1051
|/ / / | | | | | | | | | Signed-off-by: Monis Khan <mkhan@redhat.com>
* | | Merge pull request #3567 from richm/logging-needs-openshift_master_factsScott Dodson2017-03-061-0/+1
|\ \ \ | | | | | | | | logging needs openshift_master_facts before openshift_facts
| * | | logging needs openshift_master_facts before openshift_factsRich Megginson2017-03-061-0/+1
| |/ /
* | | Merge pull request #3382 from codificat/playbook2imageScott Dodson2017-03-065-20/+105
|\ \ \ | |/ / |/| | Dockerfile and docs to run containerized playbooks
| * | Dockerfile and docs to run containerized playbooksPep Turró Mauri2017-03-065-20/+105
| | | | | | | | | | | | | | | | | | | | | | | | Update openshift-ansible's Dockerfile to use playbook2image as a base, with the goal to run an arbitrary playbook from a container. The existing Dockerfile is moved to Dockerfile.rhel7 for the productized version and will be updated to use playbook2image later.
* | | Merge pull request #3566 from detiber/testConfigsJason DeTiberus2017-03-064-28/+23
|\ \ \ | |/ / |/| | separate out test tool configs from setup.cfg
| * | separate out test tool configs from setup.cfgJason DeTiberus2017-03-064-28/+23
| | | | | | | | | | | | | | | Since we are moving away from setuptools for invoking tests, lets move the configs for the different test tools into their own configs.
* | | Merge pull request #3549 from kwoodson/router_shardingScott Dodson2017-03-062-61/+54
|\ \ \ | | | | | | | | Adding support for multiple router shards.
| * | | Added ports.Kenny Woodson2017-03-022-0/+6
| | | |
| * | | Fixed router name to produce 2nd router.Kenny Woodson2017-03-021-1/+1
| | | |
| * | | Updated to work with an array of routers.Kenny Woodson2017-03-022-61/+19
| | | |
| * | | Adding support for router sharding.Kenny Woodson2017-03-022-61/+90
| | | |
* | | | Merge pull request #3325 from kwoodson/oc_projectKenny Woodson2017-03-069-1/+2280
|\ \ \ \ | |_|/ / |/| | | Adding oc_project to lib_openshift.
| * | | Lower test coverage percentage.Kenny Woodson2017-03-061-1/+1
| | | |
| * | | Mock runs differntly on travis. Fix the mock test params to be ANY.Kenny Woodson2017-03-061-3/+2
| | | |
| * | | Fixed the none namespace. Fixed tests with latest loc_oc_binary call.Kenny Woodson2017-03-063-5/+10
| | | |
| * | | Updating the namespace param to None.Kenny Woodson2017-03-062-2/+2
| | | |
| * | | Regenerated code with latest yedit changes.Kenny Woodson2017-03-061-4/+11
| | | |
| * | | Fixed tests to align with new naming.Kenny Woodson2017-03-061-1/+1
| | | |
| * | | Fixed docs. Added check for delete failures. Updated namespace to None.Kenny Woodson2017-03-063-49/+144
| | | |
| * | | Fixing lintersKenny Woodson2017-03-063-4/+3
| | | |
| * | | Adding integration test. Fixed issue with node_selector.Kenny Woodson2017-03-063-8/+97
| | | |
| * | | Adding oc_project to lib_openshift.Kenny Woodson2017-03-067-0/+2085
| | | |
* | | | Merge pull request #3563 from rhcarvalho/misc-cleanupRodolfo Carvalho2017-03-069-140/+48
|\ \ \ \ | |/ / / |/| | | Misc cleanup
| * | | Remove old commented-out testsRodolfo Carvalho2017-03-061-91/+0
| | | |
| * | | Remove redundant assertionRodolfo Carvalho2017-03-061-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | 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.
| * | | Fix testRodolfo Carvalho2017-03-061-7/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of checking if a string is True, check if 'found' is True, the string is the error message. Also, we can remove the loop and use the simpler Python 'in' construct.
| * | | Lint utils/testRodolfo Carvalho2017-03-064-26/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Do not use `print` in unit tests, send messages through the test framework instead. - Remove unused import. - Add spaces around equal sign in assigment. - Turn method into a function. - Reorganize imports according to PEP8.
| * | | Rewrap long linesRodolfo Carvalho2017-03-062-8/+22
| | | |
| * | | Remove unused argumentRodolfo Carvalho2017-03-061-1/+1
| | | | | | | | | | | | | | | | Detected by pylint. The fixture indeed doesn't require an argument.
| * | | Remove unused Makefile variablesRodolfo Carvalho2017-03-061-5/+0
|/ / /
* | | Merge pull request #3556 from dcbw/sdn-preserve-var-lib-cniScott Dodson2017-03-032-2/+2
|\ \ \ | | | | | | | | node/sdn: make /var/lib/cni persistent to ensure IPAM allocations stick around across node restart
| * | | node/sdn: make /var/lib/cni persistent to ensure IPAM allocations stick ↵Dan Williams2017-03-032-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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