| Age | Commit message (Collapse) | Author |
|
Currently, base atomic-openshift package is
installed for versioning.
This doesn't appear to be necessary.
This commit removes this step.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1504196
|
|
|
|
1504515 Correct host group for controller restart
|
|
Fixes 1504515
https://bugzilla.redhat.com/show_bug.cgi?id=1504515
|
|
|
|
The OpenShift master role already supports changing the master
configuration to refer to a project template, but there's no way to
manage that template directly. This role adds the necessary code to
generate a default template and to apply customizations using the
"yedit" module.
|
|
|
|
* Add container provider integration
* General cleanup
* Poll until service fully starts
* Add notes on multiple-provider additions
|
|
|
|
Automatic merge from submit-queue.
Set servingInfo.clientCA to ca.crt during upgrade.
https://bugzilla.redhat.com/show_bug.cgi?id=1501795
|
|
Handles spurious failures and does not get mad if it just cannot do it.
Signed-off-by: Simo Sorce <simo@redhat.com>
|
|
Initial Kuryr support
|
|
This commit enables deploying Kuryr networking on top of OpenShift in
containers. kuryr-controller is a Deployment and kuryr-cni is deployed
as DaemonSet (container will drop all CNI configuration files).
Co-Authored-By: Antoni Segura Puimedon <celebdor@gmail.com>
|
|
|
|
Currently, upgrades run a docker image pull prior to
upgrading masters and nodes for containerized installs.
If using a secure registry, and a user wishes to upgrade
their credentials due to expiry, the image pull will fail.
This commit ensures docker login credentials are updated
during upgrades, if necessary.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1503995
|
|
|
|
Add checkpoint steps for full cluster install and individual entry point playbook.
|
|
|
|
Remove unneeded master config updates during upgrades
|
|
Currently, existing clusters might not have journald
configurations applied. This may result in a rate-
limiting of important log messages on openshift-masters.
This commit ensures that journald settings are applied
during the upgrade process openshif-masters.
Fixes: https://github.com/openshift/openshift-ansible/issues/5642
|
|
Provisioning updates.
|
|
Fix prometheus role nfs
|
|
|
|
Restart all controllers to force reconfiguration during upgrade
|
|
Currently, upgrade_control_plane.yml will add any
missing sections to the openshift master's config.
These additions are only needed once. Users who
perform multiple upgrades to their clusters over
time do not need to have these variables re-inserted.
Currently, re-inserting these variables can cause
unwanted local changes.
This commit ensures that the variables are only
inserted into openshift master's config once.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1486054
|
|
|
|
Automatic merge from submit-queue.
Ensure controllerConfig.serviceServingCert is correctly set during upgrade.
A typo was originally introduced in https://github.com/openshift/openshift-ansible/pull/2449 causing an incorrect key to be configured within the master config. This commit ensures that the correct key is set during upgrades.
/cc @dlbewley
https://bugzilla.redhat.com/show_bug.cgi?id=1500981
|
|
Allow for external or internal nfs.
use facts as used for logging and metrics.
Update prometheus-alertmanager image to v0.9.1
|
|
Automatic merge from submit-queue.
Refactor openshift-management entry point
Moves checkpointing steps into path for both full cluster install and individual entry point playbook.
|
|
ingvagabund/data-migrate-of-embedded-etcd-not-allowed
Automatic merge from submit-queue.
data migration of embedded etcd not allowed
The v2->v3 migration of an embedded etcd is depricated. Instead, one needs to run:
1. `playbooks/byo/openshift-etcd/embedded2external.yml` to migrate the embedded etcd to an external one (see https://github.com/openshift/openshift-ansible/pull/5672)
2. then `playbooks/byo/openshift-etcd/migrate.yml` to migrate the v2 data to v3 data
|
|
ingvagabund/set-initial-etcd-cluster-properly-system-container-scale-up
Automatic merge from submit-queue.
Set initial etcd cluster properly during system container scale up
When a cluster is scaled up, the ETCD_INITIAL_CLUSTER must not contain etcd members that are not about to start or are not part of the etcd cluster.
Consolidating `initial_etcd_cluster` and `etcd_initial_cluster` as they do exactly the same.
Bug: 1500667
|
|
|
|
ingvagabund/default-oo_new_etcd_to_config-to-empty-list
Default groups.oo_new_etcd_to_config to an empty list
|
|
|
|
mgugino-upstream-stage/ensure_docker_containerized_lb
Automatic merge from submit-queue.
Ensure docker is installed for containerized load balancers
Currently, containerized load balancers may not have docker
configured by openshift-ansible.
This commit ensures that containerized load balancers have
the openshift_docker role applied.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1496756
|
|
- Standardize play/tasks naming
- Move install checks to separate playbook with checkpointing
- Correct 'docker_storage' tags
|
|
https://bugzilla.redhat.com/show_bug.cgi?id=1484324
|
|
Currently, containerized load balancers may not have docker
configured by openshift-ansible.
This commit ensures that containerized load balancers have
the openshift_docker role applied.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1496756
|
|
|
|
|
|
|
|
Automatic merge from submit-queue.
1499254 Ensure host was reached for proper conditional validation
If a host was unreachable during module setup, facts will not be
initialized properly and will result in later failures when stepping
through host groups. Verification that 'openshift' is defined will skip
any hosts which were previously unreachable and did not have facts
initialized.
Fixes 1499254
https://bugzilla.redhat.com/show_bug.cgi?id=1499254
|
|
|
|
Automatic merge from submit-queue.
Migrate embedded etcd to external etcd
Trello card: https://trello.com/c/9fnBfkT5/528-migrate-embedded-etcd-hosts-to-external-process?menu=filter&filter=label:committed-3.7
Tested on:
- [x] embedded etcd -> external rpm etcd
- [x] embedded etcd -> external docker etcd
One needs to set the `openshift_image_tag` before running the migration to containerized docker etcd.
|
|
Currently, debug_level is documented as a way to change
the debug output level for both masters and nodes.
debug_level does not currently have any effect.
This commit removes debug_level from openshift_facts
and properly sets openshift_master_debug_level and
openshift_node_debug_level to the value of debug_level
specified in the inventory.
This commit also reorganizes some set_fact tasks
needed during master upgrades to put all work-around
set-facts for undefined variables in one place, allowing
for easier cleanup in the future. This includes an
entry for openshift_master_debug_level.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1500164
|
|
Automatic merge from submit-queue.
Force reconciliation of role for 3.6
This is needed because in 3.6 we cannot reconcile non-cluster roles in the
bootstrap reconciliation code.
In 3.7 this is taken care of in code.
|
|
If a host was unreachable during module setup, facts will not be
initialized properly and will result in later failures when stepping
through host groups. Verification that 'openshift' is defined will skip
any hosts which were previously unreachable and did not have facts
initialized.
Fixes 1499254
https://bugzilla.redhat.com/show_bug.cgi?id=1499254
|
|
Automatic merge from submit-queue.
Allow checkpoint status to work across all groups
Conditionals placed in inventories were not being applied to localhost causing the checkpoint status to not be updated properly. Moving to the `oo_all_hosts` group will correctly pick up the conditional and apply regardless of which group (or host) may have that conditional set.
|
|
Automatic merge from submit-queue.
Fedora compat fixes for package version checks and dnf bug workaround
Fix a couple things that don't work with Fedora:
- Don't attempt to version check components, they aren't going to match the hard coded RHEL package versions.
- Work around a [bug in dnf](https://bugzilla.redhat.com/show_bug.cgi?id=1199432)'s python API that throws an error when a package spec has a `*` character in the Release field
Once we're done with review, I'll squash commits.
|
|
|