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

Kubernetes Components Update and Code Cleanup #89

Merged
merged 2 commits into from
Jan 15, 2025

Conversation

clementrey-dev
Copy link
Collaborator

@clementrey-dev clementrey-dev commented Jan 8, 2025

Description

This PR contains two main changes:

1. Kubernetes Components Version Update

Updates the following components to their latest versions:

  • kubectl: 1.25.12 -> 1.31.4
  • EKS cluster: 1.25 -> 1.31
  • Cluster Autoscaler: v1.21.0 -> v1.31.0

2. Agent Code Cleanup for flake8 validation

  • Removes unused task_id variable in run_task function
  • Removes trailing whitespace in copyright header

Checklist

  • Added tests that cover your change (if possible)
  • Added/modified documentation as required (such as the README.md, or the docs directory)
  • Manually tested
  • Made sure the title of the PR is a good description that can go into the release notes
  • (Core team) Added labels for change area (e.g. area/controlplane) and kind (e.g. kind/improvement)

BONUS POINTS checklist:

  • Backfilled missing tests for code in same general area
  • Refactored something and made the world a better place

Copy link

linux-foundation-easycla bot commented Jan 8, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@clementrey-dev clementrey-dev force-pushed the feature/upgrade-kubernetes-components branch from 7e69a80 to 331bca4 Compare January 10, 2025 14:37
@kirillsc kirillsc self-requested a review January 15, 2025 11:32
@kirillsc
Copy link
Collaborator

Looks good to me.

@kirillsc kirillsc closed this Jan 15, 2025
@kirillsc kirillsc reopened this Jan 15, 2025
Copy link
Collaborator

@kirillsc kirillsc left a comment

Choose a reason for hiding this comment

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

LGTM

@kirillsc kirillsc removed the request for review from fgogolli January 15, 2025 11:37
@clementrey-dev clementrey-dev merged commit a908450 into main Jan 15, 2025
27 of 33 checks passed
@clementrey-dev clementrey-dev deleted the feature/upgrade-kubernetes-components branch January 15, 2025 13:04
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