Bug #2985

DHCP LeaseStatus prop persisted to DB

Added by Davide Principi almost 5 years ago. Updated over 4 years ago.

Status:CLOSEDStart date:
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:nethserver-dnsmasq
Target version:v6.5
Security class: Resolution:
Affected version:v6.5-final NEEDINFO:No

Description

Hosts DB entries have a LeaseStatus prop set that must not be there, as it must be calculated each time from dnsmasq cache entries, to keep the view consistent.

Steps to reproduce:
  • create a reservation and save
  • edit, changing the Description field, then save again

The LeaseStatus prop has been persisted to DB.

Associated revisions

Revision 57b06086
Added by Davide Principi almost 5 years ago

Claim db/hosts RPM ownership. Refs #2985

Revision 4659ebc0
Added by Davide Principi almost 5 years ago

Remove LeaseStatus prop in hosts DB. Refs #2985

Requires nethserver-hosts > 1.1.0-1.ns6

History

#1 Updated by Davide Principi almost 5 years ago

  • Subject changed from : LeaseStatus prop to DHCP expired leases always visible

Some entries under DHCP > IP reservation page never disappear.

Hosts DB entries have a LeaseStatus prop set that should not be there.

#2 Updated by Davide Principi almost 5 years ago

  • Description updated (diff)

#3 Updated by Davide Principi almost 5 years ago

  • NEEDINFO changed from No to Yes

Add reproducibility infos

#4 Updated by Giacomo Sanchietti almost 5 years ago

Migration fragment to remove LeaseStatus property:

{
    use esmith::HostsDB;
    my $db = esmith::HostsDB->open();
    foreach ($db->get_all_by_prop( type => 'local' )) {
        my $l = $_->prop('LeaseStatus') || next;
        $_->delete_prop('LeaseStatus');
    }
}

#5 Updated by Davide Principi almost 5 years ago

  • Subject changed from DHCP expired leases always visible to DHCP LeaseStatus prop persisted to DB
  • Description updated (diff)
  • Status changed from NEW to TRIAGED
  • % Done changed from 0 to 20
  • NEEDINFO changed from Yes to No

#6 Updated by Davide Principi almost 5 years ago

  • Status changed from TRIAGED to ON_DEV
  • Assignee set to Davide Principi
  • % Done changed from 20 to 30

#7 Updated by Davide Principi almost 5 years ago

  • Status changed from ON_DEV to MODIFIED
  • Assignee deleted (Davide Principi)
  • % Done changed from 30 to 60

#8 Updated by Davide Principi almost 5 years ago

  • Category set to nethserver-dnsmasq
  • Status changed from MODIFIED to ON_QA
  • % Done changed from 60 to 70

In nethserver-testing (6.5):
nethserver-dnsmasq-1.4.0-1.1git4659ebc.ns6.noarch.rpm
nethserver-hosts-1.1.0-1.3git57b0608.ns6.noarch.rpm

#9 Updated by Giovanni Bezicheri almost 5 years ago

  • Assignee set to Giovanni Bezicheri

#10 Updated by Giovanni Bezicheri almost 5 years ago

  • Status changed from ON_QA to VERIFIED
  • Assignee deleted (Giovanni Bezicheri)
  • % Done changed from 70 to 90

#11 Updated by Giacomo Sanchietti over 4 years ago

  • Status changed from VERIFIED to CLOSED
  • % Done changed from 90 to 100
Released in nethserver-updates:
  • nethserver-dnsmasq-1.4.1-1.ns6.noarch.rpm
  • nethserver-hosts-1.1.1-1.ns6.noarch.rpm

Also available in: Atom PDF