)]}'
{"id":"openstack%2Ftripleo-image-elements~83323","triplet_id":"openstack%2Ftripleo-image-elements~master~I3945f98fe3fca3a8e74de798068c3abce5a98d5e","project":"openstack/tripleo-image-elements","branch":"master","topic":"set-include_service_catalog-false","hashtags":[],"change_id":"I3945f98fe3fca3a8e74de798068c3abce5a98d5e","subject":"Set include_service_catalog and sorting_method in proxy-server.conf","status":"ABANDONED","created":"2014-03-27 09:36:00.000000000","updated":"2014-06-19 07:47:48.000000000","total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"5bfa7367fb9325ab5d9e0730824bd48ea660ffd9","_number":83323,"virtual_id_number":83323,"owner":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"_account_id":4330,"name":"Tomas Sedovic","email":"tsedovic@redhat.com","username":"tomas-8c8"},{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},{"_account_id":5639,"name":"Coleman Corrigan","email":"coleman.corrigan@hp.com","username":"coleman-corrigan"},{"value":1,"date":"2014-05-06 12:56:54.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":4330,"name":"Tomas Sedovic","email":"tsedovic@redhat.com","username":"tomas-8c8"},"recommended":{"_account_id":5639,"name":"Coleman Corrigan","email":"coleman.corrigan@hp.com","username":"coleman-corrigan"},"disliked":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"value":-1,"date":"2014-05-06 12:52:47.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"value":2,"date":"2014-04-10 12:51:10.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":4330,"name":"Tomas Sedovic","email":"tsedovic@redhat.com","username":"tomas-8c8"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},{"value":1,"date":"2014-04-10 09:46:22.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":5639,"name":"Coleman Corrigan","email":"coleman.corrigan@hp.com","username":"coleman-corrigan"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":{"rejected":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"all":[{"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"date":"2014-04-10 12:51:10.000000000","_account_id":4330,"name":"Tomas Sedovic","email":"tsedovic@redhat.com","username":"tomas-8c8"},{"value":-1,"date":"2014-05-07 16:33:06.000000000","permitted_voting_range":{"min":-1,"max":0},"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},{"_account_id":5639,"name":"Coleman Corrigan","email":"coleman.corrigan@hp.com","username":"coleman-corrigan"},{"_account_id":3,"name":"Jenkins","username":"jenkins"}],"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":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},{"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},{"_account_id":4330,"name":"Tomas Sedovic","email":"tsedovic@redhat.com","username":"tomas-8c8"},{"_account_id":5639,"name":"Coleman Corrigan","email":"coleman.corrigan@hp.com","username":"coleman-corrigan"},{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2014-03-27 15:20:29.000000000","updated_by":{"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},"reviewer":{"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},"state":"REVIEWER"},{"updated":"2014-03-28 16:32:20.000000000","updated_by":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"reviewer":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"state":"REVIEWER"},{"updated":"2014-03-30 01:17:09.000000000","updated_by":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"reviewer":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"state":"REVIEWER"},{"updated":"2014-04-04 10:41:50.000000000","updated_by":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"reviewer":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"state":"REVIEWER"},{"updated":"2014-04-10 09:46:22.000000000","updated_by":{"_account_id":5639,"name":"Coleman Corrigan","email":"coleman.corrigan@hp.com","username":"coleman-corrigan"},"reviewer":{"_account_id":5639,"name":"Coleman Corrigan","email":"coleman.corrigan@hp.com","username":"coleman-corrigan"},"state":"REVIEWER"},{"updated":"2014-04-10 12:51:10.000000000","updated_by":{"_account_id":4330,"name":"Tomas Sedovic","email":"tsedovic@redhat.com","username":"tomas-8c8"},"reviewer":{"_account_id":4330,"name":"Tomas Sedovic","email":"tsedovic@redhat.com","username":"tomas-8c8"},"state":"REVIEWER"},{"updated":"2014-05-06 12:52:47.000000000","updated_by":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"reviewer":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"state":"REVIEWER"},{"updated":"2014-05-06 12:56:54.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"8adcc901fc0010b2494d7ede89952c231e5d7893","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-27 09:36:00.000000000","message":"Patch Set 1:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"927461d02396163f06ceee5230f38bf7a5fc4be1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-27 09:36:02.000000000","message":"Patch Set 1: Works for me\n\nBuild succeeded.\n\n- noop http://logs.openstack.org/23/83323/1/check/noop/5472186 : SUCCESS\n\n","accounts_in_message":[],"_revision_number":1},{"id":"4ff3254b54f10745cb5080ba4417c2dede51f75c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-27 11:01:22.000000000","message":"Patch Set 1:\n\nBuild succeeded.\n\n- check-tripleo-seed-precise http://logs.openstack.org/23/83323/1/check-tripleo/check-tripleo-seed-precise/6dc8eff : SUCCESS in 37m 53s\n- check-tripleo-ironic-seed-precise http://logs.openstack.org/23/83323/1/check-tripleo/check-tripleo-ironic-seed-precise/7a70455 : SUCCESS in 38m 35s\n- check-tripleo-undercloud-precise http://logs.openstack.org/23/83323/1/check-tripleo/check-tripleo-undercloud-precise/1955a94 : SUCCESS in 56m 57s\n- check-tripleo-ironic-undercloud-precise http://logs.openstack.org/23/83323/1/check-tripleo/check-tripleo-ironic-undercloud-precise/209fc63 : SUCCESS in 58m 02s\n- check-tripleo-overcloud-precise http://logs.openstack.org/23/83323/1/check-tripleo/check-tripleo-overcloud-precise/5441509 : SUCCESS in 1h 25m 20s\n\n","accounts_in_message":[],"_revision_number":1},{"id":"db3a880f8f6ea808810f10fa6cffffcd7154c77e","author":{"_account_id":5639,"name":"Coleman Corrigan","email":"coleman.corrigan@hp.com","username":"coleman-corrigan"},"date":"2014-03-27 11:12:56.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\n(1 inline comment)\n\nminor case issue","accounts_in_message":[],"_revision_number":1},{"id":"52157eb4e03e878200148f4ececf71c0ec959baf","author":{"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},"date":"2014-03-27 11:33:05.000000000","message":"Patch Set 1: I would prefer that you didn\u0027t merge this\n\nAs it happens, the example config and keystone documentation uses \"False\"\n\n\"False\" vs \"false\" is not consistent throughout tripleo-image-elements. However, within the swift elements, lowercase seems the norm, so agree with the -1.","accounts_in_message":[],"_revision_number":1},{"id":"210324a5be8233d129e19f49b2a18edb483a7309","author":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"date":"2014-03-27 13:43:39.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"2dd1ca77b26715a0a8922b71afa09a427cae91da","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-27 13:43:40.000000000","message":"Patch Set 2:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":2},{"id":"2e62049b3442f75e1f16cccba9d2e860c7dad71f","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-27 13:43:41.000000000","message":"Patch Set 2: Works for me\n\nBuild succeeded.\n\n- noop http://logs.openstack.org/23/83323/2/check/noop/d16e897 : SUCCESS\n\n","accounts_in_message":[],"_revision_number":2},{"id":"8b23458c43982046770818587275fa1cd5d8d98e","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-27 14:00:36.000000000","message":"Patch Set 3:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"9460912dcbebb52c133e0ed27288ad9438f9e9a5","author":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"date":"2014-03-27 14:00:36.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"ac02d8fa8457cfd0e6a4fe09556fa4cb073dff91","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-27 14:00:38.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- noop http://logs.openstack.org/23/83323/3/check/noop/8ca8757 : SUCCESS\n\n","accounts_in_message":[],"_revision_number":3},{"id":"139e17c50dc20e3eb7ee7e6391c7a6cc01d1a560","author":{"_account_id":5494,"name":"Trivial Rebase","email":"trivial-rebase@review.openstack.org","username":"trivial-rebase"},"date":"2014-03-27 14:00:42.000000000","message":"Patch Set 3:\n\nNew patchset patch-id matches previous patchset, but commit message has changed.","accounts_in_message":[],"_revision_number":3},{"id":"d781be5e0e998dfc60fa8f07ef29f4f23546e466","author":{"_account_id":5639,"name":"Coleman Corrigan","email":"coleman.corrigan@hp.com","username":"coleman-corrigan"},"date":"2014-03-27 14:02:19.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"241b4653dbd83bc354e18f9130e67e8a9101639c","author":{"_account_id":860,"name":"Donagh McCabe","email":"donagh.mccabe@gmail.com","username":"donagh-mccabe"},"date":"2014-03-27 15:20:29.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"b9b60309516bf61fd8eea862514743685e4e5a13","author":{"_account_id":4330,"name":"Tomas Sedovic","email":"tsedovic@redhat.com","username":"tomas-8c8"},"date":"2014-03-27 15:26:37.000000000","message":"Patch Set 3: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":3},{"id":"cc5722b68ec41a7967b04f57a7e74b3ea2be001b","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-03-27 15:36:31.000000000","message":"Patch Set 3:\n\nBuild succeeded.\n\n- check-tripleo-seed-precise http://logs.openstack.org/23/83323/3/check-tripleo/check-tripleo-seed-precise/0217323 : SUCCESS in 30m 31s\n- check-tripleo-ironic-seed-precise http://logs.openstack.org/23/83323/3/check-tripleo/check-tripleo-ironic-seed-precise/d9dee44 : SUCCESS in 31m 24s\n- check-tripleo-undercloud-precise http://logs.openstack.org/23/83323/3/check-tripleo/check-tripleo-undercloud-precise/bf7d9c0 : SUCCESS in 1h 00m 57s\n- check-tripleo-ironic-undercloud-precise http://logs.openstack.org/23/83323/3/check-tripleo/check-tripleo-ironic-undercloud-precise/cf8335e : SUCCESS in 57m 17s\n- check-tripleo-overcloud-precise http://logs.openstack.org/23/83323/3/check-tripleo/check-tripleo-overcloud-precise/79d4aa9 : SUCCESS in 1h 31m 13s\n\n","accounts_in_message":[],"_revision_number":3},{"id":"bd15551d653e9aa48eeec725416a38dde2f16fcf","author":{"_account_id":1501,"name":"Phil Day","email":"philip.day@hp.com","username":"philip-day"},"date":"2014-03-28 16:32:20.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"0fb7295ad64c877b87e631669c18cedd371b90ec","author":{"_account_id":360,"name":"Dan Prince","email":"dprince@redhat.com","username":"dan-prince"},"date":"2014-03-30 01:17:09.000000000","message":"Patch Set 3: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":3},{"id":"3df542f6caf0454daa67abd8db4d86596d4ed1e0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-04-04 10:41:50.000000000","message":"Patch Set 3: No score\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":3},{"id":"d33c8b91fbca97ca3945a9b5c0775dad788bc09f","author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"date":"2014-04-04 10:41:50.000000000","message":"Patch Set 3: I would prefer that you didn\u0027t merge this\n\nCould you file a bug about this on swift then? They have a non-production default value.","accounts_in_message":[],"_revision_number":3},{"id":"359ec1783d3f7d98199fd13bac41e3c1cc0400ac","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-04-04 10:41:52.000000000","message":"Patch Set 3: Works for me\n\nBuild succeeded.\n\n- noop http://logs.openstack.org/23/83323/3/check/noop/f28e395 : SUCCESS\n\n","accounts_in_message":[],"_revision_number":3},{"id":"1852435a0c1cf3059812d5540e5cbb1bc7dab9f7","author":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"date":"2014-04-04 11:45:55.000000000","message":"Patch Set 3:\n\ninclude_service_catalog is a keystone auth_token middleware attribute. Its default value is True. This is a reasonable default for most users of the keystone auth_token middleware but not for swift. I don\u0027t think it is feasible/appropriate to change the default of this keystone attribute in swift code.","accounts_in_message":[],"_revision_number":3},{"id":"d5a891f448d7dd8eb582ce3f29179fd042be616c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-04-09 16:28:19.000000000","message":"Patch Set 4:\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":4},{"id":"185b12729b970f625aa9843bcc2148cd42195abd","author":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"date":"2014-04-09 16:28:19.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"180a748876bcc12f5e07dcb0fedb8579e2a4dd49","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-04-09 16:28:21.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- noop http://logs.openstack.org/23/83323/4/check/noop/822ea15 : SUCCESS\n","accounts_in_message":[],"_revision_number":4},{"id":"7159f1efb6d4d1b2776a2cc8ceba96faa0d14250","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-04-10 00:12:38.000000000","message":"Patch Set 4:\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n\n- check-tripleo-seed-precise http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-seed-precise/f569db2 : FAILURE in 19m 21s\n- check-tripleo-ironic-seed-precise http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-ironic-seed-precise/7e61cfc : SUCCESS in 37m 14s\n- check-tripleo-undercloud-precise http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-undercloud-precise/3ecbc7b : FAILURE in 32m 35s\n- check-tripleo-ironic-undercloud-precise http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-ironic-undercloud-precise/a1fd13e : FAILURE in 56m 51s\n- check-tripleo-overcloud-precise http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-overcloud-precise/0edb7c4 : FAILURE in 20m 36s\n- check-tripleo-overcloud-f20 http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-overcloud-f20/c1f9439 : FAILURE in 54m 04s\n","accounts_in_message":[],"_revision_number":4},{"id":"62943450792fbb66d404481a3752c082e7de5461","author":{"_account_id":5639,"name":"Coleman Corrigan","email":"coleman.corrigan@hp.com","username":"coleman-corrigan"},"date":"2014-04-10 09:46:22.000000000","message":"Patch Set 4: Looks good to me, but someone else must approve\n\n","accounts_in_message":[],"_revision_number":4},{"id":"2f007c955956289102eff69d9ff87731d1c47b11","author":{"_account_id":4330,"name":"Tomas Sedovic","email":"tsedovic@redhat.com","username":"tomas-8c8"},"date":"2014-04-10 12:51:10.000000000","message":"Patch Set 4: Looks good to me (core reviewer)\n\n","accounts_in_message":[],"_revision_number":4},{"id":"e2debc7e7799057555511e9f98096761201fb5ed","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-04-23 11:53:28.000000000","message":"Patch Set 4: No score\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":4},{"id":"e05e8cdeef9c6a6b6c38df3ad641b6d5313ecd7f","author":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"date":"2014-04-23 11:53:28.000000000","message":"Patch Set 4:\n\nIs anything else needed to get this one through the review/approval process? I believe I\u0027ve complied with all requests for change. If I\u0027m mistaken, I\u0027d appreciate as a tripleo newbie, some guidance on how to progress this change. All input is much appreciated.","accounts_in_message":[],"_revision_number":4},{"id":"d6a53bdbec5a6e3e1a5c94084676b324d93e9ee0","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-04-23 12:36:07.000000000","message":"Patch Set 4: Works for me\n\nBuild succeeded.\n\n- check-oslotest-dsvm-tripleo-image-elements http://logs.openstack.org/23/83323/4/check/check-oslotest-dsvm-tripleo-image-elements/e5c173e : SUCCESS in 3m 28s\n","accounts_in_message":[],"_revision_number":4},{"id":"a5a5df43e56a2eb09f4d3e78adad97ded1c67ff1","author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"date":"2014-05-06 12:52:47.000000000","message":"Patch Set 4: Code-Review-1\n\ni agree with what lifeless was getting at. If swift recommends setting include_service_catalog to false, then that should be the default in their code.\n\nWe try to avoid overriding upstream defaults where we can. In this case, if false is really the correct value here, it should be the upstream default in swift.","accounts_in_message":[],"_revision_number":4},{"id":"b7a4b161c8d37f773864147fe953ca522dc52619","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-05-06 12:52:52.000000000","message":"Patch Set 4: -Verified\n\nStarting check jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":4},{"id":"0e53d5b66dd657cd44fad24558cb4bdba5dcd50c","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-05-06 12:56:54.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded.\n\n- check-oslotest-dsvm-tripleo-image-elements http://logs.openstack.org/23/83323/4/check/check-oslotest-dsvm-tripleo-image-elements/baa80cd : SUCCESS in 3m 57s","accounts_in_message":[],"_revision_number":4},{"id":"da29ca761c3d9fd68340381cfecafa20cab1c7d2","author":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"date":"2014-05-06 13:59:57.000000000","message":"Patch Set 4:\n\nThanks for the comments James. The issue I have is that include_service_catalog is a keystone authtoken middleware attribute and is not referenced anywhere in swift code. So I\u0027m reluctant to try and set a default value for the attribute in swift given the swift code does not use it.","accounts_in_message":[],"_revision_number":4},{"id":"821c5a555d34ae734545ccea37b9bcd8af8ad338","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2014-05-06 21:03:10.000000000","message":"Patch Set 4:\n\nBuild failed.  For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures\n\n\n- check-tripleo-seed-precise http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-seed-precise/e5915cc : SUCCESS in 39m 40s\n- check-tripleo-ironic-seed-precise http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-ironic-seed-precise/7caadd1 : SUCCESS in 40m 33s\n- check-tripleo-undercloud-precise http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-undercloud-precise/876d0e7 : SUCCESS in 51m 25s\n- check-tripleo-ironic-undercloud-precise http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-ironic-undercloud-precise/886af84 : FAILURE in 58m 27s (non-voting)\n- check-tripleo-overcloud-precise http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-overcloud-precise/02c6032 : FAILURE in 2h 05m 43s\n- check-tripleo-overcloud-f20 http://logs.openstack.org/23/83323/4/check-tripleo/check-tripleo-overcloud-f20/341cab2 : SUCCESS in 1h 17m 54s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"0068eea2176e14386b67407487079610bbbf0a28","author":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"date":"2014-05-07 16:32:26.000000000","message":"Patch Set 4:\n\nI\u0027m hoping to use passthrough to configure include_service_catalog and sorting_method in proxy-server.conf. When ready I\u0027ll abandon this change.","accounts_in_message":[],"_revision_number":4},{"id":"e77d98b136e7bf4cc470af922cdf4070576f5a50","author":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"date":"2014-05-07 16:33:06.000000000","message":"Patch Set 4: Workflow-1","accounts_in_message":[],"_revision_number":4},{"id":"e7278021002b2cdfaccacb19b8093e360738e137","author":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"date":"2014-06-19 07:47:48.000000000","message":"Abandoned\n\nSetting these attributes is now handled using the passthrough mechanism.","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"efd2bfee2ecdfb13410ada0d5aa4941b540334a7","revisions":{"e4a60a8dc8ad52bec8d685c92fe40f0a01427f97":{"kind":"REWORK","_number":1,"created":"2014-03-27 09:36:00.000000000","uploader":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"ref":"refs/changes/23/83323/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tripleo-image-elements","ref":"refs/changes/23/83323/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/1"}}},"commit":{"parents":[{"commit":"5e52c4bcdf9dcdf88737d46cd51ad8a7b348afb9","subject":"Merge \"Setting negative RamWeigher\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/5e52c4bcdf9dcdf88737d46cd51ad8a7b348afb9"}]}],"author":{"name":"Gerry Drudy","email":"gerry.drudy@hp.com","date":"2014-03-26 17:09:13.000000000","tz":0},"committer":{"name":"Gerry Drudy","email":"gerry.drudy@hp.com","date":"2014-03-27 09:38:03.000000000","tz":0},"subject":"Set include_service_catalog to False","message":"Set include_service_catalog to False\n\nThe swift docs in swift/doc/source/overview_auth.rst recommend setting\ninclude_service_catalog false (it has a default value of true) since Swift\ndoes not use the X-Service-Catalog header, there is no point in getting the\nservice catalog\n\nChange-Id: I3945f98fe3fca3a8e74de798068c3abce5a98d5e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/e4a60a8dc8ad52bec8d685c92fe40f0a01427f97"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/e4a60a8dc8ad52bec8d685c92fe40f0a01427f97"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"5e52c4bcdf9dcdf88737d46cd51ad8a7b348afb9","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"2f49f6f01ac816b5ff57c1987b590cb1dd917f33":{"kind":"REWORK","_number":2,"created":"2014-03-27 13:43:39.000000000","uploader":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"ref":"refs/changes/23/83323/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tripleo-image-elements","ref":"refs/changes/23/83323/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/2"}}},"commit":{"parents":[{"commit":"5e52c4bcdf9dcdf88737d46cd51ad8a7b348afb9","subject":"Merge \"Setting negative RamWeigher\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/5e52c4bcdf9dcdf88737d46cd51ad8a7b348afb9"}]}],"author":{"name":"Gerry Drudy","email":"gerry.drudy@hp.com","date":"2014-03-26 17:09:13.000000000","tz":0},"committer":{"name":"Gerry Drudy","email":"gerry.drudy@hp.com","date":"2014-03-27 13:44:59.000000000","tz":0},"subject":"Set include_service_catalog to False","message":"Set include_service_catalog to False\n\nThe swift docs in swift/doc/source/overview_auth.rst recommend setting\ninclude_service_catalog false (it has a default value of true) since Swift\ndoes not use the X-Service-Catalog header, there is no point in getting the\nservice catalog\n\nChange-Id: I3945f98fe3fca3a8e74de798068c3abce5a98d5e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/2f49f6f01ac816b5ff57c1987b590cb1dd917f33"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/2f49f6f01ac816b5ff57c1987b590cb1dd917f33"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"5e52c4bcdf9dcdf88737d46cd51ad8a7b348afb9","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"71be13a5f38ddbab774e0ca93031cf0b5f74fd63":{"kind":"NO_CODE_CHANGE","_number":3,"created":"2014-03-27 14:00:36.000000000","uploader":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"ref":"refs/changes/23/83323/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tripleo-image-elements","ref":"refs/changes/23/83323/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/3"}}},"commit":{"parents":[{"commit":"5e52c4bcdf9dcdf88737d46cd51ad8a7b348afb9","subject":"Merge \"Setting negative RamWeigher\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/5e52c4bcdf9dcdf88737d46cd51ad8a7b348afb9"}]}],"author":{"name":"Gerry Drudy","email":"gerry.drudy@hp.com","date":"2014-03-26 17:09:13.000000000","tz":0},"committer":{"name":"Gerry Drudy","email":"gerry.drudy@hp.com","date":"2014-03-27 14:02:11.000000000","tz":0},"subject":"Set include_service_catalog to false","message":"Set include_service_catalog to false\n\nThe swift docs in swift/doc/source/overview_auth.rst recommend setting\ninclude_service_catalog false (it has a default value of true) since Swift\ndoes not use the X-Service-Catalog header, there is no point in getting the\nservice catalog\n\nChange-Id: I3945f98fe3fca3a8e74de798068c3abce5a98d5e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/71be13a5f38ddbab774e0ca93031cf0b5f74fd63"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/71be13a5f38ddbab774e0ca93031cf0b5f74fd63"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"5e52c4bcdf9dcdf88737d46cd51ad8a7b348afb9","is_merged_in_target_branch":true}],"branch":"refs/heads/master"},"efd2bfee2ecdfb13410ada0d5aa4941b540334a7":{"kind":"REWORK","_number":4,"created":"2014-04-09 16:28:19.000000000","uploader":{"_account_id":1253,"name":"Gerry Drudy","email":"gerry.drudy@hpe.com","username":"gerry-drudy"},"ref":"refs/changes/23/83323/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/tripleo-image-elements","ref":"refs/changes/23/83323/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/tripleo-image-elements refs/changes/23/83323/4"}}},"commit":{"parents":[{"commit":"91b77ae23b423988f6fbc6fc9b18028e0a145b26","subject":"Merge \"Add support for \"signals\" to os-refresh-config\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/91b77ae23b423988f6fbc6fc9b18028e0a145b26"}]}],"author":{"name":"Gerry Drudy","email":"gerry.drudy@hp.com","date":"2014-03-26 17:09:13.000000000","tz":0},"committer":{"name":"Gerry Drudy","email":"gerry.drudy@hp.com","date":"2014-04-09 16:25:01.000000000","tz":60},"subject":"Set include_service_catalog and sorting_method in proxy-server.conf","message":"Set include_service_catalog and sorting_method in proxy-server.conf\n\nThe swift docs in swift/doc/source/overview_auth.rst recommend setting\ninclude_service_catalog false (it has a default value of true) since Swift\ndoes not use the X-Service-Catalog header, there is no point in getting the\nservice catalog\n\nThe sorting_method attribute is used by swift to determine how\nstorage nodes are chosen. Valid values are shuffle (choose nodes at random),\ntiming (by using timing measurements), or affinity (by using an explicit match)\n\nThe default sorting_method is shuffle\n\nThe SwiftSortingMethod Parameter is defined in the swift-source.yaml\nSee https://review.openstack.org/#/c/84470/\n\nChange-Id: I3945f98fe3fca3a8e74de798068c3abce5a98d5e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/efd2bfee2ecdfb13410ada0d5aa4941b540334a7"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/tripleo-image-elements/commit/efd2bfee2ecdfb13410ada0d5aa4941b540334a7"}]},"parents_data":[{"branch_name":"refs/heads/master","commit_id":"91b77ae23b423988f6fbc6fc9b18028e0a145b26","is_merged_in_target_branch":true}],"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}