Merge lp:~james-w/python-oops-dictconfig/bool-new-only into lp:python-oops-dictconfig

[James Westby]

Make new_only a BoolOption.

Hi,

This makes new_only a BoolOption so that it can be set correctly with configglue.

Thanks,

James

[Anthony Lenton (elachuni)]

Thanks!

review: Approve

[lp:~james-w/python-oops-dictconfig/bool-new-only]

16. By James Westby on 2013-02-26

Add inherit_id.

=== modified file 'oops_dictconfig/configglue_options.py'
--- oops_dictconfig/configglue_options.py	2013-02-06 17:00:45 +0000
+++ oops_dictconfig/configglue_options.py	2013-02-26 16:17:20 +0000
@@ -24,7 +24,7 @@
                 type=schema.StringOption(
                     help="Type of publisher; amqp or datedir are supported.",
                     ),
-                new_only=schema.StringOption(
+                new_only=schema.BoolOption(
                     help="Only publish if a previous publisher hasn't.",
                     ),
                 error_dir=schema.StringOption(
@@ -49,6 +49,9 @@
                 routing_key=schema.StringOption(
                     help=("For amqp: the routing key for the amqp broker."),
                     ),
+                inherit_id=schema.BoolOption(
+                    help="Re-use any oops id that is already present in the oops.",
+                    ),
                 )
         if 'help' not in kwargs:
             kwargs['help'] = "Config for an oops publisher."