)]}'
{"id":"openstack%2Fpuppet-keystone~711762","triplet_id":"openstack%2Fpuppet-keystone~stable%2Ftrain~Icf9acaa106af705fa249a2ef2abca9f9a91fba59","project":"openstack/puppet-keystone","branch":"stable/train","hashtags":[],"change_id":"Icf9acaa106af705fa249a2ef2abca9f9a91fba59","subject":"Fix performance regression due to reduced number of keystone workers","status":"MERGED","created":"2020-03-06 20:20:54.000000000","updated":"2020-03-10 17:25:10.000000000","submitted":"2020-03-10 17:22:26.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"711762-1583860946156-bd11f162","meta_rev_id":"684ad8e44f3f38f59085bb1ef1c69c896fb6242c","_number":711762,"virtual_id_number":711762,"owner":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},{"value":0,"date":"2020-03-07 16:21:43.000000000","_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-03-10 17:22:25.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"all":[{"value":2,"date":"2020-03-09 14:54:35.000000000","_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},{"value":0,"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},{"value":2,"date":"2020-03-10 15:48:55.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"all":[{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"date":"2020-03-06 20:20:54.000000000","_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},{"value":0,"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},{"value":1,"date":"2020-03-10 15:48:55.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"date":"2020-03-06 20:20:54.000000000","_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-03-06 20:20:54.000000000","updated_by":{"_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},"reviewer":{"_account_id":9954,"name":"Lance Bragstad","username":"lbragstad","inactive":true},"state":"REVIEWER"},{"updated":"2020-03-06 20:20:54.000000000","updated_by":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"reviewer":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"state":"REVIEWER"},{"updated":"2020-03-07 16:21:43.000000000","updated_by":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"reviewer":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"state":"REVIEWER"},{"updated":"2020-03-09 14:54:35.000000000","updated_by":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"reviewer":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"state":"REVIEWER"},{"updated":"2020-03-10 15:48:55.000000000","updated_by":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"reviewer":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"state":"REVIEWER"},{"updated":"2020-03-10 17:22:25.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"ac194ddd470c841b3244e90169276a13032d84d6","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-03-06 20:20:54.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"73d0c26d8b7b1cd5ae5d536038856cb8055d1324","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-03-06 21:40:01.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/cfb4f9577c774313922cc7936477ecae : SUCCESS in 6m 26s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/2621b36ea7fc42bab0211bcfd17f06d4 : SUCCESS in 6m 18s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/da34ba6d33ce4151a193579654de6b57 : SUCCESS in 6m 45s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/eb42206478344065a5b7919de1049955 : SUCCESS in 23m 56s\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/ac73ae0d931148e5968a129935eb839e : SUCCESS in 52m 19s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/2aa7b025fc154d859d21488906e21b7c : SUCCESS in 55m 36s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/963c150a9eab4f8b80d95ed2f22e6a7d : SUCCESS in 59m 01s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/81b99a5569204d358f4e6e5d91a6e397 : SUCCESS in 46m 07s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/7b54a72c9ba144acbbf2a7e8e4069c29 : SUCCESS in 47m 22s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/f8b3518f02344cd59748e9a8e8684ea0 : SUCCESS in 48m 12s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/2f0c4e3ced4d43c882283a629e3a87aa : SUCCESS in 40m 59s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/40071511f56541cc92ede09dcd43bd99 : SUCCESS in 36m 16s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/910e59ea373c4c469cbfda8a7aee8896 : SUCCESS in 37m 23s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/520b30feec93465aaf88f136010edaf6 : FAILURE in 59m 24s (non-voting)\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/ea64b16dc5b74adaa7f4ccbed0c74fe9 : SUCCESS in 5m 37s\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/a192d8ab1e384ac78986bf7e5cab529b : SUCCESS in 1h 17m 37s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"ca50d90d194e01538de1974bd0b03f8baf08651d","author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"date":"2020-03-07 16:21:43.000000000","message":"Patch Set 1:\n\nIMO this is a behavior change in a stable release that will cause a restart of keystone","accounts_in_message":[],"_revision_number":1},{"id":"2be1ed41d40a2255e1316dda0fac256e93000ca3","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2020-03-09 14:54:35.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"1d959f25816ccb3f0286875bb28e07ac89cabf01","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-03-10 15:48:55.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"968ea101a6eb6df336c9844eaead27462e92e42d","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-03-10 15:49:07.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"c81af1152c594c708bae87494f2cc01e368c0729","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-03-10 17:22:25.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/2e0226803fe64c21a845c897040b2026 : SUCCESS in 5m 35s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/dc1c77b49401425eb893349c98b17e40 : SUCCESS in 5m 02s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/9f64fdcb6db9445292ba7de9627e4211 : SUCCESS in 4m 51s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/d2b0623201c04b03be01732064ebb308 : SUCCESS in 23m 04s\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/6315098cddbe4219b4a60da41a3ef6cf : SUCCESS in 1h 04m 01s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/c3bd8092c40c4c9587f9274c4b8e431d : SUCCESS in 1h 06m 35s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/e454932d4b0f4c46b71aa643eeab1169 : SUCCESS in 1h 06m 23s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/eef4b9c52798443b95c8054642925341 : SUCCESS in 1h 02m 49s\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/68aa07d4182646e1b024854bff78eab1 : SUCCESS in 39m 35s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/66dfec2ed17b400c9098480d60b3244f : SUCCESS in 4m 01s\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/35d69fc1f7004bb1838e2df45d9bd369 : SUCCESS in 1h 30m 59s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"4baba696d0a30e3650a70ca3abfaa7d54b66965d","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-03-10 17:22:26.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":1},{"id":"684ad8e44f3f38f59085bb1ef1c69c896fb6242c","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-03-10 17:25:10.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/2bf0377b34a041f1bdebd8f7871f5d5b : SUCCESS in 2m 21s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"8a3172e5a6ffd9694ad4c3ea9ba5f86c95965792","revisions":{"8a3172e5a6ffd9694ad4c3ea9ba5f86c95965792":{"kind":"REWORK","_number":1,"created":"2020-03-06 20:20:54.000000000","uploader":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"ref":"refs/changes/62/711762/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/62/711762/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/62/711762/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/62/711762/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/62/711762/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/62/711762/1"}}},"commit":{"parents":[{"commit":"e57542a415024de86df5445a955989be839f04c4","subject":"New keystone::resource::authtoken::interface parameter","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/e57542a415024de86df5445a955989be839f04c4"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Lance Bragstad","email":"lbragstad@gmail.com","date":"2020-03-06 20:20:45.000000000","tz":-360},"subject":"Fix performance regression due to reduced number of keystone workers","message":"Fix performance regression due to reduced number of keystone workers\n\nThe puppet-keystone project consolidated keystone applications:\n\n  https://github.com/openstack/puppet-keystone/commit/ace7aeb3b71b39a59f92fbc9e7f676a70c9a797a#diff-e5968f40345cf5d3be0539fbba87f787\n\nThis effectively reduces the number of available workers to process\nkeystone requests since only half the processes are deployed now that\nthe applications are consolidated [0]. As a result, we see a performance\nregression [1] when interacting with keystone.\n\nFor example, in earlier versions, an 8 core machine would spawn four\nprocesses to serve the keystone-main application. Another four processes\nwould be spawned to serve the keystone-admin application. Now, we only\nhave 4 processes total to serve the entire keystone application. Due to\nthe difference in the number of workers deployed by default, users will\nexperience reduced performance out-of-the-box.\n\nThis patch not only brings performance back to the previous levels, it\nis also safe to implement as we are not changing the number of workers\nwhen compared to previous releases and only bringing parity.\n\n[0] https://github.com/openstack/puppet-openstacklib/blob/master/lib/facter/os_workers.rb#L33\n[1] https://bugzilla.redhat.com/show_bug.cgi?id\u003d1789495\n\nCo-Authored-By: Takashi Kajinami \u003ctkajinam@redhat.com\u003e\nDepends-on: https://review.opendev.org/#/c/705041/\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n(cherry picked from commit 8f5301d20d73a84cc10ea2591414e45cf3d1fa79)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/8a3172e5a6ffd9694ad4c3ea9ba5f86c95965792"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/8a3172e5a6ffd9694ad4c3ea9ba5f86c95965792"}]},"branch":"refs/heads/stable/train"}},"requirements":[],"submit_records":[],"submit_requirements":[]}