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)
Reviewed: https:/ /review. opendev. org/c/openstack /oslo.messaging /+/896425 /opendev. org/openstack/ oslo.messaging/ commit/ 047da4e5ed8f853 c1e69f73acaa9e6 55c88ab6e7
Committed: https:/
Submitter: "Zuul (22348)"
Branch: stable/yoga
commit 047da4e5ed8f853 c1e69f73acaa9e6 55c88ab6e7
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 cddd7de9c6c788c 0a57c8218a8 29fcfdb74b8b3f6 46d4b60929) 240b6d368ee95ab 9f479d5e02) 9469068d6dd65b0 cff398ba75) f6adb17f99deb71 830837fa97)
Change-Id: I78b08edfcb8115
(cherry picked from commit c1b606f77e69a28
(cherry picked from commit 29623702fc1ebce
(cherry picked from commit ee0bf90b6862c3e
(cherry picked from commit 45a01414689e917