[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Chris Lincoln changed: What|Removed |Added External Bug ID||Red Hat Knowledge

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Parag AN(पराग) changed: What|Removed |Added CC||panem...@gmail.com

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Dustin Black changed: What|Removed |Added Blocks||1261086

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 bigswitch changed: What|Removed |Added Flags|needinfo?(karlthered@gmail.

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 xinwu changed: What|Removed |Added Status|ASSIGNED|CLOSED

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 bigswitch changed: What|Removed |Added Flags|

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #33 from Jon Ciesla --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Jon Ciesla changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ --

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #32 from bigswitch --- New Package SCM Request === Package Name: python-networking-bigswitch Short Description: Big Switch OpenStack Neutron driver and agents Upstream URL:

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #31 from xinwu --- New Package SCM Request === Package Name: python-networking-bigswitch Short Description: Big Switch OpenStack Neutron driver and agents Upstream URL:

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Haïkel Guémar changed: What|Removed |Added Flags|needinfo?(karlthered@gmail. |fedora-review+

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 xinwu changed: What|Removed |Added Flags||fedora-cvs? -- You

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Haïkel Guémar changed: What|Removed |Added Flags|needinfo?(karlthered@gmail. |

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #29 from bigswitch --- rpmlint errors/warnings have been fixed :) Spec URL: https://bigswitch.box.com/shared/static/24hmnr6ojdgwulgpqclcvlvp5qf8tdt3.spec SRPM URL:

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 bigswitch changed: What|Removed |Added Flags|

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #26 from bigswitch --- All descriptions are updated to mention neutron. Please let us know how to proceed to merge it. Also, is it time now to start a Delorean review? Spec URL:

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #27 from Ihar Hrachyshka --- > All descriptions are updated to mention neutron. Please let us know how to > proceed to merge it. You need a sponsor that will walk you thru formal review process. Ask RDO

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #25 from Ihar Hrachyshka --- Make sure all descriptions mention neutron, since it's neutron specific. Otherwise looks good, and I believe we should proceed with merging it into Fedora and RDO. -- You are

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #23 from Ihar Hrachyshka ihrac...@redhat.com --- I believe main python package should depend on python-neutron not openstack-neutron-common. The latter should be the dependency for your agent. Summary for -doc sub package is

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #24 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Reviewer comments about service files have been addressed. Please use following links for further review. Spec URL:

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #21 from Ihar Hrachyshka ihrac...@redhat.com --- Please split %description and %package section with newlines. Summary is not ideal. Better: Big Switch Networks plugin for OpenStack Networking %{rpm_prefix} is not an ideal name

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #22 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Reviewer comments about service files have been addressed. Please use following links for further review. Spec URL:

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 bigswitch rhosp-bugs-inter...@bigswitch.com changed: What|Removed |Added Flags|

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Matthias Runge mru...@redhat.com changed: What|Removed |Added CC||mru...@redhat.com

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #17 from Ihar Hrachyshka ihrac...@redhat.com --- I think there are several issues with the service files I see at: https://github.com/xinwu/python-networking-bigswitch/blob/rpm-master/ Specifically, 1. they read

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #20 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Reviewer comments about service files have been addressed. Please use following links for further review. Spec URL:

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #19 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Hi Haïkel, I went over the following two bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1252812 https://bugzilla.redhat.com/show_bug.cgi?id=1242886 and found some minor

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #18 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Thanks a lot Ihar! I'll address the new review comments today. We don't have a Delorean pull request yet. We asked for it earlier last week and got following reply from

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #14 from xinwu xin...@bigswitch.com --- Sure, will do :) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Haïkel Guémar karlthe...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #16 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Please use following links for review Spec URL: https://bigswitch.box.com/shared/static/24hmnr6ojdgwulgpqclcvlvp5qf8tdt3.spec SRPM URL:

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #12 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Thanks a lot Ihar for the review. Your comments have been addressed. Instead of having two separate packages, we create subpackages with one spec file. Documentation is also

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #15 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Hi Haikel, I'm looking into these two right now: 1242886 1242896. I'll update once I'm done with the review. -- You are receiving this mail because: You are on the CC list

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Chandan Kumar chku...@redhat.com changed: What|Removed |Added CC||chku...@redhat.com

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #10 from Ihar Hrachyshka ihrac...@redhat.com --- I believe you should fix your branches and the way you track dependencies in upstream first. For example: - you should not depend on neutron master in kilo stable branch; - you

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Ihar Hrachyshka ihrac...@redhat.com changed: What|Removed |Added Assignee|ihrac...@redhat.com

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #9 from bigswitch rhosp-bugs-inter...@bigswitch.com --- We need to revert a change in stackforge/networking-bigswitch stable/kilo branch. https://review.openstack.org/#/c/215848/, Once that review gets merged, we will push a proper

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #8 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Thanks a lot Ihar for the review comments. I'm addressing them right now. A little explanation why we drop the .service file here. Big Switch has three pieces of code

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #7 from Ihar Hrachyshka ihrac...@redhat.com --- I will review the package formally on Mon. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #6 from Ihar Hrachyshka ihrac...@redhat.com --- I think URL should point to pypi page with the python package, not bigswitch website. It's not clear where you get the version tag from. I can't find the tag 2015.1.33 in

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Ihar Hrachyshka ihrac...@redhat.com changed: What|Removed |Added Flags||fedora-review?

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Ihar Hrachyshka ihrac...@redhat.com changed: What|Removed |Added Assignee|nob...@fedoraproject.org

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Steve Linabery slina...@redhat.com changed: What|Removed |Added CC||slina...@redhat.com

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #3 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Spec URL: https://bigswitch.box.com/shared/static/24hmnr6ojdgwulgpqclcvlvp5qf8tdt3.spec SRPM URL: https://bigswitch.box.com/shared/static/53v6lgvm7mxxh51pzpn2ykubduggv45z.rpm

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 bigswitch rhosp-bugs-inter...@bigswitch.com changed: What|Removed |Added CC|

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Chandan Kumar chku...@redhat.com changed: What|Removed |Added Blocks||1243533

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 Ihar Hrachyshka ihrac...@redhat.com changed: What|Removed |Added CC|

[Bug 1254828] Review Request: python-networking-bigswitch - big switch neutron plugin

https://bugzilla.redhat.com/show_bug.cgi?id=1254828 --- Comment #5 from bigswitch rhosp-bugs-inter...@bigswitch.com --- Reviewer's comments have been addressed. Please use following new links for further review. This rpm package doesn't include agent any more. It includes only ml2 plugin and