Skip to content

Installing kubectl in dev container. #3177

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
Merged
merged 1 commit into from
Jun 4, 2025
Merged

Installing kubectl in dev container. #3177

merged 1 commit into from
Jun 4, 2025

Conversation

CheatCodeSam
Copy link
Contributor

Description

This makes it a little easier to run just test-kind in the dev container to make sure the Kubernetes tests are working as intended.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Test updates
  • CI/CD related changes
  • Other (please describe):
    Developer Tooling

Testing

  • Added new unit tests
  • Added new integration tests
  • Ran existing test suite
  • Tested manually (please provide steps)
    Ran test container then ran just youki-dev and just test-kind and didn't have an issue starting and completing tests.

Related Issues

Additional Context

@CheatCodeSam CheatCodeSam force-pushed the k8s-in-dev-container branch from 28c1cbc to 0e8b093 Compare June 2, 2025 17:04
@utam0k utam0k added the kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. label Jun 3, 2025
YJDoc2
YJDoc2 approved these changes Jun 4, 2025
View reviewed changes
Copy link
Collaborator

@YJDoc2 YJDoc2 left a comment

Choose a reason for hiding this comment

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

LGTM. This is just dev setup stuff, don't see any major issues in adding this, so merging.

@YJDoc2 YJDoc2 merged commit efabfd6 into youki-dev:main Jun 4, 2025
58 of 62 checks passed
@github-actions github-actions bot mentioned this pull request Jun 4, 2025
Merged
@github-actions github-actions bot mentioned this pull request Jul 20, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YJDoc2 YJDoc2 YJDoc2 approved these changes

Assignees
No one assigned
Labels
kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@CheatCodeSam @YJDoc2 @utam0k