On Fri, Dec 10, 2010 at 5:34 PM, Michael Hanselmann <[email protected]> wrote:
> ---
>  lib/constants.py      |    4 ++++
>  lib/luxi.py           |   31 +++++++++++++++++++++++++++++++
>  lib/server/masterd.py |   29 +++++++++++++++++++++++++++++
>  3 files changed, 64 insertions(+), 0 deletions(-)
>
> diff --git a/lib/constants.py b/lib/constants.py
> index eae9ce6..acd4690 100644
> --- a/lib/constants.py
> +++ b/lib/constants.py
> @@ -947,6 +947,10 @@ QR_NODE = "node"
>  #: List of resources which can be queried using L{opcodes.OpQuery}
>  QR_OP_QUERY = frozenset([QR_INSTANCE, QR_NODE])
>
> +#: List of resources which can be queried using LUXI
> +QR_OP_LUXI = QR_OP_QUERY.union(frozenset([
> +  ]))
> +

Wrong nesting? Shouldn't be the frozenset the outermost call? You want
to have a frozenset after all modifications, wont you?

Rest LGTM.

Reply via email to