Skip to content
This repository was archived by the owner on Jan 23, 2026. It is now read-only.

Commit fa18287

Browse files
kirkbrauerNickCao
authored andcommitted
Fix tests broken by updated CRDs
1 parent 660a92d commit fa18287

3 files changed

Lines changed: 36 additions & 15 deletions

File tree

packages/jumpstarter-cli-admin/jumpstarter_cli_admin/get_test.py

Lines changed: 18 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
V1Alpha1ExporterStatus,
1616
V1Alpha1Lease,
1717
V1Alpha1LeaseList,
18+
V1Alpha1LeaseSelector,
1819
V1Alpha1LeaseSpec,
1920
V1Alpha1LeaseStatus,
2021
)
@@ -837,7 +838,7 @@ async def test_get_exporters_devices(_load_kube_config_mock, list_exporters_mock
837838
spec=V1Alpha1LeaseSpec(
838839
client=V1ObjectReference(name="test_client"),
839840
duration="5m",
840-
selector={"hardware": "rpi4"},
841+
selector=V1Alpha1LeaseSelector(match_labels={"hardware": "rpi4"}),
841842
),
842843
)
843844

@@ -868,7 +869,7 @@ async def test_get_exporters_devices(_load_kube_config_mock, list_exporters_mock
868869
spec=V1Alpha1LeaseSpec(
869870
client=V1ObjectReference(name="test_client"),
870871
duration="1h",
871-
selector={},
872+
selector=V1Alpha1LeaseSelector(match_labels={}),
872873
),
873874
)
874875

@@ -885,7 +886,9 @@ async def test_get_exporters_devices(_load_kube_config_mock, list_exporters_mock
885886
"name": "test_client"
886887
},
887888
"duration": "1h",
888-
"selector": {}
889+
"selector": {
890+
"matchLabels": {}
891+
}
889892
},
890893
"status": {
891894
"beginTime": "2024-01-01T21:00:00Z",
@@ -918,7 +921,8 @@ async def test_get_exporters_devices(_load_kube_config_mock, list_exporters_mock
918921
client:
919922
name: test_client
920923
duration: 1h
921-
selector: {}
924+
selector:
925+
matchLabels: {}
922926
status:
923927
beginTime: '2024-01-01T21:00:00Z'
924928
conditions:
@@ -1022,7 +1026,9 @@ async def test_get_lease(_load_kube_config_mock, get_lease_mock: AsyncMock):
10221026
},
10231027
"duration": "5m",
10241028
"selector": {
1025-
"hardware": "rpi4"
1029+
"matchLabels": {
1030+
"hardware": "rpi4"
1031+
}
10261032
}
10271033
},
10281034
"status": {
@@ -1057,7 +1063,9 @@ async def test_get_lease(_load_kube_config_mock, get_lease_mock: AsyncMock):
10571063
"name": "test_client"
10581064
},
10591065
"duration": "1h",
1060-
"selector": {}
1066+
"selector": {
1067+
"matchLabels": {}
1068+
}
10611069
},
10621070
"status": {
10631071
"beginTime": "2024-01-01T21:00:00Z",
@@ -1096,7 +1104,8 @@ async def test_get_lease(_load_kube_config_mock, get_lease_mock: AsyncMock):
10961104
name: test_client
10971105
duration: 5m
10981106
selector:
1099-
hardware: rpi4
1107+
matchLabels:
1108+
hardware: rpi4
11001109
status:
11011110
beginTime: '2024-01-01T21:00:00Z'
11021111
conditions:
@@ -1120,7 +1129,8 @@ async def test_get_lease(_load_kube_config_mock, get_lease_mock: AsyncMock):
11201129
client:
11211130
name: test_client
11221131
duration: 1h
1123-
selector: {}
1132+
selector:
1133+
matchLabels: {}
11241134
status:
11251135
beginTime: '2024-01-01T21:00:00Z'
11261136
conditions:

packages/jumpstarter-kubernetes/jumpstarter_kubernetes/__init__.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,14 @@
77
V1Alpha1ExporterStatus,
88
)
99
from .install import get_ip_address, helm_installed, install_helm_chart
10-
from .leases import LeasesV1Alpha1Api, V1Alpha1Lease, V1Alpha1LeaseList, V1Alpha1LeaseSpec, V1Alpha1LeaseStatus
10+
from .leases import (
11+
LeasesV1Alpha1Api,
12+
V1Alpha1Lease,
13+
V1Alpha1LeaseList,
14+
V1Alpha1LeaseSelector,
15+
V1Alpha1LeaseSpec,
16+
V1Alpha1LeaseStatus,
17+
)
1118
from .list import V1Alpha1List
1219

1320
__all__ = [
@@ -24,6 +31,7 @@
2431
"V1Alpha1Lease",
2532
"V1Alpha1LeaseStatus",
2633
"V1Alpha1LeaseList",
34+
"V1Alpha1LeaseSelector",
2735
"V1Alpha1LeaseSpec",
2836
"V1Alpha1List",
2937
"get_ip_address",

packages/jumpstarter-kubernetes/jumpstarter_kubernetes/test_leases.py

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
from kubernetes_asyncio.client.models import V1Condition, V1ObjectMeta, V1ObjectReference
22

3-
from jumpstarter_kubernetes import V1Alpha1Lease, V1Alpha1LeaseSpec, V1Alpha1LeaseStatus
3+
from jumpstarter_kubernetes import V1Alpha1Lease, V1Alpha1LeaseSelector, V1Alpha1LeaseSpec, V1Alpha1LeaseStatus
44

55
TEST_LEASE = V1Alpha1Lease(
66
api_version="jumpstarter.dev/v1alpha1",
@@ -16,7 +16,7 @@
1616
spec=V1Alpha1LeaseSpec(
1717
client=V1ObjectReference(name="test-client"),
1818
duration="1h",
19-
selector={"test": "label", "another": "something"},
19+
selector=V1Alpha1LeaseSelector(match_labels={"test": "label", "another": "something"}),
2020
),
2121
status=V1Alpha1LeaseStatus(
2222
begin_time="2021-10-01T00:00:00Z",
@@ -53,8 +53,10 @@ def test_lease_dump_json():
5353
},
5454
"duration": "1h",
5555
"selector": {
56-
"test": "label",
57-
"another": "something"
56+
"matchLabels": {
57+
"test": "label",
58+
"another": "something"
59+
}
5860
}
5961
},
6062
"status": {
@@ -96,8 +98,9 @@ def test_lease_dump_yaml():
9698
name: test-client
9799
duration: 1h
98100
selector:
99-
another: something
100-
test: label
101+
matchLabels:
102+
another: something
103+
test: label
101104
status:
102105
beginTime: '2021-10-01T00:00:00Z'
103106
conditions:

0 commit comments

Comments
 (0)