[queues/sharding] Add functional tests

Bug #1246757 reported by Allele Dev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Won't Fix
Medium
Victoria Martinez de la Cruz

Bug Description

The new /v1/shards transport needs some functional tests. These should cover the functionality described here: https://wiki.openstack.org/wiki/Marconi/specs/sharding/v1

The tests should be run as part of the gate , using a sharded setup

Kurt Griffiths (kgriffs)
Changed in marconi:
importance: Undecided → High
milestone: none → icehouse-3
status: New → Triaged
description: updated
Revision history for this message
Kurt Griffiths (kgriffs) wrote :

Let's tackle this in J-1

no longer affects: marconi/icehouse
Changed in marconi:
milestone: icehouse-3 → none
no longer affects: zaqar/juno
Changed in zaqar:
milestone: juno-3 → juno-rc1
importance: High → Medium
Changed in zaqar:
milestone: juno-rc1 → next
Revision history for this message
Flavio Percoco (flaper87) wrote :

Shards have been renamed to pools and a functional test suite has been added: https://git.openstack.org/cgit/openstack/zaqar/tree/tests/functional/wsgi/v1_1/test_pools.py

Changed in zaqar:
assignee: Victoria Martínez de la Cruz (vkmc) → Flavio Percoco (flaper87)
status: Triaged → Fix Released
milestone: next → none
status: Fix Released → Fix Committed
milestone: none → kilo-1
Changed in zaqar:
status: Fix Committed → Triaged
assignee: Flavio Percoco (flaper87) → Victoria Martínez de la Cruz (vkmc)
Revision history for this message
Victoria Martinez de la Cruz (vkmc) wrote :

Adding the pools tests for v1 would mean duplicating a lot of code.

But... this is not only true for pools... currently existing tests also perform really similar tasks.

I'll open a bug report so we can follow the refactor somewhere else. After the refactoring this should be covered as well :)

Revision history for this message
Victoria Martinez de la Cruz (vkmc) wrote :

I'll submit the tests for this and we can later refactor everything.

Revision history for this message
Victoria Martinez de la Cruz (vkmc) wrote :

Given that v1 is freezed it doesn't make much sense to add the missing functional tests for it. Adding those tests would mean duplicating a lot of code. For that reasons, I'm closing this bug.

Changed in zaqar:
status: Triaged → Won't Fix
Changed in zaqar:
milestone: kilo-1 → 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.