Allow botlist API call to respond to quarantined: and is_dead:

appengine/swarming/swarming_rpcs.py

 # Copyright 2015 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
 """This module defines ProtoRPC types for the Swarming Server handlers."""
 
 from protorpc import message_types
 from protorpc import messages
 
 
@@ skipping 290 matching lines @@
 ### Bot-Related Requests
 
 
 class BotsRequest(messages.Message):
   """Information needed to request bot data."""
   limit = messages.IntegerField(1, default=200)
   cursor = messages.StringField(2)
   # Must be a list of 'key:value' strings to filter the returned list of bots
   # on.
   dimensions = messages.StringField(3, repeated=True)
+  quarantined = messages.StringField(4, default='none')

[M-A Ruel, 2016/08/08 20:28:00]

I was more thinking about tree-state-bool. Since there's none, maybe define one
with
https://cloud.google.com/appengine/docs/python/tools/protorpc/messages/enumfi...
?

class ThreeStateBool(messages.Enum):  
  FALSE = 1
  TRUE = 2
  NONE = 3

I think we may already be using BoolField implicitly this way but I prefer
EnumField to StringField here.

[kjlubick, 2016/08/09 13:27:55]

Done.

+  is_dead = messages.StringField(5, default='none')
 
 
 class BotEventsRequest(messages.Message):
   """Request to get events for a bot."""
   limit = messages.IntegerField(1, default=200)
   cursor = messages.StringField(2)
   # These should be DateTimeField but endpoints + protorpc have trouble encoding
   # this message in a GET request, this is due to DateTimeField's special
   # encoding in protorpc-1.0/protorpc/message_types.py that is bypassed when
   # using endpoints-1.0/endpoints/protojson.py to add GET query parameter
@@ skipping 105 matching lines @@
 
 
 class DeletedResponse(messages.Message):
   """Indicates whether a bot was deleted."""
   deleted = messages.BooleanField(1)
 
 
 class TerminateResponse(messages.Message):
   """Returns the pseudo taskid to wait for the bot to shut down."""
   task_id = messages.StringField(1)