resource-agents icon indicating copy to clipboard operation
resource-agents copied to clipboard

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

Open oalbrigt opened this issue 5 years ago • 5 comments

oalbrigt avatar Sep 11 '20 08:09 oalbrigt

@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)"?

knakahira avatar Sep 18 '20 09:09 knakahira

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

oalbrigt avatar Sep 18 '20 09:09 oalbrigt

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

knakahira avatar Sep 18 '20 09:09 knakahira

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.

oalbrigt avatar Sep 18 '20 12:09 oalbrigt

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?

knakahira avatar Sep 23 '20 06:09 knakahira