Comment 8 for bug 2037312

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

Reviewed: https://review.opendev.org/c/openstack/oslo.messaging/+/896426
Committed: https://opendev.org/openstack/oslo.messaging/commit/d1c52bab523e151f9e94ee6bed27c964d7f57a18
Submitter: "Zuul (22348)"
Branch: stable/xena

commit d1c52bab523e151f9e94ee6bed27c964d7f57a18
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
    (cherry picked from commit c1b606f77e69a2829fcfdb74b8b3f646d4b60929)
    (cherry picked from commit 29623702fc1ebce240b6d368ee95ab9f479d5e02)
    (cherry picked from commit ee0bf90b6862c3e9469068d6dd65b0cff398ba75)
    (cherry picked from commit 45a01414689e917f6adb17f99deb71830837fa97)
    (cherry picked from commit 047da4e5ed8f853c1e69f73acaa9e655c88ab6e7)