Automated PR. Will trigger the 1.23.3 release when approved.

Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Co-authored-by: Niccolò Fei <niccolo.fei@enterprisedb.com>

Closes #5086

Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enteprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Leonardo Cecchi <leonardo.cecchi@enteprisedb.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>

Closes #4640

Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>

The k9s plugin executes commands directly in bash, meaning that some
variables are passed straight to a bash command.
One such variable is the context, which may contain spaces.
To ensure proper handling of spaces, this update quotes the context variable.

Signed-off-by: Glenn Sommer <gso@danskecommodities.com>
Signed-off-by: Jonathan Gonzalez V. <jonathan.gonzalez@enterprisedb.com>
Co-authored-by: Glenn Sommer <gso@danskecommodities.com>
Co-authored-by: Jonathan Gonzalez V. <jonathan.gonzalez@enterprisedb.com>
(cherry picked from commit 2d858241)

Update the Postgres versions used in E2E tests

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Jonathan Gonzalez V. <jonathan.gonzalez@enterprisedb.com>
Co-authored-by: postgres-versions-updater <postgres-versions-updater@users.noreply.github.com>
(cherry picked from commit 893f26a2)

Update the versions used to test the operator on public cloud providers

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: public-cloud-k8s-versions-check <public-cloud-k8s-versions-check@users.noreply.github.com>
(cherry picked from commit c440db01)

https://github.com/onsi/ginkgo `v2.19.0` -> `v2.19.1`
https://github.com/onsi/gomega `v1.33.1` -> `v1.34.0`

chore(deps): update agilepathway/pull-request-label-checker docker tag to v1.6.48 (release-1.23) (#5172)

fix(deps): update module github.com/prometheus-operator/prometheus-operator/pkg/apis/monitoring to v0.75.2 (release-1.23) (#5147)

Additionally, update the Makefile with the proper limits of the testing versions

Closes #5136

Signed-off-by: Jonathan Gonzalez V. <jonathan.gonzalez@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
(cherry picked from commit 2839384c)

Modify the ScheduledBackup reconciler to add an event to the resource
when the proposed schedule doesn't permit a subsequent execution.

This patch addresses a previous issue where the reconciler would
erroneously create a backup with an incorrect timestamp under such
conditions. The event notifies users about the schedule constraint,
enhancing the reliability of the backup process.

Closes #5108

Signed-off-by: YanniHu1996 <yantian.hu@enterprisedb.com>
Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
Co-authored-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
(cherry picked from commit 8681ade4)

Closes #5159

Signed-off-by: Jonathan Gonzalez V. <jonathan.gonzalez@enterprisedb.com>
(cherry picked from commit c8cb3cfa)

Adjusted the instance manager restart process to ensure it uses a well-known
path instead of relying on `os.Args[0]`. This prevents potential security issues
where `os.Args[0]` could be altered or replaced.

Closes #5157

Signed-off-by: Jonathan Gonzalez V. <jonathan.gonzalez@enterprisedb.com>
(cherry picked from commit a580e86e)

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit 6ac61af8)

This commit addresses an issue in CloudNativePG concerning the
initialization of new Persistent Volume Claims (PVCs) during the addition
of replicas and the creation of primary instances. In cases where the
initialization fails, Kubernetes jobs are retried. However, a partial
initialization may leave behind a PGDATA directory, causing subsequent
retries to fail.

To avoid this, the patch introduces a procedure to clean up the PGDATA
directory before retrying the initialization. If the directory is
identified as a valid PostgreSQL data directory, it is moved to a
backup location rather than deleted. This precautionary step helps
prevent accidental data loss in situations where the PGDATA directory
might have been mistakenly pre-existing in the Persistent Volume (PV).

Closes: #3365

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
(cherry picked from commit 3e7e2815)

Mention Debian distroless and UBI 8/9.

Closes #5114

Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
(cherry picked from commit aafe18f2)

Look at the `operators.coreos.com/v1` resources and check that
it they exist. If this is the case, we're in an OLM environment.
Before we were relying on the detection of the SecurityContextConstraint
 to know that we were running on an OLM environment but an OLM
environment may or may not have a SCC inside.

This helps us to improve the plugin report and properly retrieve more
useful information.

Closes #3764

---------

Signed-off-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Francesco Canovai <francesco.canovai@enterprisedb.com>
Signed-off-by: Jonathan Gonzalez V. <jonathan.gonzalez@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Francesco Canovai <francesco.canovai@enterprisedb.com>
(cherry picked from commit 91d70bc0)

Closes #5116

Signed-off-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
(cherry picked from commit 959b68c2)

Update the Postgres versions used in E2E tests

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Jonathan Gonzalez V. <jonathan.gonzalez@enterprisedb.com>
Co-authored-by: postgres-versions-updater <postgres-versions-updater@users.noreply.github.com>
(cherry picked from commit 8d6697d6)

jonasbn/github-action-spellcheck `0.38.0` -> `0.40.0` 
https://github.com/rojopolis/spellcheck-github-actions `0.38.0` -> `0.40.0`

https://github.com/operator-framework/operator-registry `v1.44.0` -> `v1.45.0`
https://github.com/redhat-openshift-ecosystem/openshift-preflight `1.9.9` -> `1.10.0`

chore(deps): update agilepathway/pull-request-label-checker docker tag to v1.6.47 (release-1.23) (#5122)

https://togithub.com/cloudnative-pg/cnpg-i `3f40829` -> `cf1ab8f`
https://togithub.com/goreleaser/goreleaser `v2.0.1` -> `v2.1.0`

https://github.com/kubernetes/api `v0.30.2` -> `v0.30.3`
https://github.com/kubernetes/apiextensions-apiserver `v0.30.2` -> `v0.30.3`
https://github.com/kubernetes/apimachinery `v0.30.2` -> `v0.30.3`
https://github.com/kubernetes/cli-runtime `v0.30.2` -> `v0.30.3`
https://github.com/kubernetes/client-go `v0.30.2` -> `v0.30.3`
https://github.com/kubernetes/utils `fe8a2dd` -> `18e509b`

This reverts commit bdd21d4d.

This update allows the reconcilePodSpec annotation to function
for both cluster and pooler objects. Documentation has been updated accordingly.

Closes: #5068

Signed-off-by: Tao Li <tao.li@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Signed-off-by: Marco Nenciarini <mnencia@kcore.it>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Jaime Silvela <jaime.silvela@enterprisedb.com>
Co-authored-by: Marco Nenciarini <mnencia@kcore.it>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
(cherry picked from commit ab5984a0)

Before declaring a cluster healthy in the E2e tests, we ensure that the
replicas are connected via streaming replication.

---------

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
Signed-off-by: Francesco Canovai <francesco.canovai@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Francesco Canovai <francesco.canovai@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit d7ce5b4c)

Closes #4422

Signed-off-by: Douglass Kirkley <dkirkley@eitccorp.com>
Co-authored-by: Douglass Kirkley <dkirkley@eitccorp.com>
(cherry picked from commit f11fbeef)

This patch makes the operator retry an immediate backup when
the previous reconciliation attempt failed.

Closes #4981

Signed-off-by: Gabriele Fedi <gabriele.fedi@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
(cherry picked from commit 694e11da)

This patch excludes immutable databases (not allowing
connections) from monitoring and alerting. Specifically, `template0`
is immutable and should not trigger the `PGDatabaseXidAge`
alert (formerly known as `PGDatabase`), as all its content is frozen.

Closes #4978

Signed-off-by: Daniel Chambre <smiyc@pm.me>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
(cherry picked from commit dad329aa)

Creating a regional cluster allows us to have a multi-az cluster,
allowing us to re-enable the drain test on GKE.

Closes #5040

Signed-off-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit 471172be)

Replica cluster promotion test could fail trying to run some queries
during an expected restart. We wait for the restart to happen before
proceeding with the test.

Closes: #5059

Signed-off-by: Francesco Canovai <francesco.canovai@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit 1f89ccbc)

This patch cleans up the stale Pods that have been terminated but
now deleted by the Kubelet during an eviction process.

Unschedulable Pods are now cleaned up during a maintenance window
when the `reusePVC` flag is `false`.

Closes: #1946
Closes: #1593

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
(cherry picked from commit 40935a1a)

chore(deps): update agilepathway/pull-request-label-checker docker tag to v1.6.46 (release-1.23) (#5091)

The instance manager was starting `pg_rewind` against new replicas
created from an online volume snapshot backup of the primary node. This
happens because the primary PGDATA was snapshotted.

Since the PGDATA was not shutdown correctly (online backup), `pg_rewind`
will start a postmaster process that will write a shutdown checkpoint
record. By doing that, the history will diverge from the primary one
while retaining the same timeline.

Fixes: #5080

---------

Signed-off-by: Leonardo Cecchi <leonardo.cecchi@enterprisedb.com>
Signed-off-by: Tao Li <tao.li@enterprisedb.com>
Co-authored-by: Tao Li <tao.li@enterprisedb.com>
(cherry picked from commit 774cad3a)

Ensure that the default value shown in user interface remains
consistent with the slash command.

Closes: #5046

Signed-off-by: Tao Li <tao.li@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
(cherry picked from commit 24c6f2ff)