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] Wordpress maintenance mode returns 503 health check fails #3019

Closed
lwj5 opened this issue Jul 3, 2020 · 6 comments
Closed

Comments

@lwj5
Copy link
Contributor

lwj5 commented Jul 3, 2020

Which chart:
wordpress

Describe the bug
When updating WordPress plugins or themes. WordPress is automatically put into maintenance mode.
wp-login.php will return 503, thus health check will fail.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'WordPress updates'
  2. Click on 'update all'
  3. See pod die

Expected behavior
The pod is not supposed to be killed, the health check URL should return 200 even in maintenance mode.

Version of Helm and Kubernetes:
Does not matter

@javsalgar
Copy link
Contributor

Hi,

I think that, in the case of a long upgrade, it may make sense to change the liveness/readiness probe of the container. In other charts we are allowing a customLivenessProbe and customReadinessProbe section that deals with this kind of situations. In this chart it is not implemented yet, so while you perform this maintenance operation I would advise you to disable the liveness/readiness probes and then reenable them. Would it make sense for your case?

@lwj5
Copy link
Contributor Author

lwj5 commented Jul 7, 2020

I see okay, thanks for the reply. I'll try to find something for a long term solution.

@lwj5 lwj5 closed this as completed Jul 7, 2020
@lwj5
Copy link
Contributor Author

lwj5 commented Jul 8, 2020

@javsalgar wp-admin/install.php might be a good health check path. It doesn't get affected by maintenance mode and fails when DB connection fails. You could consider the path change.

@javsalgar
Copy link
Contributor

Hi,

Thank you very much for the input! Feel free to create a PR with the change. If not, we will add it to our backlog (which would take a bit longer because of priorities)

@lwj5
Copy link
Contributor Author

lwj5 commented Jul 26, 2020

PR added.

@javsalgar
Copy link
Contributor

Thank you so much! We will review it very soon :)

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 a pull request may close this issue.

2 participants