summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #5394 from sosiouxme/20170912-health-role-bz1487548OpenShift Bot2017-09-131-0/+1
|\ | | | | Merged by openshift-bot
| * openshift-checks: add role symlinkLuke Meyer2017-09-121-0/+1
| | | | | | | | | | fixes bug 1487548 https://bugzilla.redhat.com/show_bug.cgi?id=1487548
* | Merge pull request #5370 from jmencak/tuned-openshift-ansible-bad_merge-fixOpenShift Bot2017-09-132-14/+4
|\ \ | | | | | | Merged by openshift-bot
| * | Re-enabling new tuned profile hierarchy (PR5089)Jiri Mencak2017-09-122-14/+4
| | |
* | | Merge pull request #5405 from ewolinetz/issue_5403Scott Dodson2017-09-131-1/+0
|\ \ \ | | | | | | | | Removing setting of pod presets
| * | | Removing setting of pod presetsEric Wolinetz2017-09-131-1/+0
| | | |
* | | | Merge pull request #5359 from mgugino-upstream-stage/version-docker-auth-configOpenShift Bot2017-09-1311-34/+111
|\ \ \ \ | |/ / / |/| | | Merged by openshift-bot
| * | | Fix: authenticated registry support for containerized hostsMichael Gugino2017-09-1111-34/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, openshift-anisble supports authentication to container registries to pull down openshift container images. The openshift_verison role uses the docker cli to gather image information from container registries before authentication credentials are provided by openshift-ansible. This commit creates the necessary token to authenticate to private registries during openshift_version. The token is generated by the role 'docker' on all hosts where docker is installed/configured when oreg_auth_users is defined. This commit also adds a read-only mount into the openshift master and node container services. This mount is '/var/lib/origin/.docker:/root/.docker:ro'. This is because the container images do not currently read the values in '/var/lib/origin/.docker' as this may be a bug upstream. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1316341
* | | | Merge pull request #5402 from ashcrow/fix-cri-o-image-nameScott Dodson2017-09-131-1/+1
|\ \ \ \ | | | | | | | | | | cri-o: Fix Fedora image name
| * | | | cri-o: Fix Fedora image nameSteve Milner2017-09-131-1/+1
|/ / / / | | | | | | | | | | | | Signed-off-by: Steve Milner <smilner@redhat.com>
* | | | Merge pull request #5379 from mtnbikenc/fix-jinja2-openshift_hostedOpenShift Bot2017-09-133-4/+4
|\ \ \ \ | | | | | | | | | | Merged by openshift-bot
| * | | | Correct logic for openshift_hosted_*_waitRussell Teague2017-09-123-4/+4
| | | | |
| * | | | Workaround Ansible Jinja2 delimiter warningRussell Teague2017-09-122-2/+2
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | This workaround prevents the warnings on using Jinja2 templating delimiters in `when:` conditions in cases where a variable is used as the conditional. This has been fixed in Ansible 2.4. https://github.com/ansible/ansible/pull/25092
* | | | Merge pull request #5386 from ashcrow/fix-wording-in-upgrade-noticeOpenShift Bot2017-09-131-1/+1
|\ \ \ \ | | | | | | | | | | Merged by openshift-bot
| * | | | upgrade: Updates warning on missing required variablesSteve Milner2017-09-121-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Steve Milner <smilner@redhat.com>
* | | | | Merge pull request #5321 from mtnbikenc/proposal-processScott Dodson2017-09-132-0/+57
|\ \ \ \ \ | | | | | | | | | | | | [Proposal] OpenShift-Ansible Proposal Process
| * | | | | [Proposal] OpenShift-Ansible Proposal ProcessRussell Teague2017-09-112-0/+57
| | | | | |
* | | | | | Merge pull request #5377 from ingvagabund/double-required-disk-space-etcd-backupOpenShift Bot2017-09-133-4/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | Merged by openshift-bot
| * | | | | | double the required disk space for etcd backupJan Chaloupka2017-09-123-4/+3
| | | | | | |
* | | | | | | Merge pull request #5365 from sosiouxme/20170908-disconnected-image-checkOpenShift Bot2017-09-1212-144/+184
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Merged by openshift-bot
| * | | | | | | openshift_health_check: allow disabling all checksLuke Meyer2017-09-122-3/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Can now set openshift_disable_check=* to disable all checks without needing to know their names. fixes bug 1462106 https://bugzilla.redhat.com/show_bug.cgi?id=1462106
| * | | | | | | docker_image_availability: fix local image searchLuke Meyer2017-09-121-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An image in the docker index may be tagged by name or by registry plus name. In order to find the image correctly locally and prevent looking for it externally, make sure all possible variations are searched.
| * | | | | | | docker_image_availability: probe registry connectivityLuke Meyer2017-09-122-122/+132
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Probe whether the host has connectivity to the registry before trying to inspect it for images, and remember the result. Also if later inspection fails due to timeout, mark registry as unreachable. Note in failure output if any registries were unreachable. Registry order should match what is configured into docker now as well. Fixes bug 1480195 https://bugzilla.redhat.com/show_bug.cgi?id=1480195
| * | | | | | | openshift_checks: add retries in pythonLuke Meyer2017-09-1210-22/+43
| |/ / / / / /
* | | | | | | Merge pull request #5343 from mtnbikenc/improve-jinja2-searchOpenShift Bot2017-09-122-23/+80
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Merged by openshift-bot
| * | | | | | | Improve searching when conditions for Jinja2 delimitersRussell Teague2017-09-112-23/+80
| | | | | | | |
* | | | | | | | Merge pull request #5350 from adelton/issue-2454OpenShift Bot2017-09-121-1/+1
|\ \ \ \ \ \ \ \ | |_|_|_|/ / / / |/| | | | | | | Merged by openshift-bot
| * | | | | | | Properly quote "true" and "false" strings for include_granted_scopes.Jan Pazdziora2017-09-111-1/+1
| | | | | | | |
* | | | | | | | Merge pull request #5352 from rhcarvalho/issue5311-harden-failure-summary-pluginScott Dodson2017-09-122-2/+29
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | | Skip failure dedup instead of crashing
| * | | | | | | Skip failure dedup instead of crashingRodolfo Carvalho2017-09-112-2/+29
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes the callback plugin behave better when dedup is not possible: work with the original list of failures instead of raising an unhandled exception and producing confusing output for users.
* | | | | | | Merge pull request #5325 from DenverJ/patch-1Scott Dodson2017-09-121-2/+2
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Fix new_master or new_node fail check during scaleup
| * | | | | | Fix new_master or new_node fail checkDenver Janke2017-09-071-2/+2
| | | | | | |
* | | | | | | Merge pull request #5353 from mgugino-upstream-stage/public_hostname_checkScott Dodson2017-09-121-0/+4
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | Add openshift_public_hostname length check
| * | | | | | Add openshift_public_hostname length checkMichael Gugino2017-09-111-0/+4
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the variable openshift_hostname is checked to ensure it is less than 63 characters. The variable openshift_public_hostname should also be checked for this length. This commit checks the length of openshift_public_hostname to ensure it is 63 characters or less. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1467790
* | | | | | Merge pull request #5356 from ↵OpenShift Bot2017-09-113-10/+14
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | ashcrow/openshift-docker-systemcontainer-image-override Merged by openshift-bot
| * | | | | | container-engine: Allow full image overrideSteve Milner2017-09-113-10/+14
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | openshift_docker_systemcontainer_image_registry_override has been replaced with openshift_docker_systemcontainer_image_override. The difference is openshift_docker_systemcontainer_image_override takes a full image path including the tag. Example: openshift_docker_systemcontainer_image_override=gscrivano/container-engine:latest Signed-off-by: Steve Milner <smilner@redhat.com>
* | | | | / Automatic commit of package [openshift-ansible] release [3.7.0-0.126.0].Jenkins CD Merge Bot2017-09-112-2/+13
| |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | Merge pull request #5340 from sdodson/bz1489913Scott Dodson2017-09-111-3/+4
|\ \ \ \ \ | | | | | | | | | | | | Revert back to hostnamectl and previous default of not setting hostname
| * | | | | Revert back to hostnamectl and previous default of not setting hostnameScott Dodson2017-09-081-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The old default was "{{ not openshift.common.version_gte_3_1_or_1_1 }}" just default to false instead, master branch will never be used with 3.0 hostname module doesn't work on atomic host Fixes BZ 1489913
* | | | | | Merge pull request #5344 from mgugino-upstream-stage/fix-pkg-versionsOpenShift Bot2017-09-111-0/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | Merged by openshift-bot
| * | | | | | Fix rpm version logic for hostsMichael Gugino2017-09-081-0/+5
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the variable openshift_pkg_version is defined by the first master unless specified by the user. The role openshift_version attempts to set this value on all other hosts after setting it on the first master. During normal installation, openshift_pkg_version is not initially defined, therefor each host will utilize the most recent rpm available. This creates a problem during later operations of a cluster if a node is added or replaced by a new node. This commit ensures that the value for openshift_pkg_version set by the first master is respected by subsequent hosts. Fixes: #5341
* | | | | | Merge pull request #5256 from ashcrow/1451023Scott Dodson2017-09-113-0/+19
|\ \ \ \ \ \ | | | | | | | | | | | | | | upgrade: Verify required network items are set
| * | | | | | inventory: Denote new required upgrade variablesSteve Milner2017-09-082-0/+4
| | | | | | |
| * | | | | | upgrade: Verify required network items are setSteve Milner2017-09-081-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When upgrading osm_cluster_network_cidr, osm_host_subnet_length, and openshift_portal_net must be set to avoid SDN initialization errors. This was found when the default parameters were changed between Openshift versions. This meant users who upgraded and did not specify either mentioned variable at install/upgrade time ended up getting SDN errors post upgrade. When osm_cluster_network_cidr, osm_host_subnet_length, and openshift_portal_net are not set the upgrade will fail telling the user that the variables must be set and how to find the current values in the current install. References: https://github.com/openshift/openshift-ansible/commit/b50b4ea0b03feb9431abd7294fe4fb6b549ddfc0 Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1451023
* | | | | | | Merge pull request #5342 from mtnbikenc/fix-include-through-symlinkScott Dodson2017-09-111-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Correct include path to not follow symlink
| * | | | | | | Correct include path to not follow symlinkRussell Teague2017-09-081-1/+1
| | |/ / / / / | |/| | | | |
* | | | | | | Merge pull request #5332 from kwoodson/amibuild_node_configOpenShift Bot2017-09-114-9/+12
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Merged by openshift-bot
| * | | | | | ami build process calls openshift-node/config.ymlKenny Woodson2017-09-074-9/+12
| | | | | | |
* | | | | | | Merge pull request #5338 from mtnbikenc/fix-pre_upgradeOpenShift Bot2017-09-112-1/+5
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | Merged by openshift-bot
| * | | | | | Fix issue with etcd_common when using pre_upgrade tagRussell Teague2017-09-082-1/+5
| | | | | | |