On 9/9/19 10:57 AM, Colleen Murphy wrote:
On Fri, Sep 6, 2019, at 21:57, Colleen Murphy wrote:
[snipped]
- CI
After skimming the meeting logs I saw the unit test timeout problem was discussed and a temporary workaround was proposed[8]. This sounded like a great idea but it seems that no one implemented it, so I did[9]. Unfortunately this will conflict with all the system-scope/default-roles patches in flight. With how many changes need to go in and how slow it will be with all of them needing to be rechecked and continually making the problem even worse, I propose we go ahead and merge the workaround ASAP and update all the in-flight changes to move the protection tests to the new location.
Alternatively, we can raise the timeouts temporarily as proposed here[11], then merge all the policy changes, then merge the protection test split.
Seems prudent (one rebase vs. many rebases), assuming the "merge all the policy changes" step can be done in a reasonable amount of time.
[snipped]
[8] http://eavesdrop.openstack.org/meetings/keystone/2019/keystone.2019-08-27-16... [9]https://review.opendev.org/680788
[11] https://review.opendev.org/680798
Colleen