- Adds flow utils for job status updates #115

r0h4n · 2017-01-17T07:47:57Z

Changes etcd_client to etcd_orm
etcd client can be found at "etcd_orm.client"

- Changes etcd_client to etcd_orm - etcd client can be found at "etcd_orm.client"

coveralls · 2017-01-17T08:19:29Z

Coverage increased (+0.2%) to 59.608% when pulling f18b3bd on r0h4n:master into 3f61eee on Tendrl:master.

shtripat · 2017-01-17T08:49:14Z

tendrl/commons/flows/base_flow.py

@@ -54,17 +56,8 @@ def run(self):
        # TODO(rohan) make a func outta the below logging common code
        msg = "Processing pre-runs for flow: %s" % self.job['run']
        LOG.info(msg)


Line no 67, 88, 100, 120 are setting the log messages but they are not written immediately to etcd. Why so?

Rohan Kanade added 3 commits January 17, 2017 13:17

- Adds flow utils for job status updates

d656f0d

- Changes etcd_client to etcd_orm - etcd client can be found at "etcd_orm.client"

- Adds flow utils for job status updates

345144d

- Changes etcd_client to etcd_orm - etcd client can be found at "etcd_orm.client"

- Adds flow utils for job status updates

f18b3bd

- Changes etcd_client to etcd_orm - etcd client can be found at "etcd_orm.client"

r0h4n merged commit 1f4bb6b into Tendrl:master Jan 17, 2017

shtripat reviewed Jan 17, 2017

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

- Adds flow utils for job status updates #115

- Adds flow utils for job status updates #115

r0h4n commented Jan 17, 2017

coveralls commented Jan 17, 2017

shtripat Jan 17, 2017

- Adds flow utils for job status updates #115

- Adds flow utils for job status updates #115

Conversation

r0h4n commented Jan 17, 2017

coveralls commented Jan 17, 2017

shtripat Jan 17, 2017

Choose a reason for hiding this comment