Skip to content

fix(jobs): align executor kind/provider with AIRCORE-397 execution model #335

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

Open
matthewgrossman wants to merge 20 commits into
base: main
Choose a base branch
from
Open

fix(jobs): align executor kind/provider with AIRCORE-397 execution model #335

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
cd7a881
fix(jobs): Separate compute and executor payload shapes
matthewgrossman Jun 15, 2026
665e334
self review
matthewgrossman Jun 15, 2026
af07f83
fixes
matthewgrossman Jun 15, 2026
0b59a34
Merge branch 'main' into AIRCORE-513-align-executorkind-with-original…
matthewgrossman Jun 15, 2026
78191cb
make update-sdk
matthewgrossman Jun 15, 2026
03120ab
add profiles
matthewgrossman Jun 15, 2026
91e6e65
fixes
matthewgrossman Jun 15, 2026
47c2516
self code review
matthewgrossman Jun 15, 2026
3b920b8
lint
matthewgrossman Jun 15, 2026
ca3bebc
fix(jobs): update config files and clean up PR review feedback
matthewgrossman Jun 15, 2026
aed8e5d
fix(jobs): fix lint errors and re-export FilesetMetadata from types.f…
matthewgrossman Jun 15, 2026
6552ef1
style: format test files
matthewgrossman Jun 15, 2026
ba8ec99
fix(lint): add unused-type-ignore-comment to ty ignore list
matthewgrossman Jun 15, 2026
a4f2afc
fix(tests): add kind to e2e executor dicts, suppress ty invalid-key i…
matthewgrossman Jun 15, 2026
ebee99c
fix(tests): convert e2e tests to subprocess executors, add kind to in…
matthewgrossman Jun 16, 2026
1cbe6a5
style: format test_job_search.py
matthewgrossman Jun 16, 2026
4eae0b1
Merge branch 'main' into AIRCORE-513-align-executorkind-with-original…
matthewgrossman Jun 16, 2026
0ff5d89
feat(jobs): thread resolved kind+profile through compile pipeline
matthewgrossman Jun 16, 2026
ae1b784
fix: resolve merge conflict in e2e/test_jobs.py
matthewgrossman Jun 16, 2026
85c29de
fix(jobs): default kind to "container" in compile signatures
matthewgrossman Jun 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 3 additions & 4 deletions e2e/test_jobs.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,8 @@
"""E2E tests for platform jobs.

These tests submit jobs with CPUExecutionProviderSpec (container + command).
The container image is omitted so that:
- On subprocess mode, the cpu→subprocess translation discards it anyway.
- On Kubernetes/Docker, the execution profile's default_task_image is used.
These tests submit jobs with ContainerExecutionProviderSpec (container + command).
The container image is omitted so that the execution profile's default_task_image
is used on Kubernetes/Docker backends.

Ported from Platform-Deploy e2e/test_jobs.py, adapted for the SDK's TypedDict
param types and filtered to tests that work without Docker.
Expand Down
Loading
Loading