summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* health checks: configure failure output in playbooksLuke Meyer2017-05-236-67/+93
| | | | | Customized the error summary to depend on the intent of the playbook run. Ensured output makes sense when failures are unrelated to running checks.
* disk/memory checks: make threshold configurableLuke Meyer2017-05-234-12/+65
|
* Show help on how to disable checks after failureRodolfo Carvalho2017-05-231-0/+22
|
* Allow disabling checks via Ansible variableRodolfo Carvalho2017-05-231-1/+8
| | | | | | | | | Example usage: $ ansible-playbook -i hosts playbooks/byo/config.yml -e openshift_disable_check=memory_availability,disk_availability Or add the variable to the inventory / hosts file.
* Verify memory and disk requirements before installRodolfo Carvalho2017-05-231-0/+12
|
* Merge pull request #4079 from sosiouxme/20170503-move-check-playbookOpenShift Bot2017-05-2321-98/+141
|\ | | | | Merged by openshift-bot
| * RPM workaround for the move of cert playbooksPep Turró Mauri2017-05-221-14/+20
| | | | | | | | | | | | | | | | The playbooks/certificate_expiry directory is being moved inside the openshift-checks tree, and a symlink there is put in place for backwards compatibility. RPM does not handle this well, so this expands the section of the specfile that already contains a workaround for this situation to also handle this move.
| * health check playbooks: relocate and expandLuke Meyer2017-05-2220-84/+121
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We are moving toward having adhoc post-install checks and so the "preflight" designation needs to be widened. Updated location to playbooks/byo/openshift-checks, added health check playbook, and updated README. Also included the certificate_expiry playbooks. Left behind symlinks and wrappers for existing checks. To conform with the direction of the rest of the repo, the openshift-checks playbooks are split into two directories, one under playbooks/common with the actual invocation and one under playbooks/byo for entrypoints that are just wrappers for the ones in common. Because the certificate_expiry playbooks are intended not just to be functional but to be examples that users modify, I did not split them similarly. That could happen later after discussion but for now I just left them whole under byo/openshift-checks/certificate_expiry.
* | Automatic commit of package [openshift-ansible] release [3.6.69-1].Scott Dodson2017-05-232-2/+80
| | | | | | | | | | | | Created by command: /usr/bin/tito tag
* | Merge pull request #3785 from ↵OpenShift Bot2017-05-2312-118/+329
|\ \ | | | | | | | | | | | | juanvallejo/jvallejo/update-docker-image-availability-check Merged by openshift-bot
| * | preflight int tests: fix for openshift_version depLuke Meyer2017-05-229-16/+76
| | |
| * | remove skopeo dependency on docker-pyjuanvallejo2017-05-192-152/+143
| | |
| * | improve error handling for missing varsjuanvallejo2017-05-193-66/+226
| | |
* | | Merge pull request #4248 from abutcher/sysctl-workaroundOpenShift Bot2017-05-232-1/+10
|\ \ \ | | | | | | | | Merged by openshift-bot
| * | | Workaround sysctl module issue with py3 by converting task to lineinfile.Andrew Butcher2017-05-192-1/+10
| | | |
* | | | Merge pull request #4243 from jarrpa/template-paramsOpenShift Bot2017-05-2228-28/+28
|\ \ \ \ | | | | | | | | | | Merged by openshift-bot
| * | | | lib/base: Allow for more complex template paramsJose A. Rivera2017-05-1928-28/+28
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | This change allows for the specification of JSON objects like lists as parameters to templates. Signed-off-by: Jose A. Rivera <jarrpa@redhat.com>
* | | | Merge pull request #3982 from jim-minter/trello131-broker-configurationOpenShift Bot2017-05-223-0/+15
|\ \ \ \ | |_|_|/ |/| | | Merged by openshift-bot
| * | | add template service broker configurableJim Minter2017-04-243-0/+15
| | | |
* | | | Merge pull request #4228 from etsauer/s3-required-varsScott Dodson2017-05-222-4/+4
|\ \ \ \ | | | | | | | | | | Removing requirement to pass aws credentials
| * | | | Removing requirement to pass aws credentialsEric Sauer2017-05-222-4/+4
| | | | |
* | | | | Merge pull request #4220 from pat2man/openshift_hosted_metrics_deployer_versionEric Wolinetz2017-05-222-0/+6
|\ \ \ \ \ | | | | | | | | | | | | Update examples to include metrics deployer version
| * | | | | Update hosts.*.example to include openshift_hosted_metrics_deployer_versionPatrick Tescher2017-05-172-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without this we can have a mismatch between the OpenShift version and the metrics version. Fixes https://github.com/openshift/origin/issues/13532
* | | | | | Merge pull request #4217 from ingvagabund/move-etcd-upgrade-code-into-roleJan Chaloupka2017-05-2122-198/+270
|\ \ \ \ \ \ | | | | | | | | | | | | | | Move etcd upgrade code into role
| * | | | | | move etcd upgrade related code into etcd_upgrade roleJan Chaloupka2017-05-1822-198/+270
| | | | | | |
* | | | | | | Merge pull request #4226 from mtnbikenc/fix-auditConfigOpenShift Bot2017-05-191-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Merged by openshift-bot
| * | | | | | | Fix auditConfig for non-HA environmentsRussell Teague2017-05-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug 1447019
* | | | | | | | Merge pull request #4134 from ewolinetz/issue4111OpenShift Bot2017-05-191-0/+8
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Merged by openshift-bot
| * | | | | | | | Adding assert to check for python-passlib on control hostewolinetz2017-05-111-0/+8
| | | | | | | | |
* | | | | | | | | Merge pull request #4211 from dmsimard/github_teamsOpenShift Bot2017-05-191-1/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Merged by openshift-bot
| * | | | | | | | | Add teams attribute to github identity providerDavid Moreau-Simard2017-05-161-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | GitHubIdentityProvider provides the ability to authenticate against GitHub organization teams [1]. This adds the attribute as optional so that users do not get the error "provider GitHubIdentityProvider contains unknown keys teams". [1]: https://docs.openshift.com/container-platform/3.5/install_config/configuring_authentication.html#GitHub
* | | | | | | | | | Merge pull request #4249 from ashcrow/router-certificate-nameScott Dodson2017-05-192-4/+4
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | inventory: rename certificates->certificate in router example
| * | | | | | | | | | inventory: rename certificates->certificate in router exampleSteve Milner2017-05-192-4/+4
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Per change 9397727e433cbd9bfd865fb5ad773c7b6b4590e8, router replaced certificates with certificate.
* | | | | | | | | | Merge pull request #4175 from codificat/20170512-img-build-updatesOpenShift Bot2017-05-195-9/+26
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|/ |/| | | | | | | | | Merged by openshift-bot
| * | | | | | | | | Fix container image build referencesPep Turró Mauri2017-05-125-9/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updating some files to account for recent changes: - Commit fbadeb4 moved the location of the Dockerfiles - playbook2image is now available from the openshift org - downstream image is building with boto 2.34 Adjusting image build scripts and documentation to match these changes.
* | | | | | | | | | Merge pull request #3630 from juanvallejo/jvallejo/add-etcd-volume-checkOpenShift Bot2017-05-192-0/+207
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|/ |/| | | | | | | | | Merged by openshift-bot
| * | | | | | | | | revert role-specific var namejuanvallejo2017-05-151-1/+1
| | | | | | | | | |
| * | | | | | | | | Merge branch 'jvallejo/add-etcd-volume-check' of ↵juanvallejo2017-05-121-5/+7
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | github.com:juanvallejo/openshift-ansible into jvallejo/add-etcd-volume-check
| | * | | | | | | | | Update variable name to standardRodolfo Carvalho2017-05-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was agreed to name role variables as `r_ROLE_NAME_VARIABLE_NAME`. Giving it a try.
| | * | | | | | | | | Make class attribute name shorterRodolfo Carvalho2017-05-111-4/+4
| | | | | | | | | | |
| | * | | | | | | | | Add module docstringRodolfo Carvalho2017-05-111-0/+2
| | | | | | | | | | |
| * | | | | | | | | | check if hostname is in list of etcd hostsjuanvallejo2017-05-121-3/+4
| |/ / / / / / / / /
| * | | | | | | | | Update checkRodolfo Carvalho2017-05-102-48/+46
| | | | | | | | | |
| * | | | | | | | | int -> floatRodolfo Carvalho2017-05-101-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We don't need to convert to int and then to float. Read it as float from the start.
| * | | | | | | | | Remove vim lineRodolfo Carvalho2017-05-101-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It has been agreed that we don't use it any longer.
| * | | | | | | | | add etcd volume checkjuanvallejo2017-05-092-0/+207
| | | | | | | | | |
* | | | | | | | | | Merge pull request #4235 from sdodson/fix-yamllintScott Dodson2017-05-191-8/+8
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / |/| | | | | | | | | Fix yamllint problems
| * | | | | | | | | Fix yamllint problemsScott Dodson2017-05-191-8/+8
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | I should've noticed before merging...
* | | | | | | | | Merge pull request #4073 from richm/logging-es-routeScott Dodson2017-05-195-0/+205
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | add ability to expose Elasticsearch as an external route
| * | | | | | | | | add ability to expose Elasticsearch as an external routeRich Megginson2017-05-185-0/+205
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the ability to expose Elastisearch as a route outside of the cluster. - `openshift_logging_es_allow_external`: True (default is False) - if this is True, Elasticsearch will be exposed as a Route - `openshift_logging_es_ops_hostname`: The external facing hostname to use for the route and the TLS server certificate (default is "es." + `openshift_master_default_subdomain`) There are other similar parameters for the TLS server cert, key, and CA cert. There are other similar parameters for when the OPS cluster is deployed e.g. `openshift_logging_es_ops_allow_external`, etc.