Skip to content
New issue

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

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

Already on GitHub? # to your account

cp(v0.33.x): feat: invalidate SSM cache upon AMI deprecation (#7301) #7342

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

jmdeal
Copy link
Contributor

@jmdeal jmdeal commented Nov 7, 2024

Fixes #N/A

Description
Cherry-picks #7301

How was this change tested?
make test

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.

@jmdeal jmdeal requested a review from a team as a code owner November 7, 2024 01:08
@jmdeal jmdeal requested a review from edibble21 November 7, 2024 01:08
@coveralls
Copy link

Pull Request Test Coverage Report for Build 11714681034

Details

  • 124 of 153 (81.05%) changed or added relevant lines in 11 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 77.805%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/fake/ssmapi.go 26 27 96.3%
pkg/controllers/controllers.go 0 2 0.0%
pkg/providers/amifamily/ami.go 15 17 88.24%
pkg/apis/v1/ec2nodeclass.go 15 18 83.33%
pkg/operator/operator.go 0 5 0.0%
pkg/controllers/providers/ssm/invalidation/controller.go 34 40 85.0%
pkg/apis/v1/zz_generated.deepcopy.go 0 10 0.0%
Totals Coverage Status
Change from base Build 11713847663: 0.005%
Covered Lines: 5430
Relevant Lines: 6979

💛 - Coveralls

@jmdeal jmdeal changed the title cp(v0.33.x): feat: invalidate SSM cache upon AMI deprecation cp(v0.33.x): feat: invalidate SSM cache upon AMI deprecation (#7301) Nov 7, 2024
@jmdeal jmdeal merged commit cfa6aa5 into aws:release-v0.33.x Nov 7, 2024
8 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants