user_id may not be a uuid

[Stefano Zilli]

Gnocchi currently requires the user_id and created_by_user_id to be a valid uuid. The checks are everywhere in the code and in the indexer sql schema itself.

In some cases tho the user_id may not be a uuid. For example, where Keystone is integrated with AD, the user_id may be the actual username making it impossible to use Gnocchi as it is.

[Chris Dent (cdent)]

I feel a bit like I'm seeing hammers to nail everywhere, but would the uuid5 trick used for resource ids work here too?

[Julien Danjou (jdanjou)]

IIRC the conversion is one way only so once it's converted and stored, there's no way to display the initial value? That might be an issue for displaying "reports".

[Marek Denis (marek-denis)]

Hi, just for the record.
As od today (and since Icehouse) user_id of federated users is designated by administrators (or, to be more specific creators of the mapping rule). It can be literally everything.

[Sam Morrison (sorrison)]

Just wondering if any movement on this bug. We really want to deploy gnocchi in prod but this is a show stopper for us.
My initial thinking is changing user and project to be a string in the DB? Is that the way forward here?

[Julien Danjou (jdanjou)]

Hey Sam,

Yes, this is probably the best plan to move forward on this. I've postponed it so far because I hoped we would have enough time to make progress on our plan to have this kind of data to be dynamically created, but I guess it'll be for a next release.

Feel free to send a patch to fix that if you are able to!

[Sam Morrison (sorrison)]

Having a look now, should be pretty straight forward, now just need to learn alembic!

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/270007
Committed: https://git.openstack.org/cgit/openstack/gnocchi/commit/?id=d4f87da5d8cfce2cdab7f6dbdcc9748783f29ccd
Submitter: Jenkins
Branch: master

commit d4f87da5d8cfce2cdab7f6dbdcc9748783f29ccd
Author: Sam Morrison <email address hidden>
Date: Wed Jan 20 15:59:54 2016 +1100

    Convert user and project IDs from UUIDs to Strings

    Change-Id: Ib7e65be82cebe78761c77db667c15fedff1df05e
    Closes-Bug: #1505984