Skip to content
This repository has been archived by the owner on Sep 12, 2024. It is now read-only.

SecondaryAAA setting is problematic #1117

Open
haozturk opened this issue Jan 17, 2023 · 0 comments
Open

SecondaryAAA setting is problematic #1117

haozturk opened this issue Jan 17, 2023 · 0 comments

Comments

@haozturk
Copy link
Collaborator

Impact of the bug
Unified/assignor.py

Describe the bug
SecondaryAAA setting fails if there is a campaign whose secondaryAAA is False and another one True when the former comes after the latter because of this line [1]

[1] https://github.com/CMSCompOps/WmAgentScripts/blob/python3-migration-fast/Unified/assignor.py#L271

How to reproduce it
Clone this workflow and assign it w/ Unified/assignor.py in this version [1]

[1] https://github.com/CMSCompOps/WmAgentScripts/tree/245ad3402dd4883bbc1bc61e138bd8645d8bbbf9

Expected behavior
For a given workflow, if there is a campaign whose secondaryAAA is true, TrustPUSitelists of the request should be True

Additional context and error message
None

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant