Skip to content

Commit c9dfaea

Browse files
committed
Fix macos runner names
1 parent 851aa83 commit c9dfaea

File tree

5 files changed

+34
-18
lines changed

5 files changed

+34
-18
lines changed

.github/workflows/test-action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -985,7 +985,7 @@ jobs:
985985
name: ${{ matrix.display_name }} ${{ matrix.tests-chunk }} ${{ matrix.transport }}${{ matrix.test-group && ' ' || '' }}${{ matrix.test-group && matrix.test-group || '' }}
986986

987987
if: ${{ !cancelled() && toJSON(fromJSON(inputs.matrix)['windows']) != '[]' }}
988-
runs-on: ${{ matrix.slug }}
988+
runs-on: ${{ matrix.runner }}
989989
# Full test runs. Each chunk should never take more than 2 hours.
990990
# Partial test runs(no chunk parallelization), 6 Hours
991991
timeout-minutes: ${{ fromJSON(inputs.testrun)['type'] == 'full' && inputs.default-timeout || 360 }}

.github/workflows/test-packages-action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -319,7 +319,7 @@ jobs:
319319

320320
test-windows:
321321
name: ${{ matrix.display_name }} ${{ matrix.pkg_type }} ${{ matrix.tests-chunk }} ${{ matrix.version }}
322-
runs-on: ${{ matrix.slug }}
322+
runs-on: ${{ matrix.runner }}
323323
timeout-minutes: 120 # 2 Hours - More than this and something is wrong
324324
if: ${{ !cancelled() && toJSON(fromJSON(inputs.matrix)['windows']) != '[]' }}
325325
strategy:

cicd/shared-gh-workflows-context.yml

Lines changed: 18 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
nox_version: "2022.8.7"
22
python_version: "3.10.19"
33
relenv_version: "0.22.4"
4+
release_branches:
5+
- "3006.x"
6+
- "3007.x"
47
pr-testrun-slugs:
58
- ubuntu-24.04-pkg
69
- ubuntu-24.04
@@ -138,12 +141,14 @@ test-salt-listing:
138141
enabled: false
139142
macos:
140143
- slug: macos-15-intel
141-
display_name: "macOS 15"
144+
display_name: "macOS 15 (Intel)"
142145
arch: x86_64
146+
runner: macos-15-intel
143147
enabled: true
144148
- slug: macos-15
145149
display_name: "macOS 15 (M1)"
146150
arch: arm64
151+
runner: macos-15
147152
enabled: true
148153
windows:
149154
- slug: windows-2022
@@ -279,32 +284,38 @@ test-salt-pkg-listing:
279284
container: "ghcr.io/saltstack/salt-ci-containers/testing:ubuntu-24.04"
280285
enabled: false
281286
macos:
282-
- slug: macos-13-intel-pkg
283-
display_name: "macOS 15"
287+
- slug: macos-15-intel-pkg
288+
display_name: "macOS 15 (Intel)"
284289
arch: x86_64
290+
runner: macos-15-intel
285291
enabled: true
286292
- slug: macos-15-pkg
287293
display_name: "macOS 15 (M1)"
288294
arch: arm64
295+
runner: macos-15
289296
enabled: true
290297
windows:
291-
- slug: windows-2022-pkg
298+
- slug: windows-2022-nsis-pkg
292299
display_name: "Windows 2022"
293300
arch: amd64
301+
runner: windows-2022
294302
pkg_type: NSIS
295303
enabled: true
296-
- slug: windows-2022-pkg
304+
- slug: windows-2022-msi-pkg
297305
display_name: "Windows 2022"
298306
arch: amd64
307+
runner: windows-2022
299308
pkg_type: MSI
300309
enabled: true
301-
- slug: windows-2025-pkg
310+
- slug: windows-2025-nsis-pkg
302311
display_name: "Windows 2025"
303312
arch: amd64
313+
runner: windows-2025
304314
pkg_type: NSIS
305315
enabled: true
306-
- slug: windows-2025-pkg
316+
- slug: windows-2025-msi-pkg
307317
display_name: "Windows 2025"
308318
arch: amd64
319+
runner: windows-2025
309320
pkg_type: MSI
310321
enabled: true

tools/testsuite/download.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ def download_artifact(
259259
"help": "The workflow run ID from where to download artifacts from",
260260
},
261261
"slug": {
262-
"help": "Slug of the test run (examples: debian-11, macos-13, windows-2022)",
262+
"help": "Slug of the test run (examples: debian-11, macos-15-intel, windows-2022)",
263263
},
264264
"repository": {
265265
"help": "The repository to query, e.g. saltstack/salt",

tools/utils/__init__.py

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,11 @@ class OS:
6060
display_name: str = attr.ib(default=None)
6161
pkg_type: str = attr.ib(default=None)
6262
enabled: bool = attr.ib(default=True)
63+
runner: str = attr.ib()
64+
65+
@runner.default
66+
def _default_runner(self):
67+
return self.slug
6368

6469
@arch.default
6570
def _default_arch(self):
@@ -90,6 +95,8 @@ def as_dict(self):
9095
"arch": self.arch,
9196
"display_name": self.display_name,
9297
"pkg_type": self.pkg_type,
98+
"enabled": self.enabled,
99+
"runner": self.runner,
93100
"fips": self.fips,
94101
"container": self.container,
95102
"job_name": self.job_name,
@@ -106,13 +113,8 @@ def job_name(self):
106113

107114
@attr.s(frozen=True, slots=True)
108115
class MacOS(OS):
109-
runner: str = attr.ib()
110116
platform: str = attr.ib(default="macos")
111117

112-
@runner.default
113-
def _default_runner(self):
114-
return self.slug
115-
116118
@property
117119
def job_name(self):
118120
return f"test-{ self.slug.replace('.', '') }"
@@ -124,6 +126,7 @@ def as_dict(self):
124126
"arch": self.arch,
125127
"display_name": self.display_name,
126128
"pkg_type": self.pkg_type,
129+
"enabled": self.enabled,
127130
"runner": self.runner,
128131
"job_name": self.job_name,
129132
}
@@ -155,6 +158,8 @@ def as_dict(self):
155158
"arch": self.arch,
156159
"display_name": self.display_name,
157160
"pkg_type": self.pkg_type,
161+
"enabled": self.enabled,
162+
"runner": self.runner,
158163
"job_name": self.job_name,
159164
}
160165

@@ -406,10 +411,10 @@ def get_platform_and_arch_from_slug(slug: str) -> tuple[str, str]:
406411
arch = "amd64"
407412
elif "macos" in slug:
408413
platform = "macos"
409-
if "macos-13" in slug and "xlarge" in slug:
410-
arch = "arm64"
411-
else:
414+
if "intel" in slug:
412415
arch = "x86_64"
416+
else:
417+
arch = "arm64"
413418
else:
414419
platform = "linux"
415420
if "arm64" in slug:

0 commit comments

Comments
 (0)