Skip to content
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

Revert 1 k8s improvements #40

Closed
wants to merge 121 commits into from
Closed

Revert 1 k8s improvements #40

wants to merge 121 commits into from

Conversation

ricomnl
Copy link
Contributor

@ricomnl ricomnl commented Jun 17, 2022

No description provided.

David Konerding and others added 30 commits November 29, 2021 22:32
initial checkin of fake k8s executor that does nothing
475da95
move some ECR methods which aren't used to aws_utils.
86934d4
Continue refactoring to enable k8s executor.
13590c6
intermediate progress on k8s
540ecf1
Merge branch 'main' into k8s
d016efd
return api response in create_job and use it to check job status
2a44753
snapshot of continued work on k8s
c4ea02e
enable k8s task
9e89532
another snapshot of k8s work.
5f68289
add eval_hash to job name
fe484c4
continue wiring up job apis
2a8f9a7
monitor job improvements for example
98b330d
add some refinements to k8s example code
4bb7079
sync point- working k8s job works
17e39c2
Return either a batch or core client API
8b57a1c
clean up workflow a bit
f00f05e
finish up basic working example
9d7b9b6
Refactor aws utils tests.
1ac8753
fix docker local job status
962c285
add first passing test.
633694c
minor test improvements
5bae273
@dakoner
keep working on tests.
98b84e8
@dakoner
intermediate work
5bbaa62
working test!
84ecb5b
in progress
b91374d
@dakoner
A bundle of working tests
7ad8f3f
@dakoner
more tests
5758204
@dakoner
intermediate work
b7ba26b
cleanup
040f3d1
make script simpler.
6656940
mattrasmus and others added 28 commits May 21, 2022 11:13
@mattrasmus
fix tests
61246f4
@mattrasmus
fix types and refactor
69d4188
@mattrasmus
Merge branch 'main' into k8s-rasmus
5994362
@mattrasmus
working on k8s executor
83787f3
@mattrasmus
add pod-level error handling
accfe70
@mattrasmus
remove parse_task_error
f8254f9
@mattrasmus
refactor k8s monitor loop
1632289
@mattrasmus
update k8s example
deba410
@mattrasmus
fix tests
b06342b
@mattrasmus
add more files for k8s example
5a653e6
@mattrasmus
remove breakpoint
abb8ad2
@ricomnl
Update docker.py
2b301d7
@ricomnl
Merge branch 'insitro:main' into main
2f3c319
@ricomnl
Update docker.py
82b26ae
Merge branch 'main' of https://github.com/ricomnl/redun
260948a
Merge remote-tracking branch 'origin/k8s-rasmus'
4ea4f21
@ricomnl
Update k8s_utils.py
0052abc
@ricomnl
Update k8s_utils.py
44fa692
Enabled incluster config retrieval
0df73ec
@ricomnl
Update file.py
bd521b6
Added support for gsutil during file staging
cb121db
Merge branch 'main' of https://github.com/ricomnl/redun
707bc4b
Added support for gsutil during file staging
dab263f
Support for pod annotations through job options
2594266
Support for pod annotations through job options, Added support for gs…
acea9bf 
…util during file staging, Enabled incluster config retrieval
@ricomnl
Merge branch 'main' into k8s_improvements
889016a
@ricomnl
Merge pull request #1 from ricomnl/k8s_improvements
3acff18 
Improvements to K8sExecutor and small fixes
@ricomnl
Revert "Improvements to K8sExecutor and small fixes"
92cf4b4
@ricomnl ricomnl closed this Jun 17, 2022
@ricomnl ricomnl deleted the revert-1-k8s_improvements branch June 17, 2022 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ricomnl @dakoner @mattrasmus