Skip to content

Commit

Permalink
fix vm-informant dbname: "neondb" -> "postgres" (#4046)
Browse files Browse the repository at this point in the history
Changes the vm-informant's postgres connection string's dbname from
"neondb" (which sometimes doesn't exist) to "postgres" (which
_hopefully_ should exist more often?).

Currently there are a handful of VMs in prod that aren't working with
autoscaling because they don't have the "neondb" database.

The vm-informant doesn't require any database in particular; it's just
connecting as `cloud_admin` to be able to adjust the file cache
settings.
  • Loading branch information
sharnoff authored Apr 18, 2023
1 parent 0bfbae2 commit 02b28ae
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Dockerfile.vm-compute-node
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ RUN set -e \

RUN set -e \
&& echo "::sysinit:cgconfigparser -l /etc/cgconfig.conf -s 1664" >> /etc/inittab \
&& CONNSTR="dbname=neondb user=cloud_admin sslmode=disable" \
&& CONNSTR="dbname=postgres user=cloud_admin sslmode=disable" \
&& ARGS="--auto-restart --cgroup=neon-postgres --pgconnstr=\"$CONNSTR\"" \
&& echo "::respawn:su vm-informant -c '/usr/local/bin/vm-informant $ARGS'" >> /etc/inittab

Expand Down

0 comments on commit 02b28ae

Please sign in to comment.