oslo.cache changes break sharding

Bug #1273377 reported by Allele Dev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Fix Released
Critical
Allele Dev
Icehouse
Fix Released
Critical
Allele Dev

Bug Description

oslo.cache no longer exposes an unset method - only unset_many exists now. As a result, the implementation of sharding is broken.

The fix needs to be committed to: marconi.queues.storage.sharding

Revision history for this message
Allele Dev (alleledev) wrote :
Kurt Griffiths (kgriffs)
Changed in marconi:
status: New → In Progress
milestone: none → icehouse-3
Allele Dev (alleledev)
Changed in marconi:
assignee: nobody → Alejandro Cabrera (cpp-cabrera)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to marconi (master)

Reviewed: https://review.openstack.org/68267
Committed: https://git.openstack.org/cgit/openstack/marconi/commit/?id=280da9d054a521652e9a3c6cb95001fed2985eab
Submitter: Jenkins
Branch: master

commit 280da9d054a521652e9a3c6cb95001fed2985eab
Author: Alejandro Cabrera <email address hidden>
Date: Tue Jan 21 16:13:23 2014 -0500

    fix(shards): guard against registering bad shard URIs

    Whenever an admin would attempt to register a new shard, the shard was
    admitted whether or not the shard URI actually existed, and whether or
    not marconi could handle shards of that type.

    This patch makes it so that a DataDriver instance is temporarily
    instantiated prior to storing the shard entry. This instance uses
    is_alive() to determine whether or not the shard can be reached/is
    supported. This applies to both PUT/PATCH. HTTP 400 is returned if the
    registration fails, as a result.

    To enable this change, the ability to create a dynamic configuration
    object was extracted from sharding.Catalog. Tests were written for
    this new function.

    The unit tests were updated accordingly, expanding the coverage of
    test_shards to include the new failure cases.

    Some changes needed to be made to handle new oslo.cache interface:
    - cache.unset doesn't exist - now using cache.unset_many
    - cache.get_cache() takes a URI, not a ConfigOpts

    Change-Id: I40509f525466da01baad2a5aef81c7b99c8d2f97
    Closes-Bug: #1258591
    Closes-Bug: #1273376
    Closes-Bug: #1273377

Changed in marconi:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in marconi:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in marconi:
milestone: icehouse-3 → 2014.1
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.