)]}'
{"id":"openstack%2Fopenstack-helm~673135","triplet_id":"openstack%2Fopenstack-helm~master~I85d85c5ab0a5111fde11a1b2a4b1fa15aca57e51","project":"openstack/openstack-helm","branch":"master","hashtags":[],"change_id":"I85d85c5ab0a5111fde11a1b2a4b1fa15aca57e51","subject":"In neutron chart, ovs bridges are created and ports are added to some of the ovs bridges according to auto_bridge_add. For now, we only support the ethernet port. This commition is to proposal bond port support. bond port is useful for network HA. For bond port, we should use ovs-vsctl add-bond instead of ovs-vsctl add-port","status":"ABANDONED","created":"2019-07-27 11:48:12.000000000","updated":"2019-08-01 07:16:04.000000000","total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"3475b7c48085e156d7f7c0d66ace60108727d31c","_number":673135,"virtual_id_number":673135,"owner":{"_account_id":30752,"name":"Qianshui Jiang","email":"qianshui.jiang@intel.com","username":"qianshui"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},{"date":"2019-07-27 11:48:22.000000000","_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":-1,"date":"2019-07-27 12:45:47.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":{"disliked":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"all":[{"value":-1,"date":"2019-07-28 02:17:43.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"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":{"all":[{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2019-07-27 11:48:22.000000000","updated_by":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"reviewer":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"state":"REVIEWER"},{"updated":"2019-07-27 12:45:47.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2019-07-28 02:17:43.000000000","updated_by":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"reviewer":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"state":"REVIEWER"}],"messages":[{"id":"c50e73202e1bd9330e1f20a8b2457b01ef263505","author":{"_account_id":30752,"name":"Qianshui Jiang","email":"qianshui.jiang@intel.com","username":"qianshui"},"date":"2019-07-27 11:48:12.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"c9c358421b6337b87cb58337c22274911c75ea48","author":{"_account_id":10068,"name":"Welcome, new contributor!","username":"welcome-message"},"date":"2019-07-27 11:48:22.000000000","message":"Patch Set 1:\n\nThank you for your first contribution to OpenStack.\n\nYour patch will now be tested automatically by OpenStack testing frameworks\nand once the automatic tests pass, it will be reviewed by other friendly\ndevelopers. They will give you feedback and may require you to refine it.\n\nPeople seldom get their patch approved on the first try, so don\u0027t be\nconcerned if requested to make corrections. Feel free to modify your patch\nand resubmit a new change-set.\n\nPatches usually take 3 to 7 days to be reviewed so be patient and be\navailable on IRC to ask and answer questions about your work. Also it\ntakes generally at least a couple of weeks for cores to get around to\nreviewing code. The more you participate in the community the more\nrewarding it is for you. You may also notice that the more you get to know\npeople and get to be known, the faster your patches will be reviewed and\neventually approved. Get to know others and become known by doing code\nreviews: anybody can do it, and it\u0027s a great way to learn the code base.\n\nThanks again for supporting OpenStack, we look forward to working with you.\n\nIRC: https://wiki.openstack.org/wiki/IRC\nWorkflow: https://docs.openstack.org/infra/manual/developers.html\nCommit Messages: https://wiki.openstack.org/wiki/GitCommitMessages","accounts_in_message":[],"_revision_number":1},{"id":"6101e683e7213188c612177cc73edfb3a620a8ca","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2019-07-27 12:45:47.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttp://docs.openstack.org/infra/manual/developers.html#automated-testing\n\n\n- openstack-tox-docs http://logs.openstack.org/35/673135/1/check/openstack-tox-docs/b030e7b/html/ : SUCCESS in 4m 15s\n- openstack-helm-lint http://logs.openstack.org/35/673135/1/check/openstack-helm-lint/049c479/ : FAILURE in 2m 09s\n- openstack-helm-cinder http://logs.openstack.org/35/673135/1/check/openstack-helm-cinder/06a9213/ : SUCCESS in 35m 30s\n- openstack-helm-cinder-pike-ubuntu_xenial http://logs.openstack.org/35/673135/1/check/openstack-helm-cinder-pike-ubuntu_xenial/3087bfe/ : SUCCESS in 35m 29s (non-voting)\n- openstack-helm-cinder-queens-ubuntu_xenial http://logs.openstack.org/35/673135/1/check/openstack-helm-cinder-queens-ubuntu_xenial/2f306c1/ : SUCCESS in 36m 47s (non-voting)\n- openstack-helm-cinder-rocky-ubuntu_xenial http://logs.openstack.org/35/673135/1/check/openstack-helm-cinder-rocky-ubuntu_xenial/27edbdc/ : SUCCESS in 37m 28s (non-voting)\n- openstack-helm-cinder-rocky-ubuntu_bionic http://logs.openstack.org/35/673135/1/check/openstack-helm-cinder-rocky-ubuntu_bionic/e3ed147/ : SUCCESS in 36m 02s (non-voting)\n- openstack-helm-cinder-rocky-opensuse_15 http://logs.openstack.org/35/673135/1/check/openstack-helm-cinder-rocky-opensuse_15/53e7193/ : SUCCESS in 36m 18s (non-voting)\n- openstack-helm-cinder-stein-ubuntu_bionic http://logs.openstack.org/35/673135/1/check/openstack-helm-cinder-stein-ubuntu_bionic/9bbabe8/ : SUCCESS in 39m 52s (non-voting)\n- openstack-helm-compute-kit http://logs.openstack.org/35/673135/1/check/openstack-helm-compute-kit/75e6661/ : SUCCESS in 43m 07s\n- openstack-helm-compute-kit-pike-ubuntu_xenial http://logs.openstack.org/35/673135/1/check/openstack-helm-compute-kit-pike-ubuntu_xenial/a4600fd/ : SUCCESS in 42m 25s (non-voting)\n- openstack-helm-compute-kit-queens-ubuntu_xenial http://logs.openstack.org/35/673135/1/check/openstack-helm-compute-kit-queens-ubuntu_xenial/61ff626/ : SUCCESS in 44m 53s (non-voting)\n- openstack-helm-compute-kit-rocky-ubuntu_xenial http://logs.openstack.org/35/673135/1/check/openstack-helm-compute-kit-rocky-ubuntu_xenial/36a8da9/ : SUCCESS in 45m 19s (non-voting)\n- openstack-helm-compute-kit-rocky-ubuntu_bionic http://logs.openstack.org/35/673135/1/check/openstack-helm-compute-kit-rocky-ubuntu_bionic/2c90cc1/ : SUCCESS in 48m 23s (non-voting)\n- openstack-helm-compute-kit-rocky-opensuse_15 http://logs.openstack.org/35/673135/1/check/openstack-helm-compute-kit-rocky-opensuse_15/4aa208e/ : FAILURE in 50m 42s (non-voting)\n- openstack-helm-compute-kit-stein-ubuntu_bionic http://logs.openstack.org/35/673135/1/check/openstack-helm-compute-kit-stein-ubuntu_bionic/b814dbf/ : SUCCESS in 44m 37s (non-voting)\n- openstack-helm-horizon http://logs.openstack.org/35/673135/1/check/openstack-helm-horizon/624f7bd/ : SUCCESS in 24m 41s\n- openstack-helm-apparmor http://logs.openstack.org/35/673135/1/check/openstack-helm-apparmor/ae82878/ : SUCCESS in 45m 42s (non-voting)","accounts_in_message":[],"_revision_number":1},{"id":"5f04be156be4a7bbc9b9fd3dfb15a9d4fdbc219d","author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"date":"2019-07-28 02:17:43.000000000","message":"Patch Set 1: Code-Review-1\n\n(3 comments)","accounts_in_message":[],"_revision_number":1},{"id":"7c5ff67d4d4ce53b3d9235833056dfd11b3b3c0e","author":{"_account_id":29668,"name":"cheng li","email":"cheng1.li@intel.com","username":"chengli3"},"date":"2019-07-31 14:25:17.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"3475b7c48085e156d7f7c0d66ace60108727d31c","author":{"_account_id":30752,"name":"Qianshui Jiang","email":"qianshui.jiang@intel.com","username":"qianshui"},"date":"2019-08-01 07:16:04.000000000","message":"Abandoned\n\nI\u0027ve update another patch, hence this patch is going to be abandoned.","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"84528f5efea549592857d83ac89b881c02ac35d4","revisions":{"84528f5efea549592857d83ac89b881c02ac35d4":{"kind":"REWORK","_number":1,"created":"2019-07-27 11:48:12.000000000","uploader":{"_account_id":30752,"name":"Qianshui Jiang","email":"qianshui.jiang@intel.com","username":"qianshui"},"ref":"refs/changes/35/673135/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm","ref":"refs/changes/35/673135/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/673135/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/673135/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm refs/changes/35/673135/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm refs/changes/35/673135/1"}}},"commit":{"parents":[{"commit":"4bb6976d2c7418dc24b969a53cbce81ec0b9142a","subject":"Merge \"Imported Translations from Zanata\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/4bb6976d2c7418dc24b969a53cbce81ec0b9142a"}]}],"author":{"name":"qianshui","email":"qianshui.jiang@intel.com","date":"2019-07-27 11:38:17.000000000","tz":480},"committer":{"name":"qianshui","email":"qianshui.jiang@intel.com","date":"2019-07-27 11:38:17.000000000","tz":480},"subject":"In neutron chart, ovs bridges are created and ports are added to some of the ovs bridges according to auto_bridge_add. For now, we only support the ethernet port. This commition is to proposal bond port support. bond port is useful for network HA. For bond port, we should use ovs-vsctl add-bond instead of ovs-vsctl add-port","message":"In neutron chart, ovs bridges are created and ports are added to some of the ovs bridges according to auto_bridge_add.\nFor now, we only support the ethernet port. This commition is to proposal bond port support. bond port is useful for network HA.\nFor bond port, we should use ovs-vsctl add-bond instead of ovs-vsctl add-port\n\nChange-Id: I85d85c5ab0a5111fde11a1b2a4b1fa15aca57e51\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/84528f5efea549592857d83ac89b881c02ac35d4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm/commit/84528f5efea549592857d83ac89b881c02ac35d4"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}