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

Invoke the integration process as service. #145

Closed
wants to merge 3 commits into from
Closed

Invoke the integration process as service. #145

wants to merge 3 commits into from

Conversation

nnDarshan
Copy link
Contributor

Reverts #133

@nnDarshan
Copy link
Contributor Author

@r0h4n @nthomas-redhat Please review

shtripat
shtripat reviewed Jan 17, 2017
View reviewed changes
Copy link
Member

@shtripat shtripat left a comment

Choose a reason for hiding this comment

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

Correct the title of the PR. Otherwise looks fine.

@nnDarshan nnDarshan changed the title Revert "Revert "Invoke the integration process as service."" Invoke the integration process as service. Jan 17, 2017
@nnDarshan
Corrected node agent startup issues
d1dd313 
tendrl-bug-id: Tendrl#142

Signed-off-by: Shubhendu <shtripat@redhat.com>
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.6%) to 54.477% when pulling ad1ca4b on nnDarshan:revert-133-revert-117-InvokeIntegrationAsService into 94eb6ce on Tendrl:master.

@nnDarshan
Invoke integration process as service
5d6585e 
tendrl-bug-id: Tendrl#116
Signed-off-by: nnDarshan <darshan.n.2024@gmail.com>
@nnDarshan nnDarshan force-pushed the revert-133-revert-117-InvokeIntegrationAsService branch from ad1ca4b to 5d6585e Compare January 19, 2017 09:33
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.02%) to 54.008% when pulling 5d6585e on nnDarshan:revert-133-revert-117-InvokeIntegrationAsService into 94eb6ce on Tendrl:master.

@nnDarshan
Fix the tendrl-node-agent
b05cd0b 
Signed-off-by: nnDarshan <darshan.n.2024@gmail.com>
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.2%) to 53.801% when pulling b05cd0b on nnDarshan:revert-133-revert-117-InvokeIntegrationAsService into 94eb6ce on Tendrl:master.

@shtripat
Copy link
Member

Some of these changes are done as part of #149. Once we merge that we can rebase this on top of that.

@r0h4n
Copy link
Contributor

r0h4n commented Feb 27, 2017

Let me know if this needs to be re-opened @nnDarshan

@r0h4n r0h4n closed this Feb 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shtripat shtripat shtripat left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nnDarshan @coveralls @shtripat @r0h4n