summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #7073 from abutcher/aws-node-hostnameOpenShift Merge Robot2018-02-122-8/+3
|\ | | | | | | | | | | | | | | Automatic merge from submit-queue. Update user-data for FQDN instance hostnames * Remove node service start from bootstrap.yml. We start and enable the node service in user-data and we want the node service to start after NetworkManager so that the instance will have its final hostname. * Restart systemd-hostnamed before restarting NetworkManager. In testing this has ensured that instances receive an FQDN hostname which is required for kube.
| * Remove node start from bootstrap.yml.Andrew Butcher2018-02-121-8/+0
| |
| * Restart systemd-hostnamed before restarting NetworkManager in node user-data.Andrew Butcher2018-02-121-0/+3
| |
* | Merge pull request #7099 from vrutkovs/etcd-containerized-upgradeOpenShift Merge Robot2018-02-121-1/+1
|\ \ | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Use r_etcd_common_etcdctl_command instead of hardcoded binary name to support containerized upgrade etcdctlv2 is used to check cluster health pre- and post-upgrade
| * | Use r_etcd_common_etcdctl_command instead of hardcoded binary name to ↵Vadim Rutkovsky2018-02-101-1/+1
| | | | | | | | | | | | support containerized upgrade
* | | Merge pull request #7022 from vrutkovs/sanitize-labelsOpenShift Merge Robot2018-02-1216-330/+462
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Verify that requested services have schedulable nodes matching the selectors Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1538445 Not sure if I should keep one task per failure or should group them by service (e.g. all logging check in one tasks)
| * | | Fix indentation to make yamllint happyVadim Rutkovsky2018-02-1011-330/+330
| | | | | | | | | | | | | | | | Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
| * | | Verify that requested services have schedulable nodes matching the selectorsVadim Rutkovsky2018-02-1015-2/+134
| |/ / | | | | | | | | | Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* | | Merge pull request #7105 from vrutkovs/additional-mounts-exampleScott Dodson2018-02-121-0/+1
|\ \ \ | | | | | | | | additional mounts: specify 'type' in container_runtime_crio_additional_mounts
| * | | additional mounts: specify 'type' in container_runtime_crio_additional_mountsVadim Rutkovsky2018-02-121-0/+1
| |/ /
* | | Merge pull request #7001 from mbruzek/cidr_pool_start_endOpenShift Merge Robot2018-02-123-19/+14
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Removing prefix, replacing with cidr, pool_start and pool_end vars The heat template was hardcoded with a /24 cidr and that limited customers to 251 ip addresses in the OpenStack subnet. This allows the user to configure the cidr and the allocation pool start and end. Addresses issue #6829 that I created last week. @tomassedovic please take a look
| * | Removing prefix and replacing with cidr, pool_start and pool_end variables.Matt Bruzek2018-02-023-19/+14
| | |
* | | Merge pull request #7093 from sdodson/bz1534917OpenShift Merge Robot2018-02-093-6/+6
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Normalize the time we wait for pods to 5s * 60 retries A few people have run into situations where it takes longer than 120seconds for the webconsole to start up so lets be a bit more foregiving and allow for up to 5 minutes in scenarios where we're waiting for pods.
| * | | Normalize the time we wait for pods to 5s * 60 retriesScott Dodson2018-02-093-6/+6
| | | |
* | | | Merge pull request #7078 from mgugino-upstream-stage/limit-hostsOpenShift Merge Robot2018-02-0930-7/+149
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Limit host scope during plays Many plays only target a select subset of hosts, especially oo_first_master for components such as logging and registry. This commit limits the scope of most plays to eliminate unnecessary task execution on node groups. This will result in great time savings for large deployments. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1516526
| * | | | Limit host scope during playsMichael Gugino2018-02-0930-7/+149
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Many plays only target a select subset of hosts, especially oo_first_master for components such as logging and registry. This commit limits the scope of most plays to eliminate unnecessary task execution on node groups. This will result in great time savings for large deployments. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1516526
* | | | | Merge pull request #7086 from spadgett/pause-for-console-deploymentOpenShift Merge Robot2018-02-091-0/+11
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Pause for console rollout /assign @sdodson
| * | | | | Pause for console rolloutSamuel Padgett2018-02-091-0/+11
| | | | | |
* | | | | | Merge pull request #7088 from mgugino-upstream-stage/fix-upgrade-cpOpenShift Merge Robot2018-02-091-1/+1
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix upgrade-control plane post_control_plane.yml An entry-point playbook was imported by mistake. This caused common init code to run again, which is undesireable. This commit changes the import to use the corresponding 'private' play which does not call the init code. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1542855
| * | | | | Fix upgrade-control plane post_control_plane.ymlMichael Gugino2018-02-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An entry-point playbook was imported by mistake. This caused common init code to run again, which is undesireable. This commit changes the import to use the corresponding 'private' play which does not call the init code. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1542855
* | | | | | Automatic commit of package [openshift-ansible] release [3.9.0-0.42.0].Justin Pierce2018-02-092-2/+53
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | Merge pull request #7058 from kwoodson/crio_additional_mountsKenny Woodson2018-02-092-0/+12
|\ \ \ \ \ | |/ / / / |/| | | | Adding ability to provide additional mounts to crio system container.
| * | | | Adding ability to provide additional mounts to crio system container.Kenny Woodson2018-02-072-0/+12
| | | | |
* | | | | Merge pull request #6712 from vrutkovs/avoid-hardcoding-image-versionOpenShift Merge Robot2018-02-0918-23/+26
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Set default image to openshift_release Previously `v3.7` was hardcoded as a default value, instead `openshift_release` should be used. A new var is introduced - `openshift_template_service_broker_image_tag`, similar to vars for other services. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1530183 TODO: * [x] Make sure all images have a variable for image prefix * [x] Figure out if `latest` is a good default for origin images * [ ] Set correct `openshift_image_tag` in CI inventory for tests to pass
| * | | | | Set default image tag to openshift_image_tag for servicesVadim Rutkovsky2018-02-0818-23/+26
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
* | | | | | Merge pull request #7077 from ewolinetz/logging_facts_sane_yaml_parseOpenShift Merge Robot2018-02-081-3/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Only try to yaml.load a file if it ends in .yml or .yaml in logging facts Addresses error seen in https://bugzilla.redhat.com/show_bug.cgi?id=1543625 Also prevents us from seeing this when rerunning a deployment for es5.x branch. When we are trying to load a configmap file to parse it for facts based on the values, we should only parse if it is a yaml or yml file. Note: if someone created a configmap entry with key that matched that, but wasn't a yaml file, it would still fail... not sure if that can be helped.
| * | | | | | Only try to yaml.load a file if it ends in .yml or .yaml in logging factsEric Wolinetz2018-02-081-3/+4
| |/ / / / /
* | | | | | Merge pull request #7074 from kwoodson/redeploy_certs_insecureOpenShift Merge Robot2018-02-081-16/+20
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Redeploy router certificates during upgrade only when secure. Wrap the upgrade logic for redeploying certificates into another block so that insecure registries do not perform any certificate tasks.
| * | | | | | Redeploy router certificates during upgrade only when secure.Kenny Woodson2018-02-081-16/+20
| | | | | | |
* | | | | | | Merge pull request #7072 from jarrpa/gluster-block-routeOpenShift Merge Robot2018-02-081-0/+19
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. GlusterFS: Fix block StorageClass heketi route Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1542861
| * | | | | | | GlusterFS: Fix block StorageClass heketi routeJose A. Rivera2018-02-081-0/+19
| | | | | | | |
* | | | | | | | Merge pull request #7014 from sdodson/xpaas-148Scott Dodson2018-02-08337-4441/+18610
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Update xPaaS content to v1.4.8
| * | | | | | | | xPaaS v1.4.8 for v3.7Scott Dodson2018-02-08112-1480/+6203
| | | | | | | | |
| * | | | | | | | xPaaS v1.4.8-1 for v3.8Scott Dodson2018-02-08112-1480/+6203
| | | | | | | | |
| * | | | | | | | xPaaS v1.4.8-1 for v3.9Scott Dodson2018-02-08112-1480/+6203
| | | | | | | | |
| * | | | | | | | Bump xpaas versionScott Dodson2018-02-081-1/+1
| | |_|_|_|/ / / | |/| | | | | |
* | | | | | | | Merge pull request #6926 from abutcher/etcd-ca-hostScott Dodson2018-02-0814-37/+50
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Determine which host is the etcd CA host
| * | | | | | | | lib_utils_oo_collect: Allow filtering on dot separated keys.Andrew Butcher2018-02-011-1/+1
| | | | | | | | |
| * | | | | | | | Determine which etcd host is the etcd_ca_host rather than assume it is the ↵Andrew Butcher2018-02-0111-13/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | first host in the etcd host group.
| * | | | | | | | Attempt to back up generated certificates on every etcd host.Andrew Butcher2018-02-011-1/+1
| | | | | | | | |
| * | | | | | | | Remove pre upgrade verification step re: etcd ca host.Andrew Butcher2018-02-011-22/+0
| | | | | | | | |
* | | | | | | | | Merge pull request #7068 from datarace/logging-fixScott Dodson2018-02-084-15/+15
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | changed oc to {{ openshift_client_binary }}
| * | | | | | | | | changed oc to {{ openshift_client_binary }}datarace2018-02-084-15/+15
| | | | | | | | | |
* | | | | | | | | | Merge pull request #7031 from mgugino-upstream-stage/fix-syscon-varsOpenShift Merge Robot2018-02-082-1/+5
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix master and node system container variables This commit corrects service names and adds DOCKER_SERVICE variable to openshift_master system container creation. This will ensure services will restart correctly and in the proper order. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1542324
| * | | | | | | | | | Fix master and node system container variablesMichael Gugino2018-02-062-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit corrects service names and adds DOCKER_SERVICE variable to openshift_master system container creation. This will ensure services will restart correctly and in the proper order. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1542324
* | | | | | | | | | | Merge pull request #7061 from gnewson/Change-etcd-checkOpenShift Merge Robot2018-02-081-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Changing the check for the number of etcd nodes This playbook is called (via std_include.yml) when the scale up playbook for either master or etcd is called. In the scenario where you are scaling up the number of masters/etcd nodes it is feasible, if not likely, that the number of etcd nodes is not 1, 3 or 5 and this check therefore causes a scale up to fail. The two example scenarios that are driving this change are: You have a cluster with 3 master nodes (each running etcd) and one of those masters fail. The master node is removed from both the OpenShift cluster and the etcd cluster and the inventory updated to reflect the state of the cluster minus the failed master node. You would then run the scale up playbook to add a new master / etcd master into the cluster using an inventory containing and etcd group of just 2 nodes. As above but the cluster has 5 master nodes. If you lose a master node and update the inventory to reflect that then the inventory will contain an etcd group with 4 nodes. @sdodson Previously submitted as https://github.com/openshift/openshift-ansible/pull/6979
| * | | | | | | | | | | Remove spaces introduced at the start of the lineGeoff Newson2018-02-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In making the initial change I introduce some spaces at the beginning of the line. Removing them.
| * | | | | | | | | | | Changing the check for the number of etcd nodesGeoff Newson2018-02-071-1/+1
| | | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #7079 from fabianvf/1524805-cloudforms-pull-policyScott Dodson2018-02-081-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Bug 1524805- CFME example now works disconnected
| * | | | | | | | | | | | Bug 1524805- CFME example now works disconnectedFabian von Feilitzsch2018-02-081-1/+1
|/ / / / / / / / / / / /