summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Make pylint disables more specificRodolfo Carvalho2017-08-241-15/+26
| | | | And beautify the code a bit.
* Handle exceptions in failure summary cb pluginRodolfo Carvalho2017-08-241-2/+11
| | | | | | | | | | | This serves two purposes: - Gracefully omit the summary if there was an error computing it, no confusion to the regular end user. - Provide a stacktrace of the error when running verbose, giving developers or users reporting bugs a better insight of what went wrong, as opposed to Ansible's opaque handling of errors in callback plugins.
* Rewrite failure summary callback pluginRodolfo Carvalho2017-08-243-119/+243
| | | | | The intent is to deduplicate similar errors that happened in many hosts, making the summary more concise.
* Handle more exceptions when running checksRodolfo Carvalho2017-08-241-19/+30
| | | | | | | | | This prevents an exception in one check from interfering with other checks. Skips checks that raise an exception in their is_active method. Whenever capturing a broad exception in the `is_action` or `run` methods, include traceback information that can be useful in bug reports.
* List known checks/tags when check name is invalidRodolfo Carvalho2017-08-242-16/+22
|
* List existing health checks when none is requestedRodolfo Carvalho2017-08-244-9/+73
| | | | | | | | This is a simple mechanism to learn what health checks are available. Note that we defer task_vars verification, so that we can compute requested_checks and resolved_checks earlier, allowing us to list checks even if openshift_facts has not run.
* Add playbook for running arbitrary health checksRodolfo Carvalho2017-08-244-1/+50
| | | | | | | | This is useful on its own, and also aids in developing/testing new checks that are not part of any playbook. Since the intent when running this playbook is to execute checks, opt for a less verbose explanation on the error summary.
* Update health check READMERodolfo Carvalho2017-08-241-5/+4
|
* Merge pull request #5161 from sdodson/image-stream-syncScott Dodson2017-08-2459-199/+1337
|\ | | | | Sync latest image stream and templates
| * Add dotnet 2.0 to v3.6Scott Dodson2017-08-224-0/+450
| |
| * Add dotnet 2.0 to v3.7Scott Dodson2017-08-228-20/+555
| |
| * Update v3.6 contentScott Dodson2017-08-2223-89/+166
| |
| * Update all image streams and templatesScott Dodson2017-08-2224-90/+166
| |
* | Merge pull request #5158 from ewolinetz/logging_issue605Scott Dodson2017-08-241-0/+4
|\ \ | | | | | | Passing memory and cpu limit for ops ES install
| * | Passing memory and cpu limit for ops ES installEric Wolinetz2017-08-221-0/+4
| |/
* | Merge pull request #5145 from sdodson/bz1480438Scott Dodson2017-08-231-31/+34
|\ \ | | | | | | If IP4_NAMESERVERS are unset then pull the value from /etc/resolv.conf
| * | If IP4_NAMESERVERS are unset then pull the value from /etc/resolv.confScott Dodson2017-08-211-31/+34
| | | | | | | | | | | | | | | | | | This will happen whenever someone has directly modified /etc/resolv.conf Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1480438
* | | Automatic commit of package [openshift-ansible] release [3.7.0-0.105.0].Jenkins CD Merge Bot2017-08-232-2/+19
| | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | Merge pull request #5100 from mgugino-upstream-stage/check-hostname-lenScott Dodson2017-08-232-3/+5
|\ \ \ | | | | | | | | Add hostname/nodename length check
| * | | Add hostname/nodename length checkMichael Gugino2017-08-162-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OpenShift will not allow nodes to register if the nodename is > 63 chars. This commit verifies that nodename is not set to a potential hostname that is greater than 63 characters. This commit also updates logic to check `openshift_hostname` to check for > 63 chars. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1467790
* | | | Merge pull request #5183 from sdodson/masterScott Dodson2017-08-231-5/+4
|\ \ \ \ | | | | | | | | | | Fix generated content
| * | | | Fix generated contentScott Dodson2017-08-231-5/+4
|/ / / /
* | | | Merge pull request #5011 from kwoodson/provisionScott Dodson2017-08-2345-364/+2499
|\ \ \ \ | | | | | | | | | | AWS Provisioning with scale groups.
| * | | | First attempt at provisioning.Kenny Woodson2017-08-2145-364/+2499
| | |/ / | |/| |
* | | | Merge pull request #5112 from kwoodson/oc_adm_csrScott Dodson2017-08-237-0/+2049
|\ \ \ \ | | | | | | | | | | Adding oc_adm_csr to lib_openshift for node approvals
| * | | | First attempt at creating the cert signer.Kenny Woodson2017-08-217-0/+2049
| | | | |
* | | | | Merge pull request #5101 from maxamillion/add-dnf-supportScott Dodson2017-08-2330-162/+165
|\ \ \ \ \ | | | | | | | | | | | | Add dnf support
| * | | | | remove out of scope variable from exception messageAdam Miller2017-08-181-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Adam Miller <maxamillion@fedoraproject.org>
| * | | | | raise AosVersionException if no expected packages found by dnf queryAdam Miller2017-08-181-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Adam Miller <maxamillion@fedoraproject.org>
| * | | | | add dnf support to roles/openshift_health_checker/library/aos_version.pyAdam Miller2017-08-1630-162/+158
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Adam Miller <maxamillion@fedoraproject.org>
* | | | | | Merge pull request #4980 from sdodson/migrate-v2Scott Dodson2017-08-238-48/+134
|\ \ \ \ \ \ | | | | | | | | | | | | | | Switch to migrating one host and scaling etcd members back up
| * | | | | | Switch to migrating one host and forming a new clusterScott Dodson2017-08-228-48/+134
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With large datasets where there are many keys with TTLs the expiry was creating a data inconsistency problem. The hope is that by performing the migration once and then forming a new cluster this is avoided. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1475351
* | | | | | Merge pull request #4761 from wozniakjan/logging_kibana_oomScott Dodson2017-08-231-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | bug 1468987: kibana_proxy OOM
| * | | | | | bug 1468987: kibana_proxy OOMJan Wozniak2017-07-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We currently set the memory allocated to the kibana-proxy container to be the same as `max_old_space_size` for nodejs. But in V8, the heap consists of multiple spaces. The old space has only memory ready to be GC and measuring the used heap by kibana-proxy code, there is at least additional 32MB needed in the code space when `max_old_space_size` peaks. Setting the default memory limit to 256MB here and also changing the default calculation of `max_old_space_size` in the image repository to be only half of what the container receives to allow some heap for other `spaces`.
* | | | | | | Merge pull request #5120 from smarterclayton/allow_gcs_registryScott Dodson2017-08-234-12/+15
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | Allow GCS object storage to be configured
| * | | | | | Allow GCS object storage to be configuredClayton Coleman2017-08-174-12/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, setting the GCS registry object storage settings resulted in an invalid configuration. This generates a registry-config secret that has the correct file if the GCS config is set.
* | | | | | | Merge pull request #5126 from ozdanborne/fix-calico-spacingScott Dodson2017-08-212-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix missing space in calico ansible roles
| * | | | | | | Fix missing space in calico ansible rolesDan Osborne2017-08-172-2/+2
| | | | | | | |
* | | | | | | | Merge pull request #4254 from dmsimard/registry_certsScott Dodson2017-08-214-49/+138
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / |/| | | | | | | Refactor openshift_hosted's docker-registry route setup
| * | | | | | | Refactor openshift_hosted's docker-registry route setupDavid Moreau-Simard2017-07-234-49/+138
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have identified an issue where a docker-registry service set up as 'reencrypt' with a provided certificate and a self-signed certificate on the pod does not authorize users to push images. If the docker-registry service is set up as 'passthrough' with the same provided certificate, everything works. In light of this, this commit essentially adds support for configuring provided certificates with a passthrough route while maintaining backwards compatibility with the other use cases. The default remains 'passthrough' with self-generated certificates. Other miscellaneous changes include: - Move fact setup that were only used in secure.yml there - Omit the hostname for the route if there are none to configure, oc_route takes care of handling the default - Replace hardcoded /etc/origin/master by openshift_master_config_dir
* | | | | | | | Automatic commit of package [openshift-ansible] release [3.7.0-0.104.0].Jenkins CD Merge Bot2017-08-202-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | | | | Merge pull request #5142 from sdodson/fix-dnsScott Dodson2017-08-201-0/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Ensure that openshift_node_facts has been called for dns_ip
| * | | | | | | | Ensure that openshift_node_facts has been called for dns_ipScott Dodson2017-08-181-0/+1
| | | | | | | | |
* | | | | | | | | Automatic commit of package [openshift-ansible] release [3.7.0-0.103.0].Jenkins CD Merge Bot2017-08-192-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | | | | | Automatic commit of package [openshift-ansible] release [3.7.0-0.102.0].Jenkins CD Merge Bot2017-08-182-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | | | | | Automatic commit of package [openshift-ansible] release [3.7.0-0.101.0].Jenkins CD Merge Bot2017-08-182-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | | | | | Automatic commit of package [openshift-ansible] release [3.7.0-0.100.0].Jenkins CD Merge Bot2017-08-182-3/+182
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | | | | | Merge pull request #4784 from abutcher/noop-scaleupScott Dodson2017-08-182-0/+27
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fail scaleup configuration when there are no new_nodes or new_masters
| * | | | | | | | | Fail within scaleup playbooks when new_{nodes,masters} host groups are empty.Andrew Butcher2017-07-182-0/+27
| | | | | | | | | |
* | | | | | | | | | Merge pull request #5132 from BrunoVernay/patch-1Scott Dodson2017-08-181-1/+1
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Display "origin 3.6" as in previous installer 3.5