Revisit MARCONI_TEST_MONGODB

Bug #1350068 reported by Kurt Griffiths
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Won't Fix
Undecided
Kurt Griffiths

Bug Description

We've had some bugs get merged because they weren't tested against mongodb. I'd like to propose removing that flag, and requiring devs and the gate to always test against mongo. Furthermore, we should audit our tests and make sure that the functional tests, at least, are excercising ALL drivers for every test (sqla, mongo, and redis).

Revision history for this message
Nataliia Uvarova (grafinya-uvarova) wrote :

Maybe, we can not remove the flag totally, but instead invert its behavior - it will disable tests on some drivers? It will allow to do hacking on marconi even during the holiday on old granny's netbook ;)
Since most of the people stick to the default (and I personally not sure if all devs are aware of this flag), the tests will be run on the gate and on most devs machines.

Revision history for this message
Kurt Griffiths (kgriffs) wrote :

Sounds good to me.

no longer affects: marconi/kilo
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to marconi (master)

Fix proposed to branch: master
Review: https://review.openstack.org/111817

Changed in marconi:
status: Triaged → In Progress
Revision history for this message
Kurt Griffiths (kgriffs) wrote :

Since we can't set vars in the gate and only py27 runs on trusty, we will have to just enable this flag in tox instead of what was proposed above.

no longer affects: marconi/juno
Changed in marconi:
importance: High → Undecided
status: In Progress → Won't Fix
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on marconi (master)

Change abandoned by Flavio Percoco (<email address hidden>) on branch: master
Review: https://review.openstack.org/111817
Reason: Superseded by Ib751048a9ed1d7bb33ef80a5ed9e9cf7bdf71d57

Thierry Carrez (ttx)
Changed in marconi:
milestone: juno-3 → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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