1. 21 Oct, 2016 1 commit
  2. 19 Oct, 2016 1 commit
  3. 13 Oct, 2016 2 commits
  4. 12 Oct, 2016 2 commits
  5. 11 Oct, 2016 1 commit
  6. 06 Oct, 2016 1 commit
  7. 05 Oct, 2016 3 commits
  8. 03 Oct, 2016 1 commit
  9. 21 Sep, 2016 3 commits
  10. 20 Sep, 2016 13 commits
  11. 19 Sep, 2016 2 commits
  12. 16 Sep, 2016 1 commit
    • Thomas Markwalder's avatar
      [5007] Addressed review comments · 1a986cf4
      Thomas Markwalder authored
      This is solution #2:
      
      src/lib/dhcpsrv/alloc_engine.h
      src/lib/dhcpsrv/alloc_engine.cc
          AllocEngine::extendLease6() - replaced logic to set the context flags with
          simply adding the original lease to the changed_leases_ list.
      
          AllocEngine::updateLeaseData() - removed logic to set the context flags.
      
      src/bin/dhcp6/dhcp6_srv.cc
          Dhcpv6Srv::createNameChangeRequests() - replaced the context flag check
          with logic which looks for candidate IA addresses in the ctx.changed_leases_
          list.  If found and the FDQN doman name has not changed, we move on to the
          next candidate.
      
      src/bin/dhcp6/tests/fqdn_unittest.cc
          TEST_F(FqdnDhcpv6SrvTest, createNameChangeRequestsNoAddr) - removed
          testing of context flag permutations
      1a986cf4
  13. 15 Sep, 2016 3 commits
    • Thomas Markwalder's avatar
      [5007] Suppress DDNS updates on DHCPv6 lease renewals unless the FQDN changes · 46f4c9fd
      Thomas Markwalder authored
      src/lib/dhcpsrv/alloc_engine.h
      src/lib/dhcpsrv/alloc_engine.cc
          - AllocEngine::extendLease6()
          - AllocEngine::updateLeaseData() - logic was added to clear the context
          DNS update flags when the renewal does not alter the lease's FQDN.
      
      src/bin/dhcp6/dhcp6_srv.h
      src/bin/dhcp6/dhcp6_srv.cc
          - Dhcpv6Srv::createNameChangeRequests() - added context as second parameter,
          and modified function to return without creating NCR(s) if both update flags
          in the context are false.
      
      src/bin/dhcp6/tests/fqdn_unittest.cc
          - TEST_F(FqdnDhcpv6SrvTest, createNameChangeRequests) - modified to verify
          combinations of context update flags
      
          - TEST_F(FqdnDhcpv6SrvTest, DISABLED_processTwoRequests) - renamed
          to TEST_F(FqdnDhcpv6SrvTest, processTwoRequestsDiffFqdn) and enabled.
          It had been disabled pending 3677 which has been completed.
      
          - TEST_F(FqdnDhcpv6SrvTest, processTwoRequestsSameFqdn)  - new test
          which verifies that client "renewing" a lease by sending a second
          request with the different FQDN generates the correct NCRs
      
          - TEST_F(FqdnDhcpv6SrvTest, DISABLED_processRequestRenew) - renamed
          to TEST_F(FqdnDhcpv6SrvTest, processRequestRenewDiffFqdn) and enabled.
          It had been disabled pending 3677 which has been completed.
      
          - TEST_F(FqdnDhcpv6SrvTest, processRequestRenewSameFqdn) - new test
          which verifies that client renewing a lease by sending a renew
          with the same FQDN does NOT generate any NCRs
      46f4c9fd
    • Marcin Siodelski's avatar
    • Marcin Siodelski's avatar
  14. 14 Sep, 2016 1 commit
  15. 13 Sep, 2016 5 commits