Skip to content

Commit 26bd94b

Browse files
author
Ghanshyam
committed
Revert "Add enforce_scope setting support for keystone"
This reverts commit 9dc2b88. Reason for revert: Devstack creation/setup the things are not yet moved to scope tokens so we need to wait for that first and then do the scope check enable globally. Change-Id: If0368aca39c1325bf90abd23831118b89e746222
1 parent 971dfbf commit 26bd94b

2 files changed

Lines changed: 0 additions & 20 deletions

File tree

lib/keystone

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -134,12 +134,6 @@ KEYSTONE_PASSWORD_HASH_ROUNDS=${KEYSTONE_PASSWORD_HASH_ROUNDS:-4}
134134
# Cache settings
135135
KEYSTONE_ENABLE_CACHE=${KEYSTONE_ENABLE_CACHE:-True}
136136

137-
# Flag to set the oslo_policy.enforce_scope. This is used to switch
138-
# the Identity API policies to start checking the scope of token. By Default,
139-
# this flag is False.
140-
# For more detail: https://docs.openstack.org/oslo.policy/latest/configuration/index.html#oslo_policy.enforce_scope
141-
KEYSTONE_ENFORCE_SCOPE=$(trueorfalse False KEYSTONE_ENFORCE_SCOPE)
142-
143137
# Functions
144138
# ---------
145139

@@ -287,11 +281,6 @@ function configure_keystone {
287281
iniset $KEYSTONE_CONF security_compliance lockout_duration $KEYSTONE_LOCKOUT_DURATION
288282
iniset $KEYSTONE_CONF security_compliance unique_last_password_count $KEYSTONE_UNIQUE_LAST_PASSWORD_COUNT
289283
fi
290-
if [[ "$KEYSTONE_ENFORCE_SCOPE" == True ]] ; then
291-
iniset $KEYSTONE_CONF oslo_policy enforce_scope true
292-
iniset $KEYSTONE_CONF oslo_policy enforce_new_defaults true
293-
iniset $KEYSTONE_CONF oslo_policy policy_file policy.yaml
294-
fi
295284
}
296285

297286
# create_keystone_accounts() - Sets up common required keystone accounts

lib/tempest

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -600,15 +600,6 @@ function configure_tempest {
600600
fi
601601
done
602602

603-
# ``enforce_scope``
604-
# If services enable the enforce_scope for their policy
605-
# we need to enable the same on Tempest side so that
606-
# test can be run with scoped token.
607-
if [[ "$KEYSTONE_ENFORCE_SCOPE" == True ]] ; then
608-
iniset $TEMPEST_CONFIG enforce_scope keystone true
609-
iniset $TEMPEST_CONFIG auth admin_system 'all'
610-
iniset $TEMPEST_CONFIG auth admin_project_name ''
611-
fi
612603
iniset $TEMPEST_CONFIG enforce_scope glance "$GLANCE_ENFORCE_SCOPE"
613604

614605
iniset $TEMPEST_CONFIG enforce_scope cinder "$CINDER_ENFORCE_SCOPE"

0 commit comments

Comments
 (0)