Project

General

Profile

Bug #24591

Incorrect error reporting for MS DHCP

Added by Lukas Zapletal 4 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Category:
DHCP
Target version:
Difficulty:
Triaged:
No
Bugzilla link:
Team Backlog:
Fixed in Releases:

Description

When request fail, error is not reported correctly and this is thrown instead:

E, [2018-08-10T19:55:09.502316 42606010] ERROR -- : undefined local variable or method `opts' for ms dhcp server:Proxy::DHCP::NativeMS::Provider
D, [2018-08-10T19:55:09.502316 42606010] DEBUG -- : undefined local variable or method `opts' for ms dhcp server:Proxy::DHCP::NativeMS::Provider (NameError)
C:/smart-proxy-1.18.0/modules/dhcp_native_ms/dhcp_native_ms_main.rb:224:in `rescue in build_lease'
C:/smart-proxy-1.18.0/modules/dhcp_native_ms/dhcp_native_ms_main.rb:212:in `build_lease'
C:/smart-proxy-1.18.0/modules/dhcp_native_ms/dhcp_native_ms_main.rb:209:in `block in all_leases'
C:/smart-proxy-1.18.0/modules/dhcp_native_ms/dhcp_native_ms_main.rb:209:in `map'
C:/smart-proxy-1.18.0/modules/dhcp_native_ms/dhcp_native_ms_main.rb:209:in `all_leases'
C:/smart-proxy-1.18.0/modules/dhcp_native_ms/dhcp_native_ms_main.rb:109:in `unused_ip'
C:/smart-proxy-1.18.0/modules/dhcp/dhcp_api.rb:34:in `block in <class:DhcpApi>'
C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/sinatra-2.0.3/lib/sinatra/base.rb:1635:in `call'
C:/Ruby24-x64/lib/ruby/gems/2.4.0/gems/sinatra-2.0.3/lib/sinatra/base.rb:1635:in `block in compile!'

Related issues

Related to Smart Proxy - Bug #18070: When adding a new dhcp reservation, ip and mac addresses should be checked not only against existing reservations but leases as wellClosed2017-01-13
Related to Foreman - Bug #25024: Failed to fetch a free IP from proxy - undefined local variable or method `opts' for ms dhcp server:Proxy::DHCP::NativeMS::ProviderDuplicate

Associated revisions

Revision 45b67ee4 (diff)
Added by Lukas Zapletal 4 months ago

Fixes #24591 - fixed debug message in MSDHCP

History

#1 Updated by The Foreman Bot 4 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/603 added

#2 Updated by Tomer Brisker 4 months ago

  • Related to Bug #18070: When adding a new dhcp reservation, ip and mac addresses should be checked not only against existing reservations but leases as well added

#3 Updated by Tomer Brisker 4 months ago

  • Found in Releases 1.15.0, 1.15.1, 1.15.3, 1.15.4, 1.16.0, 1.16.1, 1.17.0, 1.17.2 added
  • Fixed in Releases added

#4 Updated by Anonymous 4 months ago

  • Status changed from Ready For Testing to Closed

#5 Updated by Ewoud Kohl van Wijngaarden 4 months ago

  • Found in Releases 1.18.1 added
  • Found in Releases deleted ()
  • Fixed in Releases 1.20.0 added
  • Fixed in Releases deleted ()

#6 Updated by Ewoud Kohl van Wijngaarden 4 months ago

  • Target version set to 1.20.0

#7 Updated by Ewoud Kohl van Wijngaarden 4 months ago

  • Found in Releases 1.18.0 added
  • Found in Releases deleted ()

#8 Updated by Lukas Zapletal 3 months ago

  • Related to Bug #25024: Failed to fetch a free IP from proxy - undefined local variable or method `opts' for ms dhcp server:Proxy::DHCP::NativeMS::Provider added

Also available in: Atom PDF