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

Adding minion adhoc as a k8s deployment for minion deployment #8936

Merged
merged 1 commit into from
Jun 20, 2022

Conversation

xiangfu0
Copy link
Contributor

@xiangfu0 xiangfu0 commented Jun 20, 2022

Adding k8s deployment.apps/pinot-minion-stateless for minion stateless deployment.

This deployment by default uses a local disk for temp persistence.

The new deployment looks like this:

➜  kubectl get all -n pinot
NAME                                          READY   STATUS    RESTARTS      AGE
pod/pinot-broker-0                            1/1     Running   1 (88s ago)   113s
pod/pinot-controller-0                        1/1     Running   0             113s
pod/pinot-minion-stateless-5765bcc487-jr8fn   1/1     Running   1 (72s ago)   113s
pod/pinot-server-0                            1/1     Running   1 (71s ago)   113s
pod/pinot-zookeeper-0                         1/1     Running   0             113s

NAME                                TYPE           CLUSTER-IP       EXTERNAL-IP   PORT(S)                      AGE
service/pinot-broker                ClusterIP      10.110.116.28    <none>        8099/TCP                     114s
service/pinot-broker-external       LoadBalancer   10.110.57.63     <pending>     8099:31622/TCP               114s
service/pinot-broker-headless       ClusterIP      None             <none>        8099/TCP                     114s
service/pinot-controller            ClusterIP      10.107.127.32    <none>        9000/TCP                     114s
service/pinot-controller-external   LoadBalancer   10.109.193.170   <pending>     9000:31333/TCP               114s
service/pinot-controller-headless   ClusterIP      None             <none>        9000/TCP                     114s
service/pinot-server                ClusterIP      10.100.68.224    <none>        8098/TCP,80/TCP              114s
service/pinot-server-headless       ClusterIP      None             <none>        8098/TCP,80/TCP              114s
service/pinot-zookeeper             ClusterIP      10.97.195.107    <none>        2181/TCP,2888/TCP,3888/TCP   114s
service/pinot-zookeeper-headless    ClusterIP      None             <none>        2181/TCP,2888/TCP,3888/TCP   114s

NAME                                     READY   UP-TO-DATE   AVAILABLE   AGE
deployment.apps/pinot-minion-stateless   1/1     1            1           114s

NAME                                                DESIRED   CURRENT   READY   AGE
replicaset.apps/pinot-minion-stateless-5765bcc487   1         1         1       114s

NAME                                READY   AGE
statefulset.apps/pinot-broker       1/1     113s
statefulset.apps/pinot-controller   1/1     113s
statefulset.apps/pinot-server       1/1     113s
statefulset.apps/pinot-zookeeper    1/1     113s

@xiangfu0 xiangfu0 requested a review from Jackie-Jiang June 20, 2022 22:39
Copy link
Contributor

@Jackie-Jiang Jackie-Jiang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Should we consider naming it minion-stateless? I feel adhoc is a little bit confusing

@xiangfu0 xiangfu0 force-pushed the stateless-pinot-minion-k8s branch from 9754534 to b372168 Compare June 20, 2022 23:29
@xiangfu0 xiangfu0 linked an issue Jun 20, 2022 that may be closed by this pull request
@xiangfu0 xiangfu0 force-pushed the stateless-pinot-minion-k8s branch from b372168 to 1b827a9 Compare June 20, 2022 23:37
@xiangfu0 xiangfu0 merged commit 5b2b4aa into master Jun 20, 2022
@xiangfu0 xiangfu0 deleted the stateless-pinot-minion-k8s branch June 20, 2022 23:44
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.

Make Pinot minion stateless service
2 participants