pglift-cli 2.6.1__tar.gz → 3.1.0__tar.gz

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pglift_cli
-Version: 2.6.1
+Version: 3.1.0
 Summary: Command-line interface for pglift
 Project-URL: Documentation, https://pglift.readthedocs.io/
 Project-URL: Source, https://gitlab.com/dalibo/pglift/

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/src/pglift_cli/instance.py

@@ -197,7 +197,7 @@ async def apply(
             obj.lock,
             audit(dry_run=dry_run),
             diff.enabled(diff_format),
-            manager.from_manifest(actual or instance, settings=_site.SETTINGS),
+            manager.from_manifest(instance, settings=_site.SETTINGS, actual=actual),
         ):
             ret = await instances.apply(_site.SETTINGS, instance)
     if output_format == "json":

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/src/pglift_cli/model.py

@@ -22,6 +22,7 @@ from pglift.annotations import cli
 from pglift.exceptions import MutuallyExclusiveError
 from pglift.models.helpers import is_optional, optional_type
 from pglift.models.interface import PresenceState
+from pglift.settings import Settings
 from pglift.types import (
     Operation,
     field_annotation,
@@ -33,6 +34,7 @@ from . import _site, logger
 
 ModelType = type[pydantic.BaseModel]
 T = TypeVar("T", bound=pydantic.BaseModel)
+V = TypeVar("V")
 Callback = Callable[..., Any]
 ClickDecorator = Callable[[Callback], Callback]
 DEFAULT = object()
@@ -296,6 +298,15 @@ def _paramspecs_from_model(
             return DEFAULT
         return value
 
+    def default_factory(
+        _ctx: click.Context,
+        _param: click.Argument,
+        value: V,
+        *,
+        callback: Callable[[Settings, V], V],
+    ) -> V:
+        return callback(_site.SETTINGS, value)
+
     def add_state_field_callback(
         ctx: click.Context,
         _param: click.Argument,
@@ -377,6 +388,17 @@ def _paramspecs_from_model(
                     ),
                 )
 
+            elif config is not None and isinstance(config, cli.OptionalArgument):
+                attrs["required"] = False
+                attrs["callback"] = functools.partial(
+                    default_factory,
+                    callback=config.default_factory,
+                )
+
+                yield (
+                    (modelname, argname),
+                    ArgumentSpec((argname,), field, attrs, loc=(modelname,)),
+                )
             else:
                 yield (
                     (modelname, argname),
@@ -386,11 +408,7 @@ def _paramspecs_from_model(
                 )
 
         else:
-            if (
-                config
-                and isinstance(config, cli.Argument | cli.Option)
-                and config.metavar is not None
-            ):
+            if config and isinstance(config, cli.Option) and config.metavar is not None:
                 metavar = config.metavar
             else:
                 metavar = argname

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/src/pglift_cli/prometheus.py

@@ -71,7 +71,7 @@ async def apply(
     dry_run: bool,
     diff_format: diff.Format | None,
 ) -> None:
-    """Apply manifest as a Prometheus postgres_exporter."""
+    """Apply manifest as a Prometheus postgres_exporter (DEPRECATED)."""
     settings = prometheus.get_settings(_site.SETTINGS)
     name = data["name"]
     op: types.Operation = "update" if impl.exists(name, settings) else "create"
@@ -93,7 +93,7 @@ async def apply(
 @click.pass_obj
 @async_command
 async def install(obj: Obj, postgresexporter: PostgresExporter) -> None:
-    """Install the service for a (non-local) instance."""
+    """Install the service for a (non-local) instance (DEPRECATED)."""
     settings = prometheus.get_settings(_site.SETTINGS)
     with obj.lock, audit():
         async with task.async_transaction():
@@ -105,7 +105,7 @@ async def install(obj: Obj, postgresexporter: PostgresExporter) -> None:
 @click.pass_obj
 @async_command
 async def uninstall(obj: Obj, name: str) -> None:
-    """Uninstall the service."""
+    """Uninstall the service (DEPRECATED)."""
     with obj.lock, audit():
         await impl.drop(_site.SETTINGS, name)
 
@@ -121,6 +121,8 @@ async def start(obj: Obj, name: str, foreground: bool) -> None:
     The NAME argument is a local identifier for the postgres_exporter
     service. If the service is bound to a local instance, it should be
     <version>-<name>.
+
+    Deprecation warning: in a future version, only local instances will be handled.
     """
     settings = prometheus.get_settings(_site.SETTINGS)
     with obj.lock, audit():
@@ -140,6 +142,8 @@ async def stop(obj: Obj, name: str) -> None:
     The NAME argument is a local identifier for the postgres_exporter
     service. If the service is bound to a local instance, it should be
     <version>-<name>.
+
+    Deprecation warning: in a future version, only local instances will be handled.
     """
     settings = prometheus.get_settings(_site.SETTINGS)
     with obj.lock, audit():

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/src/pglift_cli/role.py

@@ -173,7 +173,7 @@ async def ls(instance: PostgreSQLInstance, output_format: OutputFormat | None) -
             partial(
                 model_dump,
                 context={"pretty": True},
-                exclude={"hba_records", "validity"},
+                exclude={"hba_records"},
             ),
         )
 
@@ -198,7 +198,7 @@ async def get(
             partial(
                 model_dump,
                 context={"pretty": True},
-                exclude={"hba_records", "validity"},
+                exclude={"hba_records"},
             ),
             box=None,
         )

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/expect/test-cli-walkthrough.t

@@ -26,7 +26,7 @@ Site settings:
   > postgresql:
   >   auth:
   >     local: md5
-  >     passfile: '$TMPDIR/.pgpass'
+  >     passfile: '$TMPDIR/pgpass/.pgpass'
   >     password_command:
   >       - jq
   >       - -r
@@ -65,7 +65,7 @@ Make sure password_command works:
     "local": "md5",
     "host": "trust",
     "hostssl": "trust",
-    "passfile": "$TMPDIR/.pgpass",
+    "passfile": "$TMPDIR/pgpass/.pgpass",
     "password_command": [
       "jq",
       "-r",
@@ -89,10 +89,11 @@ Make sure password_command works:
   $ pglift site-configure install
   INFO creating base pgBackRest configuration directory: $TMPDIR/etc/pgbackrest
   INFO installing base pgBackRest configuration
-  INFO creating pgBackRest include directory
+  INFO creating pgBackRest include directory: $TMPDIR/etc/pgbackrest/conf.d
   INFO creating pgBackRest repository backups and archive directory: $TMPDIR/pgbackrest
   INFO creating pgBackRest log directory: $TMPDIR/log/pgbackrest
   INFO creating pgBackRest spool directory: $TMPDIR/srv/pgbackrest/spool
+  INFO creating pgpass file directory: $TMPDIR/pgpass
   INFO creating PostgreSQL log directory: $TMPDIR/log/postgresql
 
   $ trap "pglift --non-interactive instance drop main; \
@@ -116,10 +117,11 @@ Audit log:
   DEBUG - pglift.util - using 'pgbackrest/pgbackrest.conf' configuration file from distribution
   INFO - pglift.util - creating base pgBackRest configuration directory: $TMPDIR/etc/pgbackrest
   INFO - pglift.pgbackrest - installing base pgBackRest configuration
-  INFO - pglift.pgbackrest - creating pgBackRest include directory
+  INFO - pglift.util - creating pgBackRest include directory: $TMPDIR/etc/pgbackrest/conf.d
   INFO - pglift.util - creating pgBackRest repository backups and archive directory: $TMPDIR/pgbackrest
   INFO - pglift.util - creating pgBackRest log directory: $TMPDIR/log/pgbackrest
   INFO - pglift.util - creating pgBackRest spool directory: $TMPDIR/srv/pgbackrest/spool
+  INFO - pglift.util - creating pgpass file directory: $TMPDIR/pgpass
   INFO - pglift.util - creating PostgreSQL log directory: $TMPDIR/log/postgresql
   INFO - pglift_cli.audit - command completed \(\d+.\d+ seconds\) (re)
 
@@ -166,13 +168,13 @@ Error cases for instance operations
   Error: instance already exists
   [1]
   $ pglift instance create 'in/va/lid' --pgbackrest-stanza=xxx --surole-password=s3per
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   Try 'pglift instance create --help' for help.
   
-  Error: Invalid value for 'NAME': String should match pattern '^[^/-]+$'
+  Error: Invalid value for '[NAME]': String should match pattern '^[^/-]+$'
   [2]
   $ pglift instance create stdby --standby-for='port 1234' --pgbackrest-stanza=stddy --surole-password=s3per
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   Try 'pglift instance create --help' for help.
   
   Error: Invalid value for '--standby-for': missing "=" after "port" in connection info string
@@ -218,7 +220,7 @@ List instances
   $ pglift instance list -o json | jq '.[] | .name, .status'
   "main"
   "running"
-  $ pglift instance list --version=14
+  $ pglift instance list --version=15
 
 Get the status of an instance:
 
@@ -235,6 +237,15 @@ Reload, restart:
 
   $ pglift instance reload
   INFO reloading PostgreSQL configuration for 1\d\/main (re)
+  $ pglift instance stop
+  INFO stopping PostgreSQL 1\d\/main (re)
+  INFO stopping Prometheus postgres_exporter 1\d-main (re)
+  $ pglift instance restart
+  INFO restarting instance 1\d\/main (re)
+  INFO restarting PostgreSQL
+  INFO stopping PostgreSQL 1\d\/main (re)
+  INFO starting PostgreSQL 1\d\/main (re)
+  INFO starting Prometheus postgres_exporter 1\d-main (re)
   $ pglift instance restart
   INFO restarting instance 1\d\/main (re)
   INFO stopping Prometheus postgres_exporter 1\d-main (re)
@@ -251,19 +262,20 @@ Stop, alter, (re)start an instance:
   $ pglift instance alter --no-data-checksums
   INFO configuring PostgreSQL
   INFO disabling data checksums
-  $ pglift instance alter --prometheus-port=$PGEPORT1 --state=started --powa-password=p0W@ --diff
+  $ pglift instance alter --prometheus-port=$PGEPORT1 -o json --diff
   INFO configuring PostgreSQL
-  INFO starting PostgreSQL 1\d\/main (re)
   INFO reconfiguring Prometheus postgres_exporter 1\d-main (re)
-  INFO restarting Prometheus postgres_exporter 1\d-main (re)
+  {
+    "change_state": "changed",
+    "diff": [
+      "--- $TMPDIR/etc/prometheus/postgres_exporter-1*-main.conf\n+++ $TMPDIR/etc/prometheus/postgres_exporter-1*-main.conf\n@@ -1,2 +1,2 @@\n DATA_SOURCE_NAME=postgresql://prometheus@:*/postgres?*\n-POSTGRES_EXPORTER_OPTS='--web.listen-address :* --log.level info'\n+POSTGRES_EXPORTER_OPTS='--web.listen-address :* --log.level info'" (glob)
+    ],
+    "pending_restart": false
+  }
+  $ pglift instance alter --state=started --powa-password=p0W@ --diff
+  INFO configuring PostgreSQL
+  INFO starting PostgreSQL 1\d\/main (re)
   INFO starting Prometheus postgres_exporter 1\d-main (re)
-  --- $TMPDIR/etc/prometheus/postgres_exporter-1*-main.conf (glob)
-  +++ $TMPDIR/etc/prometheus/postgres_exporter-1*-main.conf (glob)
-  @@ -1,2 +1,2 @@
-   DATA_SOURCE_NAME=postgresql://prometheus@:*/postgres?host=* (glob)
-  *%2Fpostgresql&sslmode=disable (glob)
-  -POSTGRES_EXPORTER_OPTS='--web.listen-address :\d+ --log.level info' (re)
-  \+POSTGRES_EXPORTER_OPTS='--web.listen-address :\d+ --log.level info' (re)
   --- /dev/null
   +++ $TMPDIR/run/prometheus/1*-main.pid (glob)
   @@ -0,0 +1 @@
@@ -330,7 +342,7 @@ Stop, alter, (re)start an instance:
 Instance environment, and program execution:
 
   $ pglift instance env | grep PASSFILE
-  PGPASSFILE=$TMPDIR/.pgpass
+  PGPASSFILE=$TMPDIR/pgpass/.pgpass
   $ pglift instance env -o json | jq '.PGBACKREST_STANZA, .PGHOST, .PGPASSWORD'
   "main"
   "$TMPDIR/run/postgresql"
@@ -378,6 +390,10 @@ PostgreSQL configuration
   $ pglift --non-interactive pgconf remove fsync
   Error: 'fsync' not found in managed configuration
   [1]
+  $ pglift --non-interactive pgconf set Port=9988
+  INFO configuring PostgreSQL
+  Error: invalid postgresql configuration, Bad line: 'Port = 9988\n'.
+  [1]
   $ pglift --non-interactive pgconf remove --dry-run log_statement
   WARNING failed to get data_checksums status: 'Data page checksum version' not found in controldata
   INFO configuring PostgreSQL
@@ -398,6 +414,7 @@ PostgreSQL configuration
   INFO reconfiguring Prometheus postgres_exporter 1\d-main (re)
   INFO restarting Prometheus postgres_exporter 1\d-main (re)
   INFO configuring pgBackRest stanza 'main' for pg1-path=\$TMPDIR\/srv\/pgsql\/1\d\/main\/data (re)
+  WARNING Due to 'port' change, the instance may be in an unstable state until it is restarted.
   port: \d+ -> 1234 (re)
   DRY RUN: no changes made
 
@@ -502,14 +519,14 @@ Add and manipulate roles:
 
   $ pglift role -i main create dba --login --pgpass --password=qwerty --in-role=pg_read_all_stats --dry-run
   INFO creating role 'dba'
-  INFO adding an entry for 'dba' in \$TMPDIR\/.pgpass \(port=\d+\) (re)
+  INFO adding an entry for 'dba' in \$TMPDIR\/pgpass\/.pgpass \(port=\d+\) (re)
   DRY RUN: no changes made
 
   $ pglift role -i main create dba --login --pgpass --password=qwerty --in-role=pg_read_all_stats
   INFO creating role 'dba'
-  INFO adding an entry for 'dba' in \$TMPDIR\/.pgpass \(port=\d+\) (re)
+  INFO adding an entry for 'dba' in \$TMPDIR\/pgpass\/.pgpass \(port=\d+\) (re)
 
-  $ cat $TMPDIR/.pgpass
+  $ cat $TMPDIR/pgpass/.pgpass
   \*:\d+:\*:dba:qwerty (re)
 
   $ pglift role get dba
@@ -539,8 +556,8 @@ Add and manipulate roles:
   
   $ pglift role alter dba --connection-limit=10 --inherit --no-pgpass --no-login --revoke=pg_read_all_stats --grant=pg_monitor --valid-until=2026-01-01 --dry-run
   INFO altering role 'dba'
-  INFO removing entry for 'dba' in \$TMPDIR\/.pgpass \(port=\d+\) (re)
-  INFO removing now empty $TMPDIR/.pgpass
+  INFO removing entry for 'dba' in \$TMPDIR\/pgpass\/.pgpass \(port=\d+\) (re)
+  INFO removing now empty $TMPDIR/pgpass/.pgpass
   DRY RUN: no changes made
 
   $ pglift role -i main create dba --password=qwerty --encrypted-password=md5azerty
@@ -552,8 +569,8 @@ Add and manipulate roles:
 
   $ pglift role alter dba --connection-limit=10 --inherit --no-pgpass --no-login --revoke=pg_read_all_stats --grant=pg_monitor --valid-until=2026-01-01
   INFO altering role 'dba'
-  INFO removing entry for 'dba' in \$TMPDIR\/.pgpass \(port=\d+\) (re)
-  INFO removing now empty $TMPDIR/.pgpass
+  INFO removing entry for 'dba' in \$TMPDIR\/pgpass\/.pgpass \(port=\d+\) (re)
+  INFO removing now empty $TMPDIR/pgpass/.pgpass
 
   $ pglift role get dba -o json
   {
@@ -567,7 +584,6 @@ Add and manipulate roles:
     "replication": false,
     "connection_limit": 10,
     "valid_until": "2026-01-01T00:00:00Z",
-    "validity": "2026-01-01T00:00:00Z",
     "memberships": [
       {
         "role": "pg_monitor"
@@ -647,7 +663,6 @@ Add and manipulate roles:
     "replication": false,
     "connection_limit": 3,
     "valid_until": null,
-    "validity": null,
     "memberships": [
       {
         "role": "pg_monitor"
@@ -703,7 +718,6 @@ Add and manipulate roles:
       "replication": false,
       "connection_limit": null,
       "valid_until": null,
-      "validity": null,
       "memberships": [],
       "hba_records": []
     },
@@ -718,7 +732,6 @@ Add and manipulate roles:
       "replication": false,
       "connection_limit": 10,
       "valid_until": "2026-01-01T00:00:00Z",
-      "validity": "2026-01-01T00:00:00Z",
       "memberships": [
         {
           "role": "pg_monitor"
@@ -737,7 +750,6 @@ Add and manipulate roles:
       "replication": false,
       "connection_limit": null,
       "valid_until": null,
-      "validity": null,
       "memberships": [],
       "hba_records": []
     },
@@ -752,7 +764,6 @@ Add and manipulate roles:
       "replication": true,
       "connection_limit": null,
       "valid_until": null,
-      "validity": null,
       "memberships": [],
       "hba_records": []
     },
@@ -767,7 +778,6 @@ Add and manipulate roles:
       "replication": false,
       "connection_limit": null,
       "valid_until": null,
-      "validity": null,
       "memberships": [
         {
           "role": "pg_monitor"
@@ -786,7 +796,6 @@ Add and manipulate roles:
       "replication": false,
       "connection_limit": null,
       "valid_until": null,
-      "validity": null,
       "memberships": [],
       "hba_records": []
     },
@@ -801,7 +810,6 @@ Add and manipulate roles:
       "replication": false,
       "connection_limit": 3,
       "valid_until": null,
-      "validity": null,
       "memberships": [
         {
           "role": "pg_monitor"
@@ -1282,11 +1290,13 @@ Cleanup.
   INFO deconfiguring Prometheus postgres_exporter 1\d-main (re)
   INFO deleting pgBackRest stanza 'main'
   INFO deconfiguring pgBackRest stanza 'main'
-  INFO removing entries matching port=\d+ from \$TMPDIR\/.pgpass (re)
-  INFO deleting PostgreSQL data and WAL directories
-  INFO deleting pgBackRest include directory
+  INFO removing entries matching port=\d+ from \$TMPDIR\/pgpass\/.pgpass (re)
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/srv\/pgsql\/1\d\/main/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/srv\/pgsql\/1\d\/main/wal (re)
+  INFO deleting pgBackRest include directory: $TMPDIR/etc/pgbackrest/conf.d
   INFO uninstalling base pgBackRest configuration
-  INFO deleting pgBackRest log directory
-  INFO deleting pgBackRest spool directory
-  INFO deleting PostgreSQL log directory
-  INFO deleting PostgreSQL socket directory (no-eol)
+  INFO deleting pgBackRest log directory: $TMPDIR/log/pgbackrest
+  INFO deleting pgBackRest spool directory: $TMPDIR/srv/pgbackrest/spool
+  INFO deleting pgpass file parent directory
+  INFO deleting PostgreSQL log directory: $TMPDIR/log/postgresql
+  INFO deleting PostgreSQL socket directory: $TMPDIR/run/postgresql (no-eol)

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/expect/test-demote.t

@@ -143,7 +143,6 @@ Check this role from the standby
     "replication": true,
     "connection_limit": null,
     "valid_until": null,
-    "validity": null,
     "memberships": [],
     "hba_records": []
   }
@@ -221,11 +220,13 @@ And finally check replication is functional
 
   INFO dropping instance 1\d\/pg1 (re)
   INFO stopping PostgreSQL 1\d\/pg1 (re)
-  INFO deleting PostgreSQL data and WAL directories
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/1\/srv\/pgsql\/1\d\/pg1\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/1\/srv\/pgsql\/1\d\/pg1\/wal (re)
   INFO dropping instance 1\d\/pg2 (re)
   INFO stopping PostgreSQL 1\d\/pg2 (re)
-  INFO deleting PostgreSQL data and WAL directories
-  INFO deleting PostgreSQL log directory
-  INFO deleting PostgreSQL socket directory
-  INFO deleting PostgreSQL log directory
-  INFO deleting PostgreSQL socket directory (no-eol)
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/2\/srv\/pgsql\/1\d\/pg2\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/2\/srv\/pgsql\/1\d\/pg2\/wal (re)
+  INFO deleting PostgreSQL log directory: $TMPDIR/1/log/postgresql
+  INFO deleting PostgreSQL socket directory: $TMPDIR/1/run/postgresql
+  INFO deleting PostgreSQL log directory: $TMPDIR/2/log/postgresql
+  INFO deleting PostgreSQL socket directory: $TMPDIR/2/run/postgresql (no-eol)

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/expect/test-help.t

@@ -12,9 +12,9 @@ Site settings
   $ export PGLIFT_RUN_PREFIX=$TMPDIR/run
 
   $ bindir=$TMPDIR/$TESTFILE/bin
-  $ mkdir -p $bindir/pgsql/17/bin
-  $ touch $bindir/pgsql/17/bin/pg_ctl
-  $ chmod +x $bindir/pgsql/17/bin/pg_ctl
+  $ mkdir -p $bindir/pgsql/18/bin
+  $ touch $bindir/pgsql/18/bin/pg_ctl
+  $ chmod +x $bindir/pgsql/18/bin/pg_ctl
   $ export PGLIFT_POSTGRESQL__BINDIR=$bindir/pgsql/{version}/bin \
   >     PGLIFT_POSTGRESQL__AUTH__PASSFILE="null" \
   >     PGLIFT_POSTGRESQL__REPLROLE="replication"
@@ -32,13 +32,23 @@ Site settings
   $ export PGLIFT_POWA='{}'
 
   $ pglift site-settings --no-defaults -o json \
-  >   | jq '.postgresql, .pgbackrest, .prometheus, .prefix, .run_prefix'
+  >   | jq '.patroni, .postgresql, .pgbackrest, .prometheus, .prefix, .run_prefix'
   {
-    "bindir": "$TMPDIR/*/bin/pgsql/{version}/bin", (glob)
+    "execpath": "$TMPDIR/test-help.t/bin/patroni",
+    "ctlpath": "$TMPDIR/test-help.t/bin/patronictl",
+    "configpath": "$TMPDIR/etc/patroni/{name}.yaml",
+    "logpath": "$TMPDIR/log/patroni",
+    "pid_file": "$TMPDIR/run/patroni/{name}.pid",
+    "postgresql": {
+      "passfile": "$TMPDIR/etc/patroni/{name}.pgpass"
+    }
+  }
+  {
+    "bindir": "$TMPDIR/test-help.t/bin/pgsql/{version}/bin",
     "versions": [
       {
-        "version": "17",
-        "bindir": "$TMPDIR/*/bin/pgsql/17/bin" (glob)
+        "version": "18",
+        "bindir": "$TMPDIR/*/bin/pgsql/18/bin" (glob)
       }
     ],
     "auth": {
@@ -218,12 +228,12 @@ Instance commands
     -o, --output-format [json]      Specify the output format.
     --help                          Show this message and exit.
   $ pglift instance create --help
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   
     Initialize a PostgreSQL instance
   
   Options:
-    --version [17|16|15|14|13]      PostgreSQL version; if unspecified,
+    --version [18|17|16|15|14]      PostgreSQL version; if unspecified,
                                     determined from site settings or most recent
                                     PostgreSQL installation available on site.
     --standby-for DSN               DSN of primary for streaming replication.
@@ -363,7 +373,7 @@ Instance commands
     List the available instances
   
   Options:
-    --version [17|16|15|14|13]  Only list instances of specified version.
+    --version [18|17|16|15|14]  Only list instances of specified version.
     -o, --output-format [json]  Specify the output format.
     --help                      Show this message and exit.
   $ pglift instance logs --help
@@ -525,7 +535,7 @@ Instance commands
     Required if there is more than one instance on system.
   
   Options:
-    --version [17|16|15|14|13]  PostgreSQL version of the new instance (default
+    --version [18|17|16|15|14]  PostgreSQL version of the new instance (default
                                 to site-configured value).
     --name TEXT                 Name of the new instance (default to old
                                 instance name).
@@ -617,7 +627,6 @@ Role commands
                                     make.
     --valid-until VALID_UNTIL       Date and time after which the role's
                                     password is no longer valid.
-    --validity VALIDITY             DEPRECATED. Use 'valid_until' instead.
     --grant ROLE                    Grant membership of the given role. (Can be
                                     used multiple times.)
     --revoke ROLE                   Revoke membership of the given role. (Can be
@@ -646,7 +655,6 @@ Role commands
                                     make.
     --valid-until VALID_UNTIL       Date and time after which the role's
                                     password is no longer valid.
-    --validity VALIDITY             DEPRECATED. Use 'valid_until' instead.
     --in-role ROLE                  Roles which this role should be a member of.
                                     (Can be used multiple times.)
     --dry-run                       Simulate change operations.
@@ -904,7 +912,7 @@ PostgreSQL configuration commands
   Options:
     --help  Show this message and exit.
 
-pg_hba.conf management commands
+HBA configuration management commands
 
   $ pglift pghba --help
   Usage: pglift pghba [OPTIONS] COMMAND [ARGS]...
@@ -1012,15 +1020,15 @@ Patroni commands:
     --help    Show this message and exit.
   
   Commands:
-    apply      Apply manifest as a Prometheus postgres_exporter.
-    install    Install the service for a (non-local) instance.
+    apply      Apply manifest as a Prometheus postgres_exporter (DEPRECATED).
+    install    Install the service for a (non-local) instance (DEPRECATED).
     start      Start postgres_exporter service NAME.
     stop       Stop postgres_exporter service NAME.
-    uninstall  Uninstall the service.
+    uninstall  Uninstall the service (DEPRECATED).
   $ pglift postgres_exporter apply --help
   Usage: pglift postgres_exporter apply [OPTIONS]
   
-    Apply manifest as a Prometheus postgres_exporter.
+    Apply manifest as a Prometheus postgres_exporter (DEPRECATED).
   
   Options:
     -f, --file MANIFEST         [required]
@@ -1032,7 +1040,7 @@ Patroni commands:
   $ pglift postgres_exporter install --help
   Usage: pglift postgres_exporter install [OPTIONS] NAME DSN PORT
   
-    Install the service for a (non-local) instance.
+    Install the service for a (non-local) instance (DEPRECATED).
   
   Options:
     --password PASSWORD        Connection password.
@@ -1046,6 +1054,9 @@ Patroni commands:
     The NAME argument is a local identifier for the postgres_exporter service.
     If the service is bound to a local instance, it should be <version>-<name>.
   
+    Deprecation warning: in a future version, only local instances will be
+    handled.
+  
   Options:
     --foreground  Start the program in foreground.
     --help        Show this message and exit.
@@ -1057,12 +1068,15 @@ Patroni commands:
     The NAME argument is a local identifier for the postgres_exporter service.
     If the service is bound to a local instance, it should be <version>-<name>.
   
+    Deprecation warning: in a future version, only local instances will be
+    handled.
+  
   Options:
     --help  Show this message and exit.
   $ pglift postgres_exporter uninstall --help
   Usage: pglift postgres_exporter uninstall [OPTIONS] NAME
   
-    Uninstall the service.
+    Uninstall the service (DEPRECATED).
   
   Options:
     --help  Show this message and exit.

pglift_cli-3.1.0/tests/expect/test-instance-name.t

@@ -0,0 +1,45 @@
+# SPDX-FileCopyrightText: 2026 Dalibo
+#
+# SPDX-License-Identifier: GPL-3.0-or-later
+
+Site settings and configuration
+
+  $ export PGLIFT_CLI__LOG_FORMAT="%(levelname)-4s %(message)s"
+  $ export PGLIFT_CLI__LOG_LEVEL=info
+  $ export PGLIFT_POSTGRESQL__AUTH__PASSFILE=$TMPDIR/pgpass/.pgpass
+  $ export PGLIFT_INSTANCE_DEFAULT_NAME=myinstance
+  $ export PGLIFT_PREFIX=$TMPDIR
+  $ export PGLIFT_RUN_PREFIX=$TMPDIR/run
+  $ alias pglift="pglift --log-level=INFO --non-interactive"
+
+  $ pglift site-configure install
+  INFO creating pgpass file directory: $TMPDIR/pgpass
+  INFO creating PostgreSQL log directory: $TMPDIR/log/postgresql
+
+(Cleanup steps)
+  $ trap "pglift --non-interactive instance drop; \
+  >   pglift --non-interactive site-configure uninstall; \
+  >   port-for -u postgres" \
+  >   EXIT
+
+Define ports:
+  $ PGPORT=$(port-for pg1)
+
+Create an instance without a name
+  $ pglift instance create --port=$PGPORT
+  INFO initializing PostgreSQL
+  INFO configuring PostgreSQL authentication
+  INFO configuring PostgreSQL
+  INFO creating PostgreSQL socket directory directory: $TMPDIR/run/postgresql
+  INFO starting PostgreSQL 1\d\/myinstance (re)
+  INFO creating instance dumps directory: \$TMPDIR\/srv\/dumps\/1\d-myinstance (re)
+
+Cleanup.
+  INFO dropping instance 1\d\/myinstance (re)
+  INFO stopping PostgreSQL 1\d\/myinstance (re)
+  INFO removing entries matching port=\d+ from \$TMPDIR\/pgpass/.pgpass (re)
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/srv\/pgsql\/1\d\/myinstance/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/srv\/pgsql\/1\d\/myinstance/wal (re)
+  INFO deleting pgpass file parent directory
+  INFO deleting PostgreSQL log directory: $TMPDIR/log/postgresql
+  INFO deleting PostgreSQL socket directory: $TMPDIR/run/postgresql (no-eol)

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/expect/test-port-validation.t

@@ -27,7 +27,7 @@
   $ pglift site-configure install
   INFO creating base pgBackRest configuration directory: $TMPDIR/etc/pgbackrest
   INFO installing base pgBackRest configuration
-  INFO creating pgBackRest include directory
+  INFO creating pgBackRest include directory: $TMPDIR/etc/pgbackrest/conf.d
   INFO creating pgBackRest repository backups and archive directory: $TMPDIR/pgbackrest-not-used
   INFO creating pgBackRest log directory: $TMPDIR/log/pgbackrest
   INFO creating pgBackRest spool directory: $TMPDIR/srv/pgbackrest/spool
@@ -57,19 +57,19 @@ With custom ports
   INFO creating instance dumps directory: \$TMPDIR\/srv\/dumps\/1\d-main (re)
   INFO starting Prometheus postgres_exporter 1\d-main (re)
   $ pglift instance create other --port=$PG2PORT --prometheus-port=$PGE1PORT
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   Try 'pglift instance create --help' for help.
   
   Error: Invalid value for '--prometheus-port': port \d+ already in use (re)
   [2]
   $ pglift instance create other --port=$PG1PORT --prometheus-port=$PGE2PORT
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   Try 'pglift instance create --help' for help.
   
   Error: Invalid value for '--port': port \d+ already in use (re)
   [2]
   $ pglift instance create other --port=$PG1PORT --prometheus-port=$PGE1PORT
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   Try 'pglift instance create --help' for help.
   
   Error: Invalid value for '--prometheus-port': port \d+ already in use (re)
@@ -79,7 +79,8 @@ With custom ports
   INFO stopping PostgreSQL 1\d\/main (re)
   INFO stopping Prometheus postgres_exporter 1\d-main (re)
   INFO deconfiguring Prometheus postgres_exporter 1\d-main (re)
-  INFO deleting PostgreSQL data and WAL directories
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/srv\/pgsql\/1\d\/main\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/srv\/pgsql\/1\d\/main\/wal (re)
 
 With a port set in postgresql.conf template
 
@@ -99,13 +100,13 @@ With a port set in postgresql.conf template
   INFO creating instance dumps directory: \$TMPDIR\/srv\/dumps\/1\d-main (re)
   INFO starting Prometheus postgres_exporter 1\d-main (re)
   $ pglift instance create other --prometheus-port=$PGE2PORT
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   Try 'pglift instance create --help' for help.
   
   Error: Invalid value for '--port': port \d+ already in use (re)
   [2]
   $ pglift instance create other --port=$PG2PORT --prometheus-port=$PGE1PORT
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   Try 'pglift instance create --help' for help.
   
   Error: Invalid value for '--prometheus-port': port \d+ already in use (re)
@@ -116,7 +117,8 @@ With a port set in postgresql.conf template
   INFO stopping PostgreSQL 1\d\/main (re)
   INFO stopping Prometheus postgres_exporter 1\d-main (re)
   INFO deconfiguring Prometheus postgres_exporter 1\d-main (re)
-  INFO deleting PostgreSQL data and WAL directories
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/srv\/pgsql\/1\d\/main\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/srv\/pgsql\/1\d\/main\/wal (re)
   $ rm -rf $PGLIFT_CONFIG_DIR/postgresql
 
 With default ports
@@ -131,7 +133,7 @@ With default ports
   INFO creating instance dumps directory: \$TMPDIR\/srv\/dumps\/1\d-main (re)
   INFO starting Prometheus postgres_exporter 1\d-main (re)
   $ pglift instance create other
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   Try 'pglift instance create --help' for help.
   
   Error: Invalid value for '--port': port 5432 already in use
@@ -142,12 +144,13 @@ With default ports
   INFO stopping PostgreSQL 1\d\/main (re)
   INFO stopping Prometheus postgres_exporter 1\d-main (re)
   INFO deconfiguring Prometheus postgres_exporter 1\d-main (re)
-  INFO deleting PostgreSQL data and WAL directories
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/srv\/pgsql\/1\d\/main\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/srv\/pgsql\/1\d\/main\/wal (re)
 
 (cleanup)
-  INFO deleting pgBackRest include directory
+  INFO deleting pgBackRest include directory: $TMPDIR/etc/pgbackrest/conf.d
   INFO uninstalling base pgBackRest configuration
-  INFO deleting pgBackRest log directory
-  INFO deleting pgBackRest spool directory
-  INFO deleting PostgreSQL log directory
-  INFO deleting PostgreSQL socket directory (no-eol)
+  INFO deleting pgBackRest log directory: $TMPDIR/log/pgbackrest
+  INFO deleting pgBackRest spool directory: $TMPDIR/srv/pgbackrest/spool
+  INFO deleting PostgreSQL log directory: $TMPDIR/log/postgresql
+  INFO deleting PostgreSQL socket directory: $TMPDIR/run/postgresql (no-eol)

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/expect/test-prometheus.t

@@ -44,8 +44,8 @@
       {
         "before_header": "$TMPDIR/etc/prometheus/postgres_exporter-test.conf",
         "after_header": "$TMPDIR/etc/prometheus/postgres_exporter-test.conf",
-        "before": "DATA_SOURCE_NAME=postgresql://:5432/monitoring\nPOSTGRES_EXPORTER_OPTS='--web.listen-address :* --log.level info'", (glob)
-        "after": "DATA_SOURCE_NAME=postgresql://prometheus@:5432/monitoring\nPOSTGRES_EXPORTER_OPTS='--web.listen-address :* --log.level info'" (glob)
+        "before": "DATA_SOURCE_NAME=postgresql://:5432/monitoring\nPOSTGRES_EXPORTER_OPTS='--web.listen-address :* --log.level info'\n", (glob)
+        "after": "DATA_SOURCE_NAME=postgresql://prometheus@:5432/monitoring\nPOSTGRES_EXPORTER_OPTS='--web.listen-address :* --log.level info'\n" (glob)
       },
       {
         "before_header": "$TMPDIR/run/prometheus/test.pid deleted",
@@ -100,4 +100,4 @@ Check port conflicts
   INFO deconfiguring Prometheus postgres_exporter test
 
 (cleanup)
-  INFO deleting PostgreSQL log directory (no-eol)
+  INFO deleting PostgreSQL log directory: $TMPDIR/log/postgresql (no-eol)

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/expect/test-standby-pgbackrest.t

@@ -60,7 +60,7 @@ Site settings and configuration
   $ pglift1 site-configure install
   INFO creating base pgBackRest configuration directory: $TMPDIR/1/etc/pgbackrest
   INFO installing base pgBackRest configuration
-  INFO creating pgBackRest include directory
+  INFO creating pgBackRest include directory: $TMPDIR/1/etc/pgbackrest/conf.d
   INFO creating pgBackRest repository backups and archive directory: $TMPDIR/test-standby-pgbackrest/backups
   INFO creating pgBackRest log directory: $TMPDIR/1/log/pgbackrest
   INFO creating pgBackRest spool directory: $TMPDIR/1/srv/pgbackrest/spool
@@ -68,7 +68,7 @@ Site settings and configuration
   $ pglift2 site-configure install
   INFO creating base pgBackRest configuration directory: $TMPDIR/2/etc/pgbackrest
   INFO installing base pgBackRest configuration
-  INFO creating pgBackRest include directory
+  INFO creating pgBackRest include directory: $TMPDIR/2/etc/pgbackrest/conf.d
   INFO creating pgBackRest log directory: $TMPDIR/2/log/pgbackrest
   INFO creating pgBackRest spool directory: $TMPDIR/2/srv/pgbackrest/spool
   INFO creating PostgreSQL log directory: $TMPDIR/2/log/postgresql
@@ -143,7 +143,7 @@ Cannot create a standby with --slot option
   >   --pgbackrest-stanza=stnz \
   >   --standby-for="host=hst" \
   >   --slot=slt
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   Try 'pglift instance create --help' for help.
   
   Error: Invalid value for '--slot': replication slots cannot be set on a standby instance
@@ -184,7 +184,7 @@ Try to create primary instance with same stanza
   $ pglift1 instance create err \
   >   --port=$PG3PORT --pgbackrest-stanza=app \
   >   --surole-password=s3per --pgbackrest-password=b@ckUp
-  Usage: pglift instance create [OPTIONS] NAME
+  Usage: pglift instance create [OPTIONS] [NAME]
   Try 'pglift instance create --help' for help.
   
   Error: Invalid value for '--pgbackrest-stanza': Stanza 'app' already bound to another instance \(datadir=\$TMPDIR\/1\/srv\/pgsql\/1\d\/pg1\/data\) (re)
@@ -301,7 +301,8 @@ new slot on the new primary
   INFO dropping instance 1\d\/pg1 (re)
   WARNING instance 1\d\/pg1 is already stopped (re)
   INFO deconfiguring pgBackRest stanza 'app'
-  INFO deleting PostgreSQL data and WAL directories
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/1\/srv\/pgsql\/1\d\/pg1\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/1\/srv\/pgsql\/1\d\/pg1\/wal (re)
   $ pglift1 instance create pg3 \
   >   --standby-for="host=$RUN_PREFIX2/postgresql port=$PG2PORT user=replication" \
   >   --standby-password=r3pl \
@@ -352,20 +353,22 @@ Restore on (new) primary:
   INFO dropping instance 1\d\/pg3 (re)
   INFO stopping PostgreSQL 1\d\/pg3 (re)
   INFO deconfiguring pgBackRest stanza 'app'
-  INFO deleting PostgreSQL data and WAL directories
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/1\/srv\/pgsql\/1\d\/pg3\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/1\/srv\/pgsql\/1\d\/pg3\/wal (re)
   INFO dropping instance 1\d\/pg2 (re)
   WARNING instance 1\d\/pg2 is already stopped (re)
   INFO deconfiguring pgBackRest stanza 'app'
-  INFO deleting PostgreSQL data and WAL directories
-  INFO deleting pgBackRest include directory
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/2\/srv\/pgsql\/1\d\/pg2\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/2\/srv\/pgsql\/1\d\/pg2\/wal (re)
+  INFO deleting pgBackRest include directory: $TMPDIR/1/etc/pgbackrest/conf.d
   INFO uninstalling base pgBackRest configuration
-  INFO deleting pgBackRest log directory
-  INFO deleting pgBackRest spool directory
-  INFO deleting PostgreSQL log directory
-  INFO deleting PostgreSQL socket directory
-  INFO deleting pgBackRest include directory
+  INFO deleting pgBackRest log directory: $TMPDIR/1/log/pgbackrest
+  INFO deleting pgBackRest spool directory: $TMPDIR/1/srv/pgbackrest/spool
+  INFO deleting PostgreSQL log directory: $TMPDIR/1/log/postgresql
+  INFO deleting PostgreSQL socket directory: $TMPDIR/1/run/postgresql
+  INFO deleting pgBackRest include directory: $TMPDIR/2/etc/pgbackrest/conf.d
   INFO uninstalling base pgBackRest configuration
-  INFO deleting pgBackRest log directory
-  INFO deleting pgBackRest spool directory
-  INFO deleting PostgreSQL log directory
-  INFO deleting PostgreSQL socket directory (no-eol)
+  INFO deleting pgBackRest log directory: $TMPDIR/2/log/pgbackrest
+  INFO deleting pgBackRest spool directory: $TMPDIR/2/srv/pgbackrest/spool
+  INFO deleting PostgreSQL log directory: $TMPDIR/2/log/postgresql
+  INFO deleting PostgreSQL socket directory: $TMPDIR/2/run/postgresql (no-eol)

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/expect/test-standby.t

@@ -111,11 +111,13 @@ Try to pause/resume WAL replay on primary
 
   INFO dropping instance 1\d\/pg1 (re)
   INFO stopping PostgreSQL 1\d\/pg1 (re)
-  INFO deleting PostgreSQL data and WAL directories
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/primary\/srv\/pgsql\/1\d\/pg1\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/primary\/srv\/pgsql\/1\d\/pg1\/wal (re)
   INFO dropping instance 1\d\/pg2 (re)
   INFO stopping PostgreSQL 1\d\/pg2 (re)
-  INFO deleting PostgreSQL data and WAL directories
-  INFO deleting PostgreSQL log directory
-  INFO deleting PostgreSQL socket directory
-  INFO deleting PostgreSQL log directory
-  INFO deleting PostgreSQL socket directory (no-eol)
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/secondary\/srv\/pgsql\/1\d\/pg2\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/secondary\/srv\/pgsql\/1\d\/pg2\/wal (re)
+  INFO deleting PostgreSQL log directory: $TMPDIR/primary/log/postgresql
+  INFO deleting PostgreSQL socket directory: $TMPDIR/primary/run/postgresql
+  INFO deleting PostgreSQL log directory: $TMPDIR/secondary/log/postgresql
+  INFO deleting PostgreSQL socket directory: $TMPDIR/secondary/run/postgresql (no-eol)

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/expect/test-transactions.t

@@ -41,7 +41,6 @@ Try to create an instance with a non-existing encoding, triggering a failure in
   INFO initializing PostgreSQL
   WARNING Command '['*/bin/pg_ctl', 'init', '-D', '$TMPDIR/srv/pgsql/test/data', '-o', '--auth-host=trust --auth-local=trust --encoding=notanencoding --locale=C --username=postgres --waldir=$TMPDIR/srv/pgsql/test/wal']' returned non-zero exit status 1. (glob)
   WARNING reverting: initializing PostgreSQL
-  INFO deleting PostgreSQL data and WAL directories
   Error: Command '['*/bin/pg_ctl', 'init', '-D', '$TMPDIR/srv/pgsql/test/data', '-o', '--auth-host=trust --auth-local=trust --encoding=notanencoding --locale=C --username=postgres --waldir=$TMPDIR/srv/pgsql/test/wal']' returned non-zero exit status 1. (glob)
   initdb: error: "notanencoding" is not a valid server encoding name
   pg_ctl: database system initialization failed

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/expect/test-upgrade.t

@@ -66,9 +66,11 @@ Create an instance with "password" local authentication method, and upgrade it:
 (cleanup)
   INFO dropping instance 1\d\/old (re)
   WARNING instance 1\d\/old is already stopped (re)
-  INFO deleting PostgreSQL data and WAL directories
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/srv\/pgsql\/1\d\/old\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/srv\/pgsql\/1\d\/old\/wal (re)
   INFO dropping instance 1\d\/new (re)
   INFO stopping PostgreSQL 1\d\/new (re)
-  INFO deleting PostgreSQL data and WAL directories
-  INFO deleting PostgreSQL log directory
-  INFO deleting PostgreSQL socket directory (no-eol)
+  INFO deleting PostgreSQL data directory: \$TMPDIR\/srv\/pgsql\/1\d\/new\/data (re)
+  INFO deleting PostgreSQL WAL directory: \$TMPDIR\/srv\/pgsql\/1\d\/new\/wal (re)
+  INFO deleting PostgreSQL log directory: $TMPDIR/log/postgresql
+  INFO deleting PostgreSQL socket directory: $TMPDIR/run/postgresql (no-eol)

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/unit/test_model.py

@@ -115,7 +115,6 @@ def test_as_parameters(runner: CliRunner) -> None:
             "--nickname=aaa",
             "--phone-numbers=12345",
         ],
-        input="alc\nalc\n",
     )
     assert result.exit_code == 0, click_result_traceback(result)
     assert json.loads(result.stderr) == {
@@ -363,3 +362,54 @@ def test_parse_params_as() -> None:
     }
     params_nested.update({f"address_{k}": v for k, v in address_params.items()})
     assert model.parse_params_as(models.Person, params_nested) == person
+
+
+def test_as_parameters_unrequired_argument(runner: CliRunner) -> None:
+    @click.command("add-product")
+    @model.as_parameters(models.Product, "create")
+    @click.option("--indent", type=int)
+    def add_product(product: models.Product, indent: int) -> None:
+        """Add a new product."""
+        click.echo(
+            product.model_dump_json(by_alias=True, indent=indent),
+            err=True,
+        )
+
+    result = runner.invoke(add_product, ["--help"])
+    assert result.exit_code == 0, click_result_traceback(result)
+    assert result.stdout == (
+        "Usage: add-product [OPTIONS] [NAME]\n"
+        "\n"
+        "  Add a new product.\n"
+        "\n"
+        "Options:\n"
+        "  --price TEXT               Price.  [required]\n"
+        "  --description DESCRIPTION  Description for the product.\n"
+        "  --indent INTEGER\n"
+        "  --help                     Show this message and exit.\n"
+    )
+
+    result = runner.invoke(
+        add_product,
+        [
+            "--price=19.5",
+            "--description=useless stuff",
+        ],
+    )
+    assert result.exit_code == 0, click_result_traceback(result)
+    assert json.loads(result.stderr) == {
+        "description": "useless stuff",
+        "name": "Unnamed product",
+        "price": 19.5,
+    }
+
+    result = runner.invoke(
+        add_product,
+        ["--price=19.5", "--description=useless stuff", "The Product Name"],
+    )
+    assert result.exit_code == 0, click_result_traceback(result)
+    assert json.loads(result.stderr) == {
+        "description": "useless stuff",
+        "name": "The Product Name",
+        "price": 19.5,
+    }

{pglift_cli-2.6.1 → pglift_cli-3.1.0}/tests/unit/test_util.py

@@ -367,6 +367,6 @@ def test_get_instance(
         ):
             util.get_instance("foo", None, settings)
     assert [call.args for call in system_lookup.call_args_list] == [
-        ("foo", "14", settings),
-        ("foo", "17", settings),
+        ("foo", "15", settings),
+        ("foo", "18", settings),
     ]