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

chore: sync v1 staging branch with main #6335

Merged
merged 52 commits into from
Jun 7, 2024
Merged

Conversation

engedaam
Copy link
Contributor

@engedaam engedaam commented Jun 7, 2024

Fixes #N/A

Description
Syncs staging/v1 with main as of 78db4f0

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 and others added 30 commits May 16, 2024 18:54
…tions/e2e/setup-cluster in the action-deps group (aws#6241)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…tions/e2e/cleanup in the action-deps group (aws#6240)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…tions/e2e/slack/notify in the action-deps group (aws#6239)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…tions/e2e/upgrade-crds in the action-deps group (aws#6237)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…tions/e2e/install-prometheus in the action-deps group (aws#6236)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…tions/e2e/install-karpenter in the action-deps group (aws#6235)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…the go-deps group across 1 directory (aws#6246)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: StableRelease <[email protected]>
Co-authored-by: Jason Deal <[email protected]>
jmdeal and others added 22 commits May 29, 2024 11:30
Co-authored-by: APICodeGen <[email protected]>
Co-authored-by: Amanuel Engeda <[email protected]>
…3 in the k8s-go-deps group (aws#6272)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Amanuel Engeda <[email protected]>
…aws#6291)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Amanuel Engeda <[email protected]>
…ons-deps group (aws#6305)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…/actions/e2e/install-karpenter in the action-deps group (aws#6308)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…n the go-deps group (aws#6307)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…/actions/e2e/run-tests-private-cluster in the action-deps group (aws#6309)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@engedaam engedaam changed the title chore: sync staging branch with main chore: sync v1 staging branch with main Jun 7, 2024
@coveralls
Copy link

coveralls commented Jun 7, 2024

Pull Request Test Coverage Report for Build 9419989776

Details

  • 170 of 211 (80.57%) changed or added relevant lines in 28 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.05%) to 82.476%

Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/batcher/createfleet.go 0 1 0.0%
pkg/batcher/describeinstances.go 0 1 0.0%
pkg/controllers/interruption/controller.go 6 7 85.71%
pkg/providers/instancetype/instancetype.go 22 23 95.65%
pkg/providers/version/version.go 1 2 50.0%
pkg/providers/instance/instance.go 19 21 90.48%
pkg/providers/pricing/pricing.go 4 7 57.14%
pkg/fake/ec2api.go 4 8 50.0%
pkg/cloudprovider/cloudprovider.go 27 32 84.38%
pkg/controllers/nodeclass/status/readiness.go 5 11 45.45%
Files with Coverage Reduction New Missed Lines %
pkg/fake/ec2api.go 1 86.89%
Totals Coverage Status
Change from base Build 9132951571: 0.05%
Covered Lines: 5530
Relevant Lines: 6705

💛 - Coveralls

@engedaam engedaam marked this pull request as ready for review June 7, 2024 16:26
@engedaam engedaam requested a review from a team as a code owner June 7, 2024 16:26
@engedaam engedaam requested a review from jmdeal June 7, 2024 16:26
Copy link
Contributor

@jonathan-innis jonathan-innis left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@engedaam engedaam merged commit 331f1ac into aws:staging/v1 Jun 7, 2024
13 checks passed
@engedaam engedaam deleted the bump-main branch July 5, 2024 07:48
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.