Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #2730 from dgoodwin/fix-ha-etcd-backup | Scott Dodson | 2016-11-09 | 1 | -1/+14 |
|\ | | | | | Fix HA etcd upgrade when facts cache has been deleted. | ||||
| * | Fix etcd backup failure due to corrupted facts. | Devan Goodwin | 2016-11-09 | 1 | -1/+14 |
| | | | | | | | | | | | | | | | | Simplest way to reproduce this issue is to attempt to upgrade having removed /etc/ansible/facts.d/openshift.fact. Actual cause in the field is not entirely known but critically it is possible for embedded_etcd to default to true, causing the etcd fact lookup to check the wrong file and fail silently, resulting in no etcd_data_dir fact being set. | ||||
* | | Revert "Revert openshift.node.nodename changes" | Scott Dodson | 2016-11-08 | 6 | -188/+40 |
| | | |||||
* | | Merge pull request #2751 from sdodson/BZ1388253 | Scott Dodson | 2016-11-08 | 6 | -40/+188 |
|\ \ | | | | | | | Revert openshift.node.nodename changes | ||||
| * | | Revert "Fix the nodeName of the OpenShift nodes on OpenStack" | Scott Dodson | 2016-11-07 | 3 | -25/+175 |
| | | | | | | | | | | | | This reverts commit aaaf82ba6032d0b1e9c36a39a7eda25b8c5f4b84. | ||||
| * | | Revert "Fix OpenStack cloud provider" | Scott Dodson | 2016-11-07 | 3 | -15/+13 |
| |/ | | | | | | | This reverts commit 1f2276fff1e41c1d9440ee8b589042ee249b95d7. | ||||
* / | Add missing symlink for node openvswitch oom fix. | Devan Goodwin | 2016-11-08 | 1 | -0/+1 |
|/ | |||||
* | Remove unused playbooks adhoc metrics_setup files #2717 | Stéphane Klein | 2016-11-07 | 6 | -249/+0 |
| | |||||
* | Where we use curl force it to use tlsv1.2 | Scott Dodson | 2016-11-02 | 2 | -2/+2 |
| | | | | | | | curl, prior to RHEL 7.2, did not properly negotiate up the TLS protocol, so force it to use tlsv1.2 Fixes bug 1390869 | ||||
* | Merge pull request #2670 from abutcher/hosted-registry-insecure | Scott Dodson | 2016-10-26 | 1 | -1/+1 |
|\ | | | | | Bug 1388016 - The insecure-registry address was removed during upgrade | ||||
| * | Default hosted_registry_insecure true when insecure registry present in ↵ | Andrew Butcher | 2016-10-26 | 1 | -1/+1 |
| | | | | | | | | existing /etc/sysconfig/docker. | ||||
* | | Merge pull request #2662 from blues-man/update_doc_upgrade_openshift | Scott Dodson | 2016-10-25 | 1 | -2/+2 |
|\ \ | | | | | | | Update link to latest versions upgrade README | ||||
| * | | Update link to latest versions upgrade README | Natale Vinto | 2016-10-25 | 1 | -2/+2 |
| |/ | |||||
* | | Merge pull request #2663 from dgoodwin/34-upgrade | Scott Dodson | 2016-10-25 | 5 | -0/+313 |
|\ \ | | | | | | | Add support for 3.4 upgrade. | ||||
| * | | Add support for 3.4 upgrade. | Devan Goodwin | 2016-10-25 | 5 | -0/+313 |
| |/ | | | | | | | | | This is a direct copy of 3.3 upgrade playbooks, with 3.3 specific hooks removed and version numbers adjusted appropriately. | ||||
* | | Merge pull request #2637 from dgoodwin/ctrl-plane-restart | Scott Dodson | 2016-10-25 | 3 | -12/+9 |
|\ \ | | | | | | | Fix and reorder control plane service restart. | ||||
| * | | Fix and reorder control plane service restart. | Devan Goodwin | 2016-10-21 | 3 | -12/+9 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This was missed in the standalone upgrade control plane playbook. However it also looks to be out of order, we should restart before reconciling and upgrading nodes. As such moved the restart directly into the control plane upgrade common code, and placed it before reconciliation. | ||||
* | | | [upgrades] Fix containerized node | Scott Dodson | 2016-10-25 | 1 | -2/+0 |
| |/ |/| | | | | | This file was removed and no longer used | ||||
* | | Move infrastructure templates into openshift_hosted_templates role. | Andrew Butcher | 2016-10-21 | 2 | -0/+6 |
| | | |||||
* | | trouble creating service signer while running upgrade dockerized | Henning Fjellheim | 2016-10-21 | 1 | -4/+4 |
| | | |||||
* | | Merge pull request #2623 from rhcarvalho/typos | Andrew Butcher | 2016-10-20 | 3 | -3/+3 |
|\ \ | | | | | | | Fix typos | ||||
| * | | Fix typos | Rodolfo Carvalho | 2016-10-19 | 3 | -3/+3 |
| | | | |||||
* | | | Merge pull request #2627 from dgoodwin/remove-pacemaker-restarts | Scott Dodson | 2016-10-20 | 4 | -132/+3 |
|\ \ \ | | | | | | | | | Drop pacemaker restart logic. | ||||
| * | | | Drop pacemaker restart logic. | Devan Goodwin | 2016-10-19 | 4 | -132/+3 |
| | |/ | |/| | | | | | | | | | | Pacemaker clusters are no longer supported, and in some cases bugs here were causing upgrade failures. | ||||
* | | | Merge pull request #2621 from dgoodwin/symlink-fix | Scott Dodson | 2016-10-19 | 5 | -17/+13 |
|\ \ \ | |_|/ |/| | | Switch from "oadm" to "oc adm" and fix bug in binary sync. | ||||
| * | | Switch from "oadm" to "oc adm" and fix bug in binary sync. | Devan Goodwin | 2016-10-19 | 5 | -17/+13 |
| |/ | | | | | | | | | | | | | | | Found bug syncing binaries to containerized hosts where if a symlink was pre-existing, but pointing to the wrong destination, it would not be corrected. Switched to using oc adm instead of oadm. | ||||
* | | [logging] Fix NFS volume binding | Scott Dodson | 2016-10-17 | 1 | -6/+6 |
| | | |||||
* | | Merge pull request #2608 from abutcher/template-with-items | Scott Dodson | 2016-10-17 | 25 | -44/+44 |
|\ \ | |/ |/| | Template with_items for upstream ansible-2.2 compat. | ||||
| * | Template with_items for upstream ansible-2.2 compat. | Andrew Butcher | 2016-10-14 | 25 | -44/+44 |
| | | |||||
* | | Merge pull request #2605 from sdodson/logging | Scott Dodson | 2016-10-14 | 1 | -6/+6 |
|\ \ | |/ |/| | [logging] Use inventory variables rather than facts | ||||
| * | Use inventory variables rather than facts | Scott Dodson | 2016-10-14 | 1 | -6/+6 |
| | | |||||
* | | Resume restarting node after upgrading node rpms. | Devan Goodwin | 2016-10-14 | 1 | -0/+4 |
| | | | | | | | | | | | | | | Error in commit 245fef16573757b6e691c448075d8564f5d569f4. As it turns out this is the only place a rpm based node can be restarted in upgrade. Restoring the restart but making it conditional to avoid the two issues reported with out of sync node restarts. | ||||
* | | upgrade: Don't check avail docker version if not already installed. | Devan Goodwin | 2016-10-14 | 1 | -2/+4 |
| | | |||||
* | | Merge pull request #2598 from detiber/2547 | Scott Dodson | 2016-10-13 | 1 | -2/+2 |
|\ \ | | | | | | | update handling of use_dnsmasq | ||||
| * | | update handling of use_dnsmasq | Jason DeTiberus | 2016-10-13 | 1 | -2/+2 |
| | | | |||||
* | | | Merge pull request #2599 from dgoodwin/node-docker-upgrade | Scott Dodson | 2016-10-13 | 1 | -0/+2 |
|\ \ \ | |_|/ |/| | | Fix standalone docker upgrade playbook skipping nodes. | ||||
| * | | Fix standalone docker upgrade playbook skipping nodes. | Devan Goodwin | 2016-10-13 | 1 | -0/+2 |
| |/ | | | | | | | | | Transition to being able to specify nodes to upgrade caused standalone nodes to get skipped in this playbook. | ||||
* / | Stop restarting node after upgrading master rpms. | Devan Goodwin | 2016-10-12 | 1 | -3/+0 |
|/ | | | | | | | | | | This looks to be causing a customer issue where some HA upgrades fail, due to a missing EgressNetworkPolicy API. We update master rpms, we don't restart services yet, but then restart node service which tries to talk to an API that does not yet exist. (pending restart) Restarting node here is very out of place and appears to not be required. | ||||
* | Set master facts for first master in node scaleup. | Andrew Butcher | 2016-10-05 | 1 | -0/+11 |
| | |||||
* | Fix default port typo. | Andrew Butcher | 2016-10-05 | 1 | -2/+2 |
| | |||||
* | Merge pull request #2511 from vishpat/nuage_ha | Andrew Butcher | 2016-10-04 | 1 | -0/+11 |
|\ | | | | | Changes for Nuage HA | ||||
| * | Filterize haproxy frontends/backends and add method for providing additional ↵ | Andrew Butcher | 2016-10-03 | 1 | -0/+11 |
| | | | | | | | | frontends/backends. | ||||
* | | Merge pull request #2441 from dgoodwin/34-upgrade-improvements | Andrew Butcher | 2016-10-04 | 22 | -465/+741 |
|\ \ | | | | | | | 3.4 Upgrade Improvements | ||||
| * | | Fix bug with service signer cert on upgrade. | Devan Goodwin | 2016-09-29 | 2 | -1/+7 |
| | | | | | | | | | | | | | | | | | | It is invalid Ansible to use a when on an include that contains plays, as it cannot be applied to plays. Issue filed upstream for a better error, or to get it working. | ||||
| * | | Use pre_upgrade tag instread of a dry run variable. | Devan Goodwin | 2016-09-29 | 4 | -20/+71 |
| | | | |||||
| * | | Move etcd backup from pre-upgrade to upgrade itself. | Devan Goodwin | 2016-09-29 | 4 | -91/+86 |
| | | | |||||
| * | | Allow a couple retries when unscheduling/rescheduling nodes in upgrade. | Devan Goodwin | 2016-09-29 | 1 | -0/+12 |
| | | | | | | | | | | | | | | | | | | | | | | | | This can fail with a transient "object has been modified" error asking you to re-try your changes on the latest version of the object. Allow up to three retries to see if we can get the change to take effect. | ||||
| * | | Skip the docker role in early upgrade stages. | Devan Goodwin | 2016-09-29 | 5 | -17/+24 |
| | | | | | | | | | | | | | | | | | | | | | | | | This improves the situation further and prevents configuration changes from accidentally triggering docker restarts, before we've evacuated nodes. Now in two places, we skip the role entirely, instead of previous implementation which only skipped upgrading the installed version. (which did not catch config issues) | ||||
| * | | Allow filtering nodes to upgrade by label. | Devan Goodwin | 2016-09-29 | 11 | -29/+71 |
| | | | |||||
| * | | Allow customizing node upgrade serial value. | Devan Goodwin | 2016-09-29 | 1 | -1/+3 |
| | | |