summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Ensure that node names are lowerecased before matchingScott Dodson2018-01-221-1/+1
| | | | | | | | The name field is always lowercase whereas the nodename may not have been in the past. Ensure that we lowercase it before looking for matching nodes. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1396350
* Merge pull request #6771 from tomassedovic/fix-readmeOpenShift Merge Robot2018-01-191-5/+1
|\ | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Drop the testing repo var from openstack readme This was introduced as a temporary fix for the origin+centos deployments. It is no longer necessary and it's confusing to newcomers.
| * Drop the testing repo var from openstack readmeTomas Sedovic2018-01-181-5/+1
| | | | | | | | | | | | This was introduced as a temporary fix for the origin+centos deployments. It is no longer necessary and it's confusing to newcomers.
* | Merge pull request #6765 from jcantrill/bz1523047_annotate_logging_prefixOpenShift Merge Robot2018-01-181-0/+1
|\ \ | | | | | | | | | | | | Automatic merge from submit-queue. bug 1523047. Annotate ops projects with an .operation prefix
| * | bug 1523047. Annotate ops projects with an .operation prefixJeff Cantrill2018-01-171-0/+1
| | |
* | | Merge pull request #6785 from spadgett/tsb-redeploy-consoleOpenShift Merge Robot2018-01-184-11/+33
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Rollout console after template service broker install The console will discover if the broker is running, but only on startup. Trigger a rollout so that the console sees the change. /assign @sdodson cc @jwforres @deads2k @bparees
| * | | Rollout console after template service broker installSamuel Padgett2018-01-184-11/+33
| | | | | | | | | | | | | | | | | | | | The console will discover if the broker is running, but only on startup. Trigger a rollout so that the console sees the change.
* | | | Merge pull request #6709 from jmencak/tuned-29OpenShift Merge Robot2018-01-181-1/+6
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Automatic profile setting for tuned 2.9 Tuned 2.9 introduces automatic and manual profile selection. If there is file /etc/tuned/profile_mode containing "manual" on a system with tuned 2.9, without this PR the appropriate OCP profiles will not be set. This PR removes the file prior to tuned restart, in addition to /etc/tuned/active_profile to make the tuned profile selection automatic.
| * | | | Automatic profile setting for tuned 2.9Jiri Mencak2018-01-121-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tuned 2.9 introduces automatic and manual profile selection. If there is a file /etc/tuned/profile_mode containing "manual" on a system with tuned 2.9, without this PR the appropriate OCP profiles will not be set. This PR removes the file prior to tuned restart, in addition to /etc/tuned/active_profile to make the tuned profile selection automatic.
* | | | | Merge pull request #6775 from spadgett/fix-task-nameOpenShift Merge Robot2018-01-182-8/+8
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Bug 1535947: Fix missing task in metrics, logging uninstall playbooks Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1535947 /assign @sdodson
| * | | | | Bug 1535947: Fix missing task in metrics, logging uninstall playbooksSamuel Padgett2018-01-182-8/+8
| | |_|_|/ | |/| | | | | | | | | | | | | Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1535947
* | | | | Merge pull request #6760 from mgugino-upstream-stage/fix-docker-atomic-proxiesOpenShift Merge Robot2018-01-182-1/+12
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Ensure atomic_proxies are configured with docker In some setups, system containers may be used in conjunction with package_docker. This commit ensures that the atomic proxies are configured while using package_docker and containerized components. This commit also ensures containerized_groups are based on openshift_is_containerized instead of 'containerized' variable directly to account for possible use of atomic hosts which users may not specify containerized in inventory directly. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1535323
| * | | | | Ensure atomic_proxies are configured with dockerMichael Gugino2018-01-172-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some setups, system containers may be used in conjunction with package_docker. This commit ensures that the atomic proxies are configured while using package_docker and containerized components. This commit also ensures containerized_groups are based on openshift_is_containerized instead of 'containerized' variable directly to account for possible use of atomic hosts which users may not specify containerized in inventory directly. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1535323
* | | | | | Merge pull request #6488 from vrutkovs/utils-test-flake8OpenShift Merge Robot2018-01-185-156/+174
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Fix flake8 errors in utils/test
| * | | | | | Fix flake8 errors in utils/testVadim Rutkovsky2018-01-045-156/+174
| | | | | | |
* | | | | | | Merge pull request #6774 from sdodson/webconsole-image-prefixScott Dodson2018-01-182-4/+4
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | Make openshift_web_console_prefix defaults like other components
| * | | | | | Make openshift_web_console_prefix defaults like other componentsScott Dodson2018-01-182-4/+4
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | Other components are like 'registry.access.redhat.com/ose-' or 'openshift/origin-' based on OCP versus Origin.
* | | | | | Merge pull request #6745 from shawn-hurley/secret-reg-authOpenShift Merge Robot2018-01-181-0/+8
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Bug 1509082 - Adding auth as a secret If username and password are defined we should use a secret for the credentials.
| * | | | | | adding check if secret auth is neededShawn Hurley2018-01-161-2/+8
| | | | | | |
| * | | | | | adding asb auth as a secret.Shawn Hurley2018-01-161-0/+2
| | | | | | |
* | | | | | | Merge pull request #6768 from weherdh/masterOpenShift Merge Robot2018-01-181-0/+6
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Add Azure to support openshift_cloudprovider_kind Try to fix this as an example. I checked it works.
| * | | | | | | Add Azure to support openshift_cloudprovider_kindweherdh2018-01-181-0/+6
| | | | | | | |
* | | | | | | | Merge pull request #6757 from sosiouxme/20180117-update-check-versionsLuke Meyer2018-01-185-107/+49
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | update health check required versions for 3.9
| * | | | | | | | health checks: update required pkg versionsLuke Meyer2018-01-172-0/+7
| | | | | | | | |
| * | | | | | | | health checks: factor out get_required_versionLuke Meyer2018-01-175-71/+32
| | | | | | | | |
| * | | | | | | | package_version check: reuse get_major_minor_versionLuke Meyer2018-01-173-38/+12
| | | | | | | | |
* | | | | | | | | Merge pull request #6589 from vrutkovs/kibana-use-sixOpenShift Merge Robot2018-01-182-14/+11
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. kibana checks: use six.moves instead of ImportError
| * | | | | | | | kibana checks: use six.moves instead of ImportErrorVadim Rutkovsky2018-01-032-14/+11
| | | | | | | | |
* | | | | | | | | Merge pull request #6752 from vrutkovs/tsb-prefixOpenShift Merge Robot2018-01-182-4/+4
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Rework default TSB prefix and imagename to match other services Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1518917
| * | | | | | | | | Rework default TSB prefix and imagename to match other servicesVadim Rutkovsky2018-01-172-4/+4
| | | | | | | | | |
* | | | | | | | | | Merge pull request #6743 from ewolinetz/logging_find_running_podsOpenShift Merge Robot2018-01-172-4/+4
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Ensure we are running oc execs against running pods We should make sure we are running `oc exec` statements against running pods only. Addresses https://bugzilla.redhat.com/show_bug.cgi?id=1534879
| * | | | | | | | | | Ensure we are running oc execs against running podsEric Wolinetz2018-01-162-4/+4
| | | | | | | | | | |
* | | | | | | | | | | Merge pull request #6747 from ewolinetz/logging_fix_pvc_storageclassOpenShift Merge Robot2018-01-171-4/+4
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Setting default storage_class_names for when calling openshift_loggin… …g_elasticsearch role Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=1521218
| * | | | | | | | | | Setting default storage_class_names for when calling ↵Eric Wolinetz2018-01-161-4/+4
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | openshift_logging_elasticsearch role
* | | | | | | | | | Merge pull request #6076 from abutcher/bz1511357Scott Dodson2018-01-172-2/+3
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Bug 1511357: Task "Reload generated facts" failed while scaling up containerized masters
| * | | | | | | | | | Pull openshift_image_tag from oo_masters_to_config rather oo_first_master.Andrew Butcher2018-01-171-1/+1
| | | | | | | | | | |
| * | | | | | | | | | Default install_result when reloading generated facts.Andrew Butcher2018-01-171-1/+2
| | |_|_|_|_|_|/ / / | |/| | | | | | | |
* | | | | | | | | | Merge pull request #6181 from mrsiano/grafanaScott Dodson2018-01-1713-0/+6059
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|/ |/| | | | | | | | | Add new grafana playbook.
| * | | | | | | | | Add new grafana playbook.Eldad Marciano2018-01-1713-0/+6059
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this patch introduce a new playbook and roles to deploy grafana automaticly using openshift-ansible capabilities. this patch will abstract the grafana deployment process specially for openshift.
* | | | | | | | | | Automatic commit of package [openshift-ansible] release [3.9.0-0.21.0].Jenkins CD Merge Bot2018-01-172-2/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | | | | | | Merge pull request #6744 from sosiouxme/20180115-fix-checksMichael Gugino2018-01-172-1/+2
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | openshift_checks: repair adhoc list-checks mode
| * | | | | | | | | | openshift_checks: repair adhoc list-checks modeLuke Meyer2018-01-162-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes bug https://bugzilla.redhat.com/show_bug.cgi?id=1509157 docker_image_availability.__init__ was loading a variable that of course was not defined for localhost, which is how this play runs. Fixed that. Would be nice if there were a more generic way to solve that class of problem but I can't think of anything for now that isn't worse. Also noticed that this play gathers facts for localhost. That's not necessary for what it's doing, so disabled that.
* | | | | | | | | | | Merge pull request #6751 from vrutkovs/apply-check-rcOpenShift Merge Robot2018-01-176-9/+19
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Check that rc != 0 for steps with 'failed_when' Some parts don't check the result rc at all, some check that 'error' is present there. Instead the safest way is to check for both Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1534538
| * | | | | | | | | | | Fix a typo in "Determine if growpart is installed"Vadim Rutkovsky2018-01-171-1/+1
| | | | | | | | | | | |
| * | | | | | | | | | | Check rc for commands with openshift_client_binary and failed_whenVadim Rutkovsky2018-01-175-8/+18
| | |_|_|_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This might mask some failures and continue instead of a failure
* | | | | | | | | | | Merge pull request #6759 from mgugino-upstream-stage/etcd-upgrade-factsScott Dodson2018-01-171-1/+4
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Add init/main.yml to etc-upgrade
| * | | | | | | | | | | Add init/main.yml to etc-upgradeMichael Gugino2018-01-171-1/+4
| | |_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, etcd upgrade playbook only calls evaluate_groups to limit the number of tasks that are run against nodes during initialization. Recently, changes have been added to allow skipping of most node tasks via providing variablized host groups during initialization code. This commit allows etcd upgrades to use common init code to ensure necessary facts are defined and proper variables are sanity checked. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1527771
* | | | | | | | | | | Merge pull request #6746 from ↵OpenShift Merge Robot2018-01-171-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | juanvallejo/jvallejo/fix-elasticsearch-diskspace-check Automatic merge from submit-queue. include elasticsearch container name Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1534988 Addresses cases where the elasticsearch pod with multiple containers does not have the `elasticsearch` container as its first one. cc @sosiouxme
| * | | | | | | | | | include elasticsearch container namejuanvallejo2018-01-161-1/+1
| | |_|_|_|_|_|/ / / | |/| | | | | | | |
* | | | | | | | | | Merge pull request #6520 from sdodson/37-38-39Scott Dodson2018-01-1715-42/+89
|\ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / |/| | | | | | | | | Faciliate 3.7 to 3.8 to 3.9 upgrade