renaming a DNS record to a previous name fails with error: list.remove(x): x not in list

Bug #1996419 reported by Alberto Donato
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MAAS
Fix Released
Medium
Unassigned
3.2
Fix Released
Medium
Unassigned

Bug Description

The following fails with the "list.remove(x): x not in list" error:

- create an A record in a domain with name "test"
- edit the entry and rename to "test-new"
- edit the entry and rename to "test" again -> fails

This is somewhat similar to https://bugs.launchpad.net/maas/+bug/1894894

Tags: dns ui
Alberto Donato (ack)
description: updated
Revision history for this message
Alberto Donato (ack) wrote :

I think this is another incarnation of LP:1894894, (see https://bugs.launchpad.net/maas/+bug/1894894/comments/3) specifically for the cause

tags: added: ui
Revision history for this message
Alberto Donato (ack) wrote :

On a related notice, the UI callse "domain.delete_address_record" to remove the entry, but this only deletes the address, not the dnsresource, so the original one ("test" in the example above) is still there, without any address.

When attempting to revert to the old name, the entry is found, so the logic tries to remove the old address (which hasn't really changed), triggering the error.

Changed in maas:
importance: High → Medium
tags: added: dns
Changed in maas:
status: Triaged → Fix Committed
Revision history for this message
Adam Collard (adam-collard) wrote :
Changed in maas:
milestone: 3.3.0 → 3.3.0-beta3
status: Fix Committed → Fix Released
Revision history for this message
Thorsten Merten (thorsten-merten) wrote :

This is fix released on the maas-ui side.

As we cannot change the status for the maas-ui project for imported items, I'll remove the dependency to maas-ui.

no longer affects: maas-ui
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.