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

Add messages to trace the ntpd commands in genesis #2292

Merged
merged 1 commit into from
Dec 8, 2016

Conversation

whowutwut
Copy link
Member

We have seen some reports from customers that hardware discovery takes up to 15 minutes in some cases in the genesis kernel. It looks like it has something to do with NTP but the actual reason is not known.

[Tue Nov 29 11:02:05 2016]cp: cannot stat '/usr/share/zoneinfo/posix/America/Los_Angeles': No such file or directory
[Tue Nov 29 11:02:07 2016]<166>Nov 29 11:02:06 xcat.genesis.doxcat: Acquired IPv4 address on enP5p7s0f1
[Tue Nov 29 11:02:07 2016]192.168.64.174/24
[Tue Nov 29 11:18:03 2016]ppc64

Between the IP address and "ppc64" it's around 16 minutes.

To help us trace the time that it takes, adding some more detailed logger messages between the ntpd calls

@whowutwut whowutwut added this to the 2.13 milestone Dec 8, 2016
@whowutwut whowutwut requested review from zet809 and daniceexi December 8, 2016 02:41
@whowutwut
Copy link
Member Author

@daniceexi As discussed on the phone, this pull request change is safer than what I did in #2278. It's also affect the xCAT-genesis-scripts, not xCAT-genesis-base, so it's a little better.

Regardless of whether we merge for 2.13. or wait for 2.13.1, I will try and run discovery tomorrow with this to see if we can see any delay in time from the ntpd code path.

@daniceexi
Copy link
Contributor

The change looks safe. So I'll merge it to master branch. If we have to respin the build before the GA, this change will be added.

@daniceexi daniceexi merged commit 0c12451 into xcat2:master Dec 8, 2016
@whowutwut whowutwut deleted the genesis_logger_ntpd branch December 8, 2016 19:12
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.

2 participants