summaryrefslogtreecommitdiffstats
path: root/utils
Commit message (Collapse)AuthorAgeFilesLines
* Add missing pip requirement to virtualenvTim Bielawa2016-08-261-1/+1
|
* Fix PyLint errors discovered when upgrading to newer versionTim Bielawa2016-08-265-32/+25
| | | | | | | | | | | | | | | | | | | | | | | * Fixes PyLint to run in the virtualenv used for all tests * Replaced 'LooseVersion' with 'parse_version' from setuptools - This is a work around for the issue in https://github.com/PyCQA/pylint/issues/73 in which pylint can not import disutils.version correctly in a virtualenv. * Removed the unused function 'delete_hosts' which was causing a pylint error as well * Removed a deprecated pylint pragma option, 'bad-builtin' * Fixed some import ordering issues it was picky about * Added another disable for a case where the PyLint suggestion would have us altering the container we would be iterating over * Add code-coverage reports to the unittests with the MINIMUM coverage percentage for success set to 70% - Current test coverage is at 76%
* Apply indentation changes to some other linesTim Bielawa2016-08-261-4/+4
| | | | Closes #2363
* Enable PEP8 tests by default in the 'make ci' target nowTim Bielawa2016-08-251-9/+1
|
* Fix PEP8 errors in cli_installer.pyTim Bielawa2016-08-251-70/+90
|
* Fix PEP8 in openshift_ansible.pyTim Bielawa2016-08-251-18/+20
|
* Fix PEP8 in oo_config.pyTim Bielawa2016-08-251-6/+3
|
* Fix PEP8 in variants.pyTim Bielawa2016-08-251-11/+13
|
* Fix PEP8 in facts_callback.pyTim Bielawa2016-08-251-4/+5
|
* Merge pull request #2341 from tbielawa/BZ1368296Scott Dodson2016-08-2410-48/+232
|\ | | | | BZ1368296 - quick install with the installer.cfg.yml in other directory failed
| * Make note about ansible/install logs messing up ci testsTim Bielawa2016-08-241-0/+4
| |
| * Move nested print_read_config_error function into it's own functionTim Bielawa2016-08-243-7/+4
| |
| * Makefile includes ci-pyflakes target nowTim Bielawa2016-08-241-1/+7
| |
| * Fix BZ1368296 by quietly recollecting facts if the cache is removedTim Bielawa2016-08-249-43/+220
| | | | | | | | | | * Add python logging * Add testing system via 'make ci'
* | a-o-i: Add variant and variant_version to migrationSamuel Munilla2016-08-241-0/+6
|/ | | | Update to also move over variant and variant_version during v1 migration
* Merge pull request #2334 from smunilla/BZ1358951Scott Dodson2016-08-235-76/+88
|\ | | | | Bug 1358951 - Error loading config, no such key: 'deployment' when using ing previously valid answers file
| * a-o-i: Remove Legacy Config UpgradeSamuel Munilla2016-08-194-74/+52
| | | | | | | | | | Remove the automatic upgrade of the 3.0 configuration file format and add some better error messaging that points the user at the docs.
| * Bug 1358951 - Error loading config, no such key: 'deployment' when using ↵Samuel Munilla2016-08-193-3/+37
| | | | | | | | | | | | | | previously valid answers file Update the quick installer to automatically convert from the old config file format to the new format.
* | a-o-i: Fix ansible_ssh_user questionSamuel Munilla2016-08-231-5/+4
|/ | | | | Since we moved 'ansible_ssh_user' under 'deployment', we need to update the criteria for when to ask the user to reenter that information.
* Merge pull request #2303 from tpoitras/typos-cli_installer.pyScott Dodson2016-08-172-80/+78
|\ | | | | Edits to fix to typos, grammar, and product branding in cli_installer
| * Fixes to typos, grammar, and product branding in cli_installerTimothy Poitras2016-08-172-80/+78
| |
* | a-o-i: Fix nosetests after removing 3.2 from installerSamuel Munilla2016-08-164-6/+14
| | | | | | | | | | Update the tests to match 3.3 being the only version. Add support for non-latest if it's specified in the config file.
* | Bug 1367323 - the "OpenShift Container Platform 3.2" variant is still listed ↵Samuel Munilla2016-08-161-1/+0
|/ | | | | | when quick install ose-3.3 Remove 3.2 from the install options in 3.3 installer.
* a-o-i: Fix broken tests from installed hosts checkSamuel Munilla2016-08-151-9/+13
| | | | | | With the latest changes to behavior around uninstalled hosts during a scaleup, nosetests were broken. This updates the tests to verify that we exit when we are expecting.
* Merge pull request #2297 from abutcher/bz#1329455Scott Dodson2016-08-151-22/+18
|\ | | | | a-o-i: Bug1329455, [quick-install] Cannot add new nodes in pre-existing env
| * a-o-i: fix bz#1329455Gan Huang2016-08-151-22/+18
| |
* | a-o-i: Mapping for 3.2 UpgradesSamuel Munilla2016-08-151-0/+6
| | | | | | | | Update the playbook mappings for 3.2
* | Merge pull request #2201 from richm/v1.3Scott Dodson2016-08-151-0/+1
|\ \ | |/ |/| add 3.3 to installer
| * add 3.3 to installerRich Megginson2016-07-311-0/+1
| |
* | a-o-i: Fix broken upgradesSamuel Munilla2016-08-122-6/+5
| | | | | | | | | | | | | | Upgrade was looking for the hosts in the old location. Quick fix to get it looking in the correct location. Fixes BZ#1357627.
* | a-o-i: Automatically Label Nodes as InfraSamuel Munilla2016-08-111-0/+11
| | | | | | | | | | | | | | In interactive mode, automatically label nodes as infrastructure nodes. Two cases are covered: 1) If all nodes are masters, all hosts are labeled infra 2) If dedicated nodes are defined, the first two (or one) nodes are labeled as infra
* | a-o-i: Restrict installed host checkSamuel Munilla2016-08-101-19/+18
| | | | | | | | Restrict installed host check to only masters and nodes.
* | Merge pull request #2278 from smunilla/BZ1353162Scott Dodson2016-08-095-15/+15
|\ \ | | | | | | a-o-i: Move ansible_ssh_user to deployment
| * | a-o-i: Update nosetests for ansible_ssh_userSamuel Munilla2016-08-093-10/+11
| | | | | | | | | | | | | | | Update the nosetests to reflect the new location of ansible_ssh_user in the config file.
| * | move ansible_ssh_user to deployment, remove ansible_config and ansible_log_pathGan Huang2016-08-092-5/+4
| | |
* | | Labeling nodes onlyGan Huang2016-08-091-1/+2
|/ /
* | Convert ansible facts callback to v2.Andrew Butcher2016-08-082-24/+30
| |
* | a-o-i: Rename OSE in Install MenuSamuel Munilla2016-08-041-1/+1
| | | | | | | | Rename 'OpenShift Enterprise' to 'OpenShift Container Platform' in the quick installer menu.
* | a-o-i: Allow Arbitrary Deployment VariablesSamuel Munilla2016-08-031-13/+11
| | | | | | | | | | This allows for arbitrary ansible variables to be set under the 'deployment' header of the installer config file.
* | a-o-i: Fix broken uninstallSamuel Munilla2016-08-022-5/+9
| | | | | | | | | | | | | | | | The uninstall method was looking for the host list according to the old quick-installer config file format. Updated to match the new arbitrary yaml config format. Fixes BZ#1359427
* | a-o-i: Remove AEP, OSE 3.0, and OSE 3.2 choicesSamuel Munilla2016-07-302-34/+5
|/ | | | | | Remove the option to select unsupported versions in the quick installer. Fixes BZ#1358101
* a-o-i: Better inventory group handlingSamuel Munilla2016-07-293-72/+49
| | | | | | | | | | A more dynamic and flexible method of writing out host groups to the ansible inventory file. To accompany this, in the quick-installer interactive mode the etcd role is automatically applied to any masters. Anyone hand writing the quick installer config will now have to explicitly specify which hosts will be etcd. Fixes #2200
* a-o-i: Set roles on standalone storageSamuel Munilla2016-07-281-3/+1
| | | | | | Sets the roles appropriately when using a standalone storage host. Fixes BZ#1353152
* Disable too many branches pylintScott Dodson2016-07-271-0/+1
|
* a-o-i: write missing openshift_node_labelsdkorn2016-07-271-0/+2
|
* Merge pull request #2228 from smunilla/arbitrary_host_varsScott Dodson2016-07-272-1/+8
|\ | | | | a-o-i: Support for arbitrary host-level variables
| * a-o-i: Support for arbitrary host-level variablesSamuel Munilla2016-07-272-1/+8
| | | | | | | | | | This allows the user to set a block containing any variables they want to set per-host instead of per-role.
* | a-o-i: Move inventory vars to the correct locationSamuel Munilla2016-07-271-16/+18
|/ | | | | | Several variables such as 'deployment_type' and 'ansible_config' were being set under a variable group for the last defined role instead of under OSEv3:vars.
* a-o-i: Persist Roles VariablesSamuel Munilla2016-07-211-0/+3
| | | | Previously, we we're saving the roles variables set during a run.
* Merge pull request #2170 from smunilla/unattended_required_factsScott Dodson2016-07-201-8/+1
|\ | | | | a-o-i: Looser facts requirements for unattended