Skip to content

Conversation

@michaelawyu
Copy link
Collaborator

Description of your changes

This PR bumps the version of the Karpenter/Azure provider (and related packages) so that we could use the latest version of the Azure Retail Prices client.

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

N/A

Special notes for your reviewer

This PR is sent to unblock the progress for cost property provider fixes.

Signed-off-by: michaelawyu <chenyu1@microsoft.com>
@codecov
Copy link

codecov bot commented Jun 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

Signed-off-by: michaelawyu <chenyu1@microsoft.com>
@ryanzhang-oss ryanzhang-oss requested a review from Copilot June 24, 2025 19:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Azure provider version and related dependencies to unblock cost property provider fixes.

  • Updated import paths in the Azure pricing tracker to point at the new provider module.
  • Bumped versions of the Azure SDK, Azure identity, karpenter-provider-azure, and numerous indirect dependencies in go.mod.

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
pkg/propertyprovider/azure/trackers/pricing.go Changed imports from github.com/Azure/karpenter to the new karpenter-provider-azure module
go.mod Upgraded versions of Azure SDK packages, karpenter-provider-azure, and other direct and indirect dependencies

@ryanzhang-oss ryanzhang-oss merged commit 23615ea into kubefleet-dev:main Jun 24, 2025
17 checks passed
@michaelawyu michaelawyu deleted the feat/bump-karpenter-version branch June 24, 2025 22:30
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.

2 participants