search

quattor / ncm-cdispd

Node Configuration Manager Configuration Dispatch Daemon
www.quattor.org
Other
2 stars 5 forks source link

ncm-cdispd code cleanup #11

Closed stdweird closed 10 years ago

stdweird commented 10 years ago

This is basically #9, with last 2 commits reset and new commit that uses the CAF::Process->stream_output to stream the ncm-ncd output (as opposed to gathering it and then re-log via splitting and throttling).

hpcugentbot commented 10 years ago

Merged build triggered.

hpcugentbot commented 10 years ago

Merged build started.

hpcugentbot commented 10 years ago

Merged build finished.

jrha commented 10 years ago

@stdweird ping?

stdweird commented 10 years ago

@jrha will do asap, but 2 lines above there's a >>8 that probably needs changing as well seen recent discussion in some other PR

jrha commented 10 years ago

Ok, cool :smile:

stdweird commented 10 years ago

@jrha done, but last commit modifies the way success is considered @ned21 comments on using the perl exitcode instead of the bitshifted one for the $result? (at least now it's consitent with the debug message that noone looks at :smile:)