openQA Project (public) - coordination #128366: [epic] further improvement after we did ensure all our database tables accomodate enough data
Added by okurz almost 3 years ago. Updated over 1 year ago.
0%
Description
In #112718 we have enabled https://www.postgresql.org/docs/current/pgstatstatements.html but it's not yet made persistent. We should make sure it is.
https://progress.opensuse.org/issues/112850 as a first draft. Need to get some feedback from @mkittler (https://suse.slack.com/archives/C02AJ1E568M/p1658223896087059) what was actually changed. The following lines might need to be included in this MR:
# Debugging for https://progress.opensuse.org/issues/112718
compute_query_id = on
pg_stat_statements.max = 10000
pg_stat_statements.track = all
Clarified, additional lines have been added. Further discussion needs to happen if we enable this feature by default all the time or just have it in the config (see discussion in https://suse.slack.com/archives/C02AJ1E568M/p1658223896087059). Putting back into new due to this
I ran into an issue on OSD webUI upgrade to Leap15.5. The postgres service failed to start with
could not access file "pg_stat_statements": No such file or directory
eventually when a salt CI pipeline triggered changes that caused the postgresql service to restart. That happened after in the above mentioned upgrade I called zypper rm -u postgresql$oldver postgresql$oldver-contrib postgresql$oldver-server which due to the -u parameter also removed postgresql-contrib which in turn removed postgresql$newver-contrib which would provide pg_stat_statements but we don't pull that in as hard requirement. Also see https://stackoverflow.com/questions/53158500/postgresql-fatal-could-not-access-file-pg-stat-statements-no-such-file-or-di about this. For now I have the configuration disabled in the config file. If we want to continue here we must ensure we have the contrib package installed as well.