Re: Func-Transmit & Async Call

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Michael DeHaan wrote:
Marco Mornati wrote:
Michael DeHaan wrote:
Marco Mornati wrote:
Anyway... here a little mod to func-transmit to make a call for getting result for provided job

if method=="list_minions":
[...]
elif method == "job_results":
   server_spec = "*"
   overlord = fc.Overlord(server_spec)
(return_code, async_results) = overlord.job_status(params.get('parameters'))
   results = {}
   results[return_code] = async_results
else:
[...]

What you receive is a dictionary with
process_status: [machine: result], [machine: result]

Is not so a brilliant result... but you can read in a bit if there are machines in error or not...

Could be useful?

Bye
Marco
This needs to be submitted using "git-format-patch/git-send-email" in order to take it. Right now this is not in patch format.

Why would "results" not just be "async_results" but has the extra hash key in there? I'd remove that.
Yes I've use a new dictionary, results, to make in the answer alse the status of the job. I see that If I make a call on all my minions I received only one process id that i need to use to check the status of all my minion. So in the answer all minions are divided, using a dict, in the current job status:

Ok, so this doesn't make sense ...

(return_code, async_results) = overlord.job_status(params.get('parameters'))
results = {
  "return_code" : return_code,
  "async_results" : async_results
}

That makes more sense.


Ok... better this way...
making the mods and send the patch...

Marco

--
Dott. Ing. Mornati Marco
Byte-Code s.r.l
via Antonio Cechov, 1
San Giuliano Milanese (MI)
E-Mail: mmornati@xxxxxxxxxxxxx


_______________________________________________
Func-list mailing list
Func-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/func-list

[Index of Archives]     [Fedora Users]     [Linux Networking]     [Fedora Legacy List]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux