Support setting per-role config with ALTER ROLE SET

[jfleury-gorgias]

Upstream PR: Squarespace/pgbedrock#73

---

⚠️ With this change, pgbedrock becomes the authority on user config and will override any existing config (or reset them if they are not set in the configuration).

[linear]

PLTSR-1003 Patch pgbedrock to support ALTER ROLES SET

This way we can use it to set per-role parameters like timeout.

Squarespace#65

[jfleury-gorgias]

I still need to make sure this works as expected on staging.

[jfleury-gorgias]

So this was successfully tested on staging.

Here are the logs from pgbedrock:

--------------------------------
--- Configuring attributes -----
--------------------------------

ALTER ROLE "gorgias" SET statement_timeout="10min"; -- Previous value: 5min
ALTER ROLE "gorgias" RESET idle_in_transaction_session_timeout; -- Previous value: 2min

And how the settings have been changed (from SELECT usename,useconfig FROM pg_shadow WHERE useconfig IS NOT NULL ORDER BY usename;):

     usename     |                            useconfig
 ----------------+------------------------------------------------------------------
  debezium       | {idle_in_transaction_session_timeout=2min}
- gorgias        | {statement_timeout=5min,idle_in_transaction_session_timeout=2min}
+ gorgias        | {statement_timeout=10min}
  gorgias_web    | {statement_timeout=55s,gin_pending_list_limit=512kB}
  gorgias_worker | {statement_timeout=55s,gin_pending_list_limit=512kB}
  periscope      | {statement_timeout=55s,idle_in_transaction_session_timeout=2min}

[jfleury-gorgias]

I created gorgias/gorgias#12995 if you want to see the change in the config.

[FlipEnergy]

This is good work, Johan. LGTM.

[jfleury-gorgias]

@FlipEnergy I just imported your commit from the upstream PR.

[deadlytea]

Would be nice to have some tests for the byte/duration parse logic before we put this in, also I'm not a fan of configs as the wording choice as it makes me think of a set of config structures, not a set of parameters, but I won't block on that we can update if we can think of something better

[jfleury-gorgias]

Would be nice to have some tests for the byte/duration parse logic before we put this in

Done and it helped me find some bugs, I should’ve done that earlier.

[deadlytea]

LGTM