)]}'
{"id":"openstack%2Fpuppet-keystone~702031","triplet_id":"openstack%2Fpuppet-keystone~master~Icf9acaa106af705fa249a2ef2abca9f9a91fba59","project":"openstack/puppet-keystone","branch":"master","hashtags":[],"change_id":"Icf9acaa106af705fa249a2ef2abca9f9a91fba59","subject":"Fix performance regression due to reduced number of keystone workers","status":"MERGED","created":"2020-01-10 19:44:55.000000000","updated":"2020-02-19 18:45:16.000000000","submitted":"2020-02-19 18:43:24.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":21,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"702031-1582137804764-caee6255","meta_rev_id":"6db282de5d200afcba5dad977d63a20613582b3f","_number":702031,"virtual_id_number":702031,"owner":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"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,"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"value":0,"date":"2020-02-01 13:52:56.000000000","_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"value":0,"_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"},{"value":0,"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2020-02-19 18:43:24.000000000","post_submit":true,"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"},{"value":0,"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"}],"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":0,"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"value":2,"date":"2020-02-10 17:07:30.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":1,"date":"2020-02-19 16:48:33.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"value":0,"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"value":0,"date":"2020-02-03 18:54:18.000000000","permitted_voting_range":{"min":0,"max":2},"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},{"value":2,"date":"2020-02-07 22:16:22.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":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},{"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"},{"value":0,"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"}],"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,"date":"2020-02-03 13:38:41.000000000","_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"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,"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"value":0,"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"value":0,"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},{"value":1,"date":"2020-02-19 17:04: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":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},{"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"},{"value":0,"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"}],"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":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},{"_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"]},{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-01-21 14:24:56.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-01-21 15:33:12.000000000","updated_by":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"reviewer":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"state":"REVIEWER"},{"updated":"2020-01-29 07:52:13.000000000","updated_by":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"reviewer":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"state":"REVIEWER"},{"updated":"2020-02-01 13:52:56.000000000","updated_by":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"reviewer":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"state":"REVIEWER"},{"updated":"2020-02-03 13:38:41.000000000","updated_by":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"reviewer":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"state":"REVIEWER"},{"updated":"2020-02-03 18:54:18.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-02-10 17:07:30.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-02-19 16:48:33.000000000","updated_by":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"reviewer":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"state":"REVIEWER"},{"updated":"2020-02-19 17:04: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-02-19 18:43:24.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":"d99ce1cf16e8630c0d9774834d284c1b0613c3dd","author":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"date":"2020-01-10 19:44:55.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"b7add4db1439a65275e6f2ec50da6956350115b3","author":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"date":"2020-01-10 19:45:58.000000000","message":"Uploaded patch set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"2664bafed65b804f8da4f1608cd1f0d80e6eda7a","author":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"date":"2020-01-10 19:58:17.000000000","message":"Uploaded patch set 3: Commit message was updated.","accounts_in_message":[],"_revision_number":3},{"id":"901b7cc635a22bf3d62f9ae1b27a273318c44cb2","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-10 21:38:31.000000000","message":"Patch Set 3: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/8486ceb8426b47759b5a7f0722f002c4 : SUCCESS in 7m 21s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/e0f443f47676499c91cf497b7e0e33c5 : SUCCESS in 4m 27s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/38a15061675d408884681878a5d17550 : SUCCESS in 5m 22s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/79d5c52ff0cf4840ae5d2bde825cd498 : FAILURE in 16m 47s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/9accdd9b147d4d24adef29526d33433c : FAILURE in 11m 10s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/b6d0047b8a0245a98b15dfd692c685bf : FAILURE in 10m 16s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/3f9f51881f6f41fbb713567912cffa7f : SUCCESS in 1h 00m 42s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/06681c9896e847f2822dce0bfc2c5536 : SUCCESS in 1h 24m 47s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/cd813124a0014d76897552978393350c : SUCCESS in 1h 14m 01s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/10789c512acb4b5ebfe1d69cf6de0f30 : SUCCESS in 54m 24s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/ab58c63d8642482388459be5cae8e8da : SUCCESS in 50m 15s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/efa8db41098b4d17b3f7d2ee20d413fa : SUCCESS in 59m 13s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/ded19ee78a6c4739a33c8fcc897d1458 : SUCCESS in 43m 37s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/3e210f90ff9342b4bc8036a4eec5a07b : SUCCESS in 51m 29s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/539d37b0ec924048971394704ab115c5 : SUCCESS in 1h 03m 44s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/eeb0e34b75124854acb66c251b3f48ff : SUCCESS in 1h 26m 34s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/cb778ac699de4ef48d130270a6fad31e : SUCCESS in 1h 11m 52s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/5bb5c78db8be4f14b52cae2ab7d89102 : SUCCESS in 1h 04m 50s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/7da50071d25445a2b1c57c4d9f11bacc : SUCCESS in 41m 21s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/1876fda2536543e3af538248687125a0 : FAILURE in 1h 03m 12s (non-voting)\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/428a50a69fc0468e9fef3733d6d7a5f0 : SUCCESS in 1h 32m 55s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"2b7f199cbec04853f3fc8859d3fcd93190b72e5e","author":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"date":"2020-01-11 18:08:20.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"c588aed2ca621a5a8d442d6478de0f651e423193","author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"date":"2020-01-11 18:23:51.000000000","message":"Patch Set 4: Code-Review-1\n\nThis seems like something deployments should be set based on it\u0027s requirements. The regression is solely based on the assumption that usage was distributed between the two ports when Keystone was deployed that way.\n\nThe more proper way, if something should be changed at all, would be changing the default value to $::os_workers_large","accounts_in_message":[],"_revision_number":4},{"id":"7688a586f0467dd218525b558d9d446b1b83d0dc","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-11 19:46:50.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/596b813e00204b53ac192d88fae059c6 : SUCCESS in 4m 23s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/18639580c6af4c0db3340433afe7954b : SUCCESS in 4m 31s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/d22706c91e2f43c9bc8bbb6e32b4936d : SUCCESS in 4m 37s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/ee99ee42356946de921f088cecd4032f : SUCCESS in 15m 55s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/548265e3886e44ee9c26db465f36a97f : SUCCESS in 10m 21s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/2b04b1f09ea74a5bb096734ddda05a28 : FAILURE in 15m 13s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/7772718616b445599f3912966869a8e2 : SUCCESS in 57m 19s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/4538e0d65f4d4f28965d200aaae6ef10 : SUCCESS in 1h 12m 58s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/99fe722c63004700ae778f9df9336cd6 : SUCCESS in 1h 02m 24s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/e3c623c8b9374c308b4fda7233d69403 : SUCCESS in 47m 20s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/a47ad82450c84792a0e9001c1d711668 : SUCCESS in 49m 38s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/242f4fa0ed594b8fa7e0ae4850c223cb : SUCCESS in 50m 43s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/5738f1be9fe34f35a631d601f3299281 : SUCCESS in 38m 08s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/401693c10da546179f0bdb9ee83a6e4a : SUCCESS in 37m 35s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/e4db160b469541fc82913f347a5a7009 : SUCCESS in 54m 46s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/74884953890d4fa2a33029ff68886a8d : SUCCESS in 1h 14m 33s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/70dd23eb95da43b393f6e11fe5bae41e : SUCCESS in 56m 48s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/db4b1ee54b434a51a838e544163ab1d4 : SUCCESS in 49m 38s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/3d1272af8cf943efbcb3a1b8fbf2c0f1 : SUCCESS in 44m 17s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/d80d46602e22480d9ee4008c7cee02bf : FAILURE in 57m 28s (non-voting)\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/64f2ce4c2b8f4e03b36000bea049000f : SUCCESS in 1h 15m 36s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"3fe19277b5fc4a53db7875cc99fc6a5dcc6fee2b","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-01-17 12:10:29.000000000","message":"Patch Set 4: Code-Review+1\n\n(3 comments)\n\n\u003e This seems like something deployments should be set based on it\u0027s\n \u003e requirements. The regression is solely based on the assumption that\n \u003e usage was distributed between the two ports when Keystone was\n \u003e deployed that way.\n \u003e \n \u003e The more proper way, if something should be changed at all, would\n \u003e be changing the default value to $::os_workers_large\n\nI think this is a wrong statement, Tobias. As explained in the commit message, the actual number of worker for that service has decreased with the merge of services - we probably want to ensure we actually have the same number of total workers for keystone - meaning this patch makes sense, and we should probably not change the os_worker value, since it seems to be something used in other locations.\n\nAdding +1 to show support for this change, but there are some questions (and, please, reformat the commit message :))","accounts_in_message":[],"_revision_number":4},{"id":"6aea6af8f56a571dc18f82c934847a8b55bb9515","author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"date":"2020-01-21 13:08:40.000000000","message":"Patch Set 4: Code-Review-1\n\n(4 comments)\n\nI do think this should be fixed here in puppet-keystone as well.  Unfortunately, I don\u0027t think any of the \"$::os_workers_*\" values are correct for what we want since they all divide the number of processes by 2 or more (even \"$::os_workers_large\").\n\nI believe the multiplication used in the patch does have some corner cases that would result in more keystone processes than desired as mentioned in my in-line comments.  I think we should used an approach based off of the \"processorcount\" (without division) with min and max caps, which is really the logic that we are trying to use.","accounts_in_message":[],"_revision_number":4},{"id":"9f356cf0e43bf5d6d4e4b252c39171dee5533fc6","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-01-21 13:24:03.000000000","message":"Patch Set 4:\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"2d8371bf555ffcffa4de0ba6fb5324e22c7d03f6","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-01-21 14:24:56.000000000","message":"Uploaded patch set 5.","accounts_in_message":[],"_revision_number":5},{"id":"f167f0cf5f89b26cd7df7c6c708082b8be435f96","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-01-21 14:25:57.000000000","message":"Patch Set 5:\n\n(2 comments)\n\nStill need to incorporate the feedback from Nathan and Cedric from PS 4. PS just cleans up the commit message and removes unrelated changes.","accounts_in_message":[],"_revision_number":5},{"id":"5136131594bceca34855d4b4b93b1903f38ad9b5","author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"date":"2020-01-21 14:32:00.000000000","message":"Patch Set 4:\n\n(2 comments)","accounts_in_message":[],"_revision_number":4},{"id":"c9fc9693a042cd494f304243d10c1a4d856382d8","author":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"date":"2020-01-21 14:37:50.000000000","message":"Patch Set 5:\n\nQuick question, do we still need https://review.opendev.org/#/c/702032/ so that default is not set to os_workers?","accounts_in_message":[],"_revision_number":5},{"id":"8720e1bb0698f857184a6a4de9794b36cb395e2d","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-01-21 14:54:37.000000000","message":"Patch Set 4:\n\n(1 comment)","accounts_in_message":[],"_revision_number":4},{"id":"a6114d7a19ce1a5c53fe21c18de748201d87b782","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-01-21 14:55:05.000000000","message":"Uploaded patch set 6.","accounts_in_message":[],"_revision_number":6},{"id":"43a039ec2b77da97f7c08a9d9a5eab2449098203","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-01-21 15:33:12.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"8254dc71d86a5f331cf87bb8baf8e0a8e47c3daf","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-21 15:59:23.000000000","message":"Patch Set 6: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/eae67e3cd7374998b09fceec3ac4abeb : SUCCESS in 5m 24s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/392fd68192004af888146587dbf5f435 : FAILURE in 8m 39s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/179298aaa970455ca51949f74ecdfc31 : FAILURE in 6m 00s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/88b9a16cc44d4871bf1c20195b1e6c5e : FAILURE in 12m 18s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/969ac72d733d4f14914d1f6d4cf38287 : FAILURE in 8m 45s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/18fafc21dfa948dfadfd9a76f62737f7 : FAILURE in 20m 09s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/420a83ee5f654305b4c7d40dcd95f360 : FAILURE in 9m 19s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/95788a7a675747d2a365b165e5c72225 : FAILURE in 12m 45s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/f97c2d9fdba6462fa5afbc507fb58f58 : FAILURE in 12m 09s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/785b387bd6b347fc8e882cd2ce67f837 : FAILURE in 10m 56s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/57e5b17223ca4f9e9ba1a995ba28c332 : FAILURE in 11m 02s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/692eea915b884870b3aa56907a457452 : FAILURE in 10m 09s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/57d7f76a5fa94a57a95a71aa97fc5f50 : FAILURE in 9m 17s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/11d59e0f744b45cdb0a9e14d38736d6f : FAILURE in 10m 24s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/aa40465758e246e9a458db3ccb3b71b9 : FAILURE in 12m 06s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/dbcf865305d44b968be9adae62f7c3c7 : FAILURE in 11m 13s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/94e6986f785241ceaa9551f52dfc8b03 : FAILURE in 9m 00s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/1cf00aa326e348b5a5722c95d350b676 : FAILURE in 11m 17s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/2865074d92ff4e5d9e99e14f4360f053 : FAILURE in 14m 51s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/ff6b95c819f44558acb5c312fc7bd0de : FAILURE in 10m 27s (non-voting)\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/6a19826578d44fdaba435b20afe7a089 : FAILURE in 58m 07s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"535bb447d946acfc81b1f0a6f64dc82c069527af","author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"date":"2020-01-23 07:39:33.000000000","message":"Patch Set 6: Code-Review+1","accounts_in_message":[],"_revision_number":6},{"id":"ab274363292e74e27c02bb326afce4709c0f1f52","author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"date":"2020-01-23 07:39:54.000000000","message":"Patch Set 6:\n\nrecheck","accounts_in_message":[],"_revision_number":6},{"id":"72c22356db99e2138a69e775e4c3ba405d77e762","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-23 08:56:10.000000000","message":"Patch Set 6:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/1b7c69cf83d04824a2bccd8a6c3f93c2 : SUCCESS in 6m 01s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/8c71314d92cb49cdbc431026120d5a0e : FAILURE in 8m 03s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/5ebf5c011e874dd6ba7f815bb720ad84 : FAILURE in 8m 10s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/6c1435cc02714d01a61f9d8c024127a4 : FAILURE in 10m 59s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/03b1c0c12aae410b9fd8788e08995567 : FAILURE in 9m 38s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/ad272739d4d4416aba93dda8b4ca2eae : FAILURE in 11m 32s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/32fbbba54dd04512963886adea21c798 : FAILURE in 10m 10s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/84de96c06ad746feb061472e1f08f3db : FAILURE in 10m 21s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/978b9213ab274fc5a3fbfc09a48399a5 : FAILURE in 11m 56s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/bf4356cdacb640b9a73783ae3e555a87 : FAILURE in 9m 17s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/ca025c955d2948dbbd0f32c522847c19 : FAILURE in 7m 31s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/7bd6098aba1f4ec6a6fe49fad0b6b7b2 : FAILURE in 8m 08s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/5ca1713856004d6598e7d1694416f7af : FAILURE in 6m 57s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/466a78dee21449499a131e8690041d80 : FAILURE in 16m 29s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/bc6ef0328b4a45898a67be89e1107250 : FAILURE in 10m 28s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/1f80b77d7a3843ae89ff41facb5f0c3c : FAILURE in 12m 48s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/eabf37c5121e45c69838f0f1e0b925a3 : FAILURE in 9m 50s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/47630ab7edcb477db27f380c62b06930 : FAILURE in 12m 03s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/01d1a36542dd4e6dba6d542cc9915712 : FAILURE in 12m 07s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/38fc7ec39c794433adb125fe4bea4095 : FAILURE in 18m 46s (non-voting)\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/1ae89a66f245430ea0e42bedfb343fb7 : FAILURE in 1h 05m 41s (non-voting)","accounts_in_message":[],"_revision_number":6},{"id":"8d1d19de4f2333ea5c76d063bef0f234e60d6bf6","author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"date":"2020-01-23 09:12:19.000000000","message":"Patch Set 6: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"cedf38644cd6ec52c53a8db6cc74b343f4ed7c2f","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-01-23 14:30:05.000000000","message":"Uploaded patch set 7.","accounts_in_message":[],"_revision_number":7},{"id":"5200a226e58bc2c84fe0416a6e5bdcf2866cfb14","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-01-23 14:30:22.000000000","message":"Patch Set 6:\n\n(1 comment)","accounts_in_message":[],"_revision_number":6},{"id":"2f78ec8b74247c39454ee8300cd84fa0b8eaf43d","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-01-23 14:36:02.000000000","message":"Patch Set 7:\n\n\u003e Quick question, do we still need https://review.opendev.org/#/c/702032/\n \u003e so that default is not set to os_workers?\n\nCédric or Nathan can double-check this, but I don\u0027t think we need the tripleo-heat-templates patch because the default here is no longer relying on os_workers.","accounts_in_message":[],"_revision_number":7},{"id":"e6224fd2c3f5130e7cb952b8319a1ad494bd6654","author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"date":"2020-01-23 15:36:16.000000000","message":"Patch Set 7:\n\n\u003e \u003e Quick question, do we still need https://review.opendev.org/#/c/702032/\n \u003e \u003e so that default is not set to os_workers?\n \u003e \n \u003e Cédric or Nathan can double-check this, but I don\u0027t think we need\n \u003e the tripleo-heat-templates patch because the default here is no\n \u003e longer relying on os_workers.\n\nI agree.","accounts_in_message":[],"_revision_number":7},{"id":"0fe359d509688fded4b20a5513a70aef75912cfe","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-01-23 16:05:04.000000000","message":"Patch Set 7: Code-Review-1\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"186eaffcad4902b77a4adb78cb3edff4fa649aea","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-01-23 16:06:45.000000000","message":"Patch Set 7:\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"ccc1f0b14f222c923c4d5553945d7d27eca23935","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-23 16:28:53.000000000","message":"Patch Set 7: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/ee5965c21ce1476582c883f4c58c0c1f : SUCCESS in 6m 55s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/ccfe1157df754267ae4365d3a7439156 : SUCCESS in 5m 44s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/ec597b19da384eb9af2be9da8fa4e567 : SUCCESS in 5m 12s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/b9793203a90d41a89fd7da063517a1ca : FAILURE in 16m 58s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/b5859dcc96e24541b7f047fe51d923eb : FAILURE in 11m 43s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/1b4f56c34f644eb59cabe9ca2d4081fb : FAILURE in 11m 37s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/99c0ed6b76064baf9c4757bf79c32230 : SUCCESS in 1h 19m 43s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/f2095ba70bed4e4eb3fbd0bd5d43a6d8 : SUCCESS in 1h 12m 14s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/c4d56ff33a3144d9b9b35098e1446bdc : SUCCESS in 1h 47m 50s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/3977ed140e964149a7e37bff6b2de0fc : SUCCESS in 1h 34m 52s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/18023d983ef14b12b3a9fc54e63d22df : SUCCESS in 1h 21m 27s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/8b58ca9cc3ec471d8464f331706329a3 : SUCCESS in 1h 37m 29s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/ed97f6bce4464565b7d7674346b2244d : SUCCESS in 57m 03s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/23dd5f0e37174f07be4e3f3367319946 : SUCCESS in 1h 17m 54s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/e0ac10bebf6e4e008e4ae32b5acf7bf2 : SUCCESS in 1h 19m 27s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/16a14c26e49b4b7f977d88f571ae8e5b : FAILURE in 1h 19m 43s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/9c766d97f21b4c61aea9d275b24163f6 : SUCCESS in 1h 12m 30s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/f1ea9d96cf304c049dc84e8f948c6753 : SUCCESS in 1h 00m 29s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/09886c555e8347bfa03abb13c226f07c : SUCCESS in 48m 55s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/eef67a4501174d2889475927e038267d : FAILURE in 58m 42s (non-voting)\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/21133bc32dd249e4b7bf8918fb35ab33 : SUCCESS in 1h 47m 02s (non-voting)","accounts_in_message":[],"_revision_number":7},{"id":"67b6c91e9f9514b59e73790c1927b49c1f39d5e1","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-01-24 23:29:27.000000000","message":"Uploaded patch set 8.","accounts_in_message":[],"_revision_number":8},{"id":"1ca9e3d08dc248ba997ff2777ffc0e5db34725be","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-25 00:30:30.000000000","message":"Patch Set 8: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/9cb1d842bd14468db096a1c81df6dab5 : SUCCESS in 5m 52s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/aefb1e3432dd4de1ab3bd7a07f977c85 : FAILURE in 5m 26s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/a904bf6534e24064b9c315cc5db0e436 : FAILURE in 5m 12s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/8dfaf1d047f543b7ac4c956fb304c264 : FAILURE in 20m 41s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/88b866b7b4c649f6833018715e8ca53f : FAILURE in 10m 44s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/0ad296828dad4f86b98ad0fd24fdfce8 : FAILURE in 11m 09s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/68e043a9548d4e6a8852567448233492 : FAILURE in 22m 34s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/2a14e42a7dc14cf286cd436cdaa520c5 : FAILURE in 8m 29s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/972681389660461cb6c96830fcbb615a : FAILURE in 9m 11s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/d2fd9a08c5884a0fa26a11977b62636b : FAILURE in 9m 36s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/231651825ee5475d806ec2e33c43abc8 : FAILURE in 9m 18s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/afe1bc4eb95e46ff92bc397fe7638309 : FAILURE in 7m 45s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/bc7bf16a2b20467c8b4ea916496b8a39 : FAILURE in 7m 04s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/0b2a8ed1ce4945748b15e8981b6df841 : FAILURE in 9m 16s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/b851c0957407432c9de73c008fd933ea : FAILURE in 8m 08s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/c10368fe75bc4fd599f4189e7c832696 : FAILURE in 11m 04s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/9b6927bf1b5f434ea78264ed7e9addaf : FAILURE in 11m 11s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/f7cceab0cc5d40c0b8d031921a827b8e : FAILURE in 8m 07s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/61f559da92de4041a580183b31666d6f : FAILURE in 12m 17s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/4ed1e39594964632afe98a5efcf0f767 : FAILURE in 10m 42s (non-voting)\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/77ff5733b7bb42859d529ca26e9b527a : FAILURE in 56m 35s (non-voting)","accounts_in_message":[],"_revision_number":8},{"id":"1d62793f9229ac1c09fb1472a805932e284b2cbc","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-01-27 13:41:22.000000000","message":"Patch Set 7:\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"fadd79fc397f7bc1b7045af5409638bd8c3dd9da","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-01-27 13:42:10.000000000","message":"Patch Set 7:\n\n(1 comment)","accounts_in_message":[],"_revision_number":7},{"id":"1725f95aa6e16eaa1c856d37535679dec9abcf41","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-01-27 15:16:07.000000000","message":"Patch Set 8:\n\nrecheck","accounts_in_message":[],"_revision_number":8},{"id":"a4e87017ab9d8d09cb5634c3ff7328fb31713889","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-27 16:26:30.000000000","message":"Patch Set 8:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/d509fa090a87470eae8395a991f734c7 : SUCCESS in 5m 06s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/a76c0d1967c74dc3a0e20413882a5cde : FAILURE in 5m 31s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/7c27edb0b22c4805ad06eb8e4237e4ca : FAILURE in 4m 53s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/1e98636a36874dedb09d629a3dd72744 : FAILURE in 17m 08s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/070e0d1e0d0846adb7ecae3e279e6e24 : FAILURE in 11m 47s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/024f13fe405b4e0ab79c9838a500a42d : FAILURE in 11m 23s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/f4f23819417b4dc69a4b7eb236ef7ecc : FAILURE in 10m 23s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/695c61945f6c46c6b7c60db890220fb7 : FAILURE in 9m 32s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/5d3811662bb346dcb966039075a21ed0 : FAILURE in 10m 00s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/1a1d937a8212422f9f06573e27bb8ad3 : FAILURE in 9m 33s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/9ae185709c174a9ba5c9b739cf2ceb90 : FAILURE in 9m 14s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/f579473ff132442a8f3e7c8438a74e2e : FAILURE in 6m 52s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/4224066e473541af8fd4c6e86e5f9c26 : FAILURE in 7m 53s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/ce271dd57d614e63a600a380e964c1b7 : FAILURE in 11m 11s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/36dbb5b87bf442c1a092b1487d6162f7 : FAILURE in 10m 54s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/54689710482642f7bcd748514c8f7d66 : FAILURE in 12m 54s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/b0cfa636936542be995e300495dadf33 : FAILURE in 9m 37s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/1ba86fa5558c4fb7977eeb022c3be6e4 : FAILURE in 11m 08s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/ba0fa6fb309b4a9f891bd024439dcf8a : FAILURE in 13m 28s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/e060193a26a74aa9a1d85ac0e5b1d4d1 : FAILURE in 11m 53s (non-voting)\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/37a3a358ed70408c973bc71175dd2a4b : FAILURE in 1h 05m 50s (non-voting)","accounts_in_message":[],"_revision_number":8},{"id":"c95ca192646c2253ef1cc9b5ae352f2af86043e2","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-01-27 16:58:16.000000000","message":"Patch Set 9: Published edit on patch set 8.","accounts_in_message":[],"_revision_number":9},{"id":"fbf8aeb60348b63e7ab403de3dff93cf01a1dc56","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-27 18:40:44.000000000","message":"Patch Set 9: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/38b4f3f3ca8d40d2925f8c557ae0452e : SUCCESS in 5m 46s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/b93b91a5292e4a0c81adb14f912d019f : SUCCESS in 5m 14s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/be3e0038bb7040d0ba7323f3e543f6d1 : SUCCESS in 9m 09s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/114a6a37ad934079ba494fc3978cb8b0 : FAILURE in 17m 10s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/fadc9f5fcf634d219bdf62b16eb02e44 : FAILURE in 17m 51s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/4be6c9f1255c4cf2b363807398dec714 : FAILURE in 11m 40s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/55f8b382335846dc9a714a69b7e78a9e : SUCCESS in 1h 08m 28s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/53e0e025594145efa73fa62cb43b1375 : SUCCESS in 1h 00m 38s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/119ca6dc59de46a992dd2c766056cba0 : SUCCESS in 1h 11m 53s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/5689d1db376c4f67b45e91af7ad657de : SUCCESS in 55m 56s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/21ac18be7db44f9980626e3773c864dc : SUCCESS in 54m 21s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/5ec715e1c2bd4290ac87abcb5b4c5cd4 : SUCCESS in 1h 00m 43s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/1d46e13c4582494f96dfa6444d8d1f56 : SUCCESS in 42m 12s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/11857528fe974e128204fd45ccd047bf : SUCCESS in 37m 32s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/6084525a420246e6a0478b018bb744c8 : SUCCESS in 57m 34s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/3eac975a236f46cbad5d633a98378ddd : SUCCESS in 1h 00m 12s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/c97bea8a29b74af39830e6a60fc947a7 : SUCCESS in 58m 45s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/8c61be8feb1b42d285fdc5574e7be866 : SUCCESS in 49m 31s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/e3e8105b810941679a3a08ad211b5e2d : SUCCESS in 39m 18s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/08d43310e9ae4a4c922d030b74efd576 : FAILURE in 56m 54s (non-voting)\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/4dfec6c8ccfa4e009483dc627ca1ad1a : SUCCESS in 1h 37m 33s (non-voting)","accounts_in_message":[],"_revision_number":9},{"id":"4d6f239e3fa8f1421075b72a2faba60b1370deb8","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-01-27 19:02:33.000000000","message":"Uploaded patch set 10.","accounts_in_message":[],"_revision_number":10},{"id":"62136d5ff36ffdd5bf776575a43e03b736cbb15c","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-27 20:34:09.000000000","message":"Patch Set 10: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/3f1dfb4a5cdb437e94a4b63a52f91e03 : SUCCESS in 5m 18s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/0b0fd061759c49bf9ddc38b100e5e239 : SUCCESS in 5m 25s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/91ed4817ead54948aa836de6510d3b7f : SUCCESS in 6m 29s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/c311524d5d3e4885a2f131aff8c98e14 : FAILURE in 26m 17s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/e0a55275b41c4f5cb1c5059683b179ee : FAILURE in 13m 47s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/abc455889d074b879ab14e768b3ed0be : FAILURE in 15m 22s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/e256c0c97f504d96965b20918954fdf8 : SUCCESS in 1h 00m 19s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/8c4946b182d849d387565e3e14a31831 : SUCCESS in 1h 11m 56s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/f4e6c44f2b704fd885c19bb553754810 : SUCCESS in 1h 14m 53s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/5ab0cc8e36804b6a8ebf9623860be14b : SUCCESS in 56m 16s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/e9eddaadceee48c4b4e1129df240d722 : SUCCESS in 52m 10s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/6c439bb9e03f4ea59c9ccbe52c0d531e : SUCCESS in 52m 48s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/12cdb862bb8e43d7b9f5c6faa79d2043 : SUCCESS in 41m 26s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/47089833bb5d4d70981da71453ed27ce : SUCCESS in 40m 31s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/3c5530a7430e409db6dd2457779a0d61 : SUCCESS in 54m 55s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/7fb424779a004a7b9ff6f7ed642fb593 : SUCCESS in 1h 18m 02s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/9b0e9161e31646699772a44c15200dce : SUCCESS in 1h 00m 10s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/5eeca96c6ffa4ecb9f96b576ab107f7e : SUCCESS in 1h 03m 57s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/ce59b8d3136b46e1b29db375f374397c : SUCCESS in 43m 03s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/b696d9b3c2254d479daa6e07c1cc6899 : FAILURE in 1h 01m 14s (non-voting)\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/1240286c1a18440ea7cbbdcb0100419c : SUCCESS in 1h 25m 31s (non-voting)","accounts_in_message":[],"_revision_number":10},{"id":"28820e1261f7738b8c63417f57b8c9d1d22c1b94","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-01-27 20:56:51.000000000","message":"Uploaded patch set 11.","accounts_in_message":[],"_revision_number":11},{"id":"7dd0e50999939842cc18a665b4d91ec7f8a43165","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-01-27 22:27:53.000000000","message":"Patch Set 11: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/2a297037f5464f4798ea185f613fefe4 : SUCCESS in 5m 01s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/ac707703e95c4efb97c9cf3c60c4177a : SUCCESS in 5m 24s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/14ad63c4205c4f8faabc963ddd6c7271 : SUCCESS in 4m 24s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/9b8f6a8b9cb745a88b3ca5cd12ec3580 : SUCCESS in 23m 15s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/cbefcf2d17ea4f06b071971ef7a440b9 : SUCCESS in 17m 47s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/3902b82949204a278aac2b28eccf3106 : FAILURE in 12m 27s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/3da0c149b52a4664a8190fc49a616131 : SUCCESS in 57m 40s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/c372789d9cce4855aac4d5b0eec62908 : SUCCESS in 1h 04m 37s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/90c1c546ae20460aab41fd7638f2b3f6 : SUCCESS in 1h 25m 28s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/3d296c9276c347d09c36b79b98d31b40 : SUCCESS in 49m 15s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/2acce92af7ba40aab6a707b6c2f0aee9 : SUCCESS in 47m 13s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/7ca4edbe9f8a468b8400aa5a3d7e325b : SUCCESS in 52m 43s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/b09a4c79b4a84d95ac2c8712720ac019 : SUCCESS in 41m 19s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/6a9d561b585947de8ad7dd81bbe491b1 : SUCCESS in 38m 26s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/af030fd1f9044edcb387260a1ce49284 : SUCCESS in 51m 44s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/4bde1b9c41b544f6973331d64b37f0e7 : SUCCESS in 1h 24m 01s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/442947d77ed14a6ba09dbce8be4f21f6 : SUCCESS in 1h 06m 11s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/6382ccac92ec4fa4a7178f24b2a8dcc8 : SUCCESS in 43m 16s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/52b2a2fe832e4aad9c63e61441c4cb33 : SUCCESS in 46m 42s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/0248ffc40d0b486aae81125063a9b71d : FAILURE in 1h 07m 33s (non-voting)\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/a011ac105c4d4062960d8e75bf9256f3 : SUCCESS in 1h 22m 21s (non-voting)","accounts_in_message":[],"_revision_number":11},{"id":"9bab3bb1907da2a3d687996528d504a60c3d7a05","author":{"_account_id":9098,"name":"Nathan Kinder","email":"nkinder@redhat.com","username":"nkinder"},"date":"2020-01-29 07:52:13.000000000","message":"Patch Set 11: Code-Review+1","accounts_in_message":[],"_revision_number":11},{"id":"21a0b71916bfdeeced0b739eb252b365fb3ac88b","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2020-01-29 14:57:31.000000000","message":"Patch Set 11: Code-Review+2","accounts_in_message":[],"_revision_number":11},{"id":"299e0aef9fdcc54e046a7ffa2db065257a4eb6d0","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-01-29 21:55:47.000000000","message":"Patch Set 11: Code-Review+1\n\nThanks for the help here, Alex.","accounts_in_message":[],"_revision_number":11},{"id":"36b54e46d0ccc52b75b1c9ce9a1dc8c4eba927f4","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2020-01-30 13:29:55.000000000","message":"Patch Set 11:\n\n(1 comment)","accounts_in_message":[],"_revision_number":11},{"id":"a279783d2856a8bdb35dd2a10c2e80082e337e93","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2020-01-30 13:33:12.000000000","message":"Patch Set 11: Code-Review-1","accounts_in_message":[],"_revision_number":11},{"id":"04bafde358e5447dedafa2aca967c4c1653bc9ff","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-01-30 14:56:35.000000000","message":"Patch Set 11:\n\nwe could actually reuse the heat one, but whatever. i can switch it out","accounts_in_message":[],"_revision_number":11},{"id":"c4cea391a0b2e06362bc668c7b671d73e5e5edad","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2020-01-30 15:52:49.000000000","message":"Patch Set 11: -Code-Review\n\n\u003e we could actually reuse the heat one, but whatever. i can switch it out\n\nUnfortunately heat one is a bit different from the one we expect here, because it returns cores/2 instead of cores.\n\nI\u0027ve submitted patches to implement the exact same logic in openstacklib[1][2].\n [1] https://review.opendev.org/#/c/705041/\n [2] https://review.opendev.org/#/c/705042/\n\nI\u0027ll remove my -1 as these could be regarded as follow-up, and I\u0027ll leave the decision for other core reviewers because we already have +2 on this.","accounts_in_message":[],"_revision_number":11},{"id":"492ff6acb44df472ceb903ee701fd3938258667b","author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"date":"2020-01-31 08:40:43.000000000","message":"Patch Set 11:\n\nProblem is that this would be backportable (even though it\u0027s a behavior change right? Will affect existing deployments) while changing openstacklib would add a dependency for puppet-keystone to a new minimum version of openstacklib (which is even worse imo)\n\nAnd I\u0027m assuming here people want to backport this?","accounts_in_message":[],"_revision_number":11},{"id":"0b14c80a92f296b9ac96a8e350bf43a0f497ae91","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-01-31 14:42:04.000000000","message":"Patch Set 11:\n\nWe could just backport the openstacklib bit and not this. One might argue this was a performance regression so backporting it may be justified.","accounts_in_message":[],"_revision_number":11},{"id":"1d66250852f1f1777301e79a176ee6ef14f52eac","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2020-02-01 13:47:33.000000000","message":"Uploaded patch set 12.","accounts_in_message":[],"_revision_number":12},{"id":"f9b3d66acecc74996d61df33bb805378390f56e8","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2020-02-01 13:50:30.000000000","message":"Uploaded patch set 13.","accounts_in_message":[],"_revision_number":13},{"id":"5316a8bef4ae3400ee31daab003349b51ec01301","author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"date":"2020-02-01 13:52:56.000000000","message":"Patch Set 13:\n\nBased on the last comment by Alex, I merged my followup patch into this one, with also addressing comment on my patch by Tobius about release note.\n\nPlease correct me if I\u0027m doing a thing different from what you expect.","accounts_in_message":[],"_revision_number":13},{"id":"0583d0908d6ba282917c8de10ee0fe77838da619","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-01 15:29:04.000000000","message":"Patch Set 13: Verified+1\n\nBuild succeeded (check pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/cdc204b6bf0847ffa89ca249202e632a : SUCCESS in 5m 16s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/1fead95c6e054c6da1326b625416e783 : SUCCESS in 5m 09s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/46a016288ce742ac94ce3b7d2367c7bd : SUCCESS in 5m 31s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/9fc7cca303e444cb928dcae3e2812cd8 : SUCCESS in 20m 40s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/5f33121494f44c70a3f765fc5ac31275 : SUCCESS in 17m 19s\n- puppet-openstack-unit-latest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/f8819391309e45c693efa07249585ae0 : FAILURE in 11m 46s (non-voting)\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/75a7dd949ea44924a9c6c9b09eeed8bf : SUCCESS in 1h 25m 57s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/28cea5fc3c3144a28372e5eb3e613b6d : SUCCESS in 1h 12m 23s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/9f2685ac0ed94361b1155e2c6262155a : SUCCESS in 1h 31m 51s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/0033ded5d4174686a441b3a9479810c7 : SUCCESS in 1h 01m 02s\n- puppet-openstack-integration-5-scenario001-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/4831f9cda0a94cf784339877fda5db17 : SUCCESS in 1h 13m 36s\n- puppet-openstack-integration-5-scenario002-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/a13aa852529b4789ade6350ebc24065b : SUCCESS in 59m 34s\n- puppet-openstack-integration-5-scenario003-tempest-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/39951f50a4f646b486e7525b79ecc71f : SUCCESS in 42m 52s\n- puppet-openstack-integration-5-scenario004-tempest-ubuntu-bionic-mimic https://zuul.opendev.org/t/openstack/build/b197dd73f3494b7fbaeff917d553eb83 : SUCCESS in 39m 10s\n- puppet-openstack-integration-5-scenario001-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/399204d0bdc4467a9b09f711822a4d16 : SUCCESS in 1h 22m 05s (non-voting)\n- puppet-openstack-integration-5-scenario002-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/7b20eb6bcc06456c909ff2ead80dda16 : SUCCESS in 51m 35s (non-voting)\n- puppet-openstack-integration-5-scenario003-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/e2f857126c164af184c1f86a18fadb72 : SUCCESS in 1h 12m 14s (non-voting)\n- puppet-openstack-integration-5-scenario004-tempest-centos-8 https://zuul.opendev.org/t/openstack/build/264a5ceed23a4fd99b696bd0cf7c0e87 : SUCCESS in 46m 56s (non-voting)\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/199522ef0ee74339a14236e18aeac614 : SUCCESS in 42m 07s\n- puppet-openstack-beaker-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/e82e1e78b73346aaa90d2b5de1951b01 : FAILURE in 59m 51s (non-voting)\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/367cff2c628341b49aa5a6aad21d276f : SUCCESS in 4m 47s\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/982f5191f55640a489c5c61fa70187d3 : SUCCESS in 1h 31m 22s (non-voting)","accounts_in_message":[],"_revision_number":13},{"id":"16babf4ba73d9971b83fb4f69b7c1288cd440062","author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"date":"2020-02-03 10:48:16.000000000","message":"Patch Set 13: Code-Review-1\n\nJust to be clear, using the new openstacklib change will make this non-backportable, is that intentional?\n\nBecause; how are we suppose to notify users that there is a new minimum openstacklib versions for this release? I would rather that you did the hardcoded approach (if you even are going to backport) and then use os_workers_keystone in master to prepare for next release","accounts_in_message":[],"_revision_number":13},{"id":"26c1e1cd9a8d6b9d4c2b81bb841a4e78d6493257","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-02-03 15:29:25.000000000","message":"Patch Set 13:\n\nAs previously mentioned we can backport the openstacklib change but we don\u0027t have to backport this one","accounts_in_message":[],"_revision_number":13},{"id":"6671ce4d01417929a1f0aaebaf313e893994db51","author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"date":"2020-02-03 18:54:18.000000000","message":"Patch Set 13: -Code-Review\n\nOk, missed that comment, that would be fair I guess","accounts_in_message":[],"_revision_number":13},{"id":"2c56d527c1cfe4091a2cadf6292f93065116c8eb","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-02-07 22:16:22.000000000","message":"Patch Set 13: Code-Review+2","accounts_in_message":[],"_revision_number":13},{"id":"49901ab2e1d3394d9f8eff9b06bed2a2236a7b4c","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2020-02-10 17:07:30.000000000","message":"Patch Set 13: Code-Review+2","accounts_in_message":[],"_revision_number":13},{"id":"dc4c8c8382e2d7e249813e5f6032505b28553716","author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"date":"2020-02-19 16:48:33.000000000","message":"Patch Set 13: Code-Review+1","accounts_in_message":[],"_revision_number":13},{"id":"4e0f6252bd5cc1d36c8f606f99198102cf5c95f3","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2020-02-19 17:04:55.000000000","message":"Patch Set 13: Workflow+1","accounts_in_message":[],"_revision_number":13},{"id":"ef988cd9613340bbcb48cc5fe887febe8cc239dd","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-19 17:05:08.000000000","message":"Patch Set 13: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":13},{"id":"1e2405ebab42ef15858dbea893d917be004a436e","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-19 18:43:24.000000000","message":"Patch Set 13: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- puppet-openstack-lint-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/f6bc20d78bda4003b030e36aedeefce5 : SUCCESS in 10m 24s\n- puppet-openstack-syntax-5-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/a240be9bff5049c8becc913a6e3cfb0a : SUCCESS in 6m 03s\n- puppet-openstack-syntax-6-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/abdc742bac2e492e8bfdd7e7267f85db : SUCCESS in 5m 25s\n- puppet-openstack-unit-5.5-centos-7 https://zuul.opendev.org/t/openstack/build/094e5c6a5cd445418be92eb9d5ed0cd0 : SUCCESS in 22m 43s\n- puppet-openstack-unit-6.11-ubuntu-bionic https://zuul.opendev.org/t/openstack/build/29a40e95a3cc4838acfb6bc87408b13a : SUCCESS in 17m 57s\n- puppet-openstack-integration-5-scenario001-tempest-centos-7-luminous https://zuul.opendev.org/t/openstack/build/df9e7752e0cd474d87b75407d33f598a : SUCCESS in 1h 00m 59s\n- puppet-openstack-integration-5-scenario002-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/6f91b47fa9c9485cbb2bf7b8634a6b86 : SUCCESS in 1h 06m 52s\n- puppet-openstack-integration-5-scenario003-tempest-centos-7 https://zuul.opendev.org/t/openstack/build/59b9f8a120774eec977de8a57519aace : SUCCESS in 1h 16m 49s\n- puppet-openstack-integration-5-scenario004-tempest-centos-7-nautilus https://zuul.opendev.org/t/openstack/build/09e7371e92f94c71ab5386f18e1e70a6 : SUCCESS in 53m 39s\n- puppet-openstack-beaker-centos-7 https://zuul.opendev.org/t/openstack/build/5c9978d42d994b66bcded238d668a12b : SUCCESS in 44m 12s\n- build-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/5393aa2836cf4c8c9e8ecbedfdeb9c15 : SUCCESS in 4m 27s\n- puppet-keystone-tripleo-standalone https://zuul.opendev.org/t/openstack/build/c0bc7dc85e9a4979bcb755b8fd490d2a : SUCCESS in 1h 33m 14s (non-voting)","accounts_in_message":[],"_revision_number":13},{"id":"ffa33cee4726db0af18c22e21f4d7a0cb896a3e3","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-19 18:43:24.000000000","message":"Change has been successfully merged by Zuul","accounts_in_message":[],"_revision_number":13},{"id":"6db282de5d200afcba5dad977d63a20613582b3f","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-02-19 18:45:16.000000000","message":"Patch Set 13:\n\nBuild succeeded (promote pipeline).\n\n- promote-openstack-releasenotes https://zuul.opendev.org/t/openstack/build/0a593068cad04db9a9653f920f07717d : SUCCESS in 1m 24s","accounts_in_message":[],"_revision_number":13}],"current_revision_number":13,"current_revision":"8f5301d20d73a84cc10ea2591414e45cf3d1fa79","revisions":{"507ff6222be48dcb91d7a1d00e7f873aeef007d8":{"kind":"REWORK","_number":1,"created":"2020-01-10 19:44:55.000000000","uploader":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"ref":"refs/changes/31/702031/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/1"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:44:54.000000000","tz":-300},"subject":"Fix performance regression due to reduced number os keystone workers","message":"Fix performance regression due to reduced number os keystone workers\n\nhttps://github.com/openstack/puppet-keystone/commit/ace7aeb3b71b39a59f92fbc9e7f676a70c9a797a#diff-e5968f40345cf5d3be0539fbba87f787\nmoved all actions to the same keystone API consoloidating the different keystone-main and keystone-admin workers that were present earlier\ninto the \"keystone\" workers. As a result, we see a performance regression[0] in actions like creating and listing users/tenants, as\nTHT/puppet-keystone deploys # of keystone workers \u003d min(max(cpucores/2,2),12) as in https://github.com/openstack/puppet-openstacklib/blob/master/lib/facter/os_workers.rb#L33\nwhich would now be only half of what was deployed earlier due to the consolidation of separate keystone-main and keystone-admin workers into one \"keystone\" worker.\nFor exampme, on an 8 core machines, in earlier versions, 4 keystone-main and 4 keystone-admin workers would be spawned, but now we only have 4 \"keystone\" workers\nand due to the difference in the number of workers deployed by default, users will be a drop in the out of box performance.\n\nThis patch not only brings performance back to the previous levels, it is also safe to implement as we are not changing the # of workers when compared to previous releases\nand oinly bringing parity.\n\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/507ff6222be48dcb91d7a1d00e7f873aeef007d8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/507ff6222be48dcb91d7a1d00e7f873aeef007d8"}]},"branch":"refs/heads/master"},"41482eb8a1ccf965ab34d2c8eedfee6efb5d9740":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2020-01-10 19:45:58.000000000","uploader":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"ref":"refs/changes/31/702031/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/2"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:45:12.000000000","tz":-300},"subject":"Fix performance regression due to reduced number os keystone workers","message":"Fix performance regression due to reduced number os keystone workers\n\nhttps://github.com/openstack/puppet-keystone/commit/ace7aeb3b71b39a59f92fbc9e7f676a70c9a797a#diff-e5968f40345cf5d3be0539fbba87f787\nmoved all actions to the same keystone API consoloidating the different keystone-main and keystone-admin workers that were present earlier\ninto the \"keystone\" workers. As a result, we see a performance regression[0] in actions like creating and listing users/tenants, as\nTHT/puppet-keystone deploys # of keystone workers \u003d min(max(cpucores/2,2),12) as in https://github.com/openstack/puppet-openstacklib/blob/master/lib/facter/os_workers.rb#L33\nwhich would now be only half of what was deployed earlier due to the consolidation of separate keystone-main and keystone-admin workers into one \"keystone\" worker.\nFor exampme, on an 8 core machines, in earlier versions, 4 keystone-main and 4 keystone-admin workers would be spawned, but now we only have 4 \"keystone\" workers\nand due to the difference in the number of workers deployed by default, users will be a drop in the out of box performance.\n\nThis patch not only brings performance back to the previous levels, it is also safe to implement as we are not changing the # of workers when compared to previous releases\nand oinly bringing parity.\n\n[0]- https://bugzilla.redhat.com/show_bug.cgi?id\u003d1789495\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/41482eb8a1ccf965ab34d2c8eedfee6efb5d9740"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/41482eb8a1ccf965ab34d2c8eedfee6efb5d9740"}]},"branch":"refs/heads/master"},"3daa22777777bc0ed106ff8b5eaa1a75724214ab":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2020-01-10 19:58:17.000000000","uploader":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"ref":"refs/changes/31/702031/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/3"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:58:07.000000000","tz":-300},"subject":"Fix performance regression due to reduced number of keystone workers","message":"Fix performance regression due to reduced number of keystone workers\n\nhttps://github.com/openstack/puppet-keystone/commit/ace7aeb3b71b39a59f92fbc9e7f676a70c9a797a#diff-e5968f40345cf5d3be0539fbba87f787\nmoved all actions to the same keystone API consoloidating the different keystone-main and keystone-admin workers that were present earlier\ninto the \"keystone\" workers. As a result, we see a performance regression[0] in actions like creating and listing users/tenants, as\nTHT/puppet-keystone deploys # of keystone workers \u003d min(max(cpucores/2,2),12) as in https://github.com/openstack/puppet-openstacklib/blob/master/lib/facter/os_workers.rb#L33\nwhich would now be only half of what was deployed earlier due to the consolidation of separate keystone-main and keystone-admin workers into one \"keystone\" worker.\nFor exampme, on an 8 core machines, in earlier versions, 4 keystone-main and 4 keystone-admin workers would be spawned, but now we only have 4 \"keystone\" workers\nand due to the difference in the number of workers deployed by default, users will be a drop in the out of box performance.\n\nThis patch not only brings performance back to the previous levels, it is also safe to implement as we are not changing the # of workers when compared to previous releases\nand oinly bringing parity.\n\n[0]- https://bugzilla.redhat.com/show_bug.cgi?id\u003d1789495\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/3daa22777777bc0ed106ff8b5eaa1a75724214ab"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/3daa22777777bc0ed106ff8b5eaa1a75724214ab"}]},"branch":"refs/heads/master"},"2cbd81a729e6faed0e2ae58ef5d5fe6ac262d9b7":{"kind":"REWORK","_number":4,"created":"2020-01-11 18:08:20.000000000","uploader":{"_account_id":16845,"name":"Sindhur","email":"smalleni@redhat.com","username":"smalleni"},"ref":"refs/changes/31/702031/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/4"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-11 18:08:13.000000000","tz":-300},"subject":"Fix performance regression due to reduced number of keystone workers","message":"Fix performance regression due to reduced number of keystone workers\n\nhttps://github.com/openstack/puppet-keystone/commit/ace7aeb3b71b39a59f92fbc9e7f676a70c9a797a#diff-e5968f40345cf5d3be0539fbba87f787\nmoved all actions to the same keystone API consoloidating the different keystone-main and keystone-admin workers that were present earlier\ninto the \"keystone\" workers. As a result, we see a performance regression[0] in actions like creating and listing users/tenants, as\nTHT/puppet-keystone deploys # of keystone workers \u003d min(max(cpucores/2,2),12) as in https://github.com/openstack/puppet-openstacklib/blob/master/lib/facter/os_workers.rb#L33\nwhich would now be only half of what was deployed earlier due to the consolidation of separate keystone-main and keystone-admin workers into one \"keystone\" worker.\nFor exampme, on an 8 core machines, in earlier versions, 4 keystone-main and 4 keystone-admin workers would be spawned, but now we only have 4 \"keystone\" workers\nand due to the difference in the number of workers deployed by default, users will be a drop in the out of box performance.\n\nThis patch not only brings performance back to the previous levels, it is also safe to implement as we are not changing the # of workers when compared to previous releases\nand oinly bringing parity.\n\n[0]- https://bugzilla.redhat.com/show_bug.cgi?id\u003d1789495\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/2cbd81a729e6faed0e2ae58ef5d5fe6ac262d9b7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/2cbd81a729e6faed0e2ae58ef5d5fe6ac262d9b7"}]},"branch":"refs/heads/master"},"3622a6da46253ddedb21e9b87131547e5d6b6d51":{"kind":"REWORK","_number":5,"created":"2020-01-21 14:24:56.000000000","uploader":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"ref":"refs/changes/31/702031/5","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/5","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/5 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/5 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/5 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/5"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"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-01-21 14:24:55.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\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/3622a6da46253ddedb21e9b87131547e5d6b6d51"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/3622a6da46253ddedb21e9b87131547e5d6b6d51"}]},"branch":"refs/heads/master"},"8b6de1b6f424f30ce44e7808be013d7d318df3eb":{"kind":"REWORK","_number":6,"created":"2020-01-21 14:55:05.000000000","uploader":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"ref":"refs/changes/31/702031/6","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/6","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/6 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/6 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/6 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/6"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"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-01-21 14:54:58.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\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/8b6de1b6f424f30ce44e7808be013d7d318df3eb"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/8b6de1b6f424f30ce44e7808be013d7d318df3eb"}]},"branch":"refs/heads/master"},"f8047aa133df80a8da84af4a3567b1c2373f1fa8":{"kind":"REWORK","_number":7,"created":"2020-01-23 14:30:05.000000000","uploader":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"ref":"refs/changes/31/702031/7","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/7","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/7 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/7 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/7 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/7"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"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-01-23 14:29:58.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\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/f8047aa133df80a8da84af4a3567b1c2373f1fa8"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/f8047aa133df80a8da84af4a3567b1c2373f1fa8"}]},"branch":"refs/heads/master"},"3c143156efe8952f1ef454eb190075d304059067":{"kind":"REWORK","_number":8,"created":"2020-01-24 23:29:27.000000000","uploader":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"ref":"refs/changes/31/702031/8","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/8","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/8 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/8 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/8 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/8"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Alex Schultz","email":"aschultz@redhat.com","date":"2020-01-24 23:29:23.000000000","tz":-420},"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\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/3c143156efe8952f1ef454eb190075d304059067"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/3c143156efe8952f1ef454eb190075d304059067"}]},"branch":"refs/heads/master"},"6a9b6169522f7a019a7e44a54fb13db2e4e7288d":{"kind":"REWORK","_number":9,"created":"2020-01-27 16:58:16.000000000","uploader":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"ref":"refs/changes/31/702031/9","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/9","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/9 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/9 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/9 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/9"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Alex Schultz","email":"aschultz@redhat.com","date":"2020-01-27 16:58:14.000000000","tz":0},"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\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/6a9b6169522f7a019a7e44a54fb13db2e4e7288d"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/6a9b6169522f7a019a7e44a54fb13db2e4e7288d"}]},"branch":"refs/heads/master"},"b9e9e1ecf3fa96c018a016648794e8982a514e50":{"kind":"REWORK","_number":10,"created":"2020-01-27 19:02:33.000000000","uploader":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"ref":"refs/changes/31/702031/10","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/10","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/10 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/10 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/10 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/10"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Alex Schultz","email":"aschultz@redhat.com","date":"2020-01-27 19:02:28.000000000","tz":-420},"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\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/b9e9e1ecf3fa96c018a016648794e8982a514e50"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/b9e9e1ecf3fa96c018a016648794e8982a514e50"}]},"branch":"refs/heads/master"},"84c03934b22c739dff4a66b026421efc47538995":{"kind":"REWORK","_number":11,"created":"2020-01-27 20:56:51.000000000","uploader":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"ref":"refs/changes/31/702031/11","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/11","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/11 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/11 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/11 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/11"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Alex Schultz","email":"aschultz@redhat.com","date":"2020-01-27 20:56:48.000000000","tz":-420},"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\nChange-Id: Icf9acaa106af705fa249a2ef2abca9f9a91fba59\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/84c03934b22c739dff4a66b026421efc47538995"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/84c03934b22c739dff4a66b026421efc47538995"}]},"branch":"refs/heads/master"},"b517316c6c75ccf308b71cc5f9f2207702296f2a":{"kind":"REWORK","_number":12,"created":"2020-02-01 13:47:33.000000000","uploader":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"ref":"refs/changes/31/702031/12","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/12","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/12 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/12 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/12 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/12"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Takashi Kajinami","email":"tkajinam@redhat.com","date":"2020-02-01 13:47:25.000000000","tz":540},"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","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/b517316c6c75ccf308b71cc5f9f2207702296f2a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/b517316c6c75ccf308b71cc5f9f2207702296f2a"}]},"branch":"refs/heads/master"},"8f5301d20d73a84cc10ea2591414e45cf3d1fa79":{"kind":"REWORK","_number":13,"created":"2020-02-01 13:50:30.000000000","uploader":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"ref":"refs/changes/31/702031/13","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-keystone","ref":"refs/changes/31/702031/13","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/13 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/13 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/13 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-keystone refs/changes/31/702031/13"}}},"commit":{"parents":[{"commit":"bc1ff1d7cb01ac02790c3302a3da6e994598d9f6","subject":"Add keystone::bootstrap class","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/bc1ff1d7cb01ac02790c3302a3da6e994598d9f6"}]}],"author":{"name":"Sai Sindhur Malleni","email":"smalleni@redhat.com","date":"2020-01-10 19:28:32.000000000","tz":-300},"committer":{"name":"Takashi Kajinami","email":"tkajinam@redhat.com","date":"2020-02-01 13:50:20.000000000","tz":540},"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","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/8f5301d20d73a84cc10ea2591414e45cf3d1fa79"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-keystone/commit/8f5301d20d73a84cc10ea2591414e45cf3d1fa79"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
