-
Notifications
You must be signed in to change notification settings - Fork 7
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
Mandate host and port in database information #174
Conversation
ba8633d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ee7a72d
Test coverage for 2344ab9
Static code analysis report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Overview
User feedback indicates that the WordPress charm might initiate the reconciliation process without a database endpoint provided in the database relation, leading to errors during database connectivity verification. Although we haven't been able to replicate this issue, updating the charm to prevent it from proceeding to reconciliation in the absence of a database endpoint in the database relation, as a precautionary measure.
Checklist
src-docs
urgent
,trivial
,complex
)