Docs: Email/TTL is not NULL for secondary zones.

Bug #1507965 reported by sonu
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Designate
Confirmed
Low
Unassigned

Bug Description

While creating secondary zones , email is set to some default value.

vyom@vyom:~$ openstack zone create temp.org. --type SECONDARY --masters vyom.org.

+----------------+--------------------------------------+
| Field | Value |
+----------------+--------------------------------------+
| action | CREATE |
| created_at | 2015-10-20T09:51:07.000000 |
| description | None |
| email | <email address hidden> |
| id | 059572a2-b963-484f-8b72-e18d8fa9774e |
| masters | vyom.org. |
| name | temp.org. |
| pool_id | 794ccc2c-d751-44fe-b57f-8894c9f5c842 |
| project_id | 237404bcb8a34bc88f1a401c7439654b |
| serial | 1 |
| status | PENDING |
| transferred_at | None |
| ttl | 3600 |
| type | SECONDARY |
| updated_at | None |
| version | 1 |
+----------------+--------------------------------------+

But according to http://docs.openstack.org/developer/designate/rest/v2/zones.html

email should be NULL for secondary zones.

Revision history for this message
Tim Simmons (timsim) wrote :

The docs are incorrect. Email/TTL are inherited from the Primary on the first transfer. Since these columns are non-nullable, there needs to be a default there.

The docs should be corrected.

summary: - Email is set to some default value for secondary zones.
+ Docs: Email/TTL is not NULL for secondary zones.
Changed in designate:
status: New → Triaged
importance: Undecided → Low
status: Triaged → Confirmed
Changed in designate:
assignee: nobody → Maciej Szankin (mszankin)
tags: added: docs
Revision history for this message
KATO Tomoyuki (kato-tomoyuki) wrote :

Hi Maciej, are you still working on this bug?

Revision history for this message
Maciej Szankin (mszankin) wrote :

Wanted, but got no time. Freeing it up for you :)

Changed in designate:
assignee: Maciej Szankin (mszankin) → nobody
Revision history for this message
KATO Tomoyuki (kato-tomoyuki) wrote :

Maciej, thanks. I'll try this one.

Changed in designate:
assignee: nobody → KATO Tomoyuki (kato-tomoyuki)
tags: added: low-hanging-fruit
Revision history for this message
Sharat Sharma (sharat-sharma) wrote :

Hi,
Assigning it to myself since there is no activity since 4 months.

Changed in designate:
assignee: KATO Tomoyuki (kato-tomoyuki) → Sharat Sharma (sharat-sharma)
status: Confirmed → In Progress
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/321538

Changed in designate:
status: In Progress → New
assignee: Sharat Sharma (sharat-sharma) → nobody
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on designate (master)

Change abandoned by Sharat Sharma (<email address hidden>) on branch: master
Review: https://review.openstack.org/321538

Tim Simmons (timsim)
Changed in designate:
status: New → Confirmed
Revision history for this message
Samantha Blanco (sblanco1) wrote :

Is this still considered an open bug? It looks like the files in the original commit no longer exist and the documentation I've seen that does discuss secondary zones (https://docs.openstack.org/developer/designate/howtos/secondary-zones.html) appears to have been updated with the correct information.

Revision history for this message
Benjamin Diaz (benjamin.diaz) wrote :

According to the current api-ref, the email parameter is now required [1]. Probably should close this bug.

[1] https://developer.openstack.org/api-ref/dns/

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.