Bug#771863: [ovs-dev] Bug#771863: Bug#771863: Service does not start or parse interfaces correctly, updating severity

2014-12-19 Thread Stig Sandbeck Mathisen
Joe Stringer joestrin...@nicira.com writes:

 I agree that this should be treated as a separate bug, yes please (I'm
 just following up as it wasn't clear to me whether you're working on
 this or not).

Ok, I'll report it as a new bug.

-- 
Stig Sandbeck Mathisen


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#771863: [ovs-dev] Bug#771863: Bug#771863: Service does not start or parse interfaces correctly, updating severity

2014-12-18 Thread Joe Stringer
On 15 December 2014 at 10:21, Gurucharan Shetty shet...@nicira.com wrote:
 On Sat, Dec 13, 2014 at 2:03 AM, Stig Sandbeck Mathisen s...@debian.org 
 wrote:
 I would like to clarify that 2.3.1 does get rid of the check on
 $RUNLEVEL which was the original bug description.

 That's great, thanks. :)

 I probably should treat this as a separate bug. Would you like me to
 file a new bug for it in the Debian BTS, in addition to submitting an
 upstream patch?

snip

I agree that this should be treated as a separate bug, yes please (I'm
just following up as it wasn't clear to me whether you're working on
this or not).


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#771863: [ovs-dev] Bug#771863: Service does not start or parse interfaces correctly, updating severity

2014-12-15 Thread Gurucharan Shetty
On Sat, Dec 13, 2014 at 2:03 AM, Stig Sandbeck Mathisen s...@debian.org wrote:
 Gurucharan Shetty shet...@nicira.com writes:

 I haven't looked at what needs to be done to handle those statements.
 I welcome a patch, preferably sent to d...@openvswitch.org after
 reading the CONTRIBUTING.md in the openvswitch repo.

 I assume the openvswitch repo is at https://github.com/openvswitch/ovs

 Do you use GitHub pull requests at all, or prefer just formatted patches
 to the mailing list?
We haven't handled GitHub pull requests so far. It does not fit well
with our current way of code-reviewing.
We would very much prefer code sent to d...@openvswitch.org using git
format-patch and git send-email (you can experiment by sending a mail
to yourself and apply it with 'git am')


 I would like to clarify that 2.3.1 does get rid of the check on
 $RUNLEVEL which was the original bug description.

 That's great, thanks. :)

 I probably should treat this as a separate bug. Would you like me to
 file a new bug for it in the Debian BTS, in addition to submitting an
 upstream patch?
I am not the maintainer of the project at debian. Ben Pfaff is the
maintainer. I only made a comment because I wrote the first version of
integration of 'interfaces' with openvswitch and wanted to clarify
that it has been fixed in 2.3.1 as a data point for the maintainer.

 --
 Stig Sandbeck Mathisen


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#771863: [ovs-dev] Bug#771863: Service does not start or parse interfaces correctly, updating severity

2014-12-13 Thread Stig Sandbeck Mathisen
Gurucharan Shetty shet...@nicira.com writes:

 I haven't looked at what needs to be done to handle those statements.
 I welcome a patch, preferably sent to d...@openvswitch.org after
 reading the CONTRIBUTING.md in the openvswitch repo.

I assume the openvswitch repo is at https://github.com/openvswitch/ovs

Do you use GitHub pull requests at all, or prefer just formatted patches
to the mailing list?

 I would like to clarify that 2.3.1 does get rid of the check on
 $RUNLEVEL which was the original bug description.

That's great, thanks. :)

I probably should treat this as a separate bug. Would you like me to
file a new bug for it in the Debian BTS, in addition to submitting an
upstream patch?

-- 
Stig Sandbeck Mathisen


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#771863: Service does not start or parse interfaces correctly, updating severity

2014-12-10 Thread Stig Sandbeck Mathisen
Control: severity -1 serious

Hello,

Since the service does not start under the new default init, and since
it also does not follow includes from /etc/network/interfaces, I would
think this justifies a serious severity. Updating the severity
accordingly.

(I feel less bad about bumping severity when there's a working patch
attached, though)

-- 
Regards,
Stig Sandbeck Mathisen


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#771863: [ovs-dev] Bug#771863: Service does not start or parse interfaces correctly, updating severity

2014-12-10 Thread Gurucharan Shetty
My previous reply did not do a good job with the context. It made it
look like OVS 2.3.1 handles source or source-directory statements.
That is not correct, it does not handle those statements. I haven't
looked at what needs to be done to handle those statements. I welcome
a patch, preferably sent to d...@openvswitch.org after reading the
CONTRIBUTING.md in the openvswitch repo.

I would like to clarify that 2.3.1 does get rid of the check on
$RUNLEVEL which was the original bug description.


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org