mrlunchbox777 / basic-setup

basic box setup
MIT License
6 stars 3 forks source link

feature: Support windows nodes for create-test-pod-info #166

Open mrlunchbox777 opened 5 months ago

mrlunchbox777 commented 5 months ago

Contact Details

mrlunchbox777@gmail.com

Feature Description

Windows nodes should be supported by create-test-pod-info.

Why?

While they are less popular, windows nodes are a supported part of the k8s ecosystem, so management functions like create-test-pod-info should support them.

What should happen?

There are several parts in the create-test-pod-info function that assume it targeting a linux node, these should be abstracted out and support both linux and windows.

Covers this TODO.

Steps to Test

  1. Target a linux and windows node with create-test-pod-info, it should create the node-shell pod and kubectl get pod -n kubesystem -l "app.kubernetes.io/name=test-podinfo" -o yaml | yq '.items[0]' should return the pod yaml rather than an error or null.

Code of Conduct