GNU bug report logs - #74058
[PATCH 0/2] [important] Set postgresql default value to #f and warn users.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 74058 in the body.
You can then email your comments to 74058 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: guix-patches <at> gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 0/2] [important] Set postgresql default value to #f and warn
 users.
Date: Mon, 28 Oct 2024 11:56:35 +0100

Nicolas Graves (2):
  gnu: postgresql-configuration: Set default postgresql to #f.
  etc: news: Warn users about postgresql default update.

 etc/news.scm               | 41 ++++++++++++++++++++++++++++++++++++++
 gnu/services/databases.scm | 10 +++++++---
 2 files changed, 48 insertions(+), 3 deletions(-)

-- 
2.46.0

Message #8 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 2/2] etc: news: Warn users about postgresql default update.
Date: Mon, 28 Oct 2024 12:13:30 +0100

* etc/news.scm: Add entry.
---
 etc/news.scm | 41 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 41 insertions(+)

diff --git a/etc/news.scm b/etc/news.scm
index 4929fa2a9a..8dc209495a 100644
--- a/etc/news.scm
+++ b/etc/news.scm
@@ -25,6 +25,7 @@
 ;; Copyright © 2024 Vivien Kraus <vivien <at> planete-kraus.eu>
 ;; Copyright © 2024 Guillaume Le Vaillant <glv <at> posteo.net>
 ;; Copyright © 2024 Zheng Junjie <873216071 <at> qq.com>
+;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;
 ;; Copying and distribution of this file, with or without modification, are
 ;; permitted in any medium without royalty provided the copyright notice and
@@ -33,6 +34,46 @@
 (channel-news
  (version 0)
 
+ (entry (commit "") ; fill-me based on the previous commit
+        (title (en "Postgresql service upgrade")
+               (fr "Mise à jour du service postgresql"))
+        (body
+         (en "This news entry concerns users of the postgresql-service-type.
+
+The default postgresql used in @code{postgresql-configuration} has been
+deprecated, and will be removed from Guix in a few months.  However, the value
+of the default @code{postgresql-configuration-postgresql} can't be directly
+changed to a newer major version of postgresql, because switching to a major
+version currently requires a manual update of the database.
+
+Because of this, the default value of postgresql-configuration-postgresql has
+been set to @code{#f}.  Current users of the service will have to set it
+manually.  If it was unset, use @code{postgresql-10} and plan an upgrade in the
+next month to a supported version of postgresql.  To upgrade, you will need to
+either dump your database using your previous version of postgresql and reload
+it in the new version or use the @command{pg_upgrade} application.  See
+@url{https://www.postgresql.org/docs/current/upgrading.html} for more
+info.")
+         (fr "Ce message concerne les utilisateurs du service-type postgresql.
+
+La précédente version par défaut de postgresql utilisée dans
+@code{postgresqsl-configuration} est obsolète et sera supprimée de Guix dans
+quelques mois.  Cependant, la valeur de la configuration par défaut
+@code{postgresql-configuration-postgresql} ne peut pas être changée
+directement pour une version majeure plus récente de postgresql, car le
+passage à une nouvelle version majeure nécessite actuellement une mise à jour
+manuelle de la base de données.
+
+Pour cette raison, la valeur par défaut de
+@code{postgresql-configuration-postgresql} est maintenant @code{#f} et les
+utilisateurs actuels du service doivent maintenant la configurer manuellement.
+Si elle n’était pas définie, utilisez @code{postgresql-10} et prévoyez une
+mise à jour dans le mois à venir vers une version plus récente de postgresql.
+Pour effectuer la mise à jour, vous devrez soit exporter votre base de données
+sous une version de postgresql puis la réimporter sous une nouvelle version,
+soit utiliser l'application @command{pg_upgrade}. Pour plus d'informations,
+consultez @url{https://www.postgresql.org/docs/current/upgrading.html}.")))
+
  (entry (commit "5966e0fdc78771c562e0f484a22f381a77908be0")
         (title
          (en "Daemon vulnerability allowing takeover of build users fixed")
-- 
2.46.0

Message #11 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH 1/2] gnu: postgresql-configuration: Set default postgresql to
 #f.
Date: Mon, 28 Oct 2024 12:13:29 +0100

* gnu/services/databases.scm (postgresql-configuration)[postgresql]:
Set default to #f.
(postgresql-services): Revert default to postgresql-10 (rationale: We
can remove this service at the same time than postgresql-10, in
something like 6 months to a year).
---
 gnu/services/databases.scm | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/gnu/services/databases.scm b/gnu/services/databases.scm
index 0933eb5e79..f8ae085339 100644
--- a/gnu/services/databases.scm
+++ b/gnu/services/databases.scm
@@ -167,8 +167,12 @@ (define contents
 (define-record-type* <postgresql-configuration>
   postgresql-configuration make-postgresql-configuration
   postgresql-configuration?
-  (postgresql         postgresql-configuration-postgresql ;file-like
-                      (default postgresql))
+  ;; Setting it to #f ensures that the user sets its current postgresql
+  ;; explicitely.  Since major upgrades currently require a manual migration
+  ;; of the database, this way the user is responsible for upgrading properly.
+  ;; MAYBE TODO: Write an action to automatically upgrade in Guix.
+  (postgresql         postgresql-configuration-postgresql ;file-like or #f
+                      (default #f))
   (port               postgresql-configuration-port
                       (default 5432))
   (locale             postgresql-configuration-locale
@@ -346,7 +350,7 @@ (define postgresql-service-type
    (default-value (postgresql-configuration))
    (description "Run the PostgreSQL database server.")))
 
-(define-deprecated (postgresql-service #:key (postgresql postgresql)
+(define-deprecated (postgresql-service #:key (postgresql postgresql-10)
                                        (port 5432)
                                        (locale "en_US.utf8")
                                        (config-file (postgresql-config-file))
-- 
2.46.0

Message #14 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [bug#74058] [PATCH 1/2] gnu: postgresql-configuration: Set
 default postgresql to #f.
Date: Wed, 06 Nov 2024 17:29:27 +0100

Hi,

Nicolas Graves <ngraves <at> ngraves.fr> skribis:

> * gnu/services/databases.scm (postgresql-configuration)[postgresql]:
> Set default to #f.
> (postgresql-services): Revert default to postgresql-10 (rationale: We
                     ^
No ‘s’.

>  (define-record-type* <postgresql-configuration>
>    postgresql-configuration make-postgresql-configuration
>    postgresql-configuration?
> -  (postgresql         postgresql-configuration-postgresql ;file-like
> -                      (default postgresql))
> +  ;; Setting it to #f ensures that the user sets its current postgresql
> +  ;; explicitely.  Since major upgrades currently require a manual migration
> +  ;; of the database, this way the user is responsible for upgrading properly.
> +  ;; MAYBE TODO: Write an action to automatically upgrade in Guix.
> +  (postgresql         postgresql-configuration-postgresql ;file-like or #f
> +                      (default #f))

How about having *no* default value in that case?  That way, existing
user configs will just fail to build, forcing users to act on it.
(Leaving it to #f would instead result in obscure backtraces.)

Ludo’.

Message #17 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [bug#74058] [PATCH 2/2] etc: news: Warn users about postgresql
 default update.
Date: Wed, 06 Nov 2024 17:33:06 +0100

Nicolas Graves <ngraves <at> ngraves.fr> skribis:

> * etc/news.scm: Add entry.

Nice.

> + (entry (commit "") ; fill-me based on the previous commit
> +        (title (en "Postgresql service upgrade")
> +               (fr "Mise à jour du service postgresql"))
> +        (body
> +         (en "This news entry concerns users of the postgresql-service-type.

s/the postgresql-service-type/@code{postgresql-service-type}/

> +The default postgresql used in @code{postgresql-configuration} has been

When used as a noun, I believe the correct spelling is “PostgreSQL”
(this needs to be changed throughout).

> +deprecated, and will be removed from Guix in a few months.  However, the value

s/from Guix//

> +of the default @code{postgresql-configuration-postgresql} can't be directly
> +changed to a newer major version of postgresql, because switching to a major
> +version currently requires a manual update of the database.
> +
> +Because of this, the default value of postgresql-configuration-postgresql has
> +been set to @code{#f}.  Current users of the service will have to set it
> +manually.  If it was unset, use @code{postgresql-10} and plan an upgrade in the
> +next month to a supported version of postgresql.  To upgrade, you will need to
> +either dump your database using your previous version of postgresql and reload
> +it in the new version or use the @command{pg_upgrade} application.  See
> +@url{https://www.postgresql.org/docs/current/upgrading.html} for more
> +info.")

Would be nice if this could be automated, although automation for this
kind of thing is a double-edge sword…

I find the instructions clear enough though, +1!

Ludo’.

Message #20 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 1/2] gnu: postgresql-configuration: Set default postgresql
 to *no*.
Date: Wed,  6 Nov 2024 18:04:28 +0100

* gnu/services/databases.scm (postgresql-configuration)[postgresql]:
Set default to *no*.
(postgresql-service): Revert default to postgresql-10 (rationale: We
can remove this service at the same time than postgresql-10, in
something like 6 months to a year).
---
 gnu/services/databases.scm | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/gnu/services/databases.scm b/gnu/services/databases.scm
index 0933eb5e79..05e2c5f996 100644
--- a/gnu/services/databases.scm
+++ b/gnu/services/databases.scm
@@ -167,8 +167,12 @@ (define contents
 (define-record-type* <postgresql-configuration>
   postgresql-configuration make-postgresql-configuration
   postgresql-configuration?
+  ;; Setting it to *no* ensures that the user sets its current postgresql
+  ;; explicitely.  Since major upgrades currently require a manual migration
+  ;; of the database, this way the user is responsible for upgrading properly.
+  ;; MAYBE TODO: Write an action to automatically upgrade in Guix.
   (postgresql         postgresql-configuration-postgresql ;file-like
-                      (default postgresql))
+                      (default *no*))
   (port               postgresql-configuration-port
                       (default 5432))
   (locale             postgresql-configuration-locale
@@ -346,7 +350,7 @@ (define postgresql-service-type
    (default-value (postgresql-configuration))
    (description "Run the PostgreSQL database server.")))
 
-(define-deprecated (postgresql-service #:key (postgresql postgresql)
+(define-deprecated (postgresql-service #:key (postgresql postgresql-10)
                                        (port 5432)
                                        (locale "en_US.utf8")
                                        (config-file (postgresql-config-file))
-- 
2.46.0

Message #23 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v2 2/2] etc: news: Warn users about postgresql default update.
Date: Wed,  6 Nov 2024 18:04:29 +0100

* etc/news.scm: Add entry.
---
 etc/news.scm | 43 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 43 insertions(+)

diff --git a/etc/news.scm b/etc/news.scm
index 4929fa2a9a..1dc8e64a72 100644
--- a/etc/news.scm
+++ b/etc/news.scm
@@ -25,6 +25,7 @@
 ;; Copyright © 2024 Vivien Kraus <vivien <at> planete-kraus.eu>
 ;; Copyright © 2024 Guillaume Le Vaillant <glv <at> posteo.net>
 ;; Copyright © 2024 Zheng Junjie <873216071 <at> qq.com>
+;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;
 ;; Copying and distribution of this file, with or without modification, are
 ;; permitted in any medium without royalty provided the copyright notice and
@@ -33,6 +34,48 @@
 (channel-news
  (version 0)
 
+ (entry (commit "") ; fill-me based on the previous commit
+        (title (en "PostgreSQL service upgrade")
+               (fr "Mise à jour du service PostgreSQL"))
+        (body
+         (en "\
+This news entry concerns users of the @code{postgresql-service-type}.
+
+The default PostgreSQL used in @code{postgresql-configuration} has been
+deprecated, and will be removed in a few months.  However, the value of the
+default @code{postgresql-configuration-postgresql} can't be directly changed
+to a newer major version of PostgreSQL, because switching to a major version
+currently requires a manual update of the database.
+
+Because of this, the default value of postgresql-configuration-postgresql has
+been set to @code{*no*}.  Current users of the service will have to set it
+manually.  If it was unset, use @code{postgresql-10} and plan an upgrade in the
+next month to a supported version of PostgreSQL.  To upgrade, you will need to
+either dump your database using your previous version of PostgreSQL and reload
+it in the new version or use the @command{pg_upgrade} application.  See
+@url{https://www.postgresql.org/docs/current/upgrading.html} for more
+info.")
+         (fr "\
+Ce message concerne les utilisateurs du service-type PostgreSQL.
+
+La précédente version par défaut de PostgreSQL utilisée dans
+@code{postgresqsl-configuration} est obsolète et sera supprimée dans quelques
+mois.  Cependant, la valeur de la configuration par défaut
+@code{postgresql-configuration-postgresql} ne peut pas être changée
+directement pour une version majeure plus récente de PostgreSQL, car le
+passage à une nouvelle version majeure nécessite actuellement une mise à jour
+manuelle de la base de données.
+
+Pour cette raison, la valeur par défaut de
+@code{postgresql-configuration-postgresql} est maintenant @code{*no*} et les
+utilisateurs actuels du service doivent la configurer manuellement. Si elle
+n’était pas définie, utilisez @code{postgresql-10} et prévoyez une mise à jour
+dans le mois à venir vers une version plus récente de PostgreSQL.  Pour
+effectuer la mise à jour, vous devrez soit exporter votre base de données sous
+une version de PostgreSQL puis la réimporter sous une nouvelle version, soit
+utiliser l'application @command{pg_upgrade}. Pour plus d'informations,
+consultez @url{https://www.postgresql.org/docs/current/upgrading.html}.")))
+
  (entry (commit "5966e0fdc78771c562e0f484a22f381a77908be0")
         (title
          (en "Daemon vulnerability allowing takeover of build users fixed")
-- 
2.46.0

Message #26 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v3 1/2] gnu: postgresql-configuration: Set default postgresql
 to *no*.
Date: Thu,  7 Nov 2024 18:09:59 +0100

* gnu/services/databases.scm (postgresql-configuration)[postgresql]:
Set default to *no*.
(postgresql-service): Revert default to postgresql-10 (rationale: We
can remove this service at the same time than postgresql-10, in
something like 6 months to a year).
---
 gnu/services/databases.scm | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/gnu/services/databases.scm b/gnu/services/databases.scm
index 0933eb5e79..c499356c3a 100644
--- a/gnu/services/databases.scm
+++ b/gnu/services/databases.scm
@@ -167,8 +167,12 @@ (define contents
 (define-record-type* <postgresql-configuration>
   postgresql-configuration make-postgresql-configuration
   postgresql-configuration?
+  ;; Setting it to '*no* ensures that the user sets its current postgresql
+  ;; explicitely.  Since major upgrades currently require a manual migration
+  ;; of the database, this way the user is responsible for upgrading properly.
+  ;; MAYBE TODO: Write an action to automatically upgrade in Guix.
   (postgresql         postgresql-configuration-postgresql ;file-like
-                      (default postgresql))
+                      (default '*no*))
   (port               postgresql-configuration-port
                       (default 5432))
   (locale             postgresql-configuration-locale
@@ -346,7 +350,7 @@ (define postgresql-service-type
    (default-value (postgresql-configuration))
    (description "Run the PostgreSQL database server.")))
 
-(define-deprecated (postgresql-service #:key (postgresql postgresql)
+(define-deprecated (postgresql-service #:key (postgresql postgresql-10)
                                        (port 5432)
                                        (locale "en_US.utf8")
                                        (config-file (postgresql-config-file))
-- 
2.46.0

Message #29 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v3 2/2] etc: news: Warn users about postgresql default update.
Date: Thu,  7 Nov 2024 18:10:00 +0100

* etc/news.scm: Add entry.
---
 etc/news.scm | 43 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 43 insertions(+)

diff --git a/etc/news.scm b/etc/news.scm
index 4929fa2a9a..078adeefb1 100644
--- a/etc/news.scm
+++ b/etc/news.scm
@@ -25,6 +25,7 @@
 ;; Copyright © 2024 Vivien Kraus <vivien <at> planete-kraus.eu>
 ;; Copyright © 2024 Guillaume Le Vaillant <glv <at> posteo.net>
 ;; Copyright © 2024 Zheng Junjie <873216071 <at> qq.com>
+;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;
 ;; Copying and distribution of this file, with or without modification, are
 ;; permitted in any medium without royalty provided the copyright notice and
@@ -33,6 +34,48 @@
 (channel-news
  (version 0)
 
+ (entry (commit "") ; fill-me based on the previous commit
+        (title (en "PostgreSQL service upgrade")
+               (fr "Mise à jour du service PostgreSQL"))
+        (body
+         (en "\
+This news entry concerns users of the @code{postgresql-service-type}.
+
+The default PostgreSQL used in @code{postgresql-configuration} has been
+deprecated, and will be removed in a few months.  However, the value of the
+default @code{postgresql-configuration-postgresql} can't be directly changed
+to a newer major version of PostgreSQL, because switching to a major version
+currently requires a manual update of the database.
+
+Because of this, the default value of postgresql-configuration-postgresql has
+been set to @code{'*no*}.  Current users of the service will have to set it
+manually.  If it was unset, use @code{postgresql-10} and plan an upgrade in the
+next month to a supported version of PostgreSQL.  To upgrade, you will need to
+either dump your database using your previous version of PostgreSQL and reload
+it in the new version or use the @command{pg_upgrade} application.  See
+@url{https://www.postgresql.org/docs/current/upgrading.html} for more
+info.")
+         (fr "\
+Ce message concerne les utilisateurs du service-type PostgreSQL.
+
+La précédente version par défaut de PostgreSQL utilisée dans
+@code{postgresqsl-configuration} est obsolète et sera supprimée dans quelques
+mois.  Cependant, la valeur de la configuration par défaut
+@code{postgresql-configuration-postgresql} ne peut pas être changée
+directement pour une version majeure plus récente de PostgreSQL, car le
+passage à une nouvelle version majeure nécessite actuellement une mise à jour
+manuelle de la base de données.
+
+Pour cette raison, la valeur par défaut de
+@code{postgresql-configuration-postgresql} est maintenant @code{'*no*} et les
+utilisateurs actuels du service doivent la configurer manuellement. Si elle
+n’était pas définie, utilisez @code{postgresql-10} et prévoyez une mise à jour
+dans le mois à venir vers une version plus récente de PostgreSQL.  Pour
+effectuer la mise à jour, vous devrez soit exporter votre base de données sous
+une version de PostgreSQL puis la réimporter sous une nouvelle version, soit
+utiliser l'application @command{pg_upgrade}. Pour plus d'informations,
+consultez @url{https://www.postgresql.org/docs/current/upgrading.html}.")))
+
  (entry (commit "5966e0fdc78771c562e0f484a22f381a77908be0")
         (title
          (en "Daemon vulnerability allowing takeover of build users fixed")
-- 
2.46.0

Message #32 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: "pelzflorian (Florian Pelz)" <pelzflorian <at> pelzflorian.de>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [bug#74058] [PATCH v3 2/2] etc: news: Warn users about
 postgresql default update.
Date: Fri, 08 Nov 2024 12:18:00 +0100

Hello Nicolas, thank you for taking care of making PostgreSQL current.
I tried to test with `make check-system TESTS="patchwork"`, but it
fails even without your change.

Anyway.

Can you include this German translation?

>  (entry (commit "") ; fill-me based on the previous commit
>         (title (en "PostgreSQL service upgrade")

                 (de "PostgreSQL-Dienst aktualisiert")


>                (fr "Mise à jour du service PostgreSQL"))
>         (body
>          (en "\
> This news entry concerns users of the @code{postgresql-service-type}.
> 
> The default PostgreSQL used in @code{postgresql-configuration} has been
> deprecated, and will be removed in a few months.  However, the value of the
> default @code{postgresql-configuration-postgresql} can't be directly changed
> to a newer major version of PostgreSQL, because switching to a major version
> currently requires a manual update of the database.
> 
> Because of this, the default value of postgresql-configuration-postgresql has
> been set to @code{'*no*}.  Current users of the service will have to set it
> manually.  If it was unset, use @code{postgresql-10} and plan an upgrade in the
> next month to a supported version of PostgreSQL.  To upgrade, you will need to
> either dump your database using your previous version of PostgreSQL and reload
> it in the new version or use the @command{pg_upgrade} application.  See
> @url{https://www.postgresql.org/docs/current/upgrading.html} for more
> info.")

         (de "Diese Neuigkeit betrifft Nutzer des @code{postgresql-service-type}.

Die in @code{postgresql-configuration} bisher vorgegebene Version von gilt als
veraltet und wird in ein paar Monaten entfernt werden.  Allerdings können wir
den Vorgabewert von @code{postgresql-configuration-postgresql} nicht einfach
für Sie auf eine neuere große Version ändern, weil bei so einem Wechsel die
Datenbank derzeit manuell aktualisiert werden muss.

Aus diesem Grund wurde der Vorgabewert von
@code{postgresql-configuration-postgresql} von nun an auf @code{'*no*} gesetzt.
Aktuelle Nutzer des Dienstes müssen sie selbst setzen.  Wenn kein Wert gesetzt
war, sollten Sie @code{postgresql-10} eintragen und dabei einplanen, dass Sie
es kommenden Monat auf eine unterstützte Version von PostgreSQL aktualisieren
müssen.  Zum Aktualisieren werden Sie entweder mit der vorherigen Version von
PostgreSQL ein Dump Ihrer Datenbank exportieren, das Sie dann in der neuen
Version laden, oder Sie benutzen dafür die Anwendung @command{pg_upgrade}.
Siehe @url{https://www.postgresql.org/docs/current/upgrading.html} für weitere
Informationen.")



Furthermore, for the French version:


>          (fr "\
> Ce message concerne les utilisateurs du service-type PostgreSQL.
> 
> La précédente version par défaut de PostgreSQL utilisée dans
> @code{postgresqsl-configuration} est obsolète et sera supprimée dans quelques
> mois.  Cependant, la valeur de la configuration par défaut
> @code{postgresql-configuration-postgresql} ne peut pas être changée
> directement pour une version majeure plus récente de PostgreSQL, car le
> passage à une nouvelle version majeure nécessite actuellement une mise à jour
> manuelle de la base de données.
> 
> Pour cette raison, la valeur par défaut de
> @code{postgresql-configuration-postgresql} est maintenant @code{'*no*} et les
> utilisateurs actuels du service doivent la configurer manuellement. Si elle
> n’était pas définie, utilisez @code{postgresql-10} et prévoyez une mise à jour
> dans le mois à venir vers une version plus récente de PostgreSQL.

In the last sentence, « version plus récente » should be supported version.




> Pour
> effectuer la mise à jour, vous devrez soit exporter votre base de données sous
> une version de PostgreSQL puis la réimporter sous une nouvelle version, soit
> utiliser l'application @command{pg_upgrade}. Pour plus d'informations,
> consultez @url{https://www.postgresql.org/docs/current/upgrading.html}.")))

Regards,
Florian

Message #35 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: "pelzflorian (Florian Pelz)" <pelzflorian <at> pelzflorian.de>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [bug#74058] [PATCH v3 2/2] etc: news: Warn users about
 postgresql default update.
Date: Sat, 09 Nov 2024 23:13:25 +0100

On 2024-11-08 12:18, pelzflorian (Florian Pelz) wrote:

> Hello Nicolas, thank you for taking care of making PostgreSQL current.
> I tried to test with `make check-system TESTS="patchwork"`, but it
> fails even without your change.

I've also checked that it fails even before
351fdf69f737d09d24499d6c1ff70a2325fff0fa, so I'm considering this out of
scope.  We should probably open an issue though.  It fails with a 500
during the http-get test.

> Anyway.
>
> Can you include this German translation?

Thanks a lot Florian, adding it now.

[...]

> In the last sentence, « version plus récente » should be supported version.

Indeed, this is actually not that easy to translate, "version supportée"
sounds awful in French.  Maybe "version prise en charge" but that doesn't
sound natural either.  Will try to find something more suitable. 

-- 
Best regards,
Nicolas Graves

Message #38 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v4 1/2] gnu: postgresql-configuration: Set default postgresql
 to *no*.
Date: Sat,  9 Nov 2024 23:23:34 +0100

* gnu/services/databases.scm (postgresql-configuration)[postgresql]:
Set default to *no*.
(postgresql-service): Revert default to postgresql-10 (rationale: We
can remove this service at the same time than postgresql-10, in
something like 6 months to a year).
---
 gnu/services/databases.scm | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/gnu/services/databases.scm b/gnu/services/databases.scm
index 0933eb5e79..c499356c3a 100644
--- a/gnu/services/databases.scm
+++ b/gnu/services/databases.scm
@@ -167,8 +167,12 @@ (define contents
 (define-record-type* <postgresql-configuration>
   postgresql-configuration make-postgresql-configuration
   postgresql-configuration?
+  ;; Setting it to '*no* ensures that the user sets its current postgresql
+  ;; explicitely.  Since major upgrades currently require a manual migration
+  ;; of the database, this way the user is responsible for upgrading properly.
+  ;; MAYBE TODO: Write an action to automatically upgrade in Guix.
   (postgresql         postgresql-configuration-postgresql ;file-like
-                      (default postgresql))
+                      (default '*no*))
   (port               postgresql-configuration-port
                       (default 5432))
   (locale             postgresql-configuration-locale
@@ -346,7 +350,7 @@ (define postgresql-service-type
    (default-value (postgresql-configuration))
    (description "Run the PostgreSQL database server.")))
 
-(define-deprecated (postgresql-service #:key (postgresql postgresql)
+(define-deprecated (postgresql-service #:key (postgresql postgresql-10)
                                        (port 5432)
                                        (locale "en_US.utf8")
                                        (config-file (postgresql-config-file))
-- 
2.46.0

Message #41 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>,
 Florian Pelz <pelzflorian <at> pelzflorian.de>
Subject: [PATCH v4 2/2] etc: news: Warn users about postgresql default update.
Date: Sat,  9 Nov 2024 23:23:35 +0100

* etc/news.scm: Add entry.

Co-authored-by: Florian Pelz <pelzflorian <at> pelzflorian.de>
---
 etc/news.scm | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 64 insertions(+)

diff --git a/etc/news.scm b/etc/news.scm
index 4929fa2a9a..ba1f204b08 100644
--- a/etc/news.scm
+++ b/etc/news.scm
@@ -25,6 +25,7 @@
 ;; Copyright © 2024 Vivien Kraus <vivien <at> planete-kraus.eu>
 ;; Copyright © 2024 Guillaume Le Vaillant <glv <at> posteo.net>
 ;; Copyright © 2024 Zheng Junjie <873216071 <at> qq.com>
+;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;
 ;; Copying and distribution of this file, with or without modification, are
 ;; permitted in any medium without royalty provided the copyright notice and
@@ -33,6 +34,69 @@
 (channel-news
  (version 0)
 
+ (entry (commit "")  ;TODO fill it based on the previous commit
+        (title (en "PostgreSQL service upgrade")
+               (de "PostgreSQL-Dienst aktualisiert")
+               (fr "Mise à jour du service PostgreSQL"))
+        (body
+         (en "\
+This news entry concerns users of the @code{postgresql-service-type}.
+
+The default PostgreSQL used in @code{postgresql-configuration} has been
+deprecated, and will be removed in a few months.  However, the value of the
+default @code{postgresql-configuration-postgresql} can't be directly changed
+to a newer major version of PostgreSQL, because switching to a major version
+currently requires a manual update of the database.
+
+Because of this, the default value of postgresql-configuration-postgresql has
+been set to @code{'*no*}.  Current users of the service will have to set it
+manually.  If it was unset, use @code{postgresql-10} and plan an upgrade in the
+next month to a supported version of PostgreSQL.  To upgrade, you will need to
+either dump your database using your previous version of PostgreSQL and reload
+it in the new version or use the @command{pg_upgrade} application.  See
+@url{https://www.postgresql.org/docs/current/upgrading.html} for more
+info.")
+         (de "\
+Diese Neuigkeit betrifft Nutzer des @code{postgresql-service-type}.
+
+Die in @code{postgresql-configuration} bisher vorgegebene Version von gilt als
+veraltet und wird in ein paar Monaten entfernt werden.  Allerdings können wir
+den Vorgabewert von @code{postgresql-configuration-postgresql} nicht einfach
+für Sie auf eine neuere große Version ändern, weil bei so einem Wechsel die
+Datenbank derzeit manuell aktualisiert werden muss.
+
+Aus diesem Grund wurde der Vorgabewert von
+@code{postgresql-configuration-postgresql} von nun an auf @code{'*no*}
+gesetzt.  Aktuelle Nutzer des Dienstes müssen sie selbst setzen.  Wenn kein
+Wert gesetzt war, sollten Sie @code{postgresql-10} eintragen und dabei
+einplanen, dass Sie es kommenden Monat auf eine unterstützte Version von
+PostgreSQL aktualisieren müssen.  Zum Aktualisieren werden Sie entweder mit
+der vorherigen Version von PostgreSQL ein Dump Ihrer Datenbank exportieren,
+das Sie dann in der neuen Version laden, oder Sie benutzen dafür die Anwendung
+@command{pg_upgrade}.  Siehe
+@url{https://www.postgresql.org/docs/current/upgrading.html} für weitere
+Informationen.")
+         (fr "\
+Ce message concerne les utilisateurs du service-type PostgreSQL.
+
+La précédente version par défaut de PostgreSQL utilisée dans
+@code{postgresqsl-configuration} est obsolète et sera supprimée dans quelques
+mois.  Cependant, la valeur de la configuration par défaut
+@code{postgresql-configuration-postgresql} ne peut pas être changée
+directement pour une version majeure prise en charge de PostgreSQL, car le
+passage à une nouvelle version majeure nécessite actuellement une mise à jour
+manuelle de la base de données.
+
+Pour cette raison, la valeur par défaut de
+@code{postgresql-configuration-postgresql} est maintenant @code{'*no*} et les
+utilisateurs actuels du service doivent la configurer manuellement. Si elle
+n’était pas définie, utilisez @code{postgresql-10} et prévoyez une mise à jour
+dans le mois à venir vers une version plus récente de PostgreSQL.  Pour
+effectuer la mise à jour, vous devrez soit exporter votre base de données sous
+une version de PostgreSQL puis la réimporter sous une nouvelle version, soit
+utiliser l'application @command{pg_upgrade}. Pour plus d'informations,
+consultez @url{https://www.postgresql.org/docs/current/upgrading.html}.")))
+
  (entry (commit "5966e0fdc78771c562e0f484a22f381a77908be0")
         (title
          (en "Daemon vulnerability allowing takeover of build users fixed")
-- 
2.46.0

Message #44 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: "pelzflorian (Florian Pelz)" <pelzflorian <at> pelzflorian.de>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [PATCH v4 2/2] etc: news: Warn users about postgresql default
 update.
Date: Sun, 10 Nov 2024 09:30:34 +0100

Hello Nicolas.

Nicolas Graves <ngraves <at> ngraves.fr> writes:
> +         (de "\
> +Diese Neuigkeit betrifft Nutzer des @code{postgresql-service-type}.
> +
> +Die in @code{postgresql-configuration} bisher vorgegebene Version von gilt als

Ooops; I deleted a word; should be:

> +Die in @code{postgresql-configuration} bisher vorgegebene Version von PostgreSQL gilt als

Sorry!

Nicolas Graves <ngraves <at> ngraves.fr> writes:
> I've also checked that it fails even before
> 351fdf69f737d09d24499d6c1ff70a2325fff0fa, so I'm considering this out of
> scope.  We should probably open an issue though.  It fails with a 500
> during the http-get test.

It happened sometime this year.  But I will not work on it now and I see
there are a few other open test-suite bugs already, so I will not open
the new bug:

https://issues.guix.gnu.org/69518

https://issues.guix.gnu.org/74286

Regards,
Florian

Message #47 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Ludovic Courtès <ludo <at> gnu.org>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [bug#74058] [PATCH v2 1/2] gnu: postgresql-configuration: Set
 default postgresql to *no*.
Date: Tue, 12 Nov 2024 10:03:08 +0100

Hi,

Nicolas Graves <ngraves <at> ngraves.fr> skribis:

> * gnu/services/databases.scm (postgresql-configuration)[postgresql]:
> Set default to *no*.
> (postgresql-service): Revert default to postgresql-10 (rationale: We
> can remove this service at the same time than postgresql-10, in
> something like 6 months to a year).
> ---
>  gnu/services/databases.scm | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/services/databases.scm b/gnu/services/databases.scm
> index 0933eb5e79..05e2c5f996 100644
> --- a/gnu/services/databases.scm
> +++ b/gnu/services/databases.scm
> @@ -167,8 +167,12 @@ (define contents
>  (define-record-type* <postgresql-configuration>
>    postgresql-configuration make-postgresql-configuration
>    postgresql-configuration?
> +  ;; Setting it to *no* ensures that the user sets its current postgresql
> +  ;; explicitely.  Since major upgrades currently require a manual migration
> +  ;; of the database, this way the user is responsible for upgrading properly.
> +  ;; MAYBE TODO: Write an action to automatically upgrade in Guix.
>    (postgresql         postgresql-configuration-postgresql ;file-like
> -                      (default postgresql))
> +                      (default *no*))

I think we misunderstood each other; when I wrote:

> How about having *no* default value in that case?

I suggested removing the (default …) bit altogether, as in:

  (define-record-type* <postgresql-configuration>
    postgresql-configuration make-postgresql-configuration
    postgresql-configuration?
    (postgresql postgresql-configuration-postgresql)  ;no default value
    …)

That way, users get a syntax error (at macro-expansion time) if they
omit the ‘postgresql’ field.

Does that make sense?

Ludo’.

Message #50 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [bug#74058] [PATCH v2 1/2] gnu: postgresql-configuration: Set
 default postgresql to *no*.
Date: Tue, 12 Nov 2024 10:51:40 +0100

On 2024-11-12 10:03, Ludovic Courtès wrote:

>
> I think we misunderstood each other; when I wrote:

Big indeed ;)

>> How about having *no* default value in that case?
>
> I suggested removing the (default …) bit altogether, as in:
>
>   (define-record-type* <postgresql-configuration>
>     postgresql-configuration make-postgresql-configuration
>     postgresql-configuration?
>     (postgresql postgresql-configuration-postgresql)  ;no default value
>     …)
>
> That way, users get a syntax error (at macro-expansion time) if they
> omit the ‘postgresql’ field.
>
> Does that make sense?

Yep, will resend a version.

-- 
Best regards,
Nicolas Graves

Message #53 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v5 1/2] gnu: postgresql-configuration: Unset default
 postgresql.
Date: Wed, 13 Nov 2024 10:17:23 +0100

* gnu/services/databases.scm (postgresql-configuration)[postgresql]:
Unset default.
(postgresql-service): Revert default to postgresql-10 (rationale: We
can remove this service at the same time than postgresql-10, in
something like 6 months to a year).
---
 gnu/services/databases.scm | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/gnu/services/databases.scm b/gnu/services/databases.scm
index 0933eb5e79..001f96830e 100644
--- a/gnu/services/databases.scm
+++ b/gnu/services/databases.scm
@@ -167,8 +167,11 @@ (define contents
 (define-record-type* <postgresql-configuration>
   postgresql-configuration make-postgresql-configuration
   postgresql-configuration?
-  (postgresql         postgresql-configuration-postgresql ;file-like
-                      (default postgresql))
+  ;; Setting no default ensures that the user sets its current postgresql
+  ;; explicitely.  Since major upgrades currently require a manual migration
+  ;; of the database, this way the user is responsible for upgrading properly.
+  ;; MAYBE TODO: Write an action to automatically upgrade in Guix.
+  (postgresql         postgresql-configuration-postgresql) ;file-like
   (port               postgresql-configuration-port
                       (default 5432))
   (locale             postgresql-configuration-locale
@@ -346,7 +349,7 @@ (define postgresql-service-type
    (default-value (postgresql-configuration))
    (description "Run the PostgreSQL database server.")))
 
-(define-deprecated (postgresql-service #:key (postgresql postgresql)
+(define-deprecated (postgresql-service #:key (postgresql postgresql-10)
                                        (port 5432)
                                        (locale "en_US.utf8")
                                        (config-file (postgresql-config-file))
-- 
2.46.0

Message #56 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v6 1/2] gnu: postgresql-configuration: Unset default
 postgresql.
Date: Wed, 13 Nov 2024 10:19:16 +0100

* gnu/services/databases.scm (postgresql-configuration)[postgresql]:
Unset default.
(postgresql-service): Revert default to postgresql-10 (rationale: We
can remove this service at the same time than postgresql-10, in
something like 6 months to a year).
---
 gnu/services/databases.scm | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/gnu/services/databases.scm b/gnu/services/databases.scm
index 0933eb5e79..001f96830e 100644
--- a/gnu/services/databases.scm
+++ b/gnu/services/databases.scm
@@ -167,8 +167,11 @@ (define contents
 (define-record-type* <postgresql-configuration>
   postgresql-configuration make-postgresql-configuration
   postgresql-configuration?
-  (postgresql         postgresql-configuration-postgresql ;file-like
-                      (default postgresql))
+  ;; Setting no default ensures that the user sets its current postgresql
+  ;; explicitely.  Since major upgrades currently require a manual migration
+  ;; of the database, this way the user is responsible for upgrading properly.
+  ;; MAYBE TODO: Write an action to automatically upgrade in Guix.
+  (postgresql         postgresql-configuration-postgresql) ;file-like
   (port               postgresql-configuration-port
                       (default 5432))
   (locale             postgresql-configuration-locale
@@ -346,7 +349,7 @@ (define postgresql-service-type
    (default-value (postgresql-configuration))
    (description "Run the PostgreSQL database server.")))
 
-(define-deprecated (postgresql-service #:key (postgresql postgresql)
+(define-deprecated (postgresql-service #:key (postgresql postgresql-10)
                                        (port 5432)
                                        (locale "en_US.utf8")
                                        (config-file (postgresql-config-file))
-- 
2.46.0

Message #59 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>,
 Florian Pelz <pelzflorian <at> pelzflorian.de>
Subject: [PATCH v6 2/2] etc: news: Warn users about postgresql default update.
Date: Wed, 13 Nov 2024 10:19:17 +0100

* etc/news.scm: Add entry.

Co-authored-by: Florian Pelz <pelzflorian <at> pelzflorian.de>
---
 etc/news.scm | 63 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 63 insertions(+)

diff --git a/etc/news.scm b/etc/news.scm
index fbf16615df..116fa0c43b 100644
--- a/etc/news.scm
+++ b/etc/news.scm
@@ -25,6 +25,7 @@
 ;; Copyright © 2024 Vivien Kraus <vivien <at> planete-kraus.eu>
 ;; Copyright © 2024 Guillaume Le Vaillant <glv <at> posteo.net>
 ;; Copyright © 2024 Zheng Junjie <873216071 <at> qq.com>
+;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;
 ;; Copying and distribution of this file, with or without modification, are
 ;; permitted in any medium without royalty provided the copyright notice and
@@ -33,6 +34,68 @@
 (channel-news
  (version 0)
 
+ (entry (commit "")  ;TODO fill it based on the previous commit
+        (title (en "PostgreSQL service upgrade")
+               (de "PostgreSQL-Dienst aktualisiert")
+               (fr "Mise à jour du service PostgreSQL"))
+        (body
+         (en "\
+This news entry concerns users of the @code{postgresql-service-type}.
+
+The default PostgreSQL used in @code{postgresql-configuration} has been
+deprecated, and will be removed in a few months.  However, the value of the
+default @code{postgresql-configuration-postgresql} can't be directly changed
+to a newer major version of PostgreSQL, because switching to a major version
+currently requires a manual update of the database.
+
+Because of this, the default value of postgresql-configuration-postgresql has
+been unset.  Current users of the service will have to set it manually.  If it
+was unset, use @code{postgresql-10} and plan an upgrade in the next month to a
+supported version of PostgreSQL.  To upgrade, you will need to either dump
+your database using your previous version of PostgreSQL and reload it in the
+new version or use the @command{pg_upgrade} application.  See
+@url{https://www.postgresql.org/docs/current/upgrading.html} for more info.")
+         (de "\
+Diese Neuigkeit betrifft Nutzer des @code{postgresql-service-type}.
+
+Die in @code{postgresql-configuration} bisher vorgegebene Version von
+PostgreSQL gilt als veraltet und wird in ein paar Monaten entfernt werden.
+Allerdings können wir den Vorgabewert von
+@code{postgresql-configuration-postgresql} nicht einfach für Sie auf eine
+neuere große Version ändern, weil bei so einem Wechsel die Datenbank derzeit
+manuell aktualisiert werden muss.
+
+Aus diesem Grund wurde der Vorgabewert von
+@code{postgresql-configuration-postgresql} von nun an entfernt.  Aktuelle
+Nutzer des Dienstes müssen sie selbst setzen.  Wenn kein Wert gesetzt war,
+sollten Sie @code{postgresql-10} eintragen und dabei einplanen, dass Sie es
+kommenden Monat auf eine unterstützte Version von PostgreSQL aktualisieren
+müssen.  Zum Aktualisieren werden Sie entweder mit der vorherigen Version von
+PostgreSQL ein Dump Ihrer Datenbank exportieren, das Sie dann in der neuen
+Version laden, oder Sie benutzen dafür die Anwendung @command{pg_upgrade}.
+Siehe @url{https://www.postgresql.org/docs/current/upgrading.html} für weitere
+Informationen.")
+         (fr "\
+Ce message concerne les utilisateurs du service-type PostgreSQL.
+
+La précédente version par défaut de PostgreSQL utilisée dans
+@code{postgresqsl-configuration} est obsolète et sera supprimée dans quelques
+mois.  Cependant, la valeur de la configuration par défaut
+@code{postgresql-configuration-postgresql} ne peut pas être changée
+directement pour une version majeure prise en charge de PostgreSQL, car le
+passage à une nouvelle version majeure nécessite actuellement une mise à jour
+manuelle de la base de données.
+
+Pour cette raison, la valeur par défaut de
+@code{postgresql-configuration-postgresql} a été supprimée et les utilisateurs
+actuels du service doivent la configurer manuellement. Si elle n’était pas
+définie, utilisez @code{postgresql-10} et prévoyez une mise à jour dans le
+mois à venir vers une version plus récente de PostgreSQL.  Pour effectuer la
+mise à jour, vous devrez soit exporter votre base de données sous une version
+de PostgreSQL puis la réimporter sous une nouvelle version, soit utiliser
+l'application @command{pg_upgrade}. Pour plus d'informations, consultez
+@url{https://www.postgresql.org/docs/current/upgrading.html}.")))
+
  (entry (commit "cfc85eb0c67a5cf10a3fbe2531b926cbb8c62489")
         (title
          (en "Linux-libre 6.10 removed due to end of upstream support")
-- 
2.46.0

Message #62 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: "pelzflorian (Florian Pelz)" <pelzflorian <at> pelzflorian.de>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [PATCH v6 2/2] etc: news: Warn users about postgresql default
 update.
Date: Thu, 14 Nov 2024 12:09:49 +0100

News LGTM (thank you for updating the German).  IIUC, doc/guix.texi
still contains:
> @table @asis
> @item @code{postgresql} (default: @code{postgresql})
> PostgreSQL package to use for the service.

Could you remove this default, like anonip-service-type also
has no default in its input field:
> @table @asis
> @item @code{anonip} (default: @code{anonip})
> The anonip package to use.
> 
> @item @code{input}
> The file name of the input log file to process.  The service creates a
> FIFO of this name.  The web server should write its logs to this FIFO.

Regards,
Florian

Message #65 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v7 1/2] gnu: postgresql-configuration: Unset default
 postgresql.
Date: Fri, 15 Nov 2024 12:03:47 +0100

* gnu/services/databases.scm (postgresql-configuration)[postgresql]:
Unset default.
(postgresql-service): Revert default to postgresql-10 (rationale: We
can remove this service at the same time than postgresql-10, in
something like 6 months to a year).
* doc/guix.texi: Remove postgresql default reference in documentation.
---
 doc/guix.texi              | 2 +-
 gnu/services/databases.scm | 9 ++++++---
 2 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index 2ab78d6237..ba4bd2cc79 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -26247,7 +26247,7 @@ Data type representing the configuration for the
 @code{postgresql-service-type}.
 
 @table @asis
-@item @code{postgresql} (default: @code{postgresql})
+@item @code{postgresql}
 PostgreSQL package to use for the service.
 
 @item @code{port} (default: @code{5432})
diff --git a/gnu/services/databases.scm b/gnu/services/databases.scm
index 0933eb5e79..001f96830e 100644
--- a/gnu/services/databases.scm
+++ b/gnu/services/databases.scm
@@ -167,8 +167,11 @@ (define contents
 (define-record-type* <postgresql-configuration>
   postgresql-configuration make-postgresql-configuration
   postgresql-configuration?
-  (postgresql         postgresql-configuration-postgresql ;file-like
-                      (default postgresql))
+  ;; Setting no default ensures that the user sets its current postgresql
+  ;; explicitely.  Since major upgrades currently require a manual migration
+  ;; of the database, this way the user is responsible for upgrading properly.
+  ;; MAYBE TODO: Write an action to automatically upgrade in Guix.
+  (postgresql         postgresql-configuration-postgresql) ;file-like
   (port               postgresql-configuration-port
                       (default 5432))
   (locale             postgresql-configuration-locale
@@ -346,7 +349,7 @@ (define postgresql-service-type
    (default-value (postgresql-configuration))
    (description "Run the PostgreSQL database server.")))
 
-(define-deprecated (postgresql-service #:key (postgresql postgresql)
+(define-deprecated (postgresql-service #:key (postgresql postgresql-10)
                                        (port 5432)
                                        (locale "en_US.utf8")
                                        (config-file (postgresql-config-file))
-- 
2.46.0

Message #68 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>,
 Florian Pelz <pelzflorian <at> pelzflorian.de>
Subject: [PATCH v7 2/2] etc: news: Warn users about postgresql default update.
Date: Fri, 15 Nov 2024 12:03:48 +0100

* etc/news.scm: Add entry.

Co-authored-by: Florian Pelz <pelzflorian <at> pelzflorian.de>
---
 etc/news.scm | 63 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 63 insertions(+)

diff --git a/etc/news.scm b/etc/news.scm
index fbf16615df..116fa0c43b 100644
--- a/etc/news.scm
+++ b/etc/news.scm
@@ -25,6 +25,7 @@
 ;; Copyright © 2024 Vivien Kraus <vivien <at> planete-kraus.eu>
 ;; Copyright © 2024 Guillaume Le Vaillant <glv <at> posteo.net>
 ;; Copyright © 2024 Zheng Junjie <873216071 <at> qq.com>
+;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;
 ;; Copying and distribution of this file, with or without modification, are
 ;; permitted in any medium without royalty provided the copyright notice and
@@ -33,6 +34,68 @@
 (channel-news
  (version 0)
 
+ (entry (commit "")  ;TODO fill it based on the previous commit
+        (title (en "PostgreSQL service upgrade")
+               (de "PostgreSQL-Dienst aktualisiert")
+               (fr "Mise à jour du service PostgreSQL"))
+        (body
+         (en "\
+This news entry concerns users of the @code{postgresql-service-type}.
+
+The default PostgreSQL used in @code{postgresql-configuration} has been
+deprecated, and will be removed in a few months.  However, the value of the
+default @code{postgresql-configuration-postgresql} can't be directly changed
+to a newer major version of PostgreSQL, because switching to a major version
+currently requires a manual update of the database.
+
+Because of this, the default value of postgresql-configuration-postgresql has
+been unset.  Current users of the service will have to set it manually.  If it
+was unset, use @code{postgresql-10} and plan an upgrade in the next month to a
+supported version of PostgreSQL.  To upgrade, you will need to either dump
+your database using your previous version of PostgreSQL and reload it in the
+new version or use the @command{pg_upgrade} application.  See
+@url{https://www.postgresql.org/docs/current/upgrading.html} for more info.")
+         (de "\
+Diese Neuigkeit betrifft Nutzer des @code{postgresql-service-type}.
+
+Die in @code{postgresql-configuration} bisher vorgegebene Version von
+PostgreSQL gilt als veraltet und wird in ein paar Monaten entfernt werden.
+Allerdings können wir den Vorgabewert von
+@code{postgresql-configuration-postgresql} nicht einfach für Sie auf eine
+neuere große Version ändern, weil bei so einem Wechsel die Datenbank derzeit
+manuell aktualisiert werden muss.
+
+Aus diesem Grund wurde der Vorgabewert von
+@code{postgresql-configuration-postgresql} von nun an entfernt.  Aktuelle
+Nutzer des Dienstes müssen sie selbst setzen.  Wenn kein Wert gesetzt war,
+sollten Sie @code{postgresql-10} eintragen und dabei einplanen, dass Sie es
+kommenden Monat auf eine unterstützte Version von PostgreSQL aktualisieren
+müssen.  Zum Aktualisieren werden Sie entweder mit der vorherigen Version von
+PostgreSQL ein Dump Ihrer Datenbank exportieren, das Sie dann in der neuen
+Version laden, oder Sie benutzen dafür die Anwendung @command{pg_upgrade}.
+Siehe @url{https://www.postgresql.org/docs/current/upgrading.html} für weitere
+Informationen.")
+         (fr "\
+Ce message concerne les utilisateurs du service-type PostgreSQL.
+
+La précédente version par défaut de PostgreSQL utilisée dans
+@code{postgresqsl-configuration} est obsolète et sera supprimée dans quelques
+mois.  Cependant, la valeur de la configuration par défaut
+@code{postgresql-configuration-postgresql} ne peut pas être changée
+directement pour une version majeure prise en charge de PostgreSQL, car le
+passage à une nouvelle version majeure nécessite actuellement une mise à jour
+manuelle de la base de données.
+
+Pour cette raison, la valeur par défaut de
+@code{postgresql-configuration-postgresql} a été supprimée et les utilisateurs
+actuels du service doivent la configurer manuellement. Si elle n’était pas
+définie, utilisez @code{postgresql-10} et prévoyez une mise à jour dans le
+mois à venir vers une version plus récente de PostgreSQL.  Pour effectuer la
+mise à jour, vous devrez soit exporter votre base de données sous une version
+de PostgreSQL puis la réimporter sous une nouvelle version, soit utiliser
+l'application @command{pg_upgrade}. Pour plus d'informations, consultez
+@url{https://www.postgresql.org/docs/current/upgrading.html}.")))
+
  (entry (commit "cfc85eb0c67a5cf10a3fbe2531b926cbb8c62489")
         (title
          (en "Linux-libre 6.10 removed due to end of upstream support")
-- 
2.46.0

Message #71 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: "pelzflorian (Florian Pelz)" <pelzflorian <at> pelzflorian.de>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [bug#74058] [PATCH v7 1/2] gnu: postgresql-configuration: Unset
 default postgresql.
Date: Fri, 15 Nov 2024 22:26:57 +0100

Hello Nicolas.  I had not tried previously and did not use postgres, but
now I applied patch 1/2 to my local guix.git checkout and pull from it
and `guix pull` gives an error  in the default value of
(define postgresql-service-type:



$ gzip -cd /var/log/guix/drvs/hr/8iqk1a4pkchbayr4x4mlm7za2kwxyn-guix-system.drv.gz
[  7/ 50] loading.....   28.0% of 25 filesice-9/psyntax.scm:2824:12: In procedure syntax-violation:
Syntax error:
gnu/services/databases.scm:349:18: postgresql-configuration: missing field initializers (postgresql) in form (postgresql-configuration)


Maybe the default value needs to be adapted;
or maybe I am missing some prerequisite patch if pull works for you.

Regards,
Florian

Message #74 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: "pelzflorian (Florian Pelz)" <pelzflorian <at> pelzflorian.de>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [bug#74058] [PATCH v7 1/2] gnu: postgresql-configuration: Unset
 default postgresql.
Date: Sat, 16 Nov 2024 01:06:09 +0100

On 2024-11-15 22:26, pelzflorian (Florian Pelz) wrote:

> Hello Nicolas.  I had not tried previously and did not use postgres, but
> now I applied patch 1/2 to my local guix.git checkout and pull from it
> and `guix pull` gives an error  in the default value of
> (define postgresql-service-type:
>
>
>
> $ gzip -cd /var/log/guix/drvs/hr/8iqk1a4pkchbayr4x4mlm7za2kwxyn-guix-system.drv.gz
> [  7/ 50] loading.....   28.0% of 25 filesice-9/psyntax.scm:2824:12: In procedure syntax-violation:
> Syntax error:
> gnu/services/databases.scm:349:18: postgresql-configuration: missing field initializers (postgresql) in form (postgresql-configuration)
>
>
> Maybe the default value needs to be adapted;
> or maybe I am missing some prerequisite patch if pull works for you.
>
> Regards,
> Florian

Thought I pulled properly, I'll retry. Thanks for pointing that out.
>
>
>

-- 
Best regards,
Nicolas Graves

Message #77 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>
Subject: [PATCH v8 1/2] gnu: postgresql-configuration: Unset default
 postgresql.
Date: Sat, 16 Nov 2024 14:48:02 +0100

* gnu/services/databases.scm (postgresql-configuration)[postgresql]:
Unset default.
(postgresql-service-type): Remove default-value.
(postgresql-service): Revert default to postgresql-10 (rationale: We
can remove this service at the same time than postgresql-10, in
something like 6 months to a year).
* doc/guix.texi: Remove postgresql default reference in documentation.
---
 doc/guix.texi              |  2 +-
 gnu/services/databases.scm | 10 ++++++----
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/doc/guix.texi b/doc/guix.texi
index 2ab78d6237..ba4bd2cc79 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -26247,7 +26247,7 @@ Data type representing the configuration for the
 @code{postgresql-service-type}.
 
 @table @asis
-@item @code{postgresql} (default: @code{postgresql})
+@item @code{postgresql}
 PostgreSQL package to use for the service.
 
 @item @code{port} (default: @code{5432})
diff --git a/gnu/services/databases.scm b/gnu/services/databases.scm
index 0933eb5e79..e8a4acc996 100644
--- a/gnu/services/databases.scm
+++ b/gnu/services/databases.scm
@@ -167,8 +167,11 @@ (define contents
 (define-record-type* <postgresql-configuration>
   postgresql-configuration make-postgresql-configuration
   postgresql-configuration?
-  (postgresql         postgresql-configuration-postgresql ;file-like
-                      (default postgresql))
+  ;; Setting no default ensures that the user sets its current postgresql
+  ;; explicitely.  Since major upgrades currently require a manual migration
+  ;; of the database, this way the user is responsible for upgrading properly.
+  ;; MAYBE TODO: Write an action to automatically upgrade in Guix.
+  (postgresql         postgresql-configuration-postgresql) ;file-like
   (port               postgresql-configuration-port
                       (default 5432))
   (locale             postgresql-configuration-locale
@@ -343,10 +346,9 @@ (define postgresql-service-type
           (service-extension
            profile-service-type
            (compose list postgresql-configuration-postgresql))))
-   (default-value (postgresql-configuration))
    (description "Run the PostgreSQL database server.")))
 
-(define-deprecated (postgresql-service #:key (postgresql postgresql)
+(define-deprecated (postgresql-service #:key (postgresql postgresql-10)
                                        (port 5432)
                                        (locale "en_US.utf8")
                                        (config-file (postgresql-config-file))
-- 
2.46.0

Message #80 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Graves <ngraves <at> ngraves.fr>
To: 74058 <at> debbugs.gnu.org
Cc: Nicolas Graves <ngraves <at> ngraves.fr>,
 Florian Pelz <pelzflorian <at> pelzflorian.de>
Subject: [PATCH v8 2/2] etc: news: Warn users about postgresql default update.
Date: Sat, 16 Nov 2024 14:48:03 +0100

* etc/news.scm: Add entry.

Co-authored-by: Florian Pelz <pelzflorian <at> pelzflorian.de>
---
 etc/news.scm | 63 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 63 insertions(+)

diff --git a/etc/news.scm b/etc/news.scm
index fbf16615df..116fa0c43b 100644
--- a/etc/news.scm
+++ b/etc/news.scm
@@ -25,6 +25,7 @@
 ;; Copyright © 2024 Vivien Kraus <vivien <at> planete-kraus.eu>
 ;; Copyright © 2024 Guillaume Le Vaillant <glv <at> posteo.net>
 ;; Copyright © 2024 Zheng Junjie <873216071 <at> qq.com>
+;; Copyright © 2024 Nicolas Graves <ngraves <at> ngraves.fr>
 ;;
 ;; Copying and distribution of this file, with or without modification, are
 ;; permitted in any medium without royalty provided the copyright notice and
@@ -33,6 +34,68 @@
 (channel-news
  (version 0)
 
+ (entry (commit "")  ;TODO fill it based on the previous commit
+        (title (en "PostgreSQL service upgrade")
+               (de "PostgreSQL-Dienst aktualisiert")
+               (fr "Mise à jour du service PostgreSQL"))
+        (body
+         (en "\
+This news entry concerns users of the @code{postgresql-service-type}.
+
+The default PostgreSQL used in @code{postgresql-configuration} has been
+deprecated, and will be removed in a few months.  However, the value of the
+default @code{postgresql-configuration-postgresql} can't be directly changed
+to a newer major version of PostgreSQL, because switching to a major version
+currently requires a manual update of the database.
+
+Because of this, the default value of postgresql-configuration-postgresql has
+been unset.  Current users of the service will have to set it manually.  If it
+was unset, use @code{postgresql-10} and plan an upgrade in the next month to a
+supported version of PostgreSQL.  To upgrade, you will need to either dump
+your database using your previous version of PostgreSQL and reload it in the
+new version or use the @command{pg_upgrade} application.  See
+@url{https://www.postgresql.org/docs/current/upgrading.html} for more info.")
+         (de "\
+Diese Neuigkeit betrifft Nutzer des @code{postgresql-service-type}.
+
+Die in @code{postgresql-configuration} bisher vorgegebene Version von
+PostgreSQL gilt als veraltet und wird in ein paar Monaten entfernt werden.
+Allerdings können wir den Vorgabewert von
+@code{postgresql-configuration-postgresql} nicht einfach für Sie auf eine
+neuere große Version ändern, weil bei so einem Wechsel die Datenbank derzeit
+manuell aktualisiert werden muss.
+
+Aus diesem Grund wurde der Vorgabewert von
+@code{postgresql-configuration-postgresql} von nun an entfernt.  Aktuelle
+Nutzer des Dienstes müssen sie selbst setzen.  Wenn kein Wert gesetzt war,
+sollten Sie @code{postgresql-10} eintragen und dabei einplanen, dass Sie es
+kommenden Monat auf eine unterstützte Version von PostgreSQL aktualisieren
+müssen.  Zum Aktualisieren werden Sie entweder mit der vorherigen Version von
+PostgreSQL ein Dump Ihrer Datenbank exportieren, das Sie dann in der neuen
+Version laden, oder Sie benutzen dafür die Anwendung @command{pg_upgrade}.
+Siehe @url{https://www.postgresql.org/docs/current/upgrading.html} für weitere
+Informationen.")
+         (fr "\
+Ce message concerne les utilisateurs du service-type PostgreSQL.
+
+La précédente version par défaut de PostgreSQL utilisée dans
+@code{postgresqsl-configuration} est obsolète et sera supprimée dans quelques
+mois.  Cependant, la valeur de la configuration par défaut
+@code{postgresql-configuration-postgresql} ne peut pas être changée
+directement pour une version majeure prise en charge de PostgreSQL, car le
+passage à une nouvelle version majeure nécessite actuellement une mise à jour
+manuelle de la base de données.
+
+Pour cette raison, la valeur par défaut de
+@code{postgresql-configuration-postgresql} a été supprimée et les utilisateurs
+actuels du service doivent la configurer manuellement. Si elle n’était pas
+définie, utilisez @code{postgresql-10} et prévoyez une mise à jour dans le
+mois à venir vers une version plus récente de PostgreSQL.  Pour effectuer la
+mise à jour, vous devrez soit exporter votre base de données sous une version
+de PostgreSQL puis la réimporter sous une nouvelle version, soit utiliser
+l'application @command{pg_upgrade}. Pour plus d'informations, consultez
+@url{https://www.postgresql.org/docs/current/upgrading.html}.")))
+
  (entry (commit "cfc85eb0c67a5cf10a3fbe2531b926cbb8c62489")
         (title
          (en "Linux-libre 6.10 removed due to end of upstream support")
-- 
2.46.0

Message #83 received at 74058 <at> debbugs.gnu.org (full text, mbox):

From: "pelzflorian (Florian Pelz)" <pelzflorian <at> pelzflorian.de>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74058 <at> debbugs.gnu.org
Subject: Re: [bug#74058] [PATCH v8 1/2] gnu: postgresql-configuration: Unset
 default postgresql.
Date: Sat, 16 Nov 2024 18:34:43 +0100

Nicolas Graves via Guix-patches via <guix-patches <at> gnu.org> writes:
> -(define-deprecated (postgresql-service #:key (postgresql postgresql)
> +(define-deprecated (postgresql-service #:key (postgresql postgresql-10)

Hello Nicolas.  Sorry, another issue.

As it should be, after createdb in `guix shell postgresql <at> 14`, reconfigure with

(service postgresql-service-type)

gives the error

guix system: Fehler: /etc/config.scm:72:21: no value specified for service of type 'postgresql'

; I change it to

(service postgresql-service-type
         (postgresql-configuration
          (postgresql postgresql-10)))

and reconfigure works again, but after rebooting,

sudo herd status

shows postgres has failed to start.

/var/log/postgresql/pg_ctl.log contains:

2024-11-15 20:37:27.366 GMT [32375] LOG:  ending log output to stderr
2024-11-15 20:37:27.366 GMT [32375] HINT:  Future log output will go to log destination "syslog".
2024-11-16 08:06:09.690 GMT [342] LOG:  ending log output to stderr
2024-11-16 08:06:09.690 GMT [342] HINT:  Future log output will go to log destination "syslog".
2024-11-16 11:48:56.075 GMT [344] LOG:  ending log output to stderr
2024-11-16 11:48:56.075 GMT [344] HINT:  Future log output will go to log destination "syslog".
2024-11-16 16:41:19.144 GMT [347] FATAL:  database files are incompatible with server
2024-11-16 16:41:19.144 GMT [347] DETAIL:  The data directory was initialized by PostgreSQL version 14, which is not compatible with this version 10.23.


Should we set the default to postgresql-14 and change /etc/news.scm to
recommend version 14, or instead recommend looking for an error in file
/var/log/postgresql/pg_ctl.log?

I used `guix shell postgresql <at> 14` because it’s the version I get with
`guix build -e '(@ (gnu packages databases) postgresql)'`.  Was that
wrong?

Regards,
Florian

Message #88 received at 74058-done <at> debbugs.gnu.org (full text, mbox):

From: "pelzflorian (Florian Pelz)" <pelzflorian <at> pelzflorian.de>
To: Nicolas Graves <ngraves <at> ngraves.fr>
Cc: 74058-done <at> debbugs.gnu.org
Subject: Re: [bug#74058] [PATCH v8 1/2] gnu: postgresql-configuration: Unset
 default postgresql.
Date: Sat, 16 Nov 2024 21:24:45 +0100

Pushed as b93434e656eba4260df82158a96c295000d3ff44.

Ohh Nicolas, I just saw the guix-devel thread which caused you to make
the patch.

https://lists.gnu.org/archive/html/guix-devel/2024-10/msg00199.html

Sorry for the misunderstanding!

Regards,
Florian

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.