summaryrefslogtreecommitdiffstats
path: root/utils
Commit message (Collapse)AuthorAgeFilesLines
* 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
* | Handle updating of scheduler config during upgradeJason DeTiberus2016-12-061-1/+1
| | | | | | | | | | | | | | | | | | | | | | - do not upgrade predicates if openshift_master_scheduler_predicates is defined - do not upgrade priorities if openshift_master_scheduler_priorities is defined - do not upgrade predicates/priorities unless they match known previous default configs - output WARNING to user if predictes/priorities are not updated during install
* | test updatesJason DeTiberus2016-12-062-4/+10
|/ | | | | | - fix nose coverage flags - add coverage support for files tested outside of the utils directory - exclude stdlib and virtualenv installed dependencies
* update tests and flake8/pylint fixesJason DeTiberus2016-11-295-23/+26
|
* Merge pull request #2783 from tbielawa/aoioutputTim Bielawa2016-11-101-0/+8
|\ | | | | Enable quiet output for all a-o-i commands
| * Enable quiet output for all a-o-i commandsTim Bielawa2016-11-101-0/+8
| | | | | | | | | | | | | | * Update: uninstall, upgrade, scaleup wf's * Verbosity can be toggled with -v flag, as before https://bugzilla.redhat.com/show_bug.cgi?id=1384294
* | Update override cluster_hostnameSamuel Munilla2016-11-101-1/+3
|/ | | | | | Update the default for the override prompt and set both openshift_master_cluster_hostname and openshift_master_cluster_public_hostname
* Merge pull request #2729 from tbielawa/BZ1390064Scott Dodson2016-11-081-4/+4
|\ | | | | Bug 1390064 - [quick install] a complete installed cluster was reported as a mix of installed and uninstalled env
| * Change the logic to just compare against masters and nodes.Tim Bielawa2016-11-071-4/+4
| |
* | Merge pull request #2735 from rhcarvalho/deadcodeScott Dodson2016-11-073-8/+0
|\ \ | | | | | | Remove some dead code
| * | a-o-i: remove dummy data_fileRodolfo Carvalho2016-11-052-6/+0
| | |
| * | a-o-i: remove script leftover from OpenShift v2Rodolfo Carvalho2016-11-051-2/+0
| |/
* / Add some tests for utils to get the coverage up.Tim Bielawa2016-11-041-1/+29
|/
* Update quick installer upgrade mappings for 3.4Samuel Munilla2016-11-031-0/+12
| | | | | | Just a quick addition for the upgrade playbooks for 3.4 Fixes Bug 1384291
* Merge pull request #2683 from smunilla/BZ1388754Scott Dodson2016-11-031-6/+22
|\ | | | | Update master_lb vs cluster_hostname workflow
| * Update master_lb vs cluster_hostname workflowSamuel Munilla2016-10-281-6/+22
| | | | | | | | | | | | Previously, the messaging around setting cluster_hostname for single master setups was confusing. This should help with that as well as removing unecessary values from the cfg file.