Re: Hint to set owner for tablespace directory

2018-10-01 Thread Maksim Milyutin
01.10.2018 15:15, Peter Eisentraut wrote: On 24/09/2018 14:50, Peter Eisentraut wrote: On 11/09/2018 17:10, Peter Eisentraut wrote: On 07/09/2018 17:59, Maksim Milyutin wrote: those directories was that user). The error message "could not set permissions on directory ..." disoriented that

Re: Hint to set owner for tablespace directory

2018-10-01 Thread Peter Eisentraut
On 24/09/2018 14:50, Peter Eisentraut wrote: > On 11/09/2018 17:10, Peter Eisentraut wrote: >> On 07/09/2018 17:59, Maksim Milyutin wrote: >>> those directories was that user). The error message "could not set >>> permissions on directory ..." disoriented that user. The need to change >>> the

Re: Hint to set owner for tablespace directory

2018-09-24 Thread Peter Eisentraut
On 11/09/2018 17:10, Peter Eisentraut wrote: > On 07/09/2018 17:59, Maksim Milyutin wrote: >> those directories was that user). The error message "could not set >> permissions on directory ..." disoriented that user. The need to change >> the owner of those directories came after careful reading

Re: Hint to set owner for tablespace directory

2018-09-11 Thread Peter Eisentraut
On 07/09/2018 17:59, Maksim Milyutin wrote: >> You are still proposing to hint that the permissions on the tablespace >> directory might be correct but that the main database instance is >> running under the wrong user. > Not exactly this way. The case that motivated me to make this patch was >

Re: Hint to set owner for tablespace directory

2018-09-07 Thread Maksim Milyutin
On 09/07/2018 06:05 PM, Peter Eisentraut wrote: On 07/09/2018 11:59, Maksim Milyutin wrote: OK. However I think it would be helpful to leave the mention about setting necessary owner for tablespace directory. My final version of hint is "You might need to fix permissions on this directory or

Re: Hint to set owner for tablespace directory

2018-09-07 Thread Peter Eisentraut
On 07/09/2018 11:59, Maksim Milyutin wrote: > OK. However I think it would be helpful to leave the mention about > setting necessary owner for tablespace directory. My final version of > hint is "You might need to fix permissions on this directory or its > parents or install the PostgreSQL

Re: Hint to set owner for tablespace directory

2018-09-07 Thread Maksim Milyutin
On 08/31/2018 04:59 PM, Tom Lane wrote: Maksim Milyutin writes: 30.08.2018 19:52, Peter Eisentraut wrote: I think the hint is backwards. When you don't have permission to chmod the tablespace directory, you probably want to fix the permissions on the tablespace directory or its parent. In

Re: Hint to set owner for tablespace directory

2018-09-03 Thread avatar28
Thanks, that should be helpful. - Currently Junior Marketing Specialist @ CrazyCall -- Sent from: http://www.postgresql-archive.org/PostgreSQL-hackers-f1928748.html

Re: Hint to set owner for tablespace directory

2018-09-02 Thread Rafia Sabih
On Fri, Aug 31, 2018 at 4:11 PM, Maksim Milyutin wrote: > On 08/31/2018 01:12 PM, Rafia Sabih wrote: > > > > On Fri, Aug 31, 2018 at 3:30 PM, Maksim Milyutin > wrote: > >> On 08/31/2018 11:55 AM, Rafia Sabih wrote: >> >> >> Adding to that this patch needs a rebase. And, please don't forget to

Re: Hint to set owner for tablespace directory

2018-08-31 Thread Tom Lane
Maksim Milyutin writes: > 30.08.2018 19:52, Peter Eisentraut wrote: >> I think the hint is backwards. When you don't have permission to chmod >> the tablespace directory, you probably want to fix the permissions on >> the tablespace directory or its parent. > In this case I propose to: > -

Re: Hint to set owner for tablespace directory

2018-08-31 Thread Maksim Milyutin
30.08.2018 19:52, Peter Eisentraut wrote: On 23/08/2018 13:24, Maksim Milyutin wrote: I have noticed the novice users are stuck trying to create tablespace over a directory whose owner is not the system postgres user. They observed the message "could not set permissions on directory ...:

Re: Hint to set owner for tablespace directory

2018-08-31 Thread Maksim Milyutin
On 08/31/2018 01:12 PM, Rafia Sabih wrote: On Fri, Aug 31, 2018 at 3:30 PM, Maksim Milyutin > wrote: On 08/31/2018 11:55 AM, Rafia Sabih wrote: Adding to that this patch needs a rebase. And, please don't forget to run 'git diff --check', as it has

Re: Hint to set owner for tablespace directory

2018-08-31 Thread Rafia Sabih
On Fri, Aug 31, 2018 at 3:30 PM, Maksim Milyutin wrote: > On 08/31/2018 11:55 AM, Rafia Sabih wrote: > > > Adding to that this patch needs a rebase. And, please don't forget to run > 'git diff --check', as it has some white-space issues. > > > git apply

Re: Hint to set owner for tablespace directory

2018-08-31 Thread Maksim Milyutin
On 08/31/2018 11:55 AM, Rafia Sabih wrote: Adding to that this patch needs a rebase. And, please don't forget to run 'git diff --check', as it has some white-space issues. Hmm, it's odd. Provided patch is fluently applied on the current master HEAD (2e39f69) and *git diff --check* doesn't

Re: Hint to set owner for tablespace directory

2018-08-31 Thread Rafia Sabih
On Fri, Aug 24, 2018 at 3:05 PM, Maksim Milyutin wrote: > On 08/24/2018 05:18 AM, Michael Paquier wrote: > > On Thu, Aug 23, 2018 at 02:24:25PM +0300, Maksim Milyutin wrote: >> >>> I want to add patch that prints hint to set required owner for the >>> tablespace directory if this is the cause of

Re: Hint to set owner for tablespace directory

2018-08-30 Thread Peter Eisentraut
On 23/08/2018 13:24, Maksim Milyutin wrote: > I have noticed the novice users are stuck trying to create tablespace > over a directory whose owner is not the system postgres user. They > observed the message "could not set permissions on directory ...: > permission denied". > > I want to add

Re: Hint to set owner for tablespace directory

2018-08-24 Thread Maksim Milyutin
On 08/24/2018 05:18 AM, Michael Paquier wrote: On Thu, Aug 23, 2018 at 02:24:25PM +0300, Maksim Milyutin wrote: I want to add patch that prints hint to set required owner for the tablespace directory if this is the cause of the problem (*errno == EPERM* after calling *chmod*). Please do not

Re: Hint to set owner for tablespace directory

2018-08-23 Thread Michael Paquier
On Thu, Aug 23, 2018 at 02:24:25PM +0300, Maksim Milyutin wrote: > I want to add patch that prints hint to set required owner for the > tablespace directory if this is the cause of the problem (*errno == EPERM* > after calling *chmod*). Please do not forget to add this patch to the next commit