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

pgsql: fix support for upper-case hostnames and report exit-reason when failing due to remaining lock file #1552

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

oalbrigt
Copy link
Contributor

No description provided.

@oalbrigt oalbrigt changed the title pgsql: fix support for upper-case hostnames and report exit-reason when failing due to remaining locak file pgsql: fix support for upper-case hostnames and report exit-reason when failing due to remaining lock file Sep 11, 2020
@knakahira
Copy link
Contributor

@oalbrigt Thank you for your good work.
But I have one question for upper-case hostname.
The pgsq RA convert upper-case hostnames to lower-case using tr command.
${NODENAME} and application_name are always lower-case even if the real host name is upper-case.
Why do you need that "lower(application_name)"?

@oalbrigt
Copy link
Contributor Author

We have a customer who's somehow able to to make it break with upper-case hostnames without this.

@knakahira
Copy link
Contributor

@oalbrigt Could you tell me how the problem happened?
I will check other upper-case hostname problems in the pgsql RA.

@oalbrigt
Copy link
Contributor Author

It's a customer of a customer, so we're having issues getting all the info regarding how they've managed it, but it seems to work fine with this patch.

@knakahira
Copy link
Contributor

Hmm, I thought about it, but I didn't know how to cause the problem.
Do you have problems if customer change the application name outside the control of pgsql RA?
Could you tell me what customer shouldn't do with current versions of pgsql RA?

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 this pull request may close these issues.

2 participants