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-nginx] Release 6.4.3-debian-12-r0 #63737

Merged
merged 1 commit into from
Mar 9, 2024

Conversation

bitnami-bot
Copy link
Contributor

This is an automated Pull Request updating:

  • App name: wordpress-nginx.
  • App version: 6.4.3.
  • Release tag: 6.4.3-debian-12-r0.

@github-actions github-actions bot added wordpress-nginx auto-merge Used by CI to auto merge release PRs authored by bitnami-bot automated labels Mar 9, 2024
@bitnami-bot bitnami-bot added the verify Execute verification workflow for these changes label Mar 9, 2024
@bitnami-bot bitnami-bot merged commit fe88269 into main Mar 9, 2024
13 checks passed
@bitnami-bot bitnami-bot deleted the automated/wordpress-nginx-6.4.3-debian-12-r0 branch March 9, 2024 11:39
Copy link

@omarjackman omarjackman left a comment

Choose a reason for hiding this comment

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

Looks like there might have been an unintentional change from /bitnami/wordpress to /bitnami/wordpress-nginx or some oversight on file permissions when trying to create it

if [[ "${#init_scripts[@]}" -gt 0 ]] && [[ ! -f "/bitnami/wordpress/.user_scripts_initialized" ]]; then
mkdir -p "/bitnami/wordpress"
if [[ "${#init_scripts[@]}" -gt 0 ]] && [[ ! -f "/bitnami/wordpress-nginx/.user_scripts_initialized" ]]; then
mkdir -p "/bitnami/wordpress-nginx"

Choose a reason for hiding this comment

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

Was this an intentional change? It causes an error when you start up with init scripts

mkdir: cannot create directory '/bitnami/wordpress-nginx': Permission denied

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Used by CI to auto merge release PRs authored by bitnami-bot automated verify Execute verification workflow for these changes wordpress-nginx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants