Store the validations in Swift

Bug #1625545 reported by Tomas Sedovic
This bug affects 2 people
Affects Status Importance Assigned to Milestone

Bug Description

This has been suggested as a followup to the initial validation implementation. Instead of having the validations called from a known disk location like we do now, we would keep them in Swift (similar to the way TripleO Heat templates are used).

Tags: validations
Revision history for this message
vikrant (ervikrant06) wrote :

Can you please help me to understand the advantage of keeping the validations in swift instead of default location and running it from swift object ?

AFAIK THT are kept in default location /usr/share/openstack-tripleo-heat-templates/and overcloud swift object is getting created during the installation of overcloud.

Revision history for this message
Tomas Sedovic (tsedovic) wrote :

Sorry for taking so long to respond.

You are correct that the initial THT are seeded from disk during the undercloud installation, but after that everything happens in Swift. When you do a deployment, it's the Swift templates that are used. If you want to add/change a template, you need to do it in Swift.

This proposes doing the same for the validations.

Here's a few reasons:

1. The UI can't access the disk but it can talk to Swift so if we want to manipulate the validations (e.g. adding new ones, showing their contents, etc.) it's easier this way

2. At some point we want to have a highly available undercloud and at that point, we don't want the disks to be the primary locations of data, because we'd have to sync everything and it would lead to problems. Having Swift be the single source of truth lets us avoid this.

3. We can have additional tools that read/process the data without having to be running on the undercloud

These are basically the same reasons that led to Swift being the primary storage for THT and this just follows suit.

I know this will add another layer of indirection and will make things more complex which is not making me happy either. But it makes sense here.

Revision history for this message
Ana Krivokapić (akrivoka) wrote :
Changed in tripleo:
status: Triaged → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.