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.

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

[bitnami/wordpress] Fix php session mgmt #28974

Merged
merged 5 commits into from
Aug 23, 2024

Conversation

jusabatier
Copy link
Contributor

Description of the change

The php-var-dir needs to be pre-populated in the emptyDir volume to avoid issues in PHP session management.

Same fix as #25697 but for wordpress

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • 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)

Signed-off-by: Julien Sabatier <[email protected]>
Signed-off-by: Julien Sabatier <[email protected]>
bitnami-bot and others added 2 commits August 22, 2024 14:08
Signed-off-by: Bitnami Containers <[email protected]>
Signed-off-by: Carlos Rodríguez Hernández <[email protected]>
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels Aug 22, 2024
@github-actions github-actions bot removed the triage Triage is needed label Aug 22, 2024
@github-actions github-actions bot removed the request for review from javsalgar August 22, 2024 21:07
@github-actions github-actions bot requested a review from migruiz4 August 22, 2024 21:07
Signed-off-by: Bitnami Containers <[email protected]>
Copy link
Member

@migruiz4 migruiz4 left a comment

Choose a reason for hiding this comment

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

Hi @jusabatier,

Thank you for your contribution! Looks good to me

@migruiz4 migruiz4 merged commit d3d8cab into bitnami:main Aug 23, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
solved verify Execute verification workflow for these changes wordpress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants