config option help for default value of default_publisher_id is wrong

Bug #1713150 reported by Matt Riedemann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Matt Riedemann

Bug Description

The config option help for "default_publisher_id" says the default value is "$my_ip" but that's not what's actually used:

        notifier = rpc.get_notifier('api', publisher_id=(
            CONF.notifications.default_publisher_id or CONF.host))

and:

    if not publisher_id:
        publisher_id = "%s.%s" % (service, host or CONF.host)

Tags: config
Matt Riedemann (mriedem)
Changed in nova:
status: New → Triaged
tags: added: config
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/498094

Changed in nova:
assignee: nobody → Matt Riedemann (mriedem)
status: Triaged → In Progress
Matt Riedemann (mriedem)
Changed in nova:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/498094
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=27cd4dd70b5a51919ce0091ab4247f65e5a27526
Submitter: Jenkins
Branch: master

commit 27cd4dd70b5a51919ce0091ab4247f65e5a27526
Author: Matt Riedemann <email address hidden>
Date: Fri Aug 25 17:13:48 2017 -0400

    Change default for [notifications]/default_publisher_id to $host

    The default used in code for CONF.notifications.default_publisher_id
    is CONF.host, which is a free-form StrOpt.

    In an upcoming change, the "my_ip" option is going to be changed from
    a StrOpt to an IPOpt, so we made the default value change for
    the default_publisher_id option before that.

    Change-Id: Ia85130f66bd5a60097e8967064559fd0962d06fd
    Closes-Bug: #1713150

Changed in nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 17.0.0.0b1

This issue was fixed in the openstack/nova 17.0.0.0b1 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.