| Age | Commit message (Collapse) | Author |
|---|
|
Automatic merge from submit-queue.
Limit host scope during plays
Many plays only target a select subset of hosts,
especially oo_first_master for components such
as logging and registry.
This commit limits the scope of most plays to
eliminate unnecessary task execution on node
groups. This will result in great time
savings for large deployments.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1516526
|
|
Many plays only target a select subset of hosts,
especially oo_first_master for components such
as logging and registry.
This commit limits the scope of most plays to
eliminate unnecessary task execution on node
groups. This will result in great time
savings for large deployments.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1516526
|
|
Automatic merge from submit-queue.
Changing the check for the number of etcd nodes
This playbook is called (via std_include.yml) when the scale up playbook for either master or etcd is called. In the scenario where you are scaling up the number of masters/etcd nodes it is feasible, if not likely, that the number of etcd nodes is not 1, 3 or 5 and this check therefore causes a scale up to fail.
The two example scenarios that are driving this change are:
You have a cluster with 3 master nodes (each running etcd) and one of those masters fail. The master node is removed from both the OpenShift cluster and the etcd cluster and the inventory updated to reflect the state of the cluster minus the failed master node. You would then run the scale up playbook to add a new master / etcd master into the cluster using an inventory containing and etcd group of just 2 nodes.
As above but the cluster has 5 master nodes. If you lose a master node and update the inventory to reflect that then the inventory will contain an etcd group with 4 nodes.
@sdodson
Previously submitted as https://github.com/openshift/openshift-ansible/pull/6979
|
|
In making the initial change I introduce some spaces at the beginning of the line. Removing them.
|
|
|
|
Hosts will need python ipaddress module installed if it was not
installed during initial installation.
Bug 1540537
https://bugzilla.redhat.com/show_bug.cgi?id=1540537
|
|
'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
|
|
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1530403
|
|
Computed as the first ip address in the portal_net
|
|
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 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
|
|
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>
|
|
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
|
|
Deprecate using Ansible tests as filters
|
|
This commit fixes a variable name to it's correct
name used in a when condition.
Also makes use of 'is defined' consistent.
|
|
Move repo and subscribe 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
|
|
This commit refactors some steps in rhel subscribe and
moves repos.yml from init/main to prerequisites.
|
|
|
|
|
|
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
|
|
Workaround the fact that package state=present with dnf fails for already installed but excluded packages.
|
|
Simply logic for this variable.
|
