diff options
author | Ben Nemec <bnemec@redhat.com> | 2020-12-22 19:34:16 +0000 |
---|---|---|
committer | Ben Nemec <bnemec@redhat.com> | 2020-12-22 19:36:14 +0000 |
commit | b8f8b17030e93f6fe64e986ebae1de4011f007d5 (patch) | |
tree | ef572ef10ff6f7b481422c0cc886ee9ff2be7ce4 | |
parent | 54f8724be281b1acfae175d7ec1c14e439269117 (diff) | |
download | oslo-messaging-b8f8b17030e93f6fe64e986ebae1de4011f007d5.tar.gz |
Fix type of direct_mandatory_flag opt
An IntOpt with a default of True is invalid. I'm a little surprised
this doesn't fail a defaults check somewhere, but it needs to be
fixed regardless.
Looking at where it is used, it appears the boolean type is correct.
This just changes the opt type to BoolOpt to match.
Change-Id: I01a38754a31c891f2b3b9c7f8135690693df5d13
Closes-Bug: 1909036
-rw-r--r-- | oslo_messaging/_drivers/impl_rabbit.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/oslo_messaging/_drivers/impl_rabbit.py b/oslo_messaging/_drivers/impl_rabbit.py index f436cbe..756641b 100644 --- a/oslo_messaging/_drivers/impl_rabbit.py +++ b/oslo_messaging/_drivers/impl_rabbit.py @@ -169,12 +169,12 @@ rabbit_opts = [ default=2, help='How often times during the heartbeat_timeout_threshold ' 'we check the heartbeat.'), - cfg.IntOpt('direct_mandatory_flag', - default=True, - help='Enable/Disable the RabbitMQ mandatory flag ' - 'for direct send. The direct send is used as reply, ' - 'so the MessageUndeliverable exception is raised ' - 'in case the client queue does not exist.'), + cfg.BoolOpt('direct_mandatory_flag', + default=True, + help='Enable/Disable the RabbitMQ mandatory flag ' + 'for direct send. The direct send is used as reply, ' + 'so the MessageUndeliverable exception is raised ' + 'in case the client queue does not exist.'), cfg.BoolOpt('enable_cancel_on_failover', default=False, help="Enable x-cancel-on-ha-failover flag so that " |