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

Set a statement_timeout for FR prod for ofn_user role #841

Open
RachL opened this issue Nov 3, 2022 · 5 comments
Open

Set a statement_timeout for FR prod for ofn_user role #841

RachL opened this issue Nov 3, 2022 · 5 comments

Comments

@RachL
Copy link
Contributor

RachL commented Nov 3, 2022

Description

FR prod does not have a statement_timeout yet. Let's set this at 240s

Acceptance Criteria & Tests

How can we test this ?

@jibees jibees self-assigned this Nov 3, 2022
@jibees
Copy link
Contributor

jibees commented Nov 3, 2022

How can we test this?

Provision a server, and check that ofn_user has the right value for statement_timeout option:

SELECT rolname, rolconfig FROM   pg_roles WHERE  rolname = 'ofn_user'; 

 rolname  |        rolconfig
----------+--------------------------
 ofn_user | {statement_timeout=240s}

@jibees jibees changed the title Set a statement_timeout for FR prod Set a statement_timeout for FR prod for ofn_user role Nov 3, 2022
@dacook
Copy link
Member

dacook commented Nov 21, 2022

Hi @jibees , is this still in development? I don't see a PR, and it doesn't look like the ofn_user has a statement_timeout in the codebase yet.

Also, I'm wondering if this is intended to be FR-specific, or could be applied to all hosts. If 240s works for FR then I think it would be suitable for all hosts.

Maybe we'd like to make a way for instance managers to update these settings individually... but better to have general rules if we can.

@jibees
Copy link
Contributor

jibees commented Nov 22, 2022

Sorry, it was a mistake!

Also, I'm wondering if this is intended to be FR-specific, or could be applied to all hosts. If 240s works for FR then I think it would be suitable for all hosts.

I wanted to put this to improve performance on fr-prod, but yes, why not actually putting this configuration to all instances.

@RachL
Copy link
Contributor Author

RachL commented Dec 8, 2022

@jibees so we keep this issue open and we change the title or we need another issue?

@RachL
Copy link
Contributor Author

RachL commented Dec 28, 2022

@jibees don't forget about this one ;)

@jibees jibees removed their assignment Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: All the things 💤
Development

No branches or pull requests

3 participants