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

Restructure Azure backend into versions #2952

Merged
merged 9 commits into from
Sep 21, 2023

Conversation

zalegrala
Copy link
Contributor

@zalegrala zalegrala commented Sep 20, 2023

What this PR does:

In #2882 we upgraded the SDK version for Azure. This started showing periodic segmentation fault errors in the Go runtime for unknown reasons. After a bit of research an no answers, here we resurect the original Azure backend code into a v1 package and move the new SDK into a v2 package behind a configuration flag. This will allow us to make a slower migration and gain some confidence about the approach and the new SDK before forcing production workloads onto the new code path.

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

Copy link
Member

@joe-elliott joe-elliott left a comment

Choose a reason for hiding this comment

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

I like the safety of this call. It gives us the chance to experiment with the new sdk to prove out stability given the current panics in CI.

Needs changelog/docs

cc @LasseHels

Heads up on this change. I believe (@zalegrala has been chatting with you so you're somewhat aware). Internally we are going to run the v2 sdk in our dev envs and on non critical components in prod. After we've gained some confidence in stability we can completely swap over to v2. Thank you for all your work here!

@zalegrala zalegrala marked this pull request as ready for review September 21, 2023 19:54
@zalegrala zalegrala merged commit 6c7e07d into grafana:main Sep 21, 2023
15 checks passed
@zalegrala zalegrala deleted the azureV2Structure branch September 21, 2023 21:54
@LasseHels
Copy link
Contributor

@joe-elliott @zalegrala Good call! We are excited to try this out in dev as well. The Tempo PVC bug is our most persistent bug, so we'll be celebrating when we finally don't have to nurse the PVCs anymore 🎉.

@zalegrala
Copy link
Contributor Author

Nice that will be good. We'll be rolling this change out over the next few weeks and likely running a couple less critical components on the v2 work. Thank s for your efforts here. Based on the upstream Azure bug, it sounds like we only would experience the issue in testing, but I'd rather err on the side of caution.

@zalegrala zalegrala mentioned this pull request Jul 18, 2024
3 tasks
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