Commit 8f07dc0d authored by Michael Hanselmann's avatar Michael Hanselmann
Browse files

watcher: Separate function for writing instance status file

For now this will do another query to the master daemon, but with the
split for node groups this issue will go away.
Signed-off-by: default avatarMichael Hanselmann <>
Reviewed-by: default avatarIustin Pop <>
parent d7c42723
......@@ -166,9 +166,7 @@ def GetClusterData():
instances = {}
# write the instance status file
up_data = "".join(["%s %s\n" % (fields[0], fields[1]) for fields in result])
utils.WriteFile(file_name=constants.INSTANCE_STATUS_FILE, data=up_data)
_UpdateInstanceStatus(client, constants.INSTANCE_STATUS_FILE)
for fields in result:
(name, status, autostart, snodes) = fields
......@@ -437,6 +435,25 @@ def ParseOptions():
return (options, args)
def _UpdateInstanceStatus(cl, filename):
"""Get a list of instances on this cluster.
@todo: Think about doing this per nodegroup, too
op = opcodes.OpInstanceQuery(output_fields=["name", "status"], names=[],
job_id = cl.SubmitJob([op])
(result, ) = cli.PollJob(job_id, cl=cl, feedback_fn=logging.debug)
logging.debug("Got instance data, writing status file %s", filename)
utils.WriteFile(filename, data="".join("%s %s\n" % (name, status)
for (name, status) in result))
def Main():
"""Main function.
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment