Skip to content

feat(migrations): Add migration for inputs.openstack server_diagnotics option#19122

Merged
srebhan merged 1 commit into
influxdata:masterfrom
skartikey:feat/migrate-openstack-server-diagnotics
Jun 18, 2026
Merged

feat(migrations): Add migration for inputs.openstack server_diagnotics option#19122
srebhan merged 1 commit into
influxdata:masterfrom
skartikey:feat/migrate-openstack-server-diagnotics

Conversation

@skartikey

@skartikey skartikey commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

The server_diagnotics option in inputs.openstack was deprecated in v1.32.0 for removal in v1.40.0, replaced by adding serverdiagnostics to enabled_services. This adds a migration that performs that rewrite on telegraf config migrate, materializing the previous default service set first when enabled_services is otherwise unset. The option removal ships separately (#19130) so users can run the migration while still on v1.39.x.

Checklist

  • No AI generated code was used in this PR

Related issues

related to #19086

@telegraf-tiger telegraf-tiger Bot added the feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin label Jun 17, 2026
@skartikey skartikey added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Jun 18, 2026
…s option

The server_diagnotics option was deprecated in v1.32.0 for removal in
v1.40.0, replaced by adding 'serverdiagnostics' to 'enabled_services'.
Remove it and add a migration that appends 'serverdiagnostics' to
'enabled_services' when set to true, materializing the default service set
first when the list is otherwise unset.
@telegraf-tiger

Copy link
Copy Markdown
Contributor

@skartikey skartikey added this to the v1.39.1 milestone Jun 18, 2026

@srebhan srebhan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice! Thanks @skartikey!

@srebhan srebhan merged commit d6e7c52 into influxdata:master Jun 18, 2026
29 checks passed
skartikey added a commit that referenced this pull request Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants