Comment 2 for bug 2037312

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.messaging (master)

Reviewed: https://review.opendev.org/c/openstack/oslo.messaging/+/896451
Committed: https://opendev.org/openstack/oslo.messaging/commit/c1b606f77e69a2829fcfdb74b8b3f646d4b60929
Submitter: "Zuul (22348)"
Branch: master

commit c1b606f77e69a2829fcfdb74b8b3f646d4b60929
Author: Jay Faulkner <email address hidden>
Date: Mon Sep 25 10:28:48 2023 -0700

    Add is_admin to safe fields list for notifications

    We encountered bug 2037312 in unit tests when attempting to get this
    change rolled out. Heat apparently will attempt to set is_admin using
    policy logic if it's not passed in for a new context; this breaks as the
    context we are requested doesn't have all the needed information to
    exercise the policy logic.

    is_admin is just a bool; it's not sensitive; easiest route forward is to
    add it to the safe list

    Closes-bug: 2037312
    Change-Id: I78b08edfcb8115cddd7de9c6c788c0a57c8218a8