Skip to content

docs(cnpg-i): deprecate backup metrics and fields in Cluster status - #8091

Merged
fcanovai merged 6 commits into
mainfrom
dev/8051
Jul 22, 2025
Merged

fcanovai merged 6 commits into
mainfrom
dev/8051

Conversation

@gbartolini

@gbartolini gbartolini commented Jul 21, 2025 •

Copy link
Copy Markdown
Contributor

Due to the transition to a plugin-based, backup-agnostic architecture in CloudNativePG, we are deprecating:

  • firstRecoverabilityPoint, firstRecoverabilityPointByMethod, lastSuccessfulBackup, lastSuccessfulBackupByMethod, and lastFailedBackup in Cluster.status
  • cnpg_collector_first_recoverability_point, cnpg_collector_last_failed_backup_timestamp, cnpg_collector_last_available_backup_timestamp metrics

If you have migrated to a plugin-based backup and recovery solution such as Barman Cloud, these fields/metrics are no longer synchronized and will not be updated. Users still relying on the in-core support for Barman Cloud and volume snapshots can continue to use these fields for the time being.

Closes #8051

@gbartolini
gbartolini requested a review from jsilvela as a code owner July 21, 2025 17:20
@gbartolini
gbartolini requested a review from a team July 21, 2025 17:20
@cnpg-bot cnpg-bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.22 labels Jul 21, 2025
@github-actions

Copy link
Copy Markdown
Contributor

❗ By default, the pull request is configured to backport to all release branches.

  • To stop backporting this pr, remove the label: backport-requested ◀️ or add the label 'do not backport'
  • To stop backporting this pr to a certain release branch, remove the specific branch label: release-x.y

@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jul 21, 2025
@gbartolini gbartolini removed backport-requested ◀️ This pull request should be backported to all supported releases release-1.22 size:M This PR changes 30-99 lines, ignoring generated files. labels Jul 21, 2025
@dosubot dosubot Bot added do not backport This PR must not be backported - it will be in the next minor release documentation 📖 Improvements or additions to documentation size:M This PR changes 30-99 lines, ignoring generated files. labels Jul 21, 2025
@mnencia mnencia added backport-requested ◀️ This pull request should be backported to all supported releases and removed do not backport This PR must not be backported - it will be in the next minor release labels Jul 21, 2025
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 21, 2025
@gbartolini gbartolini changed the title docs(cnpg-i)!: deprecate backup fields in Cluster status docs(cnpg-i): deprecate backup fields in Cluster status Jul 22, 2025
@gbartolini gbartolini changed the title docs(cnpg-i): deprecate backup fields in Cluster status docs(cnpg-i): deprecate backup metrics and fields in Cluster status Jul 22, 2025
@dosubot dosubot Bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Jul 22, 2025
Comment thread api/v1/cluster_types.go Resolved
@fcanovai
fcanovai force-pushed the dev/8051 branch 2 times, most recently from a14c912 to 7a35226 Compare July 22, 2025 13:12
@fcanovai
fcanovai requested a review from NiccoloFei as a code owner July 22, 2025 13:33
@fcanovai
fcanovai requested a review from litaocdl as a code owner July 22, 2025 13:33
@fcanovai
fcanovai force-pushed the dev/8051 branch 2 times, most recently from 2e21919 to e44c7be Compare July 22, 2025 14:11
gbartolini and others added 6 commits July 22, 2025 16:17
Deprecates `firstRecoverabilityPoint`, `firstRecoverabilityPointByMethod`,
`lastSuccessfulBackup`, `lastSuccessfulBackupByMethod`, and `lastFailedBackup`
in `Cluster.status` due to the transition to a plugin-based, backup-agnostic
architecture in CloudNativePG.

BREAKING CHANGE: If you have migrated to a plugin-based backup and recovery
solution such as Barman Cloud, these fields are no longer synchronized and will
not be updated. Users still relying on the in-core support for Barman Cloud
and volume snapshots can continue to use these fields for the time being.

Closes #8051

Signed-off-by: Gabriele Bartolini <[email protected]>
Signed-off-by: Marco Nenciarini <[email protected]>
Signed-off-by: Gabriele Bartolini <[email protected]>
Signed-off-by: Francesco Canovai <[email protected]>
@fcanovai

Copy link
Copy Markdown
Contributor

/ok-to-merge

@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label Jul 22, 2025
@fcanovai
fcanovai merged commit 278677a into main Jul 22, 2025
@fcanovai
fcanovai deleted the dev/8051 branch July 22, 2025 14:38
cnpg-bot pushed a commit that referenced this pull request Jul 22, 2025
…#8091)

Due to the transition to a plugin-based, backup-agnostic architecture in
CloudNativePG, we are deprecating:

- `firstRecoverabilityPoint`, `firstRecoverabilityPointByMethod`,
`lastSuccessfulBackup`, `lastSuccessfulBackupByMethod`, and
`lastFailedBackup` in `Cluster.status`
- `cnpg_collector_first_recoverability_point`,
`cnpg_collector_last_failed_backup_timestamp`,
`cnpg_collector_last_available_backup_timestamp` metrics

If you have migrated to a plugin-based backup and recovery solution such
as Barman Cloud, these fields/metrics are no longer synchronized and
will not be updated. Users still relying on the in-core support for
Barman Cloud and volume snapshots can continue to use these fields for
the time being.

Closes #8051

Signed-off-by: Gabriele Bartolini <[email protected]>
Signed-off-by: Marco Nenciarini <[email protected]>
Signed-off-by: Francesco Canovai <[email protected]>
Co-authored-by: Marco Nenciarini <[email protected]>
Co-authored-by: Francesco Canovai <[email protected]>
(cherry picked from commit 278677a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-requested ◀️ This pull request should be backported to all supported releases documentation 📖 Improvements or additions to documentation lgtm This PR has been approved by a maintainer ok to merge 👌 This PR can be merged size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs]: Deprecate Cluster status fields related to in-tree backup implementation

4 participants