| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
'name' param is required and can't be skipped. 'when' is used instead
|
|\
| |
| |
| |
| |
| |
| |
| |
| | |
Automatic merge from submit-queue.
Don't deploy the console if disabled or registry subtype
Whenever openshift_deployment_subtype == registry or the console is explicitly listed as a disabled feature don't deploy.
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1538974
|
| |
| |
| |
| | |
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1538974
|
|\ \
| |/
|/|
| |
| |
| |
| |
| |
| | |
Automatic merge from submit-queue.
Add kube service ipaddress to no_proxy list
Computed as the first ip address in the portal_net
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1511870
|
| |
| |
| |
| | |
Computed as the first ip address in the portal_net
|
|/
|
|
| |
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1530403
|
|\
| |
| | |
Allow 5 etcd hosts
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Automatic merge from submit-queue.
[1506750] Ensure proper hostname check override
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1506750
|
| | | |
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Currently, openshift_facts requires pyyaml to be installed.
This package is installed via init/base_packages.yml, which
is currently called after init/facts.yml. This results
in a situation where installs will fail due to missing
python dependency.
This commit splits init/facts.yml into two, and
allows base_packages.yml to be run before the
openshift_facts.py plugin is executed.
|
|/
|
|
|
|
|
|
| |
Currently, users have no way to run preqrequisites.yml
on just newly added nodes during scaleup.
This commit ensures only the new nodes are changed during
scaleup as well as ensure prerequisites are run.
|
|
|
|
|
|
|
|
|
|
|
|
| |
This variable may or may not be defined by the users.
During deployments, it will be set to '-{{ openshift_version }}'
if undefined.
During upgrades, it will remain undefined.
This commit ensures that if the variable is undefined,
empty strings '' are set.
|
|\
| |
| | |
Remove become statements
|
| |
| |
| |
| |
| | |
This commit removes become:no statements that break
the installer in various ways.
|
| |
| |
| |
| |
| | |
This commit limits common init code to exclude
oo_nodes_to_config during upgrade_control_plane runs.
|
|/
|
|
|
|
|
|
|
|
|
|
|
| |
This commit changes how we handle openshift_version role.
Most of the version initialization code is only run
on the first master now. All other hosts have values
set from the master.
Aftwards, we run some basic RPM queries to ensure
that the correct version is available on the other nodes.
Containerized needs to do their own image checks elsewhere.
|
|\
| |
| | |
upgrades: set openshift_client_binary fact when running on oo_first_master host
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This sets openshift_client_binary var for the first master,
as some roles use this var along with first_master_client_binary.
Not sure if its worth setting this var for the faulty roles instead though.
Signed-off-by: Vadim Rutkovsky <vrutkovs@redhat.com>
|
|/
|
|
|
|
|
| |
This commit ensures base packages are installed
for oo_all_hosts, which is what we were doing previously.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1530516
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In Ansible 2.2, the include_role directive came into existence as
a Tech Preview. It is still a Tech Preview through Ansible 2.4
(and in current devel branch), but with a noteable change. The
default behavior switched from static: true to static: false
because that functionality moved to the newly introduced
import_role directive (in order to stay consistent with include*
being dynamic in nature and `import* being static in nature).
The dynamic include is considerably more memory intensive as it will
dynamically create a role import for every host in the inventory
list to be used. (Also worth noting, there is at the time of this
writing an object allocation inefficiency in the dynamic include
that can in certain situations amplify this effect considerably)
This change is meant to mitigate the pressure on memory for the
Ansible control host.
We need to evaluate where it makes sense to dynamically include roles
and revert back to dynamic inclusion if and where it makes sense to do
so.
|
|
|
|
|
|
|
| |
Move openshift_deployment_type check into sanity_check
action plugin. Remove compatibility for deployment_type.
deployment_type has been deprecated for some time now.
|
|
|
|
|
|
| |
Move more checks outside of init/main.yml for
speeding up upgrades and other operational plays that
need to run.
|
|
|
|
|
|
|
| |
This commit moves sanity_checks tasks into a custom
action plugin that is only run against a single host.
This will result in a large reduction of tasks during initialization
|
|
|
|
|
| |
We set these variables using facts in init, no need
to duplicate the logic all around the codebase.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This commit relocates filter_plugings to lib_utils,
changes the namespacing to prevent unintended use of
older versions that may be present in filter_plugins/
directory on existing installs.
Add lib_utils to meta depends for roles
Also consolidate some plugins into lib_utils from
various other areas.
Update rpm spec, obsolete plugin rpms.
|
|\
| |
| | |
Fix rhel_subscribe boolean
|
| |
| |
| |
| |
| |
| |
| | |
This commit fixes a variable name to it's correct
name used in a when condition.
Also makes use of 'is defined' consistent.
|
|\ \
| |/
|/| |
Deprecate using Ansible tests as filters
|
| | |
|
|\ \
| | |
| | | |
Move repo and subscribe to prerequisites
|
| |/
| |
| |
| |
| | |
This commit refactors some steps in rhel subscribe and
moves repos.yml from init/main to prerequisites.
|
|\ \
| |/
|/|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Automatic merge from submit-queue.
1506750 Validate node hostname and IP address
- Replaces use of `pause` with a `fail` task. `pause` only runs on one host, therefore the check was not run for all nodes
- Adds check for valid openshift_ip
Fixes 1506750
https://bugzilla.redhat.com/show_bug.cgi?id=1506750
|
| |
| |
| |
| |
| |
| | |
Fixes 1506750
https://bugzilla.redhat.com/show_bug.cgi?id=1506750
|
|/ |
|
|
|
|
|
| |
Removing some remaining embedded etcd facts except
for the migration plays.
|
| |
|
| |
|
|
|
|
|
|
| |
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).
|
|
|
|
|
| |
This commit removes openshift.common.cli_image in
favor of openshift_cli_image.
|
|
|
|
| |
Simplify the variable is_master_system_container
|
|\
| |
| | |
Remove is_openvswitch_system_container from facts
|
| |
| |
| |
| | |
Simply logic for this variable.
|
|\ \
| |/
|/| |
Workaround the fact that package state=present with dnf fails for already installed but excluded packages.
|
| |
| |
| |
| | |
list the package with --installed.
|
|/
|
|
| |
Simplify the logic for this variable.
|
|\
| |
| | |
Cleanup etcd runtime variable.
|
| |
| |
| |
| | |
Cleaning out reference to etcd_runtime in openshift facts.
|
|\ \
| | |
| | | |
Fix node system container var
|
| |/
| |
| |
| | |
Fix logic, make it simpler.
|