)]}'
{"id":"openstack%2Fpuppet-openstacklib~375146","triplet_id":"openstack%2Fpuppet-openstacklib~master~Ia588817fde9203a134bc9eaad9a5bd5453d7639a","project":"openstack/puppet-openstacklib","branch":"master","topic":"os_workers-fact","hashtags":[],"change_id":"Ia588817fde9203a134bc9eaad9a5bd5453d7639a","subject":"Create os_workers fact","status":"MERGED","created":"2016-09-22 22:34:53.000000000","updated":"2016-09-23 19:09:20.000000000","submitted":"2016-09-23 19:09:20.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"74037d46e2fc758efe81eb32acb1ceb2f7ad0b16","_number":375146,"virtual_id_number":375146,"owner":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":2,"date":"2016-09-23 19:09:20.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"_account_id":8971,"name":"Fuel CI","username":"fuel-ci","inactive":true,"tags":["SERVICE_USER"]},{"value":0,"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},{"value":0,"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"}],"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"},"recommended":{"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},"all":[{"value":1,"date":"2016-09-23 16:19:15.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},{"value":2,"date":"2016-09-23 16:16:13.000000000","_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"_account_id":8971,"name":"Fuel CI","username":"fuel-ci","inactive":true,"tags":["SERVICE_USER"]},{"value":1,"date":"2016-09-23 16:16:43.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},{"value":2,"date":"2016-09-23 17:44:24.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"}],"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":"","value":1,"default_value":0,"optional":true},"Workflow":{"approved":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"all":[{"value":0,"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"value":0,"_account_id":8971,"name":"Fuel CI","username":"fuel-ci","inactive":true,"tags":["SERVICE_USER"]},{"value":0,"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},{"value":1,"date":"2016-09-23 17:44:24.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},{"_account_id":8971,"name":"Fuel CI","username":"fuel-ci","inactive":true,"tags":["SERVICE_USER"]},{"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2016-09-23 16:16:13.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":"2016-09-23 16:16:43.000000000","updated_by":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"reviewer":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"state":"REVIEWER"},{"updated":"2016-09-23 16:19:15.000000000","updated_by":{"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},"reviewer":{"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},"state":"REVIEWER"},{"updated":"2016-09-23 16:57:49.000000000","updated_by":{"_account_id":8971,"name":"Fuel CI","username":"fuel-ci","inactive":true,"tags":["SERVICE_USER"]},"reviewer":{"_account_id":8971,"name":"Fuel CI","username":"fuel-ci","inactive":true,"tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2016-09-23 17:44:24.000000000","updated_by":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"reviewer":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"state":"REVIEWER"},{"updated":"2016-09-23 19:09:20.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"aa33827fb512f72c560f8f8b996c0780dd3eb63d","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2016-09-22 22:34:53.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"8bf6249a1fb5c6f892a1bf5fd931389d43f20688","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2016-09-22 22:36:49.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.","accounts_in_message":[],"_revision_number":2},{"id":"80d274b176500b8e8732732007331ca70a2421bb","author":{"_account_id":8971,"name":"Fuel CI","username":"fuel-ci","inactive":true,"tags":["SERVICE_USER"]},"date":"2016-09-22 22:36:54.000000000","message":"Patch Set 1:\n\nBuild Failed \n\n* master.puppet-openstack.fuel-library.pkgs.ubuntu.review_in_fuel_library https://ci.fuel-infra.org/job/master.puppet-openstack.fuel-library.pkgs.ubuntu.review_in_fuel_library/2870/ : ABORTED\n\n* master.puppet-openstack.fuel-library.pkgs.ubuntu.smoke_neutron https://ci.fuel-infra.org/job/master.puppet-openstack.fuel-library.pkgs.ubuntu.smoke_neutron/4963/ : ABORTED\n\n* master.puppet-openstack.fuel.noop https://ci.fuel-infra.org/job/master.puppet-openstack.fuel.noop/5026/ : ABORTED","accounts_in_message":[],"_revision_number":1},{"id":"96388dcd590adc2ba17e672f13ddeea97450dedc","author":{"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},"date":"2016-09-22 23:00:35.000000000","message":"Patch Set 2:\n\nThe code looks fine to me but on a SPARC T7-2 (the smallest T7 Oracle sells), $::os_workers will still end up being set to 128 (512 cores on these systems).  It might make sense to add a check on $::hardwareisa \u003d\u003d sparc and just hardcode the fact to being \u00278\u0027 (which seems to be what you\u0027re going for on 32 core Intel systems).","accounts_in_message":[],"_revision_number":2},{"id":"120371778d485db04c8f5d0276c1e4e756883d55","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2016-09-22 23:18:47.000000000","message":"Patch Set 2:\n\nwe can also adjust the math to make more sense on sparc but I don\u0027t know the range of cores for sparc or do a min/max to not let it exceed 8 at all. we\u0027ll technically get higher with hyperthreading in like a quad 8 core with hyper threading would be 16 ((4x8x2)/2)","accounts_in_message":[],"_revision_number":2},{"id":"15c3f1818943842f0d0b063a2079f0227da30901","author":{"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},"date":"2016-09-22 23:31:53.000000000","message":"Patch Set 2:\n\nThe thing with SPARC is admins can do physical resource partitioning (Solaris LDOMs) and crank a domain down to a single core (it would be really ill-advised but they *could* do it).\n\nMaybe something like:\n\nmin($::processorcount, 8) so if an admin does something really dumb we don\u0027t try to exceed the number of cores, but we cap it at 8 in case the system is not partitioned (or partitioned sensibly).","accounts_in_message":[],"_revision_number":2},{"id":"d6fe39fa377869a8e4e0a3d7a43b3d43bd03aed5","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2016-09-22 23:32:15.000000000","message":"Patch Set 2: Workflow-1","accounts_in_message":[],"_revision_number":2},{"id":"d989f8ec2fab60f9328f49575b58d59e101d87ae","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2016-09-22 23:35:37.000000000","message":"Patch Set 2:\n\nwell we can do a min(max($::processorcount/4, 2), 8) basically.  The goal of the $::processorcount/4 is to not prevent it from scaling up but just doing so at a much slower rate.  We kinda want it to scale up just not too crazy.  Of course this is our conservative opinionated approach since everyone is free to define whatever values they want.","accounts_in_message":[],"_revision_number":2},{"id":"15c649d91e50bf14afcfd2247d8a9b63643cc26b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-22 23:53:46.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see http://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n- gate-puppet-openstacklib-puppet-lint http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-lint/2d73c4a/ : FAILURE in 3m 06s\n- gate-puppet-openstacklib-puppet-syntax-3-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-syntax-3-centos-7/49e811c/ : SUCCESS in 3m 24s\n- gate-puppet-openstacklib-puppet-syntax-4-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-syntax-4-centos-7/801c9f1/ : FAILURE in 4m 37s\n- gate-puppet-openstacklib-puppet-unit-3.6-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-unit-3.6-centos-7/0e4bbf9/ : FAILURE in 4m 36s\n- gate-puppet-openstacklib-puppet-unit-3.8-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-unit-3.8-centos-7/7910a35/ : FAILURE in 5m 30s\n- gate-puppet-openstacklib-puppet-unit-4.5-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-unit-4.5-centos-7/bd00286/ : SUCCESS in 5m 22s\n- gate-puppet-openstacklib-puppet-unit-latest-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-unit-latest-centos-7/9683fd6/ : SUCCESS in 6m 45s (non-voting)\n- gate-puppet-openstacklib-puppet-beaker-rspec-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-beaker-rspec-centos-7/5156644/ : FAILURE in 5m 25s\n- gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial/9d0c3a3/ : FAILURE in 3m 19s\n- gate-puppet-openstack-integration-3-scenario001-tempest-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/95f4781/ : SUCCESS in 29m 28s\n- gate-puppet-openstack-integration-3-scenario002-tempest-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-3-scenario002-tempest-centos-7/620df65/ : SUCCESS in 36m 04s\n- gate-puppet-openstack-integration-3-scenario003-tempest-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/2cab4e5/ : SUCCESS in 39m 07s\n- gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-xenial/ca743d0/ : SUCCESS in 27m 30s\n- gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-3-scenario002-tempest-ubuntu-xenial/9fde551/ : SUCCESS in 33m 02s\n- gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-3-scenario003-tempest-ubuntu-xenial/fa777e1/ : SUCCESS in 28m 23s\n- gate-puppet-openstacklib-releasenotes http://docs-draft.openstack.org/46/375146/2/check/gate-puppet-openstacklib-releasenotes/2d175b7//releasenotes/build/html/ : SUCCESS in 1m 24s\n- gate-tripleo-ci-centos-7-nonha-multinode http://logs.openstack.org/46/375146/2/check/gate-tripleo-ci-centos-7-nonha-multinode/d928e7c/ : SUCCESS in 1h 14m 21s","accounts_in_message":[],"_revision_number":2},{"id":"d73ce80cc0feff186354736ab9e886466dce4033","author":{"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},"date":"2016-09-23 00:02:24.000000000","message":"Patch Set 2:\n\nThat sounds like it\u0027ll work just fine.","accounts_in_message":[],"_revision_number":2},{"id":"0578d4a837ca23531ca319d1777beb9a3caf55ef","author":{"_account_id":8971,"name":"Fuel CI","username":"fuel-ci","inactive":true,"tags":["SERVICE_USER"]},"date":"2016-09-23 00:40:21.000000000","message":"Patch Set 2:\n\nBuild Successful \n\n* master.puppet-openstack.fuel-library.pkgs.ubuntu.review_in_fuel_library https://ci.fuel-infra.org/job/master.puppet-openstack.fuel-library.pkgs.ubuntu.review_in_fuel_library/2871/ : SUCCESS\n\n* master.puppet-openstack.fuel-library.pkgs.ubuntu.smoke_neutron https://ci.fuel-infra.org/job/master.puppet-openstack.fuel-library.pkgs.ubuntu.smoke_neutron/4964/ : SUCCESS\n\n* master.puppet-openstack.fuel.noop https://ci.fuel-infra.org/job/master.puppet-openstack.fuel.noop/5027/ : SUCCESS","accounts_in_message":[],"_revision_number":2},{"id":"157f14300e11992b518c797fef85fb8843a2ab53","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2016-09-23 14:49:18.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"bc97c510dcf7c14afd1ec800204e439b0dc6fce1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-23 15:43:44.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-puppet-openstacklib-puppet-lint http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-lint/a1e9420/ : SUCCESS in 2m 56s\n- gate-puppet-openstacklib-puppet-syntax-3-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-syntax-3-centos-7/cd4e9bb/ : SUCCESS in 3m 02s\n- gate-puppet-openstacklib-puppet-syntax-4-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-syntax-4-centos-7/edfc12a/ : SUCCESS in 4m 31s\n- gate-puppet-openstacklib-puppet-unit-3.6-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-unit-3.6-centos-7/d34a0d3/ : SUCCESS in 6m 27s\n- gate-puppet-openstacklib-puppet-unit-3.8-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-unit-3.8-centos-7/79605bc/ : SUCCESS in 9m 49s\n- gate-puppet-openstacklib-puppet-unit-4.5-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-unit-4.5-centos-7/4bd36c5/ : SUCCESS in 9m 15s\n- gate-puppet-openstacklib-puppet-unit-latest-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-unit-latest-centos-7/ba9c7ab/ : SUCCESS in 9m 18s (non-voting)\n- gate-puppet-openstacklib-puppet-beaker-rspec-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-beaker-rspec-centos-7/7f33c24/ : SUCCESS in 6m 03s\n- gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial http://logs.openstack.org/46/375146/2/check/gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial/fbda1e6/ : SUCCESS in 5m 22s\n- gate-puppet-openstack-integration-4-scenario001-tempest-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/d34f68f/ : SUCCESS in 40m 26s\n- gate-puppet-openstack-integration-4-scenario002-tempest-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-4-scenario002-tempest-centos-7/ed461cd/ : SUCCESS in 39m 55s\n- gate-puppet-openstack-integration-4-scenario003-tempest-centos-7 http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/7963971/ : SUCCESS in 25m 35s\n- gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/47c84db/ : SUCCESS in 23m 02s\n- gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial/a0b7abd/ : SUCCESS in 27m 47s\n- gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/2/check/gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial/7679322/ : SUCCESS in 25m 37s\n- gate-puppet-openstacklib-releasenotes http://docs-draft.openstack.org/46/375146/2/check/gate-puppet-openstacklib-releasenotes/34ea0c9//releasenotes/build/html/ : SUCCESS in 1m 13s\n- gate-tripleo-ci-centos-7-nonha-multinode http://logs.openstack.org/46/375146/2/check/gate-tripleo-ci-centos-7-nonha-multinode/60afd73/ : SUCCESS in 46m 57s","accounts_in_message":[],"_revision_number":2},{"id":"7883c913f9d3984132ba0c1296ac184e009c553a","author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"date":"2016-09-23 16:14:01.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"eb81902be23d2a749881143332010e9de37f0a2b","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2016-09-23 16:16:13.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"a5b03e0202e7c721e9391f01e49bcc3d1b27bcc9","author":{"_account_id":8318,"name":"Matthew J Black","email":"mjblack@gmail.com","username":"mjblack"},"date":"2016-09-23 16:16:43.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"08627e9cd1041197f638e876fa81b267e2320ab6","author":{"_account_id":10353,"name":"Drew Fisher","email":"drew.fisher@oracle.com","username":"drew-fisher"},"date":"2016-09-23 16:19:15.000000000","message":"Patch Set 3: Code-Review+1","accounts_in_message":[],"_revision_number":3},{"id":"22a0000b1a666e28802a977c1059fb15d2176b3b","author":{"_account_id":8971,"name":"Fuel CI","username":"fuel-ci","inactive":true,"tags":["SERVICE_USER"]},"date":"2016-09-23 16:57:49.000000000","message":"Patch Set 2:\n\nBuild Successful \n\n* master.puppet-openstack.fuel-library.pkgs.ubuntu.review_in_fuel_library https://ci.fuel-infra.org/job/master.puppet-openstack.fuel-library.pkgs.ubuntu.review_in_fuel_library/2888/ : SUCCESS\n\n* master.puppet-openstack.fuel-library.pkgs.ubuntu.smoke_neutron https://ci.fuel-infra.org/job/master.puppet-openstack.fuel-library.pkgs.ubuntu.smoke_neutron/4981/ : SUCCESS\n\n* master.puppet-openstack.fuel.noop https://ci.fuel-infra.org/job/master.puppet-openstack.fuel.noop/5044/ : SUCCESS","accounts_in_message":[],"_revision_number":2},{"id":"f92e51fc2f8d724261b538afc83fce6ea9ea9cc1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-23 17:30:49.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-puppet-openstacklib-puppet-lint http://logs.openstack.org/46/375146/3/check/gate-puppet-openstacklib-puppet-lint/c871486/ : SUCCESS in 3m 54s\n- gate-puppet-openstacklib-puppet-syntax-3-centos-7 http://logs.openstack.org/46/375146/3/check/gate-puppet-openstacklib-puppet-syntax-3-centos-7/4ec6734/ : SUCCESS in 5m 27s\n- gate-puppet-openstacklib-puppet-syntax-4-centos-7 http://logs.openstack.org/46/375146/3/check/gate-puppet-openstacklib-puppet-syntax-4-centos-7/a3f1030/ : SUCCESS in 3m 36s\n- gate-puppet-openstacklib-puppet-unit-3.6-centos-7 http://logs.openstack.org/46/375146/3/check/gate-puppet-openstacklib-puppet-unit-3.6-centos-7/473d8d2/ : SUCCESS in 5m 41s\n- gate-puppet-openstacklib-puppet-unit-3.8-centos-7 http://logs.openstack.org/46/375146/3/check/gate-puppet-openstacklib-puppet-unit-3.8-centos-7/9562a1e/ : SUCCESS in 8m 53s\n- gate-puppet-openstacklib-puppet-unit-4.5-centos-7 http://logs.openstack.org/46/375146/3/check/gate-puppet-openstacklib-puppet-unit-4.5-centos-7/1b25be8/ : SUCCESS in 4m 47s\n- gate-puppet-openstacklib-puppet-unit-latest-centos-7 http://logs.openstack.org/46/375146/3/check/gate-puppet-openstacklib-puppet-unit-latest-centos-7/2f239c3/ : SUCCESS in 4m 12s (non-voting)\n- gate-puppet-openstacklib-puppet-beaker-rspec-centos-7 http://logs.openstack.org/46/375146/3/check/gate-puppet-openstacklib-puppet-beaker-rspec-centos-7/89cb34f/ : SUCCESS in 7m 39s\n- gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial http://logs.openstack.org/46/375146/3/check/gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial/0c4dcd4/ : SUCCESS in 7m 02s\n- gate-puppet-openstack-integration-4-scenario001-tempest-centos-7 http://logs.openstack.org/46/375146/3/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/8934593/ : SUCCESS in 39m 26s\n- gate-puppet-openstack-integration-4-scenario002-tempest-centos-7 http://logs.openstack.org/46/375146/3/check/gate-puppet-openstack-integration-4-scenario002-tempest-centos-7/edd9052/ : SUCCESS in 35m 06s\n- gate-puppet-openstack-integration-4-scenario003-tempest-centos-7 http://logs.openstack.org/46/375146/3/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/a2cc090/ : SUCCESS in 43m 56s\n- gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/3/check/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/010fdfa/ : SUCCESS in 24m 47s\n- gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/3/check/gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial/47bfbfe/ : SUCCESS in 35m 13s\n- gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/3/check/gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial/d52cc35/ : SUCCESS in 25m 34s\n- gate-puppet-openstacklib-releasenotes http://docs-draft.openstack.org/46/375146/3/check/gate-puppet-openstacklib-releasenotes/d6f51c0//releasenotes/build/html/ : SUCCESS in 1m 28s\n- gate-tripleo-ci-centos-7-nonha-multinode http://logs.openstack.org/46/375146/3/check/gate-tripleo-ci-centos-7-nonha-multinode/5c8bbd9/ : SUCCESS in 1h 10m 30s","accounts_in_message":[],"_revision_number":3},{"id":"527d639a8c246a5c17ec203ae8d8b7807c397e6b","author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"date":"2016-09-23 17:44:24.000000000","message":"Patch Set 3: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":3},{"id":"93f51d063a8101ac2982b6379fa0dfe43f1d99bb","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-23 17:46:19.000000000","message":"Patch Set 3: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"f4e84f00d60eac0d0f3456af6eb84b99bcd708eb","author":{"_account_id":8971,"name":"Fuel CI","username":"fuel-ci","inactive":true,"tags":["SERVICE_USER"]},"date":"2016-09-23 18:21:44.000000000","message":"Patch Set 3:\n\nBuild Failed \n\n* master.puppet-openstack.fuel-library.pkgs.ubuntu.review_in_fuel_library https://ci.fuel-infra.org/job/master.puppet-openstack.fuel-library.pkgs.ubuntu.review_in_fuel_library/2893/ : FAILURE\n\n* master.puppet-openstack.fuel-library.pkgs.ubuntu.smoke_neutron https://ci.fuel-infra.org/job/master.puppet-openstack.fuel-library.pkgs.ubuntu.smoke_neutron/4986/ : SUCCESS\n\n* master.puppet-openstack.fuel.noop https://ci.fuel-infra.org/job/master.puppet-openstack.fuel.noop/5049/ : SUCCESS","accounts_in_message":[],"_revision_number":3},{"id":"fa0eb5b5d11c5831d1f897cc1b3dd8b28a3cedfb","date":"2016-09-23 19:09:20.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":3},{"id":"f42f1b77c1254687aad3b6478c23ad5e13807b1e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2016-09-23 19:09:20.000000000","message":"Patch Set 3: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-puppet-openstacklib-puppet-lint http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstacklib-puppet-lint/8199a6b/ : SUCCESS in 3m 09s\n- gate-puppet-openstacklib-puppet-syntax-3-centos-7 http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstacklib-puppet-syntax-3-centos-7/e1d6629/ : SUCCESS in 3m 45s\n- gate-puppet-openstacklib-puppet-syntax-4-centos-7 http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstacklib-puppet-syntax-4-centos-7/a412a2d/ : SUCCESS in 5m 25s\n- gate-puppet-openstacklib-puppet-unit-3.6-centos-7 http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstacklib-puppet-unit-3.6-centos-7/af7ea13/ : SUCCESS in 5m 06s\n- gate-puppet-openstacklib-puppet-unit-3.8-centos-7 http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstacklib-puppet-unit-3.8-centos-7/b8f3c82/ : SUCCESS in 5m 03s\n- gate-puppet-openstacklib-puppet-unit-4.5-centos-7 http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstacklib-puppet-unit-4.5-centos-7/2d7e645/ : SUCCESS in 4m 36s\n- gate-puppet-openstacklib-puppet-beaker-rspec-centos-7 http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstacklib-puppet-beaker-rspec-centos-7/712feb2/ : SUCCESS in 8m 11s\n- gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial/e34dc56/ : SUCCESS in 5m 35s\n- gate-puppet-openstack-integration-4-scenario001-tempest-centos-7 http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/ba5c12f/ : SUCCESS in 34m 54s\n- gate-puppet-openstack-integration-4-scenario002-tempest-centos-7 http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstack-integration-4-scenario002-tempest-centos-7/aaf6f38/ : SUCCESS in 25m 25s\n- gate-puppet-openstack-integration-4-scenario003-tempest-centos-7 http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/74fb770/ : SUCCESS in 24m 53s\n- gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/3696ac3/ : SUCCESS in 26m 11s\n- gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial/23e818c/ : SUCCESS in 30m 33s\n- gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial http://logs.openstack.org/46/375146/3/gate/gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial/1b0311c/ : SUCCESS in 23m 08s\n- gate-puppet-openstacklib-releasenotes http://docs-draft.openstack.org/46/375146/3/gate/gate-puppet-openstacklib-releasenotes/d46fef5//releasenotes/build/html/ : SUCCESS in 2m 05s\n- gate-tripleo-ci-centos-7-nonha-multinode http://logs.openstack.org/46/375146/3/gate/gate-tripleo-ci-centos-7-nonha-multinode/2ed2169/ : SUCCESS in 1h 12m 30s","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"1b6c0a7272f175cffd42c94835bdf31eaf5a3d57","revisions":{"c3a509d584fa6c4fa408786fb8bafb08ecdb0795":{"kind":"REWORK","_number":1,"created":"2016-09-22 22:34:53.000000000","uploader":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"ref":"refs/changes/46/375146/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-openstacklib","ref":"refs/changes/46/375146/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/1"}}},"commit":{"parents":[{"commit":"7c889eccbee0c057a8776de1e17b2633a508edb7","subject":"Prepare 9.2.0 (newton b3)","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/7c889eccbee0c057a8776de1e17b2633a508edb7"}]}],"author":{"name":"Alex Schultz","email":"aschultz@redhat.com","date":"2016-09-22 22:32:14.000000000","tz":-360},"committer":{"name":"Alex Schultz","email":"aschultz@redhat.com","date":"2016-09-22 22:34:52.000000000","tz":-360},"subject":"Create os_workers fact","message":"Create os_workers fact\n\nThis change adds a new fact that can be used as the default values for\nany worker configuration. When used this fact will reduce the number of\ndefault workers configured for services. Previously we were using\n$::processorcount which could lead to excessive memory consumption or\nconnection limits being execeeded when services were deployed on\nbaremetal with large number of processors. The os_workers fact uses the\nlarger of two following values, \u00272\u0027 or \u0027# processors / 4\u0027\n\nChange-Id: Ia588817fde9203a134bc9eaad9a5bd5453d7639a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/c3a509d584fa6c4fa408786fb8bafb08ecdb0795"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/c3a509d584fa6c4fa408786fb8bafb08ecdb0795"}]},"branch":"refs/heads/master"},"ff2a87e2147722dcb5d747a30a6906b02a1b8cb1":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2016-09-22 22:36:49.000000000","uploader":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"ref":"refs/changes/46/375146/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-openstacklib","ref":"refs/changes/46/375146/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/2"}}},"commit":{"parents":[{"commit":"e6e3ee03d087b01f4915fe963068a876ef97fdc3","subject":"Merge \"Add support for setting a timeout for service_validation\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/e6e3ee03d087b01f4915fe963068a876ef97fdc3"}]}],"author":{"name":"Alex Schultz","email":"aschultz@redhat.com","date":"2016-09-22 22:32:14.000000000","tz":-360},"committer":{"name":"Alex Schultz","email":"aschultz@redhat.com","date":"2016-09-22 22:36:45.000000000","tz":-360},"subject":"Create os_workers fact","message":"Create os_workers fact\n\nThis change adds a new fact that can be used as the default values for\nany worker configuration. When used this fact will reduce the number of\ndefault workers configured for services. Previously we were using\n$::processorcount which could lead to excessive memory consumption or\nconnection limits being execeeded when services were deployed on\nbaremetal with large number of processors. The os_workers fact uses the\nlarger of two following values, \u00272\u0027 or \u0027# processors / 4\u0027\n\nChange-Id: Ia588817fde9203a134bc9eaad9a5bd5453d7639a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/ff2a87e2147722dcb5d747a30a6906b02a1b8cb1"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/ff2a87e2147722dcb5d747a30a6906b02a1b8cb1"}]},"branch":"refs/heads/master"},"1b6c0a7272f175cffd42c94835bdf31eaf5a3d57":{"kind":"REWORK","_number":3,"created":"2016-09-23 16:14:01.000000000","uploader":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"ref":"refs/changes/46/375146/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-openstacklib","ref":"refs/changes/46/375146/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/375146/3"}}},"commit":{"parents":[{"commit":"e6e3ee03d087b01f4915fe963068a876ef97fdc3","subject":"Merge \"Add support for setting a timeout for service_validation\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/e6e3ee03d087b01f4915fe963068a876ef97fdc3"}]}],"author":{"name":"Alex Schultz","email":"aschultz@redhat.com","date":"2016-09-22 22:32:14.000000000","tz":-360},"committer":{"name":"Alex Schultz","email":"aschultz@redhat.com","date":"2016-09-23 16:13:46.000000000","tz":-360},"subject":"Create os_workers fact","message":"Create os_workers fact\n\nThis change adds a new fact that can be used as the default values for\nany worker configuration. When used this fact will reduce the number of\ndefault workers configured for services. Previously we were using\n$::processorcount which could lead to excessive memory consumption or\nconnection limits being execeeded when services were deployed on\nbaremetal with large number of processors. The os_workers fact uses the\nlarger of two following values, \u00272\u0027 or \u0027# processors / 4\u0027 but will not\nexceed \u00278\u0027.\n\nChange-Id: Ia588817fde9203a134bc9eaad9a5bd5453d7639a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/1b6c0a7272f175cffd42c94835bdf31eaf5a3d57"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/1b6c0a7272f175cffd42c94835bdf31eaf5a3d57"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}