Skip to content

Conversation

@jonathan-innis
Copy link
Contributor

Fixes #N/A

Description

Fix the upgrade test failure by updating this test to the latest version that contains the helm fix

How was this change tested?

make presubmit

Does this change impact docs?

  • Yes, PR includes docs updates
  • Yes, issue opened: #
  • No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@jonathan-innis jonathan-innis requested a review from a team as a code owner September 8, 2025 20:56
@netlify
Copy link

netlify bot commented Sep 8, 2025

Deploy Preview for karpenter-docs-prod canceled.

Name Link
🔨 Latest commit 62811b0
🔍 Latest deploy log https://app.netlify.com/projects/karpenter-docs-prod/deploys/68bf430d4949f30008d5dbca

Copy link
Contributor

@jigisha620 jigisha620 left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@jonathan-innis jonathan-innis enabled auto-merge (squash) September 8, 2025 20:57
@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2025

Preview deployment ready!

Preview URL: https://pr-8469.d18coufmbnnaag.amplifyapp.com

Built from commit 62811b0c722f86cc99171ae9d381d30a78b668d5

@jonathan-innis jonathan-innis merged commit b8b4b03 into aws:main Sep 9, 2025
23 of 24 checks passed
@coveralls
Copy link

Pull Request Test Coverage Report for Build 17563890894

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 13 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.1%) to 67.122%

Files with Coverage Reduction New Missed Lines %
pkg/fake/iamapi.go 5 57.69%
pkg/providers/instanceprofile/instanceprofile.go 8 78.86%
Totals Coverage Status
Change from base Build 17561158422: -0.1%
Covered Lines: 7617
Relevant Lines: 11348

💛 - Coveralls

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