Skip to content

Conversation

@p0rtale
Copy link
Contributor

@p0rtale p0rtale commented Jun 30, 2025

Added a new config_applied var for clusterwide config application status

  • Tests
  • Changelog
  • Documentation

@p0rtale p0rtale requested a review from a1div0 June 30, 2025 10:24
@p0rtale p0rtale self-assigned this Jun 30, 2025
@p0rtale p0rtale requested a review from Satbek June 30, 2025 12:36
@p0rtale p0rtale closed this Jul 1, 2025
@p0rtale p0rtale reopened this Jul 2, 2025
@p0rtale p0rtale force-pushed the config-applied-metric branch 2 times, most recently from 6f05c79 to 04f469f Compare July 2, 2025 12:10
@p0rtale p0rtale changed the title config: add metric for clusterwide config application status config: add config_applied var Jul 2, 2025
@p0rtale p0rtale force-pushed the config-applied-metric branch 2 times, most recently from 1f877ce to 015e3fe Compare July 3, 2025 05:03
Copy link
Contributor

@Satbek Satbek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Add please a changelog

@p0rtale p0rtale force-pushed the config-applied-metric branch from 015e3fe to 1d8ed9b Compare July 3, 2025 07:48
@p0rtale p0rtale requested a review from Satbek July 3, 2025 07:48
@Satbek Satbek merged commit 860a59e into master Jul 3, 2025
31 of 32 checks passed
@Satbek Satbek deleted the config-applied-metric branch July 3, 2025 08:56
@olegrok
Copy link
Contributor

olegrok commented Jul 9, 2025

@Satbek
Do you plan to add this metric to https://github.com/tarantool/cartridge-metrics-role ?

@Satbek
Copy link
Contributor

Satbek commented Jul 9, 2025

@Satbek Do you plan to add this metric to https://github.com/tarantool/cartridge-metrics-role ?

tarantool/metrics#511 added here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants