Keystone performance degradation in 9.1 MU

Bug #1628534 reported by Pavel
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
Critical
Unassigned

Bug Description

Looks like we got Keystone's performance degradation in 9.1 MU.

Here you can see fresh test results for 9.0 (build 945)
---------------------------
<substitute this part with TestRail host address>/index.php?/runs/view/22023&group_by=cases:section_id&group_order=asc
---------------------------
Test run was performed on ENV-14 Lab of Performance QA Team.

And here you can see fresh test results for 9.1 RC2 (snapshot 315):
---------------------------
- short run: <substitute this part with TestRail host address>/index.php?/runs/view/22009&group_by=cases:section_id&group_order=asc
- long run: <substitute this part with TestRail host address>/index.php?/runs/view/22004&group_by=cases:section_id&group_order=asc
---------------------------
As you can see we have about 30% degradation for both authentication and validation time in compare to 9.0 release.
The results are almost the same for both runs. Test runs were performed on ENV-12 Lab of Performance QA Team.

The same tests were also performed nearly 2 weeks ago for the earlier 9.1 snapshot (form 14.09.2016-16:43:22)
---------------------------
<substitute this part with TestRail host address>/index.php?/runs/view/20843&group_by=cases:section_id&group_order=asc
---------------------------
No performance regression as you can see.
Test run was performed on ENV-13 Lab of Performance QA Team.

ENV-12, ENV-14 and ENV-13 have the same hardware and configuraton of controlers. Hardware description can be found here:
---------------------------
<substitute this part with JIRA host address>/wiki/display/QA/Scale+lab+HW
---------------------------

A bit later I'll provide results of test run for 9.1 on ENV-14 and comparative stats of hardware resources consuming for 9.0 and 9.1 during Keystone tests were run.

Changed in mos:
milestone: none → 9.1
importance: Undecided → High
assignee: nobody → MOS Keystone (mos-keystone)
Pavel (p.petrov)
description: updated
description: updated
description: updated
description: updated
description: updated
description: updated
description: updated
Pavel (p.petrov)
description: updated
Revision history for this message
Boris Bobrov (bbobrov) wrote :
Revision history for this message
Pavel (p.petrov) wrote :
Pavel (p.petrov)
description: updated
description: updated
Revision history for this message
Boris Bobrov (bbobrov) wrote :

The only patch that could affect us is https://review.fuel-infra.org/#/c/26523/. Pavel, could you please run the performance tests with the patch reverted? I have prepared the revert: https://review.fuel-infra.org/#/c/26985/.

Changed in mos:
assignee: MOS Keystone (mos-keystone) → Pavel (p.petrov)
Changed in mos:
importance: High → Critical
Pavel (p.petrov)
description: updated
Changed in mos:
status: New → Confirmed
Revision history for this message
Boris Bobrov (bbobrov) wrote :

On the env provided by Pavel we have reverted both patches merged in the last 2 weeks, https://review.fuel-infra.org/26664 and https://review.fuel-infra.org/26523. It has not fixed the issue, which probably means that these 2 patches are not the root cause.

Revision history for this message
Boris Bobrov (bbobrov) wrote :

I have inserted some debug prints to keystone and we ran the tests. From the logs from keystone perspective 90-percentile is 0.255000s. Which is much lower than what Pavel's tool have.

Because of that, i will stop debugging keystone. Instead, we shall bisect 9.1 build to find the first bad build.

Boris Bobrov (bbobrov)
Changed in mos:
assignee: Pavel (p.petrov) → Boris Bobrov (bbobrov)
Revision history for this message
Boris Bobrov (bbobrov) wrote :
Changed in mos:
status: Confirmed → In Progress
Changed in mos:
status: In Progress → Fix Committed
Boris Bobrov (bbobrov)
Changed in mos:
status: Fix Committed → In Progress
Revision history for this message
Max Yatsenko (myatsenko) wrote :

There are patched that change number of threads of keystone to 1 :

https://review.openstack.org/#/c/381032/
https://review.openstack.org/#/c/381034/

Revision history for this message
Boris Bobrov (bbobrov) wrote :

All patches were merged

Changed in mos:
status: In Progress → Fix Committed
Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

Performance team verified the fix on MOS 9.1 RC3.

Changed in mos:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 10.0.0rc1

This issue was fixed in the openstack/fuel-library 10.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 10.0.0

This issue was fixed in the openstack/fuel-library 10.0.0 release.

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/keystone (mcp/newton)

Fix proposed to branch: mcp/newton
Change author: Boris Bobrov <email address hidden>
Review: https://review.fuel-infra.org/33560

Boris Bobrov (bbobrov)
Changed in mos:
assignee: Boris Bobrov (bbobrov) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.