Bug 1205729 - TFTP Servername can be set incorrectly
Summary: TFTP Servername can be set incorrectly
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Installation
Version: 6.0.4
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: Unspecified
Assignee: Dmitri Dolguikh
QA Contact: Kedar Bidarkar
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-03-25 14:34 UTC by Stephen Benjamin
Modified: 2017-02-23 20:14 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-08-12 13:56:40 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 9896 0 None None None 2016-04-22 15:13:35 UTC

Description Stephen Benjamin 2015-03-25 14:34:20 UTC
/etc/foreman-proxy/settings.d/tftp.yaml has tftp_servername set always, but the logic in the foreman-proxy module looks for ipaddress_eth0, and then if not present, ipaddress.  

This totally breaks on el7/newer Fedora, where you're not guaranteed to have eth0 anymore, and if you have something like the "docker0" interface, because then that becomes $::ipaddress. All your hosts next-server is set to the docker0 interface IP.

Why set this parameter tftp_servername at all?   Seems to work without it.

Comment 1 Stephen Benjamin 2015-03-25 14:34:22 UTC
Created from redmine issue http://projects.theforeman.org/issues/9896

Comment 3 Bryan Kearney 2015-04-17 10:06:27 UTC
Moving to POST since upstream bug http://projects.theforeman.org/issues/9896 has been closed
-------------
Anonymous
Applied in changeset commit:puppet-foreman_proxy|c762f632d4bb4045491c60d02f1b568fbeb133ab.

Comment 6 Kedar Bidarkar 2015-05-11 11:54:03 UTC
Now tftp-servername parameter is not set automatically.

[root@xxx ~]# katello-installer --help | grep -i tftp-servername
    --capsule-tftp-servername     Defines the TFTP server name to use, overrides the name in the subnet declaration (default: nil)


By default it is now set to: nil


VERIFIED with Sat6.1-GA-SNAP3-compose2

Comment 7 Bryan Kearney 2015-08-11 13:20:37 UTC
This bug is slated to be released with Satellite 6.1.

Comment 8 Bryan Kearney 2015-08-12 13:56:40 UTC
This bug was fixed in version 6.1.1 of Satellite which was released on 12 August, 2015.


Note You need to log in before you can comment on or make changes to this bug.