Skip to content

Conversation

@cdrage
Copy link
Member

@cdrage cdrage commented Jan 15, 2026

fix(tests): improve test compatibility for macOS and podman

What type of PR is this?

/kind cleanup

What this PR does / why we need it:

  • Remove deprecated 'version' attribute that we no longer use across all
    fixture files (confirmed they still convert 1-1)

  • Added macOS compatibility, using grealpath instead since it's a bit
    different on macOS vs Linux

  • Container connection on build for machines without podman or docker

  • Fixed typos

  • Fixes tests usually having warnings that no longer do

  • Fix bug with state validation

TLDR; CI is finally passing again after some clean up

Which issue(s) this PR fixes:

N/A, spring clean up fixing up Kompose testing again for any newcomers

Special notes for your reviewer:

Signed-off-by: Charlie Drage charlie@charliedrage.com

#### What type of PR is this?

<!--
Add one of the following kinds:
/kind bug
/kind documentation
/kind feature
-->

/kind cleanup

#### What this PR does / why we need it:

- Remove deprecated 'version' attribute that we no longer use across all
  fixture files (confirmed they still convert 1-1)

- Added macOS compatibility, using grealpath instead since it's a bit
  different on macOS vs Linux

- Container connection on build for machines without podman or docker

- Fixed typos

- Fixes tests usually having warnings that no longer do

- Fix bug with state validation

TLDR; CI is finally passing again after some clean up

#### Which issue(s) this PR fixes:
<!--
*Automatically closes linked issue when PR is merged.
Usage: `Fixes #<issue number>`, or `Fixes (paste link of issue)`.
-->

N/A, spring clean up fixing up Kompose testing again for any newcomers

#### Special notes for your reviewer:

Signed-off-by: Charlie Drage <charlie@charliedrage.com>
@k8s-ci-robot k8s-ci-robot added kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jan 15, 2026
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cdrage

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jan 15, 2026
@k8s-ci-robot k8s-ci-robot requested a review from TessaIO January 15, 2026 20:34
Copy link
Member Author

@cdrage cdrage left a comment

Choose a reason for hiding this comment

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

Merging this to unblock the dependabot PR's so I can get a release out :)

@cdrage cdrage merged commit 465f63c into kubernetes:main Jan 15, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants