summaryrefslogtreecommitdiffstats
path: root/roles
Commit message (Collapse)AuthorAgeFilesLines
* openshift_checks: refactor logging checksLuke Meyer2017-08-0215-616/+699
| | | | | | | Turn failure messages into exceptions that tests can look for without depending on text meant for humans. Turn logging_namespace property into a method. Get rid of _exec_oc and just use logging.exec_oc.
* openshift_checks: add property to track 'changed'Luke Meyer2017-08-0212-40/+44
| | | | | | | | | | Introduced the 'changed' property for checks that can make changes to track whether they did or not. Rather than the check's own logic having to track this and include it in the result hash, just set the property and have the action plugin insert it in the result hash after running (even if there is an exception). Cleared out a lot of crufty "changed: false" hash entries.
* Merge pull request #4968 from sdodson/cfme-imagesOpenShift Bot2017-08-022-1/+13
|\ | | | | Merged by openshift-bot
| * Use enterprise images for CFME enterprise deploymentsScott Dodson2017-08-012-1/+13
| |
* | Missing spaceKarthik Prabhakar2017-08-021-1/+1
| |
* | Merge pull request #4958 from sdodson/logging-metrics-imagetagScott Dodson2017-08-022-2/+2
|\ \ | | | | | | Switch logging and metrics OCP image tag from 3.6.0 to v3.6
| * | Switch logging and metrics OCP image tag from 3.6.0 to v3.6Scott Dodson2017-08-012-2/+2
| |/
* | Merge pull request #4592 from juanvallejo/jvallejo/verify-fluentd-logging-configOpenShift Bot2017-08-0211-109/+563
|\ \ | | | | | | Merged by openshift-bot
| * | add fluentd logging driver config checkjuanvallejo2017-08-0111-109/+563
| | |
* | | Merge pull request #4916 from richm/mux-client-modeOpenShift Bot2017-08-028-13/+27
|\ \ \ | | | | | | | | Merged by openshift-bot
| * | | use mux_client_mode instead of use_mux_clientRich Megginson2017-08-018-13/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of the `openshift_logging_use_mux_client` boolean parameter, use `openshift_logging_mux_client_mode` which will allow us to support different mux client use cases: The value `maximal` will cause Fluentd to perform as much of the processing as possible at the local node. This currently means all of the processing *except* for the Kubernetes metadata processing, which will be done by mux. This is the currently recommended mode to use due to current scaling issues. The value `minimal` means that Fluentd will do *no* processing at all, and send the raw logs to mux for processing. This is currently not recommended to use due to current scaling issues. Ansible will warn you if you try to use this mode. `MUX_ALLOW_EXTERNAL` is no longer needed in the mux dc. mux now always operates to process external logs. The ansible setting `openshift_logging_mux_allow_external` is still required in order to set up the mux service to accept connections from outside of the cluster.
* | | | Merge pull request #4949 from carlpett/patch-1OpenShift Bot2017-08-011-1/+1
|\ \ \ \ | | | | | | | | | | Merged by openshift-bot
| * | | | Fix aws_secret_key checkCalle Pettersson2017-07-311-1/+1
| | | | |
* | | | | Merge pull request #4961 from jarrpa/minor-namespace-fixScott Dodson2017-08-011-1/+1
|\ \ \ \ \ | | | | | | | | | | | | GlusterFS: Check for namespace if deploying a StorageClass
| * | | | | GlusterFS: Check for namespace if deploying a StorageClassJose A. Rivera2017-08-011-1/+1
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1476197 Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
* | | | | Merge pull request #4894 from tbielawa/reg-dns-scaleup-bz1469336OpenShift Bot2017-08-015-6/+16
|\ \ \ \ \ | |_|_|_|/ |/| | | | Merged by openshift-bot
| * | | | Use existing OPENSHIFT_DEFAULT_REGISTRY setting during masters scaleupTim Bielawa2017-07-285-6/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes a bug which reported that AFTER a cluster upgrade from OCP 3.5 to 3.6, any masters which are later added via the scaleup playbooks are setting a value for OPENSHIFT_DEFAULT_REGISTRY which is inconsistent with the already configured masters. * OPENSHIFT_DEFAULT_REGISTRY value is saved from oo_first_master * The new scaleup master has a fact set noting it is a scaleup host * The saved OPENSHIFT_DEFAULT_REGISTRY value is used instead of the default for 3.6, which is to use a hard-coded registry value of 'docker-registry.default.svc:5000' Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1469336
* | | | | Merge pull request #4700 from sdodson/rate-limitOpenShift Bot2017-08-011-4/+4
|\ \ \ \ \ | |_|/ / / |/| | | | Merged by openshift-bot
| * | | | Add rate limit configurabilityScott Dodson2017-07-061-4/+4
| | | | |
* | | | | Merge pull request #4912 from abutcher/openshift-node-factsOpenShift Bot2017-08-014-28/+50
|\ \ \ \ \ | | | | | | | | | | | | Merged by openshift-bot
| * | | | | Move node facts to new openshift_node_facts role.Andrew Butcher2017-07-274-28/+50
| | | | | |
* | | | | | Merge pull request #4900 from ↵OpenShift Bot2017-07-3116-151/+170
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | ingvagabund/move-common-openshift-node-systemd-unit-tasks-into-separete-files Merged by openshift-bot
| * | | | | | move common tasks to a single file included by both systemd_units.ymlJan Chaloupka2017-07-2816-151/+170
| | | | | | |
* | | | | | | Merge pull request #4952 from jarrpa/namespace_fixScott Dodson2017-07-311-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | GlusterFS: Fix variable names in defaults.
| * | | | | | | GlusterFS: Fix variable names in defaults.Jose A. Rivera2017-07-311-1/+1
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1476197 Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
* | | | | | | Merge pull request #4919 from srampal/contiv_1.1.1Scott Dodson2017-07-3110-6/+113
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | Some automation fixes for contiv GUI/auth_proxy & vxlan mode
| * | | | | | Fixes for auth_proxy, vxlan modeSanjeev Rampal2017-07-2710-6/+113
| | | | | | |
* | | | | | | Merge pull request #4933 from kwoodson/policy_to_bindingOpenShift Bot2017-07-296-92/+88
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | Merged by openshift-bot
| * | | | | | Remove cluster in favor of rolebindings.Kenny Woodson2017-07-286-92/+88
| | | | | | |
* | | | | | | Merge pull request #4943 from jcantrill/file_buffer_masterScott Dodson2017-07-286-0/+68
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Impl fluentd file buffer
| * | | | | | | Impl fluentd file bufferNoriko Hosoi2017-07-286-0/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (cherry picked from commit 231a7c4ce27ff944d9e14169062f487f57c94dd6)
* | | | | | | | Merge pull request #4918 from sdodson/fix-sysconfig-checkOpenShift Bot2017-07-281-2/+2
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Merged by openshift-bot
| * | | | | | | | Tolerate non existence of /etc/sysconfig/atomic-openshift-masterScott Dodson2017-07-271-2/+2
| | | | | | | | |
* | | | | | | | | Merge pull request #4931 from ewolinetz/bz1476195Scott Dodson2017-07-282-0/+21
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | | Updating metrics role to create serviceaccounts and roles immediately
| * | | | | | | | Updating metrics role to create serviceaccounts and roles immediatelyewolinetz2017-07-282-0/+21
| | |/ / / / / / | |/| | | | | |
* | | | | | | | Merge pull request #4879 from jcantrill/oc_pvcOpenShift Bot2017-07-287-10/+177
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Merged by openshift-bot
| * | | | | | | | add selector and storage class name to oc_pvc moduleJeff Cantrill2017-07-277-10/+177
| | |_|_|_|/ / / | |/| | | | | |
* | | | | | | | Merge pull request #4878 from mtnbikenc/refactor-bios-versionOpenShift Bot2017-07-281-5/+1
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | Merged by openshift-bot
| * | | | | | | Refactor openshift_facts BIOS vendor discoveryRussell Teague2017-07-261-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Ansible 2.4, `get_file_content` is moving from ansible.module_utils.facts to ansible.module_utils.facts.utils However, the method is no longer needed as the BIOS vendor fact can be retrieved from a standard Ansible fact, `ansible_system_vendor`. i.e.: "ansible_system_vendor": "Google",
* | | | | | | | GlusterFS: Use default namespace when not native.Jose A. Rivera2017-07-281-1/+1
| |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1476197 Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
* | | | | | | Merge pull request #4899 from rhcarvalho/misc-cleanup-normalize-inputOpenShift Bot2017-07-282-6/+19
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Merged by openshift-bot
| * | | | | | | Normalize list of checks passed to action pluginRodolfo Carvalho2017-07-271-5/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows users to pass either a string with comma-separated check names or a proper list of strings, whatever is more convenient in the context. For consistency, the same normalization is applied to the 'openshift_disable_check' variable.
| * | | | | | | Make LoggingCheck.run return the correct typeRodolfo Carvalho2017-07-271-1/+4
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The run method is expected to return a dict. Even though we should not run LoggingCheck by itself, it is still possible to do it and without this change we get an unhandled exception.
* | | | | | | Revert "Add health checks to upgrade playbook"Rodolfo Carvalho2017-07-282-98/+5
| | | | | | |
* | | | | | | Merge pull request #4372 from ↵Rodolfo Carvalho2017-07-282-5/+98
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | juanvallejo/jvallejo/verify-disk-memory-before-upgrade Add health checks to upgrade playbook
| * | | | | | fixes after rebasing with #4485juanvallejo2017-07-262-3/+2
| | | | | | |
| * | | | | | add pre-flight checks to ugrade pathjuanvallejo2017-07-262-5/+99
| | |_|_|/ / | |/| | | |
* | | | | | Merge pull request #4897 from ↵Scott Dodson2017-07-272-10/+14
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | ingvagabund/backport-4403-to-openshift-node-upgrade-role backport 'Add systemctl daemon-reload handler to openshift_node' #4403 to openshift_node_upgrade
| * | | | | backport 'Add systemctl daemon-reload handler to openshift_node' #4403 to ↵Jan Chaloupka2017-07-272-10/+14
| | | | | | | | | | | | | | | | | | | | | | | | openshift_node_upgrade
* | | | | | Merge pull request #4843 from kwoodson/sc_require_provScott Dodson2017-07-273-12/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove default and force argument for storageclass provisioner.