Sorry for the double post, I forgot the subject. On Thu, Oct 12, 2017 at 2:29 PM, Bence Romsics <bence.roms...@gmail.com> wrote: > Hi Horizon Folks, > > After the PTG I'm back from vacation and I'd like to ask for reviews > for bp/neutron-trunk-ui: > > https://review.openstack.org/#/q/status:open+topic:bp/neutron-trunk-ui > > When you'll hit that link you'll see a Jenkins-1 on some of the > patches, but please don't be put off by that. We know the importance > of tests and we are going to fix/write the missing test coverage of > course. > > However we'd like to avoid re-writing the tests several times, that's > why I'm asking for a bit non-standard two-round review: > > 1) Please review the functional parts ignoring the (missing/outdated) > tests firsts. > 2) When you're satisfied with (1) we'll write the tests and that can > be reviewed too. > > We believe the patch series contains a functionally complete Horizon > panel. The only missing part is the tests. > > If you'd like to try the new trunks panel introduced by the patch > series this is how you can do it: > > a) Add trunks to your local.conf: > enable_plugin neutron https://git.openstack.org/openstack/neutron > enable_service q-trunk > > b) stack.sh > > c) Verify if (a) was successful by listing the 'trunk' api extension: > $ openstack extension show trunk > > d) Fetch the full patch series. Final patch: > https://review.openstack.org/489186 > > The new panel should automatically appear as Project/Network/Trunks. > > Background info on Neutron trunks: > > * https://docs.openstack.org/neutron/pike/admin/config-trunking.html > * https://wiki.openstack.org/wiki/Neutron/TrunkPort > > Thanks in advance, > Bence Romsics > freenode: rubasov
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev