Skip to content

wip: enable all released modules tests on ci for windows #6018

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
Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

wip: enable all released modules tests on ci for windows #6018

wants to merge 17 commits into from
+284 −131

Conversation

@koba1t
Copy link
Member

@koba1t koba1t commented Nov 17, 2025

fix #4001

@k8s-ci-robot
Copy link
Contributor

k8s-ci-robot commented Nov 17, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: koba1t

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

The pull request process is described here

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. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Nov 17, 2025
@k8s-ci-robot k8s-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 17, 2025
@koba1t koba1t changed the title enable all modules tests and ci for windows CI: enable all modules tests on ci for windows Nov 17, 2025
@koba1t koba1t force-pushed the chore/enable_tests_and_ci_for_Windows branch from 48fce68 to 64df232 Compare November 17, 2025 14:14
@k8s-ci-robot
Copy link
Contributor

k8s-ci-robot commented Nov 17, 2025

This PR has multiple commits, and the default merge method is: merge.
You can request commits to be squashed using the label: tide/merge-method-squash

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@k8s-ci-robot k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 17, 2025
@koba1t koba1t force-pushed the chore/enable_tests_and_ci_for_Windows branch from ba4f115 to f6920ff Compare November 17, 2025 14:57
@k8s-ci-robot k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 17, 2025
@koba1t koba1t force-pushed the chore/enable_tests_and_ci_for_Windows branch 4 times, most recently from d1cd5fd to 1a3aa5f Compare November 17, 2025 21:07
@koba1t koba1t added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Nov 17, 2025
@koba1t koba1t force-pushed the chore/enable_tests_and_ci_for_Windows branch 3 times, most recently from acd97e9 to c1bd618 Compare November 17, 2025 21:39
@k8s-ci-robot k8s-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Nov 18, 2025
@koba1t koba1t force-pushed the chore/enable_tests_and_ci_for_Windows branch 5 times, most recently from ec7b657 to 5aee981 Compare November 18, 2025 14:16
@k8s-ci-robot k8s-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 18, 2025
@koba1t koba1t force-pushed the chore/enable_tests_and_ci_for_Windows branch 5 times, most recently from 9756e38 to 61fa6b1 Compare November 19, 2025 16:29
@koba1t koba1t changed the title CI: enable all released modules tests on ci for windows wip: enable all released modules tests on ci for windows Nov 19, 2025
@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Nov 19, 2025
@koba1t koba1t force-pushed the chore/enable_tests_and_ci_for_Windows branch from 61fa6b1 to d72e5bb Compare November 26, 2025 16:59
koba1t added 17 commits November 27, 2025 02:01
@koba1t
disable repospec_test.go for windows due to a huge amount of errors
4d72317
@koba1t
skip docker test when disabled by env var
aab1cba
@koba1t
fix filepath separater for prevent stackoverflow in Windows
9afc69e
@koba1t
add a handler for when windows volume path like 'C:'
1cb90c4
@koba1t
fix origin path for windows
73465cc
@koba1t
skip helm args tests for windows
c5566ca
@koba1t
skip remote ssh loader tests on windows due to that didn't supported yet
0ee3195
@koba1t
disable exec plugin tests for windows:
d8b1b76
@koba1t
tmp: fix base filepath
63d7946
@koba1t
skip exec tests for windows
64b9c7b
@koba1t
fix path handing in test for windows
444ec19
@koba1t
fix dir name handling in Windows for IPv6 addr
7d21b61
@koba1t
skip one Plugin test for windows due to error message is differ
537ee68
@koba1t
tmp: fix path handling with cross-platform
42e9d62
@koba1t
disable tests for windows due to no compiler toolchain
96271e6
@koba1t
improve path handling for cross platform
335f794
@koba1t
wip
736c34e
@koba1t koba1t force-pushed the chore/enable_tests_and_ci_for_Windows branch from d72e5bb to 736c34e Compare November 26, 2025 17:01
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Nov 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@varshaprasad96 varshaprasad96 Awaiting requested review from varshaprasad96

Assignees

No one assigned

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. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix breaking api tests on windows

2 participants

@koba1t @k8s-ci-robot