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.

Sign up for GitHub

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

Jump to bottom

feat: updated cloud-tty tools [TTY][MINOR] #1815

Merged
merged 11 commits into from
Nov 26, 2024
Merged

feat: updated cloud-tty tools [TTY][MINOR] #1815

merged 11 commits into from
Nov 26, 2024

Conversation

Ani1357
Copy link
Contributor

@Ani1357 Ani1357 commented Nov 18, 2024 •
edited
Loading

This PR updates the tools used in the CloudTTY image.

  • yq: from v4.40.2 to v4.44.5
  • kubectl: from v1.28.4 to v1.31.2
  • k9s: from v0.28.2 to v0.32.7
  • helm: from v3.13.2 to v3.16.3
  • velero cli: from v1.13.0 to v1.14.0
  • tekton cli: from v0.33.0 to v0.38.1
  • kubectl-cnpg: from v1.21.1 to v1.24.1

Current image was build via this pipeline. For testing you can:

  • use this API branch with the update image
  • set the apl-tty image version to latest
  • you can run it locally docker run -it linode/apl-tty bash

CasLubbers
CasLubbers approved these changes Nov 21, 2024
View reviewed changes
Copy link
Contributor

@CasLubbers CasLubbers left a comment

Choose a reason for hiding this comment

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

Tested that the shell still works ✅

ferruhcihan
ferruhcihan approved these changes Nov 25, 2024
View reviewed changes
Copy link
Contributor

@ferruhcihan ferruhcihan left a comment

Choose a reason for hiding this comment

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

LGTM (we should remove APL-369 branch name from the .github/workflows/cloudtty-build-push.yml file before merging it)

@Ani1357 Ani1357 merged commit 26f388e into main Nov 26, 2024
5 checks passed
@Ani1357 Ani1357 deleted the APL-369 branch November 26, 2024 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ferruhcihan ferruhcihan ferruhcihan approved these changes

@CasLubbers CasLubbers CasLubbers approved these changes

@merll merll Awaiting requested review from merll

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Ani1357 @CasLubbers @ferruhcihan @j-zimnowoda