summaryrefslogtreecommitdiffstats
path: root/utils
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #6488 from vrutkovs/utils-test-flake8OpenShift Merge Robot2018-01-184-155/+173
|\ | | | | | | | | Automatic merge from submit-queue. Fix flake8 errors in utils/test
| * Fix flake8 errors in utils/testVadim Rutkovsky2018-01-044-155/+173
| |
* | Don't use from ansible.module_utils.six as its no longer available in ↵Vadim Rutkovsky2018-01-121-5/+1
|/ | | | Ansible 2.4
* Remove all references to pacemaker (pcs, pcsd) and ↵Andrew Butcher2017-12-051-2/+0
| | | | | | | openshift.master.cluster_method. With pacemaker removed there is no longer a need for openshift.master.cluster_method. We only have one option.
* Playbook Consolidation - openshift-nodeRussell Teague2017-11-211-1/+1
|
* Merge pull request #6091 from sosiouxme/20171110-ini-parse-failureOpenShift Merge Robot2017-11-141-0/+4
|\ | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. ansible.cfg: error when inventory does not parse As of Ansible 2.4 this option exists: https://github.com/ansible/ansible/issues/15035#issuecomment-329672168 The kind of error you get later when this is your real problem tends to be *very* confusing. Much better to just fail right away. I cannot think of any scenario in which it would be preferable to proceed when inventory parsing fails.
| * ansible.cfg: error when inventory does not parseLuke Meyer2017-11-101-0/+4
| | | | | | | | | | | | | | | | As of Ansible 2.4 this option exists: https://github.com/ansible/ansible/issues/15035#issuecomment-329672168 The kind of error you get later when this is your real problem tends to be *very* confusing. Much better to just fail right away.
* | Remove upgrade playbooks for 3.3 through 3.5Russell Teague2017-11-081-12/+6
|/
* Merge pull request #2910 from richm/preserve-ansible_inventory_pathOpenShift Merge Robot2017-10-311-2/+4
|\ | | | | | | | | Automatic merge from submit-queue. preserve the oo-install ansible_inventory_path value
| * preserve the oo-install ansible_inventory_path valueRich Megginson2017-05-181-2/+4
| |
* | Cleanup old deployment typesMichael Gugino2017-09-201-1/+0
| | | | | | | | | | | | | | | | | | | | | | Previously, openshift-ansible supported various types of deployments using the variable "openshift_deployment_type" Currently, openshift-ansible only supports two deployment types, "origin" and "openshift-enterprise". This commit removes all logic and references to deprecated deployment types.
* | Display "origin 3.6" as in previous installer 3.5Bruno Vernay2017-08-181-1/+1
| | | | | | | | Previous installer release v3.5 used to display the 3 variants. I keep OSE as the default, but proposes to add back the "origin v3.6" variant.
* | Quick Installer should specify which config file to edit.jkaurredhat2017-08-041-1/+1
| | | | | | | | | | | | Fixes bugzilla : https://bugzilla.redhat.com/show_bug.cgi?id=1460353 Signed-off-by: jkaurredhat <jkaur@redhat.com>
* | Merge pull request #3877 from wkshi/bugfix-1422541-masterScott Dodson2017-07-271-0/+2
|\ \ | | | | | | Fix BZ#1422541 BZ#1440586 BZ#1440612 BZ#1440617 on master branch
| * | fix BZ1422541 on master branchWenkai Shi2017-04-071-0/+2
| | |
* | | ansible.cfg: improve ssh ControlPathLuke Meyer2017-07-111-0/+6
| |/ |/| | | | | | | | | | | | | | | | | | | AWS hosts and user names frequently make the default Ansible ControlPath too long. This prevents ssh persistent connections, significantly impacting runtime. More detail: https://github.com/ansible/ansible/issues/11536 This config change shortens the ControlPath to make this less likely. It would be better to change it to the %C hash, as extremely long hostnames or usernames could still encounter this problem, but that is not yet available with RHEL's openssh. This at least improves the situation.
* | Fix variants for v3.6Russell Teague2017-05-021-3/+4
| | | | | | | | Fixes bug 1447204
* | Standardize all Origin versioning on 3.6Russell Teague2017-04-251-0/+6
| |
* | Support unicode output when dumping yamlRussell Teague2017-04-031-4/+10
|/
* Merge pull request #3569 from detiber/consolidateToxJason DeTiberus2017-03-068-254/+3
|\ | | | | Consolidate root/utils tests
| * Update docs for test consolidation and remove the MakefileJason DeTiberus2017-03-062-163/+3
| |
| * Consolidate root/utils testsJason DeTiberus2017-03-066-91/+0
| | | | | | | | - Consolidate tests between the root of the repo and utils
* | Remove commented-out test codeRodolfo Carvalho2017-03-061-39/+0
|/
* Remove old commented-out testsRodolfo Carvalho2017-03-061-91/+0
|
* 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.
* Remove unused Makefile variablesRodolfo Carvalho2017-03-061-5/+0
|
* Fix Quick Installer failed due to a Python method failureTim Bielawa2017-02-241-1/+6
| | | | | | | | | | Fixes issues dumping various objects to YAML. This change updates the quick installer fact caching callback to use the improved AnsibleDumper when writing out the facts to storage. * https://bugzilla.redhat.com/show_bug.cgi?id=1421053 * https://bugzilla.redhat.com/show_bug.cgi?id=1421061 * https://bugzilla.redhat.com/show_bug.cgi?id=1421059
* Update link to project homepageRodolfo Carvalho2017-02-211-1/+1
|
* Remove pytest-related dependencies from setup.pyRodolfo Carvalho2017-02-211-3/+0
| | | | | | | | | | | | | The `setup_requires` line prevents builds in internal environments without Internet access / pythonX-pytest-runner RPM installed. In fact, we're running tests with `pytest`, outside of `setup.py`, so we don't need those dependencies there. Note: we decided not to run tests through `python setup.py pytest` because pytest-runner was limited in how we can pass arguments to pytest / enable plugins. E.g., I could not get the coverage plugin working when running `python setup.py pytest`.
* Raise the bar on coverage requirementsRodolfo Carvalho2017-02-201-1/+1
|
* Accept extra positional arguments in toxRodolfo Carvalho2017-02-201-1/+1
| | | | | | Example: tox -e py35-ansible22-unit -- --collect-only
* Replace nose with pytest (utils)Rodolfo Carvalho2017-02-206-17/+42
|
* Clean up utils/README.mdRodolfo Carvalho2017-02-201-25/+16
| | | | Remove old way of running tests, plus mirror edits.
* Remove dead code in installerRodolfo Carvalho2017-02-192-35/+0
| | | | | Found using `vulture`, confirmed that code that was once used became unused in later commits after its introduction.
* Update variant_versionSteve Milner2017-02-141-1/+2
| | | | 3.5 added and 3.4 moved to LEGACY
* Create v3_5 upgrade playbooksRussell Teague2017-01-301-24/+6
|
* flake8 mccabe dependency fixRussell Teague2017-01-251-5/+2
|
* Temporary work-around for flake8 vs maccabe version conflictTim Bielawa2017-01-241-0/+2
|
* cleanup: Removed debug prints from testsSteve Milner2017-01-112-2/+0
|
* remove debug statement from testJason DeTiberus2017-01-101-1/+0
|
* increase test coverageJason DeTiberus2017-01-101-1/+1
|
* More toxificationJason DeTiberus2017-01-106-26/+57
| | | | | | | | | | | | - Move pylint tests to tox - Move yamllint tests to tox - Create separate tox config (and setup.py) for root - bump ansible requirement - unify pylint config - add docs - remove git directory containing old testing tools - install python-six if not present for openshift-facts - add python-six as a dependency for openshift-ansible-utils
* add test for utils to bump coverageJason DeTiberus2017-01-101-0/+73
|
* Merge pull request #3061 from tbielawa/scaleup-bz1390135Scott Dodson2017-01-091-0/+14
|\ | | | | The scaleup subcommand does not support the unattended option
| * The scaleup subcommand does not support the unattended optionTim Bielawa2017-01-091-0/+14
| | | | | | | | | | | | * 'atomic-openshift-installer' will exit now if -u is given with the 'scaleup' subcommand * https://bugzilla.redhat.com/show_bug.cgi?id=1390135
* | Merge pull request #3021 from tbielawa/fix_aoi_manTim Bielawa2017-01-093-6/+7
|\ \ | |/ |/| Update aoi manpage with correct operation count
| * Update manpage version.Tim Bielawa2017-01-092-3/+4
| |
| * Update aoi manpage with correct operation countTim Bielawa2016-12-282-4/+4
| |
* | Hush the sudo privs check in oo-installerTim Bielawa2017-01-041-1/+1
|/ | | | | | | * Don't print words when checking for sudo access * Just 'echo' an empty string with no new-line Closes #3003