| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| | |
Merged by openshift-bot
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
GitHubIdentityProvider provides the ability to authenticate against
GitHub organization teams [1].
This adds the attribute as optional so that users do not get the error
"provider GitHubIdentityProvider contains unknown keys teams".
[1]: https://docs.openshift.com/container-platform/3.5/install_config/configuring_authentication.html#GitHub
|
|\ \
| | |
| | | |
Merged by openshift-bot
|
| | | |
|
| |/
|/|
| |
| | |
Will allow for default image resolution to be used.
|
| |
| |
| |
| |
| |
| |
| | |
In a project where contributors are free to use whatever editor they
want and we have linting tools that verify the proper formatting of
Python files, it should not be required to have a vim-specific line in
Python files.
|
| | |
|
| |
| |
| |
| |
| |
| | |
In ansible 2.3 "[WARNING]: when statements should not include jinja2
templating delimiters such as {{ }} or {% %}. Found: {{
g_glusterfs_hosts is not defined }}"
|
|/ |
|
| |
|
| |
|
| |
|
|
|
|
| |
Detected by pylint. The fixture indeed doesn't require an argument.
|
| |
|
|
|
|
|
|
| |
Origin 1.6 and future releases will be versioned as Origin 3.6+. This
unifies a point of inconsistency and brings Origin in line with the full
release history. Update version numbers to match that 3.x is preferred.
|
| |
|
| |
|
|
|
|
|
|
|
| |
And remove explicit dependencies on nose, replacing with pytest.
The former is the way forward, for it is a better maintained test
library, and a transitive dependency of `molecule`, the test framework
we're using to add integration tests to this repo (work in progress).
|
|
|
|
| |
In the intention to remove the dependency on nose and use only pytest.
|
|
|
|
|
| |
This removes irrelevant differences between the two test files, making
it easier to observe similarities.
|
|
|
|
|
|
|
|
| |
This is intented to make it easier to read tests top-down.
The test methods in both files are sorted somehow logically, and in the
same order in both files, to help visualizing the similarities between
them with diff tools.
|
|
|
|
|
| |
Consolidate tests for bad input into a single file instead of having
the same code in two files.
|
|
|
|
|
| |
And reorganize them in such a way to show that there are a bunch of
duplicated tests that are exactly the same.
|
|
|
|
|
|
| |
http://stackoverflow.com/a/10097543/4804690
Additionally, avoid using platform-specific '/' and '..'.
|
| |
|
| |
|
|
|
|
| |
openshift_master_facts available within the role
|
| |
|
| |
|
|
|
|
| |
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
|
|
|
|
| |
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
* Added checks to make ci for yaml linting
* Modified y(a)ml files to pass lint checks
|
|\
| |
| | |
Fix metricsPublicURL only being set correctly on first master.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- 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
|
|\ \
| | |
| | | |
Scheduler var fix
|
| |/
| |
| |
| |
| |
| | |
- 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.
|
|/ |
|
| |
|
|
|
|
|
|
|
|
| |
Move the values in kube_admission_plugin_config up one level per
the new format from 1.3:
"The kubernetesMasterConfig.admissionConfig.pluginConfig should be moved
and merged into admissionConfig.pluginConfig."
|
| |
|
| |
|
| |
|