Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

makedhcp misleading error message #5649

Closed
Obihoernchen opened this issue Sep 20, 2018 · 4 comments
Closed

makedhcp misleading error message #5649

Obihoernchen opened this issue Sep 20, 2018 · 4 comments
Assignees
Milestone

Comments

@Obihoernchen
Copy link
Member

Obihoernchen commented Sep 20, 2018

If you do net set xcatmaster for your and the node is in a remote network which isn't directly attached to the xCAT server the following error is shown:

[root@xcat ~]# makedhcp <node>
Error: [xcat]: Unable to determine the tftpserver for <node>

This is misleading because tftpserver was set for all networks.
The real issue was the missing xcatmaster attribute.

The following code throws this error:
https://github.com/xcat2/xcat-core/blob/master/xCAT-server/lib/xcat/plugins/dhcp.pm#L530-L558

This code looks for xcatmaster, not tftpserver. Therefore, in my opinion the message should include xcatmaster instead of tftpserver.

@zet809
Copy link

zet809 commented Sep 21, 2018

hi, @Obihoernchen , thanks very much for point this out, the xcatmaster shall be the tftpserver of the compute node, but the message is really misleading.

@zet809
Copy link

zet809 commented Sep 21, 2018

I have modify the warning message as Unable to determine the tftpserver for <mynode>, please makesure "xcatmaster" is set correctly in PR #5651

@zet809 zet809 self-assigned this Sep 21, 2018
@zet809 zet809 added this to the 2.14.4 milestone Sep 21, 2018
@Obihoernchen
Copy link
Member Author

Obihoernchen commented Sep 21, 2018

cool thanks 👍

@Obihoernchen
Copy link
Member Author

#5651 merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants