[kudu-CR] Adding a subsection to NTP troubleshooting section to cover frequent cases of NTP crashes when the Kudu cluster is deployed on Microsoft Azure

2018-02-05 Thread Mahdi Askari (Code Review) 
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 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mahdi Askari 
Gerrit-Reviewer: Todd Lipcon 
Gerrit-Comment-Date: Mon, 05 Feb 2018 23:47:30 +
Gerrit-HasComments: Yes

[kudu-CR] Adding a subsection to NTP troubleshooting section to cover frequent cases of NTP crashes when the Kudu cluster is deployed on Microsoft Azure

2018-01-09 Thread Mahdi Askari (Code Review) 
Hello Alex Rodoni, Kudu Jenkins, Todd Lipcon,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8960

to look at the new patch set (#3).

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
..

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

Change-Id: I5d638aa26137d3a795fce1d0f3f744d4969bdf22
---
M docs/troubleshooting.adoc
1 file changed, 30 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/60/8960/3
--
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: newpatchset
Gerrit-Change-Id: I5d638aa26137d3a795fce1d0f3f744d4969bdf22
Gerrit-Change-Number: 8960
Gerrit-PatchSet: 3
Gerrit-Owner: Mahdi Askari 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon

[kudu-CR] Adding a subsection to NTP troubleshooting section to cover frequent cases of NTP crashes when the Kudu cluster is deployed on Microsoft Azure

2018-01-09 Thread Mahdi Askari (Code Review) 
Hello Alex Rodoni, Kudu Jenkins, Todd Lipcon,

I'd like you to reexamine a change. Please visit

http://gerrit.cloudera.org:8080/8960

to look at the new patch set (#2).

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
..

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

Change-Id: I5d638aa26137d3a795fce1d0f3f744d4969bdf22
---
M docs/troubleshooting.adoc
1 file changed, 30 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/60/8960/2
--
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: newpatchset
Gerrit-Change-Id: I5d638aa26137d3a795fce1d0f3f744d4969bdf22
Gerrit-Change-Number: 8960
Gerrit-PatchSet: 2
Gerrit-Owner: Mahdi Askari 
Gerrit-Reviewer: Alex Rodoni 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon

[kudu-CR] Adding a subsection to NTP troubleshooting section to cover frequent cases of NTP crashes when the Kudu cluster is deployed on Microsoft Azure

2018-01-07 Thread Mahdi Askari (Code Review) 
Mahdi Askari has uploaded this change for review. ( 
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
..

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

Change-Id: I5d638aa26137d3a795fce1d0f3f744d4969bdf22
---
M docs/troubleshooting.adoc
1 file changed, 24 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/60/8960/1
--
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: newchange
Gerrit-Change-Id: I5d638aa26137d3a795fce1d0f3f744d4969bdf22
Gerrit-Change-Number: 8960
Gerrit-PatchSet: 1
Gerrit-Owner: Mahdi Askari