Skip to content
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.

docs: add vanilla Kubernetes docs #159

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Conversation

tumido
Copy link
Member

@tumido tumido commented Nov 28, 2023

Description of the change

Existing or Associated Issue(s)

Additional Information

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the values.yaml and added to the README.md. The pre-commit utility can be used to generate the necessary content. Use pre-commit run -a to apply changes.
  • JSON Schema template updated and re-generated the raw schema via pre-commit hook.
  • List tests pass for Chart using the Chart Testing tool and the ct lint command.

runAsUser: 1001
runAsGroup: 1001
fsGroup: 1001
postgresql:
Copy link
Member

Choose a reason for hiding this comment

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

This is the setting that I've just verified that works on IKS.
it forces the PostgreSQL run as postgres user and group (uid and guide 26) and it enables the volumePermissions initContinainer that fixes the permissions on the files in the data volume

primary:
  podSecurityContext:
    enabled: true
    fsGroup: 26
  containerSecurityContext:
    enabled: true
    runAsUser: 26
volumePermissions:
  enabled: true
  image:
    registry: registry.redhat.io
    repository: rhel9/postgresql-15
    tag: latest

Copy link
Member Author

Choose a reason for hiding this comment

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

updated

Signed-off-by: Tomas Coufal <tcoufal@redhat.com>
@tumido tumido requested a review from kadel December 1, 2023 15:40
@tumido tumido marked this pull request as ready for review December 1, 2023 15:40
@tumido tumido requested a review from a team as a code owner December 1, 2023 15:40
@tumido tumido requested a review from a team December 1, 2023 15:40
Copy link
Contributor

@schwesig schwesig left a comment

Choose a reason for hiding this comment

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

/lgtm

@kadel kadel merged commit ab7ec4d into janus-idp:main Dec 5, 2023
2 of 3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants