FloatingIP PTR should have managed_tenant_id enforced

Bug #1437060 reported by Endre Karlson
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Designate
Fix Released
Medium
Endre Karlson

Bug Description

If you set a FIP PTR atm the reverse zone will be created as teh context tenant if no config option is set. We should blow up with a ConfigurationError if the managed_tenant_id is not set and disallow the set operation

Revision history for this message
Endre Karlson (endre-karlson) wrote :

We'll set the default for the option instead to 00000000-0000-0000-0000-000000000000 and log a warning on central startup.

Changed in designate:
assignee: nobody → Endre Karlson (endre-karlson)
Changed in designate:
milestone: none → liberty-1
importance: Undecided → Medium
status: New → Confirmed
status: Confirmed → Won't Fix
status: Won't Fix → Triaged
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to designate (master)

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

Changed in designate:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to designate (master)

Reviewed: https://review.openstack.org/168310
Committed: https://git.openstack.org/cgit/openstack/designate/commit/?id=bdab598b6a58c37473c45602d2d8a92a78bed3f3
Submitter: Jenkins
Branch: master

commit bdab598b6a58c37473c45602d2d8a92a78bed3f3
Author: Endre Karlson <email address hidden>
Date: Fri Mar 27 12:45:43 2015 +0100

    Fix ownership issue with Zone's created on set fip

    When a user sets a FloatingIP a Reverse Zone is created that's owned by the
    system but if no managed_resource_tenant_id option was set then the Zone would
    get owned by the context.tenant_id when create_domain() is called in
    set_floatingip.

    We fix it by changing the default managed_resource_tenant_id to
    "00000000-0000-0000-0000-000000000000".

    Closes-Bug: #1437060

    Change-Id: I491309d92a9e3fba2f42af1ae85372b6258d53f3

Changed in designate:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in designate:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in designate:
milestone: liberty-1 → 1.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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