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.

Sign up for GitHub

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

Jump to bottom

[bitnami/sonarqube] Fix issue with volume-permissions #27113

Merged
merged 5 commits into from
Jun 13, 2024
Merged

[bitnami/sonarqube] Fix issue with volume-permissions #27113

merged 5 commits into from
Jun 13, 2024

Conversation

migruiz4
Copy link
Member

Description of the change

Fix issues with Sonarqube volumePermissions container:

  • Removes the subPath which was causing permission issues in the final container
  • Removes the volumePermissions.containerSecurityContext.enabled conditional, which was causing the volume-permissions container to run as user 1001 instead of 0.

Applicable issues

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

@bitnami-bot bitnami-bot added the verify Execute verification workflow for these changes label Jun 12, 2024
@github-actions github-actions bot requested a review from rafariossaa June 12, 2024 11:12
@migruiz4 migruiz4 requested a review from dgomezleon June 12, 2024 11:13
bitnami-bot and others added 4 commits June 12, 2024 11:15
@bitnami-bot
Update CHANGELOG.md
9e221e9 
Signed-off-by: Bitnami Containers <[email protected]>
@migruiz4
[bitnami/sonarqube] Restore volumePermissions.containerSecurityContex…
b8d08e6 
…t.enabled

Signed-off-by: Miguel Ruiz <[email protected]>
@bitnami-bot
Update CHANGELOG.md
4011936 
Signed-off-by: Bitnami Containers <[email protected]>
@bitnami-bot
Update README.md with readme-generator-for-helm
fa2e2f2 
Signed-off-by: Bitnami Containers <[email protected]>
@migruiz4 migruiz4 merged commit d265a6f into main Jun 13, 2024
10 checks passed
@migruiz4 migruiz4 deleted the sonarqube-fix-vp branch June 13, 2024 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dgomezleon dgomezleon dgomezleon approved these changes

@rafariossaa rafariossaa Awaiting requested review from rafariossaa

Assignees

@rafariossaa rafariossaa

@dgomezleon dgomezleon

Labels
bitnami solved sonarqube verify Execute verification workflow for these changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[bitnami/sonarqube] When volumePermissions is enabled and defined other user, plugins failed to install
4 participants
@migruiz4 @dgomezleon @rafariossaa @bitnami-bot