summaryrefslogtreecommitdiffstats
path: root/roles/openshift_master
Commit message (Collapse)AuthorAgeFilesLines
* Remove unneeded embedded etcd logicMichael Gugino2017-12-081-17/+1
| | | | | Removing some remaining embedded etcd facts except for the migration plays.
* Merge pull request #6278 from mgugino-upstream-stage/service_typeMichael Gugino2017-12-0810-58/+58
|\ | | | | Remove openshift.common.service_type
| * Remove openshift.common.service_typeMichael Gugino2017-12-0710-58/+58
| | | | | | | | | | | | | | | | This commit removes openshift.common.service_type in favor of openshift_service_type. This commit also removes r_openshift_excluder_service_type from plays in favor of using the role's defaults.
* | Fix incorrect register name master registry authMichael Gugino2017-12-071-1/+1
|/ | | | | | Corrects register variable name to registry auth support. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1511374
* Implement container_runtime playbooks and changesMichael Gugino2017-12-071-4/+0
| | | | | | This commit refactors some duplicate code, removes usage of set_fact where not needed, and reorganizes container_runtime role to use include_role.
* Remove all references to pacemaker (pcs, pcsd) and ↵Andrew Butcher2017-12-054-73/+1
| | | | | | | openshift.master.cluster_method. With pacemaker removed there is no longer a need for openshift.master.cluster_method. We only have one option.
* Merge pull request #6306 from mgugino-upstream-stage/fix-hosted-varsScott Dodson2017-12-052-1/+2
|\ | | | | Fix hosted vars
| * Fix hosted varsMichael Gugino2017-12-052-1/+2
| | | | | | | | | | | | Remove hosted vars from openshift_facts. The current pattern is causing a bunch of undesired sideffects.
* | Fix oreg_auth_credentials_create register varMichael Gugino2017-12-051-2/+6
|/ | | | | | | | | | | | There is a variable collision for the *oreg_auth_credentials_create variables in openshift_master and openshift_node registry_create tasks. This commit ensures standard and alternative oreg auth credential placement tasks don't use the same register variable. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1520866
* Merge pull request #6297 from mgugino-upstream-stage/crt_roleScott Dodson2017-12-043-8/+10
|\ | | | | Implement container runtime role
| * Implement container runtime roleMichael Gugino2017-12-013-8/+10
| |
* | Merge pull request #6069 from kwoodson/upgrade_scale_groupsScott Dodson2017-12-011-0/+1
|\ \ | |/ |/| Initial upgrade for scale groups.
| * Initial upgrade for scale groups.Kenny Woodson2017-11-291-0/+1
| |
* | retry package operationsLuke Meyer2017-11-302-0/+7
|/ | | | | | When a package install/update fails due to network blips or other spotty availability, retry it. If the failure is a real failure (e.g. package is really not there) it still fails after 3 tries (Ansible default).
* Remove openshift_facts project_cfg_factsMichael Gugino2017-11-282-6/+12
| | | | This simplifies some variable logic.
* Remove dns_port factMichael Gugino2017-11-282-3/+5
| | | | Simply dns_port variable by remove from openshift_facts.
* Merge pull request #6259 from adelton/issue-6255Michael Gugino2017-11-281-1/+1
|\ | | | | dist.iteritems() no longer exists in Python 3.
| * dist.iteritems() no longer exists in Python 3.Jan Pazdziora2017-11-271-1/+1
| |
* | Simplify is_master_system_container logicMichael Gugino2017-11-273-5/+7
|/ | | | Simplify the variable is_master_system_container
* Merge pull request #6231 from mtnbikenc/include-deprecation-openshift-masterRussell Teague2017-11-223-15/+14
|\ | | | | Include Deprecation - openshift-master
| * Include Deprecation - openshift-masterRussell Teague2017-11-223-15/+14
| |
* | Fix system_images_registry variableMichael Gugino2017-11-222-3/+9
|/ | | | Simplify the logic for this variable.
* Merge pull request #6084 from mgugino-upstream-stage/combine-master-upgradeOpenShift Merge Robot2017-11-158-3/+298
|\ | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Combine master upgrade play with role Currently, there are plays importing tasks directly from openshift_master role. This has caused numerous bugs and code duplicaiton in the past. This commit combines the upgrade into openshift_master role utilizing include_role syntax.
| * Combine master upgrade play with roleMichael Gugino2017-11-098-3/+298
| | | | | | | | | | | | | | | | | | Currently, there are plays importing tasks directly from openshift_master role. This has caused numerous bugs and code duplicaiton in the past. This commit combines the upgrade into openshift_master role utilizing include_role syntax.
* | Merge pull request #5968 from mgugino-upstream-stage/version-scrubScott Dodson2017-11-154-55/+1
|\ \ | | | | | | Removed old version code
| * | Removed old version codeMichael Gugino2017-11-144-55/+1
| | | | | | | | | | | | | | | | | | This commit removes any references to versions < 1.5/3.5 We assume the version is always greater than or equal to 1.5/3.5.
* | | Merge pull request #6094 from ↵OpenShift Merge Robot2017-11-152-0/+21
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgugino-upstream-stage/syscontainers-docker-login-module Automatic merge from submit-queue. Alternative method to create docker registry auth creds Currently, the command 'docker login' is run when using oreg with authentication. On some hosts, such as hosts configured to use system containers, the docker service is not running. 'docker login' will fail without the docker service running. This commit adds a module to idempotently add credentials to the registry credentials file without the use of 'docker login' Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1511374
| * | Alternative method to create docker registry auth credsMichael Gugino2017-11-142-0/+21
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the command 'docker login' is run when using oreg with authentication. On some hosts, such as hosts configured to use system containers, the docker service is not running. 'docker login' will fail without the docker service running. This commit adds a module to idempotently add credentials to the registry credentials file without the use of 'docker login' Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1511374
* | Regex anchors changed to match protocol start and ports.Kenny Woodson2017-11-131-2/+2
|/
* Merge pull request #6016 from kwoodson/bootstrap_enhancements-v3Kenny Woodson2017-11-084-158/+20
|\ | | | | Bootstrap enhancements.
| * Bootstrap enhancements.Kenny Woodson2017-11-084-158/+20
| |
* | Retry restarting master controllersMichael Gugino2017-11-061-3/+6
| | | | | | | | | | | | | | | | | | Currently, master controller services may fail to restart if master api services are not fully initialized. This commit enables retry of master controllers. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1509837
* | Revert "Bootstrap enhancements."Clayton Coleman2017-11-054-20/+81
| |
* | Merge pull request #6006 from mgugino-upstream-stage/elb-v2OpenShift Merge Robot2017-11-034-81/+20
|\ \ | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Bootstrap enhancements. This includes the elb work.
| * | Bootstrap enhancements.Kenny Woodson2017-11-034-81/+20
| |/
* | Fix master upgrade version detect and systemd enableMichael Gugino2017-11-031-0/+11
|/ | | | | Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1508755 Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1509158
* Merge pull request #5972 from smarterclayton/default_mtuOpenShift Merge Robot2017-11-011-2/+2
|\ | | | | | | | | | | | | | | | | Automatic merge from submit-queue. MTU for bootstrapping should default to openshift_node_sdn_mtu The default value needs to be consistently used. @kwoodson
| * MTU for bootstrapping should default to openshift_node_sdn_mtuClayton Coleman2017-10-311-2/+2
| | | | | | | | The default value needs to be consistently used.
* | Merge pull request #5971 from smarterclayton/retry_createOpenShift Merge Robot2017-11-011-0/+3
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Retry service account bootstrap kubeconfig creation Pulled back from the larger refactor so this job can be enabled. @kwoodson back porting so I can turn the job on
| * | Retry service account bootstrap kubeconfig creationClayton Coleman2017-10-311-0/+3
| |/ | | | | | | Pulled back from the larger refactor so this job can be enabled.
* | Merge pull request #5264 from AlexanderZagaynov/cors_escapeOpenShift Merge Robot2017-11-011-2/+3
|\ \ | |/ |/| | | | | | | | | | | | | | | Automatic merge from submit-queue. escape corsAllowedOrigins regexp strings and anchor them `corsAllowedOrigins` parameter got interpreted by OpenShift/Kubernetes as a regular expression (there is a bug about that: https://bugzilla.redhat.com/show_bug.cgi?id=1482903). It leads to some vague behaviour, like for `127.0.0.1` value `127a0b0c1` will be matched as valid, as well as `localhost.example.com` for `localhost`. I've added regexp escaping here, as well as value anchoring to the begin and end of the string. I've also added case-insensitive flag `(?i)` to match values like `LocalHost` for `localhost`.
| * fix comment and make it visible to end-userAlexander Zagaynov2017-09-051-2/+3
| |
| * escape also custom_cors_originsAlexander Zagaynov2017-08-301-1/+1
| |
| * add comment on regexp specificsAlexander Zagaynov2017-08-301-1/+1
| |
| * escape corsAllowedOrigins regexp strings and anchor themAlexander Zagaynov2017-08-301-1/+1
| |
* | Merge pull request #5936 from sdodson/arbitrary-fw-rulesOpenShift Merge Robot2017-10-311-1/+3
|\ \ | | | | | | | | | | | | Automatic merge from submit-queue. Add arbitrary firewall port config to master too
| * | Add arbitrary firewall port config to master tooScott Dodson2017-10-301-1/+3
| | |
* | | Merge pull request #5929 from ↵OpenShift Merge Robot2017-10-312-10/+15
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ingvagabund/remove-master-service-during-non-ha-to-ha-upgrade Automatic merge from submit-queue. remove master.service during the non-ha to ha upgrade Bug: 1506165
| * | | remove master.service during the non-ha to ha upgradeJan Chaloupka2017-10-302-10/+15
| | | |
* | | | Merge pull request #5924 from smarterclayton/bootstrap_cleanupOpenShift Merge Robot2017-10-312-1/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. GCP cannot use growpart utils from AWS due to conflicting package Allow cloud provider specific packages to be specified. Also, fix a wait condition in openshift_gcp to allow masters to be bootstrapped nodes. @kwoodson