You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2022/06/01 23:01:13 UTC
[GitHub] [pinot] xiangfu0 opened a new pull request, #8816: Adding acl support for helm chart
xiangfu0 opened a new pull request, #8816:
URL: https://github.com/apache/pinot/pull/8816
Adding ACL support in helmChart.
helmChart values.yaml section:
```
pinotAuth:
enabled: true
controllerFactoryClass: org.apache.pinot.controller.api.access.BasicAuthAccessControlFactory
brokerFactoryClass: org.apache.pinot.broker.broker.BasicAuthAccessControlFactory
configs:
- access.control.principals=admin,user
- access.control.principals.admin.password=verysecret
- access.control.principals.user.password=secret
- access.control.principals.user.tables=baseballStats,otherstuff
- access.control.principals.user.permissions=READ
```
Generated controller configs:
```
apiVersion: v1
kind: ConfigMap
metadata:
name: pinot-controller-config
data:
pinot-controller.conf: |-
controller.helix.cluster.name=pinot-quickstart
controller.port=9000
controller.data.dir=/var/pinot/controller/data
controller.zk.str=pinot-zookeeper:2181
pinot.set.instance.id.to.hostname=true
controller.task.scheduler.enabled=true
controller.admin.access.control.factory.class=org.apache.pinot.controller.api.access.BasicAuthAccessControlFactory
pinot.controller.access.control.principals=admin,user
pinot.controller.access.control.principals.admin.password=verysecret
pinot.controller.access.control.principals.user.password=secret
pinot.controller.access.control.principals.user.tables=baseballStats,otherstuff
pinot.controller.access.control.principals.user.permissions=READ
```
Generated broker configs:
```
apiVersion: v1
kind: ConfigMap
metadata:
name: pinot-broker-config
data:
pinot-broker.conf: |-
pinot.broker.client.queryPort=8099
pinot.broker.routing.table.builder.class=random
pinot.set.instance.id.to.hostname=true
pinot.broker.access.control.class=org.apache.pinot.broker.broker.BasicAuthAccessControlFactory
pinot.broker.access.control.principals=admin,user
pinot.broker.access.control.principals.admin.password=verysecret
pinot.broker.access.control.principals.user.password=secret
pinot.broker.access.control.principals.user.tables=baseballStats,otherstuff
pinot.broker.access.control.principals.user.permissions=READ
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
Re: [PR] Adding acl support for helm chart [pinot]
Posted by "pospospos2007 (via GitHub)" <gi...@apache.org>.
pospospos2007 commented on PR #8816:
URL: https://github.com/apache/pinot/pull/8816#issuecomment-1837088165
May I ask a question? After I do enabled login for pinot, when I load new data on pinot, the table is not able to read anymore? Does anyone know what happend? I got error:
Got permanent error status code: 403 while downloading segment from: http://xxxx:9000/segments/table/
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
Re: [PR] Adding acl support for helm chart [pinot]
Posted by "Jackie-Jiang (via GitHub)" <gi...@apache.org>.
Jackie-Jiang commented on PR #8816:
URL: https://github.com/apache/pinot/pull/8816#issuecomment-1839866064
@xiangfu0 ^^
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org
[GitHub] [pinot] npawar merged pull request #8816: Adding acl support for helm chart
Posted by GitBox <gi...@apache.org>.
npawar merged PR #8816:
URL: https://github.com/apache/pinot/pull/8816
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org