Mahdi Askari has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8960 )

Change subject: Adding a subsection to NTP troubleshooting section to cover 
frequent cases of NTP crashes when the Kudu cluster is deployed on Microsoft 
Azure
......................................................................


Patch Set 3:

(7 comments)

Comments replied.  Please review.

http://gerrit.cloudera.org:8080/#/c/8960/1/docs/troubleshooting.adoc
File docs/troubleshooting.adoc:

http://gerrit.cloudera.org:8080/#/c/8960/1/docs/troubleshooting.adoc@125
PS1, Line 125: ==== NTP Clock on secure clusters with Centrify
> nit: trailing whitespace
Done


http://gerrit.cloudera.org:8080/#/c/8960/1/docs/troubleshooting.adoc@127
PS1, Line 127: - CentrifyDC (Linux package which connects the host to an Active 
Directory) bypasses NTP program.
> nit: can you wrap this to 80-100 coluns as done elsewhere in the file?
Done


http://gerrit.cloudera.org:8080/#/c/8960/1/docs/troubleshooting.adoc@129
PS1, Line 129:
> can you format file names appropriately using `...`?
Done


http://gerrit.cloudera.org:8080/#/c/8960/1/docs/troubleshooting.adoc@129
PS1, Line 129:
> this raw link sitting in the documentation doesn't look good
Done


http://gerrit.cloudera.org:8080/#/c/8960/1/docs/troubleshooting.adoc@129
PS1, Line 129:
> typo
Done


http://gerrit.cloudera.org:8080/#/c/8960/1/docs/troubleshooting.adoc@129
PS1, Line 129: nk:https:/
> is Centrify Azure-specific? Should this be a separate troubleshooting point
It is not Azure Specific. Can be any cloud or on-prem. I will move it to its 
own section.


http://gerrit.cloudera.org:8080/#/c/8960/1/docs/troubleshooting.adoc@135
PS1, Line 135:
> how can this script be automated to run at startup on every startup? Seems
I will have a look and see how we can automate this in start up.



--
To view, visit http://gerrit.cloudera.org:8080/8960
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5d638aa26137d3a795fce1d0f3f744d4969bdf22
Gerrit-Change-Number: 8960
Gerrit-PatchSet: 3
Gerrit-Owner: Mahdi Askari <ma...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mahdi Askari <ma...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Comment-Date: Mon, 05 Feb 2018 23:47:30 +0000
Gerrit-HasComments: Yes

Reply via email to