summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fixes to 'make ci'Tim Bielawa2016-12-092-23/+17
| | | | | | | | | | | | | * Fixed: A grep filter was capturing the actual pylint check rc's * Changed: pylint used to print out messages for locally-disabled and file-ignored items * Changed: pylint output format is now 'parseable' * Cleaned up: Pylint was emitting deprecation messages for 'required-attributes' and 'ignore-iface-methods'. They have been removed from the pylintrc file
* Merge pull request #2959 from mtnbikenc/plugin_lintTim Bielawa2016-12-091-5/+19
|\ | | | | Resolved lint issues for version_requirement plugin
| * Resolved lint issuesRussell Teague2016-12-091-5/+19
|/
* Merge pull request #2942 from mtnbikenc/ansible-versionTim Bielawa2016-12-0811-33/+48
|\ | | | | Minimum Ansible version check
| * Minimum Ansible version checkRussell Teague2016-12-081-0/+46
| | | | | | | | | | | | Moved version check to callback pluging to allow checking the version prior to playbook syntax checking. This prevents errors on syntax-checking with older versions of Ansible.
| * Removed verify_ansible_version playbook refsRussell Teague2016-12-0810-33/+2
|/
* Merge pull request #2902 from dgoodwin/fix-metrics-urlScott Dodson2016-12-089-38/+38
|\ | | | | Fix metricsPublicURL only being set correctly on first master.
| * Fix metricsPublicURL only being set correctly on first master.Devan Goodwin2016-12-019-38/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | Problem was caused by facts not being set for that master. To fix this patch cleans up the calculation of metricsPublicURL in general. Because this value is used in openshift_master to template into the master config file, we now define these facts more clearly in openshift_master_facts, and add a dependency on this to openshift_metrics. The calculation of default sub-domain is also changed to remove it from system facts (as neither of these are facts about the system) and instead use plain variables.
* | Merge pull request #2937 from dgoodwin/upgrade-prepullScott Dodson2016-12-084-21/+22
|\ \ | | | | | | Pre-pull master/node/ovs images during upgrade.
| * | Pre-pull master/node/ovs images during upgrade.Devan Goodwin2016-12-074-21/+22
| | | | | | | | | | | | | | | | | | | | | | | | We did this for install but not upgrade, leading to situations where the service restarts after upgrade could take much longer than expected as docker pulls down the new image. Now the images are present when we restart services and should allow them to come back online much more quickly, equivalent to rpm service restarts.
* | | Merge pull request #2948 from dagwieers/patch-3Scott Dodson2016-12-081-0/+2
|\ \ \ | | | | | | | | Silence warnings when using rpm directly
| * | | Silence warnings when using rpm directlyDag Wieers2016-12-081-0/+2
| | | |
* | | | Merge pull request #2943 from detiber/fixOpenShiftReleaseScott Dodson2016-12-084-3/+12
|\ \ \ \ | | | | | | | | | | Properly handle x.y.z formatted versions for openshift_release
| * | | | Remove Hostname from 1.1 and 1.2 predicatesJason DeTiberus2016-12-072-3/+0
| | | | |
| * | | | Properly handle x.y.z formatted versions for openshift_releaseJason DeTiberus2016-12-074-0/+12
| | | | |
* | | | | Merge pull request #2952 from tbielawa/fix-coverage-eraseScott Dodson2016-12-081-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Fix coverage not appending new data
| * | | | | Fix coverage not appending new dataTim Bielawa2016-12-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Adds '--cover-erase' flag to the 'ci-unittests' target Fixes bug introduced in d7b4265f329591817bac381e2772ae14ac337cea: "Can't add arcs to existing line data" error message
* | | | | | Merge pull request #2950 from dgoodwin/drop-32-upgradeScott Dodson2016-12-083-84/+0
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Drop 3.2 upgrade playbooks.
| * | | | | Drop 3.2 upgrade playbooks.Devan Goodwin2016-12-083-84/+0
| | |/ / / | |/| | |
* | | | | Merge pull request #2946 from dagwieers/patch-1Scott Dodson2016-12-081-0/+2
|\ \ \ \ \ | | | | | | | | | | | | Silence warnings when using some commands directly
| * | | | | Silence warnings when using rpm directlyDag Wieers2016-12-081-0/+2
| |/ / / /
* | | | | Merge pull request #2947 from dagwieers/patch-2Scott Dodson2016-12-081-0/+2
|\ \ \ \ \ | |/ / / / |/| | | | Silence warnings when using rpm directly
| * | | | Silence warnings when using rpm directlyDag Wieers2016-12-081-0/+2
|/ / / /
* | | | Merge pull request #2934 from sdodson/etcd3-v2Scott Dodson2016-12-071-7/+4
|\ \ \ \ | |/ / / |/| | | etcd_upgrade: Simplify package installation
| * | | etcd_upgrade: Simplify package installationScott Dodson2016-12-071-7/+4
| | | |
* | | | Merge pull request #2939 from tbielawa/gh2933Jason DeTiberus2016-12-071-6/+15
|\ \ \ \ | | | | | | | | | | Speed up 'make ci' and trim the output
| * | | | Speed up 'make ci' and trim the outputTim Bielawa2016-12-071-6/+15
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The virtualenv is conditionally rebuilt now if the test requirements file is modified. This will save upwards of 30+ seconds in iterative 'make ci' runs. The pylint output is now trimmed to exclude disabled tests. The order of the 'ci' target prerequisites has been changed to run the fastest tests first. Closes #2933
* | | | Merge pull request #2892 from detiber/upgradeSchedulerScott Dodson2016-12-079-269/+573
|\ \ \ \ | |/ / / |/| | | Scheduler upgrades
| * | | add comments and remove debug codeJason DeTiberus2016-12-071-2/+8
| | | |
| * | | Handle updating of scheduler config during upgradeJason DeTiberus2016-12-067-257/+549
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - do not upgrade predicates if openshift_master_scheduler_predicates is defined - do not upgrade priorities if openshift_master_scheduler_priorities is defined - do not upgrade predicates/priorities unless they match known previous default configs - output WARNING to user if predictes/priorities are not updated during install
| * | | Fix templatingJason DeTiberus2016-12-061-9/+9
| | | |
| * | | test updatesJason DeTiberus2016-12-062-4/+10
| |/ / | | | | | | | | | | | | | | | - fix nose coverage flags - add coverage support for files tested outside of the utils directory - exclude stdlib and virtualenv installed dependencies
* | | Merge pull request #2928 from sdodson/etcd3-v2Scott Dodson2016-12-064-19/+21
|\ \ \ | |/ / |/| | Fix etcd upgrades to etcd 3.x
| * | Always install latest etcd for containerized hostsScott Dodson2016-12-061-3/+5
| | |
| * | etcd_upgrade : Use different variables for rpm vs container versionsScott Dodson2016-12-061-10/+10
| | |
| * | Switch back to using etcd rather than etcd3Scott Dodson2016-12-062-6/+6
|/ /
* | Merge pull request #2920 from detiber/schedulerVarFixAndrew Butcher2016-12-053-20/+18
|\ \ | | | | | | Scheduler var fix
| * | Update scheduler predicate/priorities varsJason DeTiberus2016-12-012-20/+16
| | | | | | | | | | | | | | | | | | - Introduce additional variables for current scheduler config and default values to better determine if the values we are getting are user-defined, coming from config or are the default values.
| * | fix tagsJason DeTiberus2016-12-011-0/+2
| | |
* | | Merge pull request #2915 from sdodson/BZ1401425Scott Dodson2016-12-051-0/+4
|\ \ \ | | | | | | | | node_dnsmasq - restart dnsmasq if it's not currently running
| * | | node_dnsmasq - restart dnsmasq if it's not currently runningScott Dodson2016-12-051-0/+4
| | | | | | | | | | | | | | | | | | | | Fixes Bug 1401425 Fixes BZ1401425
* | | | Merge pull request #2914 from abutcher/upgrade-admission-checkScott Dodson2016-12-054-2/+6
|\ \ \ \ | |/ / / |/| | | Conditionalize master config update for admission_plugin_config.
| * | | Conditionalize master config update for admission_plugin_config.Andrew Butcher2016-12-054-2/+6
|/ / /
* | | Merge pull request #2908 from tremble/upgrade_factsScott Dodson2016-12-051-0/+1
|\ \ \ | | | | | | | | upgrade_control_plane.yml: systemd_units.yaml needs the master facts
| * | | upgrade_control_plane.yml: systemd_units.yaml nees the master factsMark Chappell2016-12-021-0/+1
| | | |
* | | | Merge pull request #2907 from tremble/masterAndrew Butcher2016-12-022-2/+2
|\ \ \ \ | |/ / / |/| | | openshift-master/restart : use openshift.common.hostname instead of inventory_hostname
| * | | openshift-master/restart : use openshift.common.hostname instead of ↵Mark Chappell2016-12-022-2/+2
|/ / / | | | | | | | | | | | | | | | | | | inventory_hostname When using a dynamic inventory inventory_hostname isn't guaranteed to be usable. We should use openshift.common.hostname which already copes with this
* | | Merge pull request #2888 from sdodson/dnsmasq-no-strict-orderScott Dodson2016-12-014-4/+4
|\ \ \ | | | | | | | | openshift_node_dnsmasq - Remove strict-order option from dnsmasq
| * | | openshift_node_dnsmasq - Remove strict-order option from dnsmasqScott Dodson2016-12-014-4/+4
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | strict-order forces dnsmasq to iterate through nameservers in order. If one of the nameservers is down this will slow things down while dnsmasq waits for a timeout. Also, this option prevents dnsmasq from querying other nameservers if the first one returns a negative result. While I think it's odd to have a nameserver that returns negative results for a query that another returns positive results for this does seem to fix the issue in testing. Fixes Bug 1399577
* | | Merge pull request #2898 from smunilla/GH2738_misconfigured_etcd_varScott Dodson2016-12-011-0/+2
|\ \ \ | |_|/ |/| | Explictly set etcd vars for byo scaleup