summaryrefslogtreecommitdiffstats
path: root/utils
Commit message (Collapse)AuthorAgeFilesLines
* 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.
* 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
* python3 support, add tox for better local testing against multiple python ↵Jason DeTiberus2016-12-2013-83/+102
| | | | versions
* YAML LintingRussell Teague2016-12-122-3/+14
| | | | | * Added checks to make ci for yaml linting * Modified y(a)ml files to pass lint checks
* Make 'cover-erase' a config file setting. Move VENT target to pre-req for ↵Tim Bielawa2016-12-092-6/+7
| | | | all ci-* targets
* Fixes to 'make ci'Tim Bielawa2016-12-091-14/+15
| | | | | | | | | | | | | * Fixed: A grep filter was capturing the actual pylint check rc's * Changed: pylint used to print out messages for locally-disabled and file-ignored items * Changed: pylint output format is now 'parseable' * Cleaned up: Pylint was emitting deprecation messages for 'required-attributes' and 'ignore-iface-methods'. They have been removed from the pylintrc file
* Fix coverage not appending new dataTim Bielawa2016-12-081-1/+1
| | | | | | | * Adds '--cover-erase' flag to the 'ci-unittests' target Fixes bug introduced in d7b4265f329591817bac381e2772ae14ac337cea: "Can't add arcs to existing line data" error message
* Merge pull request #2939 from tbielawa/gh2933Jason DeTiberus2016-12-071-6/+15
|\ | | | | Speed up 'make ci' and trim the output
| * Speed up 'make ci' and trim the outputTim Bielawa2016-12-071-6/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | The virtualenv is conditionally rebuilt now if the test requirements file is modified. This will save upwards of 30+ seconds in iterative 'make ci' runs. The pylint output is now trimmed to exclude disabled tests. The order of the 'ci' target prerequisites has been changed to run the fastest tests first. Closes #2933