)]}'
{"id":"openstack%2Fneutron~2279","triplet_id":"openstack%2Fneutron~master~If7e9ad2dcb8124e7d82ac837c5b62c0d09b00bcd","project":"openstack/neutron","branch":"master","topic":"bp/services-insertion-wrapper","hashtags":[],"change_id":"If7e9ad2dcb8124e7d82ac837c5b62c0d09b00bcd","subject":"Implementation of the BP services-insertion-wrapper inside the Cisco Plugin","status":"MERGED","created":"2011-12-14 00:56:21.000000000","updated":"2011-12-14 00:56:21.000000000","submitted":"2011-12-14 00:56:21.000000000","submitter":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"b1945fabd0a4ac57055f1eb990265ce4c0ba2ab8","_number":2279,"virtual_id_number":2279,"owner":{"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"all":[{"value":0,"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},{"value":1,"date":"2011-12-14 00:56:21.000000000","_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":829,"name":"Ying Liu","username":"yinliu2"},{"value":0,"_account_id":141,"name":"Rohit Agarwalla","email":"roagarwa@cisco.com","username":"rohitagarwalla"},{"value":0,"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"}],"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":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"all":[{"value":0,"date":"2011-12-14 00:56:21.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},{"value":0,"date":"2011-12-14 00:56:21.000000000","permitted_voting_range":{"min":0,"max":1},"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":829,"name":"Ying Liu","username":"yinliu2"},{"value":0,"_account_id":141,"name":"Rohit Agarwalla","email":"roagarwa@cisco.com","username":"rohitagarwalla"},{"value":2,"date":"2011-12-14 00:56:21.000000000","_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"}],"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":{"all":[{"value":0,"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":829,"name":"Ying Liu","username":"yinliu2"},{"value":0,"_account_id":141,"name":"Rohit Agarwalla","email":"roagarwa@cisco.com","username":"rohitagarwalla"},{"value":0,"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},{"value":0,"_account_id":3,"name":"Jenkins","username":"jenkins"},{"value":0,"_account_id":829,"name":"Ying Liu","username":"yinliu2"},{"value":0,"_account_id":141,"name":"Rohit Agarwalla","email":"roagarwa@cisco.com","username":"rohitagarwalla"},{"value":0,"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Gate Blocker Fix / Urgent Change"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":3,"name":"Jenkins","username":"jenkins"},{"_account_id":141,"name":"Rohit Agarwalla","email":"roagarwa@cisco.com","username":"rohitagarwalla"},{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},{"_account_id":829,"name":"Ying Liu","username":"yinliu2"},{"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2011-12-14 00:56:21.000000000","updated_by":{"_account_id":141,"name":"Rohit Agarwalla","email":"roagarwa@cisco.com","username":"rohitagarwalla"},"reviewer":{"_account_id":141,"name":"Rohit Agarwalla","email":"roagarwa@cisco.com","username":"rohitagarwalla"},"state":"REVIEWER"},{"updated":"2011-12-14 00:56:21.000000000","updated_by":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"reviewer":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"state":"REVIEWER"},{"updated":"2011-12-14 00:56:21.000000000","updated_by":{"_account_id":829,"name":"Ying Liu","username":"yinliu2"},"reviewer":{"_account_id":829,"name":"Ying Liu","username":"yinliu2"},"state":"REVIEWER"},{"updated":"2011-12-14 00:56:21.000000000","updated_by":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"reviewer":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"state":"REVIEWER"}],"messages":[{"id":"17ff495838089ce62326465335160e45ec67f29a","author":{"_account_id":141,"name":"Rohit Agarwalla","email":"roagarwa@cisco.com","username":"rohitagarwalla"},"date":"2011-12-14 00:56:21.000000000","message":"Patch Set 2: (1 inline comment)\n\nDid a quick review on the db portion. looks good with one inline comment. thanks for adding the service db tests.","accounts_in_message":[],"_revision_number":2},{"id":"55c6c77eb5cec98cac39d14402881bf691da7d83","author":{"_account_id":829,"name":"Ying Liu","username":"yinliu2"},"date":"2011-12-14 00:56:21.000000000","message":"Patch Set 2: Looks good to me, but someone else must approve\n\nLooks good to me.","accounts_in_message":[],"_revision_number":2},{"id":"17de7b4da993d4b792be5e2153b9f775cb5b0968","author":{"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},"date":"2011-12-14 00:56:21.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"38e559a4ab4f426123c64bedbd30c382fb192a6d","author":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"date":"2011-12-14 00:56:21.000000000","message":"Patch Set 3: Looks good to me, approved\n\nOk folks, as discussed at the meeting, we\u0027re shipping this, with a promise from Edgar that there will be some improvement in E-3 (as well as a more general discussion after that about the \"right\" final resting place for this code).","accounts_in_message":[],"_revision_number":3},{"id":"47d7514cf71a91a2ab8271ca44fd87c53379e209","author":{"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},"date":"2011-12-14 00:56:21.000000000","message":"Uploaded patch set 3.","accounts_in_message":[],"_revision_number":3},{"id":"32bd939228a58b523b454c6ed13920698a946aa5","author":{"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},"date":"2011-12-14 00:56:21.000000000","message":"Patch Set 3:\n\nRemoved old code from api.py on Patch 3\nThanks Rohit for highlighting this.","accounts_in_message":[],"_revision_number":3},{"id":"2f876037c2d41a0a10dee53df0b56a5abc3d5aee","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-12-14 00:56:21.000000000","message":"Patch Set 3: Verified\n\nBuild Successful \n \nhttps://jenkins.openstack.org/job/quantum-pep8/88/ : SUCCESS \nhttps://jenkins.openstack.org/job/gate-quantum-merge/19/ : SUCCESS \nhttps://jenkins.openstack.org/job/quantum/113/ : SUCCESS","accounts_in_message":[],"_revision_number":3},{"id":"6df1dbfef819c5f890f61997df2c3e5237f7fac9","author":{"_account_id":3,"name":"Jenkins","username":"jenkins"},"date":"2011-12-14 00:56:21.000000000","message":"Change has been successfully merged into the git repository.","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"fa912dfcd91ea7a7e4a014d7179e7232357dd40c","revisions":{"17a1c78e77d9712816037d6476f4a358a380b3f4":{"kind":"REWORK","_number":1,"created":"2011-12-14 00:56:21.000000000","uploader":{"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},"ref":"refs/changes/79/2279/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/79/2279/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/79/2279/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/79/2279/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/79/2279/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/79/2279/1"}}},"commit":{"parents":[{"commit":"1a048e944e6b0fc41948db9ac971d0acdb26ea70","subject":"Merge \"bug #891246: Fix paths in agent Makefile\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/1a048e944e6b0fc41948db9ac971d0acdb26ea70"}]}],"author":{"name":"Edgar Magana","email":"eperdomo@cisco.com","date":"2011-12-13 07:41:59.000000000","tz":-480},"committer":{"name":"Edgar Magana","email":"eperdomo@cisco.com","date":"2011-12-13 07:46:57.000000000","tz":-480},"subject":"Implementation of the BP services-insertion-wrapper inside the Cisco Plugin","message":"Implementation of the BP services-insertion-wrapper inside the Cisco Plugin\n\nThis utility offers a simplify way to insert and remove network services\nin the path of the traffic to the server VMs, by splitting the network\ninto two, and having the service bridge between the two networks,\nin the process applying the service. This model is called In-Path\n(Bump in the Wire)\n\nChange-Id: If7e9ad2dcb8124e7d82ac837c5b62c0d09b00bcd\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/17a1c78e77d9712816037d6476f4a358a380b3f4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/17a1c78e77d9712816037d6476f4a358a380b3f4"}]},"branch":"refs/heads/master"},"be242d1596203399a796871e3cf9f3e3f16e862a":{"kind":"REWORK","_number":2,"created":"2011-12-14 00:56:21.000000000","uploader":{"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},"ref":"refs/changes/79/2279/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/79/2279/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/79/2279/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/79/2279/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/79/2279/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/79/2279/2"}}},"commit":{"parents":[{"commit":"1a048e944e6b0fc41948db9ac971d0acdb26ea70","subject":"Merge \"bug #891246: Fix paths in agent Makefile\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/1a048e944e6b0fc41948db9ac971d0acdb26ea70"}]}],"author":{"name":"Edgar Magana","email":"eperdomo@cisco.com","date":"2011-12-13 07:41:59.000000000","tz":-480},"committer":{"name":"Edgar Magana","email":"eperdomo@cisco.com","date":"2011-12-13 08:03:15.000000000","tz":-480},"subject":"Implementation of the BP services-insertion-wrapper inside the Cisco Plugin","message":"Implementation of the BP services-insertion-wrapper inside the Cisco Plugin\n\nThis utility offers a simplify way to insert and remove network services\nin the path of the traffic to the server VMs, by splitting the network\ninto two, and having the service bridge between the two networks,\nin the process applying the service. This model is called In-Path\n(Bump in the Wire)\n\nChange-Id: If7e9ad2dcb8124e7d82ac837c5b62c0d09b00bcd\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/be242d1596203399a796871e3cf9f3e3f16e862a"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/be242d1596203399a796871e3cf9f3e3f16e862a"}]},"branch":"refs/heads/master"},"fa912dfcd91ea7a7e4a014d7179e7232357dd40c":{"kind":"REWORK","_number":3,"created":"2011-12-14 00:56:21.000000000","uploader":{"_account_id":1616,"name":"Edgar Magana Perdomo","username":"eperdomo","inactive":true},"ref":"refs/changes/79/2279/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/neutron","ref":"refs/changes/79/2279/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/neutron refs/changes/79/2279/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/neutron refs/changes/79/2279/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/neutron refs/changes/79/2279/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/neutron refs/changes/79/2279/3"}}},"commit":{"parents":[{"commit":"c8f2630ad5e447f1969b7dcfc1d8f10f7a14afb7","subject":"Merge \"Fixing bug/903829 Making setup_server.py not try to install quantum.conf.sample\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/c8f2630ad5e447f1969b7dcfc1d8f10f7a14afb7"}]}],"author":{"name":"Edgar Magana","email":"eperdomo@cisco.com","date":"2011-12-13 07:41:59.000000000","tz":-480},"committer":{"name":"Edgar Magana","email":"eperdomo@cisco.com","date":"2011-12-13 21:35:11.000000000","tz":-480},"subject":"Implementation of the BP services-insertion-wrapper inside the Cisco Plugin","message":"Implementation of the BP services-insertion-wrapper inside the Cisco Plugin\n\nThis utility offers a simplify way to insert and remove network services\nin the path of the traffic to the server VMs, by splitting the network\ninto two, and having the service bridge between the two networks,\nin the process applying the service. This model is called In-Path\n(Bump in the Wire)\n\nChange-Id: If7e9ad2dcb8124e7d82ac837c5b62c0d09b00bcd\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/fa912dfcd91ea7a7e4a014d7179e7232357dd40c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/neutron/commit/fa912dfcd91ea7a7e4a014d7179e7232357dd40c"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}