[openstack-dev] [sahara][qa] Unblocking Sahara gate

Ghanshyam Mann ghanshyam.mann at nectechnologies.in
Thu Oct 13 06:41:42 UTC 2016


I think this is issue more the way plugin was using the config options.

Sahara tests plugin should have used the options in their own way or implement same way it was done in Tempest (define self. data-processing etc) instead of partially relying on tempest defined variable.

Honestly saying, I do not prefer of adding the automapping in tempest side. 

[1]  will fix the issue and unblock the sahara tests. And I saw sahara plugin is also branchless so should be ok with just this patch ?

..1 - https://review.openstack.org/#/c/385701/


Thanks & Regards,
Ghanshyam Mann

> -----Original Message-----
> From: Luigi Toscano [mailto:ltoscano at redhat.com]
> Sent: 13 October 2016 00:37
> To: openstack-dev at lists.openstack.org
> Subject: [openstack-dev] [sahara][qa] Unblocking Sahara gate
> 
> Hi all,
> 
> the removal of the Sahara tests from Tempest [1] broke the tests now in
> sahara-tests, and basically I underestimated the fall-out of the removal.
> Apart from a minor issue due to the wrong exception [2], the key issue
> comes from the option groups in tempest.conf, which are defined as data-
> processing and data-processing-feature-available, while the code tries to
> access CONF.data_processing. Previously this was handled by some magic
> mapping [3], which can't be used, as it is, by plugins.
> I came up with this solution [4] after discussing with Andrea Frittoli. The
> patches only fails with the automation negative tests, which are on the way
> of the Dodo anyway [5].
> The alternative fix for sahara would involve patching the configured
> tempest.conf in few branches, in addition to the fixes to sahara-tests, and I'd
> frankly prefer a more general solution.
> 
> My question is: can you please approve [5] and [4], so that Sahara gates can
> be unlocked?
> 
> 
> [1] https://review.openstack.org/#/c/380082/
> [2] https://review.openstack.org/#/c/385336/
> [3]
> http://git.openstack.org/cgit/openstack/tempest/tree/tempest/config.py?
> h=13.0.0#n1239
> [4] https://review.openstack.org/#/c/385460/
> [5] https://review.openstack.org/#/c/380982/
> 
> Ciao
> --
> Luigi
> 
> __________________________________________________________
> ________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-
> request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev



More information about the OpenStack-dev mailing list