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.

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

Managed Yarn AppMaster & ClusterManager. Credentials and ContainerId implementations. #46

Merged
merged 3 commits into from
Jan 7, 2020

Conversation

tanmaykm
Copy link
Member

@tanmaykm tanmaykm commented Jan 7, 2020

  • Implement Credentials with ability to interpret stored tokens. This is how we can pick up credentials from locations stored in to by Yarn.
  • Implement ContainerId. Use this to interpret IDs of containers, apps and attempts provided by Yarn. We do not need to serialize it via JULIA_YARN_CID any more.
  • Implement managed Yarn AppMaster
  • Update Yarn ClusterManager to work in managed mode.

Running as a managed app master would allow Julia Applications to be run completely inside Yarn, unlike the case of unmanaged app master where the master process hangs outside the cluster. The submitting process can terminate safely, leaving the submited Julia process running inside the cluster. The Julia process running inside the Yarn cluster can then start other worker processes as it desires - all running inside Yarn. Status of the submitted application can be queried anytime, or viewed on the Yarn status board.

implement Credentials with ability to interpret stored tokens
- add method to extract containerid and appid from environment set by yarn
- now that we can uderstand the container id that yarn sets up (above), use that instead of serializing via JULIA_YARN_CID
- allow multiple args to be specified for launch command as a vector of strings, useful in composing command line for execution
@coveralls
Copy link

coveralls commented Jan 7, 2020

Coverage Status

Coverage increased (+4.5%) to 54.34% when pulling c52f95c on tan/yarn into 57579d0 on master.

@tanmaykm tanmaykm merged commit 9aa868b into master Jan 7, 2020
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