summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Using the currently attached pvc for an ES dc if available, otherwise ↵Eric Wolinetz2017-11-011-2/+2
| | | | falling back to current logic
* 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 #5974 from tbielawa/bz1504535OpenShift Merge Robot2017-11-011-5/+19
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Don't use possibly undefined variables in error messages In an error reporting task we were referencing a variable which may not have been defined. Even though NFS server detection was working correctly the undefined variable reference caused an AnsibleUndefinedVariable error. * Fixes CFME failing to deploy using external NFS storage https://bugzilla.redhat.com/show_bug.cgi?id=1504535
| * | Don't use possibly undefined variables in error messagesTim Bielawa2017-11-011-5/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In an error reporting task we were referencing a variable which may not have been defined. Even though NFS server detection was working correctly the undefined variable reference caused an AnsibleUndefinedVariable error. * Fixes CFME failing to deploy using external NFS storage https://bugzilla.redhat.com/show_bug.cgi?id=1504535
* | | Merge pull request #5938 from ↵Scott Dodson2017-11-013-2/+36
|\ \ \ | | | | | | | | | | | | | | | | staebler/1506976-watch_serviceinstance_in_view_role Add rules to the view ClusterRole for service catalog.
| * | | Add rules to the view ClusterRole for service catalog.staebler2017-10-303-2/+36
| | | |
* | | | 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 #5912 from ewolinetz/update_console_flagsScott Dodson2017-11-011-1/+1
|\ \ \ \ \ | |_|_|/ / |/| | | | Updating console OPENSHIFT_CONSTANTS flag for TSB
| * | | | Updating console OPENSHIFT_CONSTANTS flag for TSBEric Wolinetz2017-10-271-1/+1
| | | | |
* | | | | Merge pull request #5955 from ↵OpenShift Merge Robot2017-11-012-0/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mgugino-upstream-stage/remove-docker-etc-containers-regs Automatic merge from submit-queue. Docker: make use of new etc/containers/registries.conf optional Currently, not all versions of docker support using /etc/containers/registries.conf This commit makes the use of that file optional.
| * | | | | Docker: make use of new etc/containers/registries.conf optionalMichael Gugino2017-10-312-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, not all versions of docker support using /etc/containers/registries.conf This commit makes the use of that file optional.
* | | | | | Automatic commit of package [openshift-ansible] release [3.7.0-0.189.0].Jenkins CD Merge Bot2017-11-012-2/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Created by command: /usr/bin/tito tag --debug --accept-auto-changelog --keep-version --debug
* | | | | | Merge pull request #5953 from sdodson/bind-dynamicOpenShift Merge Robot2017-10-311-1/+1
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Switch from bind-interfaces to bind-dynamic
| * | | | | Switch from bind-interfaces to bind-dynamicScott Dodson2017-10-311-1/+1
| | | | | |
* | | | | | Merge pull request #5956 from smarterclayton/delete_build_istanceClayton Coleman2017-10-311-0/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | Ensure GCP image build instance gets cleaned up on teardown
| * | | | | | Ensure GCP image build instance gets cleaned up on teardownClayton Coleman2017-10-311-0/+6
| | | | | | |
* | | | | | | Merge pull request #5957 from kwoodson/doc_fix_cert_requiredScott Dodson2017-10-311-0/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Stating that certificate it is required when doing SSL on ELB.
| * | | | | | | Stating that certificate it is required when doing SSL on ELB.Kenny Woodson2017-10-311-0/+1
|/ / / / / / /
* | | | | | | Merge pull request #5949 from mgugino-upstream-stage/ha_controller_ttlOpenShift Merge Robot2017-10-312-4/+0
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Remove unused osm_controller_lease_ttl This variable is no longer used and references should be removed. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1507449
| * | | | | | Remove unused osm_controller_lease_ttlMichael Gugino2017-10-312-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This variable is no longer used and references should be removed. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1507449
* | | | | | | Merge pull request #4571 from wongma7/efs-followMichael Gugino2017-10-317-17/+22
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Add blank storageClassName to provisioner PVC & PV template
| * | | | | | Give PV & PVC empty storage class to avoid being assigned default gp2Matthew Wong2017-10-272-0/+2
| | | | | | |
| * | | | | | Use oc_project to ensure openshift_provisioners_project presentMatthew Wong2017-10-271-12/+5
| | | | | | |
| * | | | | | Fix yaml formattingMatthew Wong2017-10-274-5/+15
| | | | | | |
* | | | | | | Merge pull request #5936 from sdodson/arbitrary-fw-rulesOpenShift Merge Robot2017-10-312-1/+9
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Add arbitrary firewall port config to master too
| * | | | | | | Add arbitrary firewall port config to master tooScott Dodson2017-10-302-1/+9
| | | | | | | |
* | | | | | | | 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-317-9/+28
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * | | | | | | | Delete images located in a family named {{ prefix }}imagesClayton Coleman2017-10-301-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allows CI jobs to create images specifically for this cluster and then remove them afterwards.
| * | | | | | | | Use global IP to indicate node should pick DNSClayton Coleman2017-10-301-0/+1
| | | | | | | | |
| * | | | | | | | Remove project metadata prefixed with the cluster prefixClayton Coleman2017-10-301-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allows other scripts to create metadata that is cleaned up safely.
| * | | | | | | | Use openshift.node.registry_url instead of oreg_urlClayton Coleman2017-10-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Consistent with other use
| * | | | | | | | Allow master node group to wait for stable on GCPClayton Coleman2017-10-301-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Masters can be in a bootstrap config, but allow them to wait for stable so that we don't have inventory issues.
| * | | | | | | | GCP cannot use AWS growpart packageClayton Coleman2017-10-303-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the required package optional
* | | | | | | | | Merge pull request #5932 from mgugino-upstream-stage/retry-docker-credsOpenShift Merge Robot2017-10-314-0/+13
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Add retry logic to docker auth credentials This commit enables retry on docker login commands. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1506931
| * | | | | | | | Add retry logic to docker auth credentialsMichael Gugino2017-10-304-0/+13
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit enables retry on docker login commands. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1506931
* | | | | | | | Merge pull request #2910 from richm/preserve-ansible_inventory_pathOpenShift Merge Robot2017-10-311-2/+4
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. preserve the oo-install ansible_inventory_path value
| * | | | | | | | preserve the oo-install ansible_inventory_path valueRich Megginson2017-05-181-2/+4
| | | | | | | | |
* | | | | | | | | Merge pull request #5930 from mgugino-upstream-stage/retry-journald-restartScott Dodson2017-10-311-3/+5
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Retry restarting journald
| * | | | | | | | | Retry restarting journaldMichael Gugino2017-10-301-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sometimes journald fails to restart cleanly during upgrades. This commit retries restarting 3 times. Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1506141
* | | | | | | | | | Merge pull request #5919 from pecameron/cache-sizeOpenShift Merge Robot2017-10-312-0/+4
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. dnsmasq cache-size dns-forward-max change On very large clusters dnsmasq performance suffers due to limited cache-size and dns-forward-max values. bug: 1482847 https://bugzilla.redhat.com/show_bug.cgi?id=1482847
| * | | | | | | | | | dnsmasq cache-size dns-forward-max changePhil Cameron2017-10-302-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On very large clusters dnsmasq performance suffers due to limited cache-size and dns-forward-max values. bug: 1482847 https://bugzilla.redhat.com/show_bug.cgi?id=1482847
* | | | | | | | | | | Merge pull request #5722 from qinpingli/sc_openstackOpenShift Merge Robot2017-10-313-2/+8
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Automatic merge from submit-queue. Create default storageclass for cloudprovider openstack With this patch, a default storageclass will be created on openstack cloudprovider after installation.
| * | | | | | | | | | | Modify StorageClass name to standardPing Qin2017-10-301-1/+1
| | | | | | | | | | | |