Commit Graph

13 Commits

Author SHA1 Message Date
Gael Chamoulaud 98314886b3 Deletes validations directory
All the validations directory contents have been migrated into Ansible
Roles then this directory could be now deleted.

Change-Id: I41328b2f1e6d8e8d695c0794557567f07c509874
Implements: blueprint validation-framework
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
2019-07-01 11:32:28 +02:00
Gael Chamoulaud c08460c707 Fix ansible lint errors
The gate is now using the last ansible-lint version (4.0.1) which have new
ids/tags and new rules. The old tags (ANSIBLE*) should remains backwardly
compatible but this patch is using the new ones.

This patch fixes all the lint issues.

Change-Id: Ic151f3e969b8e3729394fd477bb8341d1cb75fd7
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
2019-01-07 15:33:28 +01:00
Jose Luis Franco Arza 864ab1d64d Run validation only if overcloud exists.
When no overcloud is deployed, the openstack-endpoint.yaml validations
fail. This patch re-orders couple of tasks and runs the failing task for
unavailable endpoints only when an overcloud exists.

Change-Id: I174b5a239a7c1ab7b93323654a417db8f087bc27
Story: #2002061
2018-05-31 11:46:09 +02:00
Carlos Camacho c8cefbd9e9 Use clean keystone URL in the keystone validation
Inventory variable overcloud_keystone_url gets its
value from stack's output 'KeystoneURL'
which might include the keystone version.
We need to get rid off the version, otherwise
undercloud validations fail at the time of
checking the Keystone endpoint [0].

[0] - 545e4924e3/validations/openstack-endpoints.yaml (L28)

Co-Authored-By: Jose Luis Franco <jfrancoa@redhat.com>
Story: #2001897

Change-Id: I3e900afd840a4a0504a573a1eabe80911f8f7881
2018-05-08 11:29:36 +02:00
Florian Fuchs 496ab519fe Fix overcloud services connectivity validation
The overcloud services validation uses the Keystone API version 2 to generate a
token. This doesn't work anymore since Queens.

This patch changes the validation to use  API version 3 instead.

Closes-Bug: #1757187

Change-Id: I4e080521a334fca4999b9bc6e3e1bcacd24c476b
2018-03-21 12:08:45 +01:00
Carlos Camacho 2b24b3dc55 Invert validation task order
We should check that the Horizon endpoint is
configured correctly before running the validation.

Change-Id: Ib0b1c7e68711222e0f8326a268fde7b100a92e87
Related-Bug: 1742947
2018-01-18 15:08:04 +01:00
Gael Chamoulaud 6406ef8bd8 Add ansible-lint script
This patch adds tools/ansible-lint.sh script to lint validations playbooks.
Moreover, it adds `changed_when: False` to commands that should not change
things if nothing needs doing [ANSIBLE0012].

Change-Id: I4fb432bfbd2b73740c29011d08a09058b6c45348
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
2017-12-01 09:29:51 +01:00
marios 94290a9eaa Remove the pre-update group and add a post-upgrade group
The post-upgrade group will be invoked from
I32330a05fa142fc92d73f68103429c4c2c1816ab

Change-Id: I31075498a9d395603ed5e40d65ea3db6d89adc6a
2017-07-07 18:17:02 +03:00
marios 0b608b2e6a Adds pre-upgrade/update to the overcloud endpoint validation
This adds the pre-upgrade and pre-update groups to the existing
overcloud endpoint validation which checks horizon and keystone

Change-Id: Iebccce31a7a12f72f22305a977085d1a1d886b81
2017-06-14 15:45:52 +03:00
Jenkins cc5e584724 Merge "Don't rely on overcloudrc" 2016-12-16 13:37:32 +00:00
Tomas Sedovic 8927704efa Don't rely on overcloudrc
We were reading overcloudrc for the openstack-endpoints validation.
However, the name of tha file may differ if the deployment plan is
called something else than "overcloud" and more importantly, the GUI
deployments don't create the file on the undercloud at all.

This updates the Ansible inventory to read the admin password and
keystone url for the specified plan if available and the validation to
use those values instead of relying on overcloudrc.

Change-Id: I9ff26016c43888267968ef2588f686f7de95bda3
Closes-Bug: #1643808
2016-12-01 16:47:25 +01:00
MikeG451 875882a41c Remove TODO in Validation description, revise description
Move TODO text so it will not appear in description and GUI.
Revise description to better describe behavior of validation.

Change-Id: I580e26dfb74c299340e1cba6a4c1aca10edf180a
Closes-bug: #1643541
2016-11-29 16:39:42 +00:00
Tomas Sedovic b8eda48e12 Validation to verify connectivity to overcloud
Change-Id: Ibdc304cba260f180308ac5cebe9f86f589299629
Co-Authored-By: John Browning <jecbrowning@gmail.com>
2016-10-17 08:33:04 +00:00