)]}'
{"id":"openstack%2Fpuppet-openstacklib~437846","triplet_id":"openstack%2Fpuppet-openstacklib~stable%2Focata~I9e56f8e2de542b20fe9e6995506cff5bb435e220","project":"openstack/puppet-openstacklib","branch":"stable/ocata","topic":"bug/1664561","hashtags":[],"change_id":"I9e56f8e2de542b20fe9e6995506cff5bb435e220","subject":"Allow integer port in os_transport_url","status":"MERGED","created":"2017-02-24 09:08:33.000000000","updated":"2017-02-25 16:00:14.000000000","submitted":"2017-02-25 16:00:14.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"171dcaed7f901f0c0c2173a823eb1947647e7582","_number":437846,"virtual_id_number":437846,"owner":{"_account_id":20775,"name":"Carlos Camacho","email":"ccamacho@redhat.com","username":"ccamacho"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":2,"date":"2017-02-25 16:00:14.000000000","post_submit":true,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":21513,"name":"lixinhu","email":"lixinhu@inspur.com","username":"lixinhu"},{"value":0,"_account_id":18795,"name":"mkarpin","email":"mkarpin@mirantis.com","username":"nkarpin"}],"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":18795,"name":"mkarpin","email":"mkarpin@mirantis.com","username":"nkarpin"},"recommended":{"_account_id":21513,"name":"lixinhu","email":"lixinhu@inspur.com","username":"lixinhu"},"all":[{"value":0,"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":1,"date":"2017-02-24 10:42:18.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":21513,"name":"lixinhu","email":"lixinhu@inspur.com","username":"lixinhu"},{"value":2,"date":"2017-02-24 12:29:19.000000000","_account_id":18795,"name":"mkarpin","email":"mkarpin@mirantis.com","username":"nkarpin"}],"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":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"all":[{"value":1,"date":"2017-02-25 14:57:54.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":21513,"name":"lixinhu","email":"lixinhu@inspur.com","username":"lixinhu"},{"value":0,"_account_id":18795,"name":"mkarpin","email":"mkarpin@mirantis.com","username":"nkarpin"}],"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":18795,"name":"mkarpin","email":"mkarpin@mirantis.com","username":"nkarpin"},{"_account_id":21513,"name":"lixinhu","email":"lixinhu@inspur.com","username":"lixinhu"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2017-02-24 10:42:18.000000000","updated_by":{"_account_id":21513,"name":"lixinhu","email":"lixinhu@inspur.com","username":"lixinhu"},"reviewer":{"_account_id":21513,"name":"lixinhu","email":"lixinhu@inspur.com","username":"lixinhu"},"state":"REVIEWER"},{"updated":"2017-02-24 12:29:19.000000000","updated_by":{"_account_id":18795,"name":"mkarpin","email":"mkarpin@mirantis.com","username":"nkarpin"},"reviewer":{"_account_id":18795,"name":"mkarpin","email":"mkarpin@mirantis.com","username":"nkarpin"},"state":"REVIEWER"},{"updated":"2017-02-25 14:57:54.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":"2017-02-25 16:00:14.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"c8550a582d9d89ba10b2a753848e8a02b85d8d9c","author":{"_account_id":20775,"name":"Carlos Camacho","email":"ccamacho@redhat.com","username":"ccamacho"},"date":"2017-02-24 09:08:33.000000000","message":"Patch Set 1: Cherry Picked from branch master.","accounts_in_message":[],"_revision_number":1},{"id":"c66e995ed3f469ef191e4a33f66c0f6539572f11","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-02-24 10:13:26.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- gate-puppet-openstacklib-puppet-lint http://logs.openstack.org/46/437846/1/check/gate-puppet-openstacklib-puppet-lint/dd2e081/ : SUCCESS in 1m 33s\n- gate-puppet-openstacklib-puppet-syntax-3-centos-7 http://logs.openstack.org/46/437846/1/check/gate-puppet-openstacklib-puppet-syntax-3-centos-7/482ac31/ : SUCCESS in 2m 07s\n- gate-puppet-openstacklib-puppet-syntax-4-centos-7 http://logs.openstack.org/46/437846/1/check/gate-puppet-openstacklib-puppet-syntax-4-centos-7/3736599/ : SUCCESS in 1m 54s\n- gate-puppet-openstacklib-puppet-unit-3.6-centos-7 http://logs.openstack.org/46/437846/1/check/gate-puppet-openstacklib-puppet-unit-3.6-centos-7/3b5dca1/ : SUCCESS in 5m 03s\n- gate-puppet-openstacklib-puppet-unit-3.8-centos-7 http://logs.openstack.org/46/437846/1/check/gate-puppet-openstacklib-puppet-unit-3.8-centos-7/9632594/ : SUCCESS in 5m 29s\n- gate-puppet-openstacklib-puppet-unit-4.5-centos-7 http://logs.openstack.org/46/437846/1/check/gate-puppet-openstacklib-puppet-unit-4.5-centos-7/7a18357/ : SUCCESS in 4m 29s\n- gate-puppet-openstacklib-puppet-beaker-rspec-centos-7 http://logs.openstack.org/46/437846/1/check/gate-puppet-openstacklib-puppet-beaker-rspec-centos-7/1a1af94/ : SUCCESS in 5m 00s\n- gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial http://logs.openstack.org/46/437846/1/check/gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial/d18411c/ : SUCCESS in 4m 41s\n- gate-puppet-openstack-integration-4-scenario001-tempest-centos-7 http://logs.openstack.org/46/437846/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/c02c12c/ : SUCCESS in 37m 54s\n- gate-puppet-openstack-integration-4-scenario002-tempest-centos-7 http://logs.openstack.org/46/437846/1/check/gate-puppet-openstack-integration-4-scenario002-tempest-centos-7/52bd741/ : SUCCESS in 49m 34s\n- gate-puppet-openstack-integration-4-scenario003-tempest-centos-7 http://logs.openstack.org/46/437846/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/d2d6a86/ : SUCCESS in 38m 18s\n- gate-puppet-openstack-integration-4-scenario004-tempest-centos-7 http://logs.openstack.org/46/437846/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/5066191/ : SUCCESS in 25m 31s\n- gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial http://logs.openstack.org/46/437846/1/check/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/ed093c2/ : SUCCESS in 27m 21s\n- gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial http://logs.openstack.org/46/437846/1/check/gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial/7a14995/ : SUCCESS in 49m 55s\n- gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial http://logs.openstack.org/46/437846/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial/42bca93/ : SUCCESS in 40m 05s\n- gate-puppet-openstack-integration-4-scenario004-tempest-ubuntu-xenial http://logs.openstack.org/46/437846/1/check/gate-puppet-openstack-integration-4-scenario004-tempest-ubuntu-xenial/70eaa00/ : SUCCESS in 26m 39s\n- gate-puppet-openstacklib-releasenotes http://docs-draft.openstack.org/46/437846/1/check/gate-puppet-openstacklib-releasenotes/8925ec6//releasenotes/build/html/ : SUCCESS in 1m 13s\n- gate-tripleo-ci-centos-7-nonha-multinode http://logs.openstack.org/46/437846/1/check/gate-tripleo-ci-centos-7-nonha-multinode/4d41ac8/ : SUCCESS in 1h 03m 36s","accounts_in_message":[],"_revision_number":1},{"id":"f075ca4fe5959ff47ebc80ce6944aae9800e5a7c","author":{"_account_id":21513,"name":"lixinhu","email":"lixinhu@inspur.com","username":"lixinhu"},"date":"2017-02-24 10:42:18.000000000","message":"Patch Set 1: Code-Review+1","accounts_in_message":[],"_revision_number":1},{"id":"7eb21bdf4c9764512faf57566439f17e65a044a5","author":{"_account_id":18795,"name":"mkarpin","email":"mkarpin@mirantis.com","username":"nkarpin"},"date":"2017-02-24 12:29:19.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"21c499684ab6850d97904c6ce64f9891eabb0063","author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"date":"2017-02-25 14:57:54.000000000","message":"Patch Set 1: Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"23d8443bb5a3038010acd0c09b09e45ceb71e2c1","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-02-25 14:58:05.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.\nhttp://status.openstack.org/zuul/","accounts_in_message":[],"_revision_number":1},{"id":"78b88b18a95f151c9fc23cdf97539fd3d17c1b80","date":"2017-02-25 16:00:14.000000000","message":"Change has been successfully merged into the git repository by Jenkins","accounts_in_message":[],"_revision_number":1},{"id":"1c076920befc6bbeabc246240820df155aa39c11","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2017-02-25 16:00:14.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\n\n- gate-puppet-openstacklib-puppet-lint http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstacklib-puppet-lint/a3729c6/ : SUCCESS in 2m 27s\n- gate-puppet-openstacklib-puppet-syntax-3-centos-7 http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstacklib-puppet-syntax-3-centos-7/c663bbd/ : SUCCESS in 2m 18s\n- gate-puppet-openstacklib-puppet-syntax-4-centos-7 http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstacklib-puppet-syntax-4-centos-7/87a9aea/ : SUCCESS in 1m 35s\n- gate-puppet-openstacklib-puppet-unit-3.6-centos-7 http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstacklib-puppet-unit-3.6-centos-7/576aab6/ : SUCCESS in 4m 23s\n- gate-puppet-openstacklib-puppet-unit-3.8-centos-7 http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstacklib-puppet-unit-3.8-centos-7/a4f31ce/ : SUCCESS in 4m 10s\n- gate-puppet-openstacklib-puppet-unit-4.5-centos-7 http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstacklib-puppet-unit-4.5-centos-7/aa3b25d/ : SUCCESS in 4m 36s\n- gate-puppet-openstacklib-puppet-beaker-rspec-centos-7 http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstacklib-puppet-beaker-rspec-centos-7/77767d3/ : SUCCESS in 4m 24s\n- gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstacklib-puppet-beaker-rspec-ubuntu-xenial/00f51b9/ : SUCCESS in 4m 08s\n- gate-puppet-openstack-integration-4-scenario001-tempest-centos-7 http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstack-integration-4-scenario001-tempest-centos-7/7fd4141/ : SUCCESS in 37m 09s\n- gate-puppet-openstack-integration-4-scenario002-tempest-centos-7 http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstack-integration-4-scenario002-tempest-centos-7/7e95264/ : SUCCESS in 39m 41s\n- gate-puppet-openstack-integration-4-scenario003-tempest-centos-7 http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/e6a2312/ : SUCCESS in 35m 17s\n- gate-puppet-openstack-integration-4-scenario004-tempest-centos-7 http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstack-integration-4-scenario004-tempest-centos-7/ea15345/ : SUCCESS in 32m 14s\n- gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/f9d183a/ : SUCCESS in 38m 57s\n- gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial/728b379/ : SUCCESS in 49m 54s\n- gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstack-integration-4-scenario003-tempest-ubuntu-xenial/35aff69/ : SUCCESS in 38m 02s\n- gate-puppet-openstack-integration-4-scenario004-tempest-ubuntu-xenial http://logs.openstack.org/46/437846/1/gate/gate-puppet-openstack-integration-4-scenario004-tempest-ubuntu-xenial/0cc9722/ : SUCCESS in 28m 36s\n- gate-puppet-openstacklib-releasenotes http://docs-draft.openstack.org/46/437846/1/gate/gate-puppet-openstacklib-releasenotes/cd3f73f//releasenotes/build/html/ : SUCCESS in 1m 38s\n- gate-tripleo-ci-centos-7-nonha-multinode http://logs.openstack.org/46/437846/1/gate/gate-tripleo-ci-centos-7-nonha-multinode/1000f3f/ : SUCCESS in 1h 02m 01s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"acffece0d02fe6ec97161dd3716ddcffa170380a","revisions":{"acffece0d02fe6ec97161dd3716ddcffa170380a":{"kind":"REWORK","_number":1,"created":"2017-02-24 09:08:33.000000000","uploader":{"_account_id":20775,"name":"Carlos Camacho","email":"ccamacho@redhat.com","username":"ccamacho"},"ref":"refs/changes/46/437846/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/puppet-openstacklib","ref":"refs/changes/46/437846/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/437846/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/437846/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/437846/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/puppet-openstacklib refs/changes/46/437846/1"}}},"commit":{"parents":[{"commit":"c2fb5aafa38a9512d2c4ebf227357084042eea5c","subject":"Update .gitreview for stable/ocata","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/c2fb5aafa38a9512d2c4ebf227357084042eea5c"}]}],"author":{"name":"Carlos Camacho","email":"ccamacho@redhat.com","date":"2017-02-21 10:37:22.000000000","tz":60},"committer":{"name":"Carlos Camacho","email":"ccamacho@redhat.com","date":"2017-02-24 09:08:33.000000000","tz":0},"subject":"Allow integer port in os_transport_url","message":"Allow integer port in os_transport_url\n\nCurrently we only allow to use strings\nas the data type for ports.\n\nDue to this we need to cast the data type\nin the puppet modules because from THT\nthis is configured as a Number.\n\nThis submission allow to use either string\nor numbers for the port parameter in the\nos_transport_url function.\n\nChange-Id: I9e56f8e2de542b20fe9e6995506cff5bb435e220\nCloses-Bug: #1664561\n(cherry picked from commit b3d0590022717c50be02dda0b1283ae3c4312206)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/acffece0d02fe6ec97161dd3716ddcffa170380a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/puppet-openstacklib/commit/acffece0d02fe6ec97161dd3716ddcffa170380a"}]},"branch":"refs/heads/stable/ocata"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
