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

Reading jobs from etcd doesnt adhere to the api job structure #40

Closed
shtripat opened this issue Nov 7, 2016 · 4 comments
Closed

Reading jobs from etcd doesnt adhere to the api job structure #40

shtripat opened this issue Nov 7, 2016 · 4 comments
Assignees

Comments

@shtripat
Copy link
Member

shtripat commented Nov 7, 2016

The new job structure is as per https://github.com/Tendrl/bridge_common/blob/master/etc/samples/tendrl_api_job.sample.json.

Based on this rpc.py which reads and processes the jobs from etcd job queue should be modified to figure out the actual flow name and invoke the same.

@shtripat
Copy link
Member Author

shtripat commented Nov 7, 2016

I strongly feel we should refer a flow only from API job which can internally invoke multiple atoms based on its need. Flows can be more flexibly defined with different combinations of atoms for different api needs.

@sankarshanmukhopadhyay
Copy link
Contributor

This has been open and under consideration for a month (ish) now - what's the decision? @r0h4n

@shtripat
Copy link
Member Author

This is done in previous hackthon. @r0h4n can you please confirm and close this?

@r0h4n
Copy link
Contributor

r0h4n commented Dec 13, 2016

This one has gone through many iterations and is stable now. More details https://github.com/Tendrl/common/blob/master/etc/samples/tendrl_api_job.sample.json

@r0h4n r0h4n closed this as completed Dec 13, 2016
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

No branches or pull requests

3 participants