Bug 1107670

Summary: DHCP validation errors when proxy returns nil
Product: Red Hat Satellite Reporter: Bryan Kearney <bkearney>
Component: ProvisioningAssignee: Greg Sutcliffe <gsutclif>
Status: CLOSED CURRENTRELEASE QA Contact: Tazim Kolhar <tkolhar>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.3CC: bbuckingham, cwelton, dcleal, gsutclif, jmontleo, lzap, stbenjam, tkolhar
Target Milestone: UnspecifiedKeywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
URL: http://projects.theforeman.org/issues/5941
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-09-11 12:26:36 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Bryan Kearney 2014-06-10 12:36:34 UTC
If the proxy has no record of a given lease, the DHCP .valid? method will bomb out, instead of correctly returning false.

Comment 1 Bryan Kearney 2014-06-10 12:36:36 UTC
Created from redmine issue http://projects.theforeman.org/issues/5941

Comment 3 Bryan Kearney 2014-06-10 13:03:39 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/5941 has been closed

Comment 6 Tazim Kolhar 2014-08-27 12:28:46 UTC
please provide verification steps

Comment 7 Dominic Cleal 2014-09-01 13:05:45 UTC
Developer verified via console, but it's internal code that shouldn't really be here.

irb(main):005:0* h.dhcp_record     
=> redacted-52:54:00:2d:c5:9b/192.168.100.5
irb(main):006:0> d = h.dhcp_record
=> redacted-52:54:00:2d:c5:9b/192.168.100.5
irb(main):007:0> d.mac = '53:54:00:2d:c5:9b'
=> "53:54:00:2d:c5:9b"
irb(main):008:0> d
=> redacted-53:54:00:2d:c5:9b/192.168.100.5
irb(main):009:0> d.valid?
=> false

Comment 8 Tazim Kolhar 2014-09-02 07:47:50 UTC
VERIFIED

as discussed with the developer taking this as verified

Comment 9 Bryan Kearney 2014-09-11 12:26:36 UTC
This was delivered with Satellite 6.0 which was released on 10 September 2014.