Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Move nested print_read_config_error function into it's own function | Tim Bielawa | 2016-08-24 | 3 | -7/+4 |
| | |||||
* | Makefile includes ci-pyflakes target now | Tim Bielawa | 2016-08-24 | 1 | -1/+7 |
| | |||||
* | Fix BZ1368296 by quietly recollecting facts if the cache is removed | Tim Bielawa | 2016-08-24 | 9 | -43/+220 |
| | | | | | * Add python logging * Add testing system via 'make ci' | ||||
* | Merge pull request #2334 from smunilla/BZ1358951 | Scott Dodson | 2016-08-23 | 5 | -76/+88 |
|\ | | | | | Bug 1358951 - Error loading config, no such key: 'deployment' when using ing previously valid answers file | ||||
| * | a-o-i: Remove Legacy Config Upgrade | Samuel Munilla | 2016-08-19 | 4 | -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 Munilla | 2016-08-19 | 3 | -3/+37 |
| | | | | | | | | | | | | | | previously valid answers file Update the quick installer to automatically convert from the old config file format to the new format. | ||||
* | | Merge pull request #2333 from smunilla/fix_ssh_question | Scott Dodson | 2016-08-23 | 1 | -5/+4 |
|\ \ | | | | | | | a-o-i: Fix ansible_ssh_user question | ||||
| * | | a-o-i: Fix ansible_ssh_user question | Samuel Munilla | 2016-08-23 | 1 | -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 #2339 from dgoodwin/plugin-order-warning | Scott Dodson | 2016-08-23 | 1 | -0/+13 |
|\ \ | | | | | | | Add warning at end of 3.3 upgrade if pluginOrderOverride is found. | ||||
| * | | Add warning at end of 3.3 upgrade if pluginOrderOverride is found. | Devan Goodwin | 2016-08-22 | 1 | -0/+13 |
| | | | |||||
* | | | Merge pull request #2348 from dgoodwin/ded-etcd-fix | Scott Dodson | 2016-08-23 | 2 | -1/+5 |
|\ \ \ | | | | | | | | | Dedicated Etcd Upgrade Fix | ||||
| * | | | Don't run node config upgrade hook if host is not a node. | Devan Goodwin | 2016-08-23 | 1 | -1/+1 |
| | | | | |||||
| * | | | Better error if no OpenShift RPMs are available. | Devan Goodwin | 2016-08-23 | 1 | -0/+4 |
| | | | | |||||
* | | | | Merge pull request #2346 from abutcher/ca-bundle | Scott Dodson | 2016-08-23 | 1 | -0/+26 |
|\ \ \ \ | |/ / / |/| | | | Link ca to ca-bundle when ca-bundle does not exist. | ||||
| * | | | Link ca to ca-bundle when ca-bundle does not exist. | Andrew Butcher | 2016-08-23 | 1 | -0/+26 |
|/ / / | |||||
* | | | Merge pull request #2344 from openshift/revert-2325-fileglobglob | Scott Dodson | 2016-08-23 | 3 | -2/+2 |
|\ \ \ | | | | | | | | | Revert "Due to problems with with_fileglob lets avoid using it for now" | ||||
| * | | | Revert "Due to problems with with_fileglob lets avoid using it for now" | Scott Dodson | 2016-08-23 | 3 | -2/+2 |
|/ / / | |||||
* | | | Merge pull request #2340 from lhuard1A/ansible_virt_module | Jason DeTiberus | 2016-08-22 | 5 | -41/+34 |
|\ \ \ | | | | | | | | | Replace some virsh commands by native virt_XXX ansible module | ||||
| * | | | Replace some virsh commands by native virt_XXX ansible module | Lénaïc Huard | 2016-08-22 | 5 | -41/+34 |
| | | | | |||||
* | | | | Merge pull request #2336 from sdodson/etcd-uninstall | Scott Dodson | 2016-08-22 | 1 | -1/+4 |
|\ \ \ \ | |/ / / |/| | | | Fix etcd uninstall | ||||
| * | | | Fix etcd uninstall | Scott Dodson | 2016-08-19 | 1 | -1/+4 |
| | | | | |||||
* | | | | Merge pull request #2330 from lhuard1A/open_node_ports | Jason DeTiberus | 2016-08-19 | 4 | -0/+15 |
|\ \ \ \ | | | | | | | | | | | Open OpenStack security group for the service node port range | ||||
| * | | | | Open OpenStack security group for the service node port range | Lénaïc Huard | 2016-08-19 | 4 | -0/+15 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With OpenShift 3.2, creating a service accessible from the outside of the cluster thanks to `nodePort` automatically opens the “local” `iptables` firewall to allow incoming connection on the `nodePort` of the service. In order to benefit from this improvement, the OpenStack security group shouldn’t block those incoming connections. This change opens, on the OS nodes, the port range dedicated to service node ports. | ||||
* | | | | | Merge pull request #2329 from lhuard1A/fix_nodeonmaster | Jason DeTiberus | 2016-08-19 | 1 | -1/+1 |
|\ \ \ \ \ | |_|/ / / |/| | | | | Fix the “node on master” feature | ||||
| * | | | | Fix the “node on master” feature | Lénaïc Huard | 2016-08-19 | 1 | -1/+1 |
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | What we want to do is to add the master as a node if: * `g_nodeonmaster` is set to true, and * we are not in the case where we want to add new nodes. The second test was done by only checking whether `g_new_node_hosts` was defined. This was wrong because, in all cloud-provider setups, this variable was set with the default value of “empty list” (`[]`). The test has been changed to use the `bool` filter so that it correctly evaluates to false (and hence, effectively add the master as a node) when `g_new_node_hosts` is the empty list. | ||||
* | | | | Merge pull request #2325 from sdodson/fileglobglob | Scott Dodson | 2016-08-19 | 3 | -2/+2 |
|\ \ \ \ | | | | | | | | | | | Due to problems with with_fileglob lets avoid using it for now | ||||
| * | | | | Due to problems with with_fileglob lets avoid using it for now | Scott Dodson | 2016-08-18 | 3 | -2/+2 |
| | | | | | |||||
* | | | | | Merge pull request #2331 from dgoodwin/docker-upgrade-symlink | Scott Dodson | 2016-08-19 | 1 | -0/+1 |
|\ \ \ \ \ | | |_|/ / | |/| | / | |_|_|/ |/| | | | Fix standalone Docker upgrade missing symlink. | ||||
| * | | | Fix standalone Docker upgrade missing symlink. | Devan Goodwin | 2016-08-19 | 1 | -0/+1 |
| | | | | |||||
* | | | | Automatic commit of package [openshift-ansible] release [3.3.13-1]. | Troy Dawson | 2016-08-19 | 2 | -2/+8 |
| | | | | |||||
* | | | | Merge pull request #2327 from lhuard1A/openstack_ansible_2.1 | Scott Dodson | 2016-08-19 | 4 | -25/+27 |
|\ \ \ \ | | | | | | | | | | | Fix warnings in OpenStack provider with ansible 2.1 | ||||
| * | | | | Fix warnings in OpenStack provider with ansible 2.1 | Lénaïc Huard | 2016-08-19 | 4 | -25/+27 |
| | |_|/ | |/| | | | | | | | | | | Some expressions now need to be enclosed inside `{{…}}`. | ||||
* | | | | Merge pull request #2323 from sdodson/bz1367937 | Scott Dodson | 2016-08-18 | 1 | -1/+1 |
|\ \ \ \ | |/ / / |/| | | | Mount /sys rw | ||||
| * | | | Mount /sys rw | Scott Dodson | 2016-08-18 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1367937 | ||||
* | | | | Merge pull request #2319 from openshift/sdodson-patch-3 | Scott Dodson | 2016-08-17 | 1 | -1/+1 |
|\ \ \ \ | | | | | | | | | | | Update uninstall.yml | ||||
| * | | | | Update uninstall.yml | Scott Dodson | 2016-08-17 | 1 | -1/+1 |
|/ / / / | | | | | | | | | Fixes #2317 | ||||
* | | | | Merge pull request #2314 from sdodson/bz1356823 | Scott Dodson | 2016-08-17 | 1 | -1/+1 |
|\ \ \ \ | | | | | | | | | | | Fix padding on registry config | ||||
| * | | | | Fix padding on registry config | Scott Dodson | 2016-08-17 | 1 | -1/+1 |
| |/ / / | |||||
* | | | | Automatic commit of package [openshift-ansible] release [3.3.12-1]. | Troy Dawson | 2016-08-17 | 2 | -2/+23 |
| | | | | |||||
* | | | | Merge pull request #2303 from tpoitras/typos-cli_installer.py | Scott Dodson | 2016-08-17 | 2 | -80/+78 |
|\ \ \ \ | |/ / / |/| | | | Edits to fix to typos, grammar, and product branding in cli_installer | ||||
| * | | | Fixes to typos, grammar, and product branding in cli_installer | Timothy Poitras | 2016-08-17 | 2 | -80/+78 |
| | | | | |||||
* | | | | Merge pull request #2309 from smunilla/fix_nosetests_3.3 | Scott Dodson | 2016-08-16 | 4 | -6/+14 |
|\ \ \ \ | | | | | | | | | | | a-o-i: Fix nosetests after removing 3.2 from installer | ||||
| * | | | | a-o-i: Fix nosetests after removing 3.2 from installer | Samuel Munilla | 2016-08-16 | 4 | -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. | ||||
* | | | | | Merge pull request #2310 from dgoodwin/reconcile-fix | Scott Dodson | 2016-08-16 | 1 | -43/+46 |
|\ \ \ \ \ | |_|_|/ / |/| | | | | Reconcile roles after master upgrade, but before nodes. | ||||
| * | | | | Reconcile roles after master upgrade, but before nodes. | Devan Goodwin | 2016-08-16 | 1 | -43/+46 |
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prevents the network egress bug causing node restart to fail during 3.3 upgrade. (even though a separate fix is incoming for this) Only catch is preventing the openshift_cli role, which requires docker, from triggering a potential upgrade, which we still don't want at this point. To avoid we use the same variable to protect docker installed version as we use in pre.yml. | ||||
* | | | | Merge pull request #2220 from jkhelil/fix_key_error_when_ssh_failing | Scott Dodson | 2016-08-16 | 3 | -3/+5 |
|\ \ \ \ | |/ / / |/| | | | fixing openshift key error in case of node failure during run (ssh is… | ||||
| * | | | fixing openshift key error in case of node failure during run (ssh issue) | jawed | 2016-08-09 | 3 | -3/+5 |
| | | | | |||||
* | | | | Merge pull request #2307 from smunilla/BZ1367323 | Scott Dodson | 2016-08-16 | 1 | -1/+0 |
|\ \ \ \ | | | | | | | | | | | Bug 1367323 - the "OpenShift Container Platform 3.2" variant is still listed when quick install ose-3.3 | ||||
| * | | | | Bug 1367323 - the "OpenShift Container Platform 3.2" variant is still listed ↵ | Samuel Munilla | 2016-08-16 | 1 | -1/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when quick install ose-3.3 Remove 3.2 from the install options in 3.3 installer. | ||||
* | | | | | Merge pull request #2306 from smunilla/BZ1367199 | Scott Dodson | 2016-08-16 | 1 | -1/+1 |
|\ \ \ \ \ | |/ / / / |/| | | | | Bug 1367199 - iptablesSyncPeriod should default to 30s OOTB |