Skip to content

Add ops file to enable scheduler metrics#1300

Open
johha wants to merge 1 commit intodevelopfrom
scheduler-metrics-ops-file
Open

Add ops file to enable scheduler metrics#1300
johha wants to merge 1 commit intodevelopfrom
scheduler-metrics-ops-file

Conversation

@johha
Copy link
Contributor

@johha johha commented Jan 15, 2026

Please take a moment to review the questions before submitting the PR

🚫 We only accept PRs to develop branch. If this is an exception, please specify why 🚫

WHAT is this change about?

Add ops file enable-scheduler-metrics.yml to enable metrics on scheduler VMs for cloud_controller_clock and cc_deployment_updater jobs.

What customer problem is being addressed? Use customer persona to define the problem e.g. Alana is unable to...

Operators want to see metrics (e.g. about db connection pool) from scheduler VM jobs.

Please provide any contextual information.

cloudfoundry/capi-release#600
cloudfoundry/cloud_controller_ng#4749

Has a cf-deployment including this change passed cf-acceptance-tests?

  • YES
  • NO

Does this PR introduce a breaking change? Please take a moment to read through the examples before answering the question.

  • YES - please choose the category from below. Feel free to provide additional details.
  • NO

Types of breaking changes:

  1. causes app or operator downtime
  2. increases VM footprint of cf-deployment - e.g. new jobs, new add ons, increases # of instances etc.
  3. modifies, deletes or moves the name of a job or instance group in the main manifest
  4. modifies the name or deletes a property of a job or instance group in the main manifest
  5. changes the name of credentials in the main manifest
  6. requires out-of-band manual intervention on the part of the operator
  7. modifies the ops-file path, changes the type, changes the values or removes ops-files from the following folders
    • ./operations/ or ./operations/experimental
    • ./addons
    • ./backup-and-restore/

If you're promoting an experimental Ops-file (or removing one), Please follow the Ops-file workflows.

Ops files changes in the following folders are considered as NON BREAKING CHANGES
./community, ./example-vars-files, ./test

How should this change be described in cf-deployment release notes?

Alana can now enable metrics for the scheduler VMs. When this ops file is applied, the prom_scraper job will scrape the metric endpoints for the cloud_controller_clock and cc_deployment_updater jobs using mTLS.

Does this PR introduce a new BOSH release into the base cf-deployment.yml manifest or any ops-files?

  • YES
  • NO

Does this PR make a change to an experimental or GA'd feature/component?

  • experimental feature/component
  • GA'd feature/component

Please provide Acceptance Criteria for this change?

Please specify either bosh cli or cf cli commands for our team (and cf operators) to verify the changes.

Few examples

  1. For a PR with a new job in the manifest, bosh instances can verify the job is running after upgrade. You can provide additional commands to verify the job is running as specified.
  2. For a PR with new variables, bosh variables | grep <var-name> command can verify the variable exists. This is the simplest varification but you can also provide additional commands to test that the variable holds the desired value.

What is the level of urgency for publishing this change?

  • Urgent - unblocks current or future work
  • Slightly Less than Urgent

Tag your pair, your PM, and/or team!

It's helpful to tag a few other folks on your team or your team alias in case we need to follow up later.

@johha johha force-pushed the scheduler-metrics-ops-file branch from bb2ae9d to ef673bd Compare February 13, 2026 13:13
@johha johha marked this pull request as ready for review February 13, 2026 13:16
@jochenehret jochenehret requested review from a team February 13, 2026 13:35
jochenehret
jochenehret previously approved these changes Feb 13, 2026
@ard-wg-gitbot
Copy link
Contributor

Hello friend, it looks like your pull request has failed one or more of our checks. Please take a look! 👀

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