Skip to content

INT-7151 alleviate "permission denied" errors in logs#227

Merged
jflinchbaugh merged 3 commits intomainfrom
INT-7151-fs-perms
Sep 6, 2022
Merged

INT-7151 alleviate "permission denied" errors in logs#227
jflinchbaugh merged 3 commits intomainfrom
INT-7151-fs-perms

Conversation

@jflinchbaugh
Copy link
Contributor

* set fs group, uid, gid of pod running the container
@jflinchbaugh jflinchbaugh requested review from a team and amiralig September 2, 2022 14:56
| `persistence.logCSI` | A YAML block for defining CSI Storage Driver configuration for the Log PV. The entire block is taken as you write it. Should support _any_ csi driver that your cluster has installed. | `nil` |
| `persistence.affinity.nodeSelectorTerms` | A YAML block for defining the affinity node selection. This block is taken as you write it. | `nil` |
| `resources` | Resource requests and limits for the IQ pod in the cluster. | See `values.yaml` for suggested minimum recommended values. |
| `podSecurityContext` | `securityContext` for the whole pod. | Default matches the stock container. |
Copy link
Contributor Author

@jflinchbaugh jflinchbaugh Sep 2, 2022

Choose a reason for hiding this comment

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

here's the only line added to the table for the field we're fixing. the other changes in this table are only whitespace.

@jflinchbaugh jflinchbaugh marked this pull request as draft September 2, 2022 16:20
podSecurityContext:
runAsUser: 1000
runAsGroup: 1000
fsGroup: 1000
Copy link
Contributor Author

Choose a reason for hiding this comment

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

here's the fsGroup that the customers want uncommented all the time.

@jflinchbaugh jflinchbaugh marked this pull request as ready for review September 6, 2022 14:07
Copy link
Contributor

@eduard-tita eduard-tita left a comment

Choose a reason for hiding this comment

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

LGTM

@eduard-tita eduard-tita requested a review from a team September 6, 2022 14:30
@nblair
Copy link

nblair commented Sep 6, 2022

@jflinchbaugh can you comment on how this relates to #226, perhaps on that issue/PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants