Skip to content

K8S: fix decorator attribute handling - #753

Merged
savingoyal merged 1 commit into
Netflix:kubernetes-prfrom
oavdeev:fix-k8s-attr-bug
Oct 12, 2021
Merged

savingoyal merged 1 commit into
Netflix:kubernetes-prfrom
oavdeev:fix-k8s-attr-bug

Conversation

@oavdeev

@oavdeev oavdeev commented Oct 12, 2021

Copy link
Copy Markdown
Collaborator

No description provided.

@savingoyal
savingoyal merged commit 987a6ee into Netflix:kubernetes-pr Oct 12, 2021
@oavdeev
oavdeev deleted the fix-k8s-attr-bug branch October 12, 2021 21:42
savingoyal added a commit that referenced this pull request Oct 15, 2021
* Refactor @resources decorator

@resources decorator is shared by all compute related decorators -
@Batch, @lambda, @K8s, @titus. This patch moves it out of
batch_decorator.py so that other decorators can cleanly reference
it.

* Update __init__.py

* Refactor @Batch decorator

* more change

* more changes

* more changes

* @kubernetes

* Kubernetes

* More changes

* More changes

* more changes

* some more changes

* more changes

* add disk space

* Add todos

* some fixes

* add k8s testing context

* more changes

* some more changes

* minor fixups

* better error handling for evicted pods (#711)

* fixes for pod/job metadata race conditions (#704)

* K8S: label value sanitizer (#719)

* rename name_space to namespace for k8s plugin (#750)

* fix k8s attribute handling bug (#753)

* tweak k8s test resources (to run on kind) (#754)

* add k8s api retries (#756)

* update done marker

* Use linux binaries in @conda when run in k8s (#758)

Conda environment should pack linux python binary when run on MacOS
to avoid an error

metaflow_PlayListFlow_osx-64_179c56284704ca8e53622f848a3df27cdd1f4327/bin/python: cannot execute binary file: Exec format error

* fix comment

* fix merge conflict

* update char

Co-authored-by: Oleg Avdeev <oleg.v.avdeev@gmail.com>
Co-authored-by: Roman Kindruk <36699371+sappier@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants