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

Fix issue 1638: Genesis not wait long enough time to have NICs get IP… #1640

Merged
merged 1 commit into from
Aug 5, 2016
Merged

Fix issue 1638: Genesis not wait long enough time to have NICs get IP… #1640

merged 1 commit into from
Aug 5, 2016

Conversation

zet809
Copy link

@zet809 zet809 commented Aug 5, 2016

… address
Fix issue #1638

@zet809 zet809 added this to the 2.12.2 milestone Aug 5, 2016
@daniceexi
Copy link
Contributor

@whowutwut could you confirm that the top of rack switch has NTP enabled?

@gurevichmark gurevichmark merged commit d1c4b9e into xcat2:master Aug 5, 2016
@gurevichmark
Copy link
Contributor

Changes look good to me.

@whowutwut
Copy link
Member

whowutwut commented Aug 5, 2016

Sorry, was merged before I got a chance to comment.

One thought that I had here is whether we should keep the existing code and simply add another fail over test... The behavior right now is that we simply do not get an IP address because we did not wait long enough.

However, I think there are problems even after Genesis regarding port-fast, so I think maybe we should have left the existing code. If it fails to get an IP address, print a message to inform the user of possible options to resolve it. I think even if genesis gets the IP address in HW discovery, it doesn't necessarily mean that the OS will be able to provision correctly unless some configuration is fixed on the switch side.

@daniceexi
Copy link
Contributor

@whowutwut Do you mean we can recreate this issue even when the STP on the edge switch is disabled. If so I think we need figure out why this happens first.

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

Successfully merging this pull request may close these issues.

4 participants