fix: add SkipNameValidation when creating controller in retry block by mateusoliveira43 · Pull Request #9123 · cloudnative-pg/cloudnative-pg

@dosubot dosubot bot added the size:S

This PR changes 10-29 lines, ignoring generated files.

label

Nov 10, 2025

@dosubot dosubot bot added size:M

This PR changes 30-99 lines, ignoring generated files.

and removed size:S

This PR changes 10-29 lines, ignoring generated files.

labels

Nov 11, 2025
Signed-off-by: Mateus Oliveira <66965232+mateusoliveira43@users.noreply.github.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>

leonardoce

@dosubot dosubot bot added the lgtm

This PR has been approved by a maintainer

label

Nov 17, 2025

cnpg-bot pushed a commit that referenced this pull request

Nov 17, 2025
…9123)

If `runSubCommand` fails after it creates instance-cluster controller,
all subsequent runs will fail with `controller with name
instance-cluster already exists` error. To fix this, the patch adds
`SkipNameValidation` to the controller options for subsequent runs.

Also adds error logs every time an error is returned.

See: #9159

Signed-off-by: Mateus Oliveira <66965232+mateusoliveira43@users.noreply.github.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit dfd3e3c)

leonardoce pushed a commit that referenced this pull request

Nov 17, 2025
…9123)

If `runSubCommand` fails after it creates instance-cluster controller,
all subsequent runs will fail with `controller with name
instance-cluster already exists` error. To fix this, the patch adds
`SkipNameValidation` to the controller options for subsequent runs.

Also adds error logs every time an error is returned.

See: #9159

Signed-off-by: Mateus Oliveira <66965232+mateusoliveira43@users.noreply.github.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit dfd3e3c)

leonardoce pushed a commit that referenced this pull request

Nov 17, 2025
…9123)

If `runSubCommand` fails after it creates instance-cluster controller,
all subsequent runs will fail with `controller with name
instance-cluster already exists` error. To fix this, the patch adds
`SkipNameValidation` to the controller options for subsequent runs.

Also adds error logs every time an error is returned.

See: #9159

Signed-off-by: Mateus Oliveira <66965232+mateusoliveira43@users.noreply.github.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit dfd3e3c)
(cherry picked from commit 4ee77d0)

mnencia pushed a commit that referenced this pull request

Jan 20, 2026
…9123)

If `runSubCommand` fails after it creates instance-cluster controller,
all subsequent runs will fail with `controller with name
instance-cluster already exists` error. To fix this, the patch adds
`SkipNameValidation` to the controller options for subsequent runs.

Also adds error logs every time an error is returned.

See: #9159

Signed-off-by: Mateus Oliveira <66965232+mateusoliveira43@users.noreply.github.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Co-authored-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
(cherry picked from commit dfd3e3c)
(cherry picked from commit 4ee77d0)