[kudu-CR] k8s: minor updates to the StatefulSet

2021-02-26 Thread Andrew Wong (Code Review)
Andrew Wong has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17098 )

Change subject: k8s: minor updates to the StatefulSet
..

k8s: minor updates to the StatefulSet

I walked through some of the Kubernetes README and noticed the ports
were off. This updates to use the typical defaults we use for masters
and tservers.

It also updates to use 4 tservers, which is closer to what would be used
in production, given 3-4-3 replication.

Change-Id: I50b0084f70d30187bcca4e356e38c35b2486c611
Reviewed-on: http://gerrit.cloudera.org:8080/17098
Tested-by: Kudu Jenkins
Reviewed-by: Hao Hao 
Reviewed-by: Alexey Serbin 
---
M kubernetes/README.adoc
M kubernetes/kudu-services.yaml
M kubernetes/kudu-statefulset.yaml
3 files changed, 31 insertions(+), 25 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Hao Hao: Looks good to me, approved
  Alexey Serbin: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I50b0084f70d30187bcca4e356e38c35b2486c611
Gerrit-Change-Number: 17098
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Andrew Wong 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)


[kudu-CR] k8s: minor updates to the StatefulSet

2021-02-26 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17098 )

Change subject: k8s: minor updates to the StatefulSet
..


Patch Set 1: Code-Review+2

Thank you for addressing these!


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I50b0084f70d30187bcca4e356e38c35b2486c611
Gerrit-Change-Number: 17098
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong 
Gerrit-Reviewer: Alexey Serbin 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 26 Feb 2021 21:58:05 +
Gerrit-HasComments: No


[kudu-CR] k8s: minor updates to the StatefulSet

2021-02-26 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17098 )

Change subject: k8s: minor updates to the StatefulSet
..


Patch Set 1: Code-Review+2


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I50b0084f70d30187bcca4e356e38c35b2486c611
Gerrit-Change-Number: 17098
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong 
Gerrit-Reviewer: Hao Hao 
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Fri, 26 Feb 2021 20:12:13 +
Gerrit-HasComments: No


[kudu-CR] k8s: minor updates to the StatefulSet

2021-02-20 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17098


Change subject: k8s: minor updates to the StatefulSet
..

k8s: minor updates to the StatefulSet

I walked through some of the Kubernetes README and noticed the ports
were off. This updates to use the typical defaults we use for masters
and tservers.

It also updates to use 4 tservers, which is closer to what would be used
in production, given 3-4-3 replication.

Change-Id: I50b0084f70d30187bcca4e356e38c35b2486c611
---
M kubernetes/README.adoc
M kubernetes/kudu-services.yaml
M kubernetes/kudu-statefulset.yaml
3 files changed, 31 insertions(+), 25 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/98/17098/1
--
To view, visit http://gerrit.cloudera.org:8080/17098
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I50b0084f70d30187bcca4e356e38c35b2486c611
Gerrit-Change-Number: 17098
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong