publish_errors won't work at all if log notifier is enabled
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| oslo.messaging |
Invalid
|
Medium
|
Tiantian Gao | ||
Bug Description
When the publish_errors option is enabled, it will not work if the log notifier is enabled. The logic should be that this code doesn't run *only* for the log notifier. It should still work for all other notifiers if more than one is configured. Right now if multiple are configured and the log notifier is one of them, this feature doesn't work.
class PublishErrorsHa
def emit(self, record):
if ('openstack.
return
Note that publish_errors is broken for other reasons, too: https:/
| affects: | openstack-common → oslo |
| Mark McLoughlin (markmc) wrote | #1 |
| Changed in oslo: | |
| status: | New → Incomplete |
| status: | Incomplete → Confirmed |
| importance: | Undecided → Medium |
| Changed in oslo: | |
| assignee: | nobody → Jon Grimm (jgrimm) |
| OpenStack Infra (hudson-openstack) wrote Fix proposed to oslo-incubator (master) | #2 |
| Changed in oslo: | |
| status: | Confirmed → In Progress |
| affects: | oslo-incubator → oslo.messaging |
| Changed in oslo.messaging: | |
| assignee: | Jon Grimm (jgrimm) → nobody |
| status: | In Progress → Triaged |
| OpenStack Infra (hudson-openstack) wrote Fix proposed to oslo.messaging (master) | #3 |
| Changed in oslo.messaging: | |
| assignee: | nobody → Tiantian Gao (gtt116) |
| status: | Triaged → In Progress |
| Changed in oslo.messaging: | |
| status: | In Progress → Invalid |
| OpenStack Infra (hudson-openstack) wrote Change abandoned on oslo.messaging (master) | #4 |
For reference, the check was added originally by: https:/