Skip to content

vmscrapeconfig: add absent sd configs#2283

Merged
vrutkovs merged 1 commit into
masterfrom
add-lacking-sd-configs
Jun 15, 2026
Merged

vmscrapeconfig: add absent sd configs#2283
vrutkovs merged 1 commit into
masterfrom
add-lacking-sd-configs

Conversation

@AndrewChubatiuk

@AndrewChubatiuk AndrewChubatiuk commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

fixes #2265


Summary by cubic

Adds support for new service discovery types in VMScrapeConfig to reach full parity with VictoriaMetrics sd_configs, and fixes a potential deadlock in the operator metrics collector. Addresses #2265.

  • New Features

    • Support consulAgentSDConfigs, dockerSDConfigs, dockerSwarmSDConfigs, marathonSDConfigs, yandexCloudSDConfigs.
    • Generate YAML for these configs in internal/controller/operator/factory/vmscrapes/scrapeconfig.go.
    • Update CRDs and docs; add deepcopy methods and tests.
  • Bug Fixes

    • Prevent deadlock in operator_object_status collector by not holding the mutex during metric emission.

Written for commit 0b263ce. Summary will update on new commits.

Review in cubic

@AndrewChubatiuk AndrewChubatiuk changed the title Add lacking sd configs vmscrapeconfig: add absent sd configs Jun 13, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1 issue found across 10 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread api/operator/v1beta1/vmscrapeconfig_types.go
…mSDConfigs, marathonSDConfigs, and yandexCloudSDConfigs
@AndrewChubatiuk AndrewChubatiuk force-pushed the add-lacking-sd-configs branch from e017a5d to 0b263ce Compare June 13, 2026 17:27
@vrutkovs vrutkovs merged commit 8a32be0 into master Jun 15, 2026
7 checks passed
@vrutkovs vrutkovs deleted the add-lacking-sd-configs branch June 15, 2026 08:33
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.

feature: add yandexcloud_sd_configs in VMScrapeConfig

2 participants