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

K8 update #972

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

K8 update #972

wants to merge 11 commits into from

Conversation

ct-clmsn
Copy link
Contributor

@ct-clmsn ct-clmsn commented May 4, 2019

added kubernetes remote deployment scripts.

@hkaiser hkaiser added this to the 0.0.1 milestone May 4, 2019
hkaiser
hkaiser reviewed May 4, 2019
View reviewed changes
python/phylanx/ast/physl.py Outdated Show resolved Hide resolved
hkaiser
hkaiser reviewed May 4, 2019
View reviewed changes
Copy link
Member

@hkaiser hkaiser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks a lot!

@ct-clmsn
Copy link
Contributor Author

ct-clmsn commented May 5, 2019 •
edited by parsa
Loading

@hkaiser and now to finalize RandomForest! I will add an issue that needs to address recovering output from the remote system. I need to return a data structure similar to a Python version of a future; meaning when a user deploys code to the remote system, a .get() should be provided to collect results. This should be fairly straightforward; it will require polling on the remote host's Kubernetes server's log to get the remote Phylanx process's stdout.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hkaiser hkaiser hkaiser left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
type: compatibility issue type: enhancement
Projects
None yet
Milestone
0.0.1
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ct-clmsn @hkaiser