Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v9.5.x] AzureMonitor: Ensure legacy properties containing template variables are correctly migrated #68790

Merged
merged 1 commit into from
May 22, 2023

Conversation

grafanabot
Copy link
Contributor

Backport 0be29b4 from #68697

…are correctly migrated (#68697)

* Update migration ordering

- Update migration in applyTemplateVariables to account for interpolation from expression datasource
- Update mock to ensure overrides are passed through correctly
- Update tests
- Update migrateQuery function name for clarity

* Fix tests

* Remove comment

(cherry picked from commit 0be29b4)
@grafanabot grafanabot requested a review from aangelisc May 22, 2023 10:14
@grafanabot grafanabot added backport A backport PR no-changelog Skip including change in changelog/release notes type/bug area/frontend datasource/Azure Azure Monitor Datasource labels May 22, 2023
@aangelisc aangelisc added this to the 9.5.x milestone May 22, 2023
@aangelisc aangelisc enabled auto-merge (squash) May 22, 2023 10:20
@github-actions
Copy link
Contributor

Backend code coverage report for PR #68790
No changes

@github-actions
Copy link
Contributor

Frontend code coverage report for PR #68790

Plugin Main PR Difference
azuremonitor 84.61% 83.51% -1.10%

@aangelisc aangelisc merged commit d16b23c into v9.5.x May 22, 2023
20 checks passed
@aangelisc aangelisc deleted the backport-68697-to-v9.5.x branch May 22, 2023 11:35
@zerok zerok added add to changelog and removed no-changelog Skip including change in changelog/release notes labels Jun 29, 2023
@zerok zerok modified the milestones: 9.5.x, 9.5.6 Jun 30, 2023
@zerok zerok modified the milestones: 9.5.6, 9.5.x, 9.5.7 Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants