Re: [PR] [improve][sec] Add group pulsar and add user pulsar to it instead of root [pulsar]

2023-12-12 Thread via GitHub
yaalsn commented on PR #21084: URL: https://github.com/apache/pulsar/pull/21084#issuecomment-1853069662 @michaeljmarshall Sorry, I should disscuss more. -- 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 t

Re: [PR] [improve][sec] Add group pulsar and add user pulsar to it instead of root [pulsar]

2023-12-07 Thread via GitHub
michaeljmarshall commented on PR #21084: URL: https://github.com/apache/pulsar/pull/21084#issuecomment-1846535442 Why did we merge this without addressing my questions on the mailing list? https://lists.apache.org/thread/34t3npt0368hppv1gk8wc808ofl786j3 I appreciate that you bro

Re: [PR] [improve][sec] Add group pulsar and add user pulsar to it instead of root [pulsar]

2023-12-07 Thread via GitHub
lhotari commented on PR #21084: URL: https://github.com/apache/pulsar/pull/21084#issuecomment-1845982713 This PR #13376 provides more context and the reference about why root group must be used for OpenShift support: * https://engineering.bitnami.com/articles/running-non-root-containers-

Re: [PR] [improve][sec] Add group pulsar and add user pulsar to it instead of root [pulsar]

2023-12-07 Thread via GitHub
lhotari commented on PR #21084: URL: https://github.com/apache/pulsar/pull/21084#issuecomment-1845972457 > Currently, the user pulsar is in the root group, it would be better to use the non-root group to keep more safety. I think we need to revert and first do a PIP if we are doing th

Re: [PR] [improve][sec] Add group pulsar and add user pulsar to it instead of root [pulsar]

2023-12-07 Thread via GitHub
zymap merged PR #21084: URL: https://github.com/apache/pulsar/pull/21084 -- 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-unsubscr...@pulsar.apac

Re: [PR] [improve][sec] Add group pulsar and add user pulsar to it instead of root [pulsar]

2023-12-06 Thread via GitHub
codecov-commenter commented on PR #21084: URL: https://github.com/apache/pulsar/pull/21084#issuecomment-1843995912 ## [Codecov](https://app.codecov.io/gh/apache/pulsar/pull/21084?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

Re: [PR] [improve][sec] Add group pulsar and add user pulsar to it instead of root [pulsar]

2023-12-06 Thread via GitHub
zymap commented on PR #21084: URL: https://github.com/apache/pulsar/pull/21084#issuecomment-1843991639 Failed to build the image: ``` Failed to execute goal io.fabric8:docker-maven-plugin:0.43.3:build (default) on project latest-version-image: Unable to build image [apachepulsar/pulsa

Re: [PR] [improve][sec] Add group pulsar and add user pulsar to it instead of root [pulsar]

2023-12-06 Thread via GitHub
zymap closed pull request #21084: [improve][sec] Add group pulsar and add user pulsar to it instead of root URL: https://github.com/apache/pulsar/pull/21084 -- 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

Re: [PR] [improve][sec] Add group pulsar and add user pulsar to it instead of root [pulsar]

2023-10-05 Thread via GitHub
github-actions[bot] commented on PR #21084: URL: https://github.com/apache/pulsar/pull/21084#issuecomment-1749877224 The pr had no activity for 30 days, mark with Stale label. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an