* [PATCH] openvswtich: start openvswitch at the boot time
@ 2013-01-31 7:18 lei.yang
2013-01-31 13:39 ` Bruce Ashfield
0 siblings, 1 reply; 8+ messages in thread
From: lei.yang @ 2013-01-31 7:18 UTC (permalink / raw)
To: meta-virtualization, bruce.ashfield
From: Lei Yang <lei.yang@windriver.com>
When I define the bridge, I get below error
root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
Jan 25 17:34:07|00002|stream_unix|ERR|/tmp/stream-unix.1487.0: connection to /var/run/openvswitch/db.sock
failed: No such file or directory
........
........
This is because openvswitch damon isn't running,so we'd better start it at the boot time.
The split-pacakage ${PN}-switch ${PN}-controller provide us to start nessary damon with update-rc.d
so we add them to the RDEPENDS.
Since openvswtich depends on openvswtich kernel module,so auto load the kernel module too.
Signed-off-by: Lei Yang <lei.yang@windriver.com>
Tested-by: Lei Yang <lei.yang@windriver.com>
---
recipes-kernel/linux/linux-yocto_3.4.bbappend | 2 +-
.../openvswitch/openvswitch_1.4.3.bb | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/recipes-kernel/linux/linux-yocto_3.4.bbappend b/recipes-kernel/linux/linux-yocto_3.4.bbappend
index 5b163ff..918302a 100644
--- a/recipes-kernel/linux/linux-yocto_3.4.bbappend
+++ b/recipes-kernel/linux/linux-yocto_3.4.bbappend
@@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
file://vswitch.scc \
file://lxc.scc \
"
-
+module_autoload_openvswitch = "openvswitch"
diff --git a/recipes-networking/openvswitch/openvswitch_1.4.3.bb b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
index 88f0114..6764073 100644
--- a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
+++ b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
@@ -7,7 +7,7 @@ LICENSE = "Apache-2"
DEPENDS += "bridge-utils openssl python perl"
RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
- python perl perl-module-strict"
+ python perl perl-module-strict ${PN}-switch ${PN}-controller"
RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
RDEPENDS_${PN}-pki = "${PN}"
--
1.7.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH] openvswtich: start openvswitch at the boot time
2013-01-31 7:18 [PATCH] openvswtich: start openvswitch at the boot time lei.yang
@ 2013-01-31 13:39 ` Bruce Ashfield
2013-02-01 3:54 ` lei yang
0 siblings, 1 reply; 8+ messages in thread
From: Bruce Ashfield @ 2013-01-31 13:39 UTC (permalink / raw)
To: lei yang; +Cc: meta-virtualization@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 2612 bytes --]
On Thu, Jan 31, 2013 at 2:18 AM, <lei.yang@windriver.com> wrote:
> From: Lei Yang <lei.yang@windriver.com>
>
> When I define the bridge, I get below error
>
> root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
> Jan 25 17:34:07|00002|stream_unix|ERR|/tmp/stream-unix.1487.0: connection
> to /var/run/openvswitch/db.sock
> failed: No such file or directory
> ........
> ........
>
> This is because openvswitch damon isn't running,so we'd better start it at
> the boot time.
> The split-pacakage ${PN}-switch ${PN}-controller provide us to start
> nessary damon with update-rc.d
>
s/pacakage/package/
s/nessary/necessary/
s/damon/daemon/
> so we add them to the RDEPENDS.
>
> Since openvswtich depends on openvswtich kernel module,so auto load the
> kernel module too.
>
s/openvswtich/openvswitch/
>
> Signed-off-by: Lei Yang <lei.yang@windriver.com>
> Tested-by: Lei Yang <lei.yang@windriver.com>
>
We can drop the Tested-by: since it is your patch :)
I can make the changes below if you agree with them, and save us another
revision
of the patch.
Cheers,
Bruce
> ---
> recipes-kernel/linux/linux-yocto_3.4.bbappend | 2 +-
> .../openvswitch/openvswitch_1.4.3.bb | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> index 5b163ff..918302a 100644
> --- a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> +++ b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> @@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
> file://vswitch.scc \
> file://lxc.scc \
> "
> -
> +module_autoload_openvswitch = "openvswitch"
> diff --git a/recipes-networking/openvswitch/openvswitch_1.4.3.bbb/recipes-networking/openvswitch/
> openvswitch_1.4.3.bb
> index 88f0114..6764073 100644
> --- a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> +++ b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> @@ -7,7 +7,7 @@ LICENSE = "Apache-2"
> DEPENDS += "bridge-utils openssl python perl"
>
> RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
> - python perl perl-module-strict"
> + python perl perl-module-strict ${PN}-switch
> ${PN}-controller"
> RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
> RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
> RDEPENDS_${PN}-pki = "${PN}"
> --
> 1.7.1
>
>
--
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"
[-- Attachment #2: Type: text/html, Size: 4497 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] openvswtich: start openvswitch at the boot time
2013-01-31 13:39 ` Bruce Ashfield
@ 2013-02-01 3:54 ` lei yang
2013-02-01 14:10 ` Bruce Ashfield
0 siblings, 1 reply; 8+ messages in thread
From: lei yang @ 2013-02-01 3:54 UTC (permalink / raw)
To: Bruce Ashfield; +Cc: meta-virtualization@yoctoproject.org
On 01/31/2013 09:39 PM, Bruce Ashfield wrote:
>
>
>
> On Thu, Jan 31, 2013 at 2:18 AM, <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>> wrote:
>
> From: Lei Yang <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>>
>
> When I define the bridge, I get below error
>
> root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
> Jan 25 17:34:07|00002|stream_unix|ERR|/tmp/stream-unix.1487.0:
> connection to /var/run/openvswitch/db.sock
> failed: No such file or directory
> ........
> ........
>
> This is because openvswitch damon isn't running,so we'd better
> start it at the boot time.
> The split-pacakage ${PN}-switch ${PN}-controller provide us to
> start nessary damon with update-rc.d
>
>
> s/pacakage/package/
> s/nessary/necessary/
> s/damon/daemon/
>
> so we add them to the RDEPENDS.
>
> Since openvswtich depends on openvswtich kernel module,so auto
> load the kernel module too.
>
>
> s/openvswtich/openvswitch/
>
>
> Signed-off-by: Lei Yang <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>>
> Tested-by: Lei Yang <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>>
>
>
> We can drop the Tested-by: since it is your patch :)
>
> I can make the changes below if you agree with them, and save us
> another revision
> of the patch.
>
Agree, I don't trust "set spell" again in vim :-( , it doesn't help me
for this spelling error
Lei
> Cheers,
>
> Bruce
>
> ---
> recipes-kernel/linux/linux-yocto_3.4.bbappend | 2 +-
> .../openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb> | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> index 5b163ff..918302a 100644
> --- a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> +++ b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> @@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
> file://vswitch.scc \
> file://lxc.scc \
> "
> -
> +module_autoload_openvswitch = "openvswitch"
> diff --git a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> index 88f0114..6764073 100644
> --- a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> +++ b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> @@ -7,7 +7,7 @@ LICENSE = "Apache-2"
> DEPENDS += "bridge-utils openssl python perl"
>
> RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
> - python perl perl-module-strict"
> + python perl perl-module-strict ${PN}-switch
> ${PN}-controller"
> RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
> RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
> RDEPENDS_${PN}-pki = "${PN}"
> --
> 1.7.1
>
>
>
>
> --
> "Thou shalt not follow the NULL pointer, for chaos and madness await
> thee at its end"
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] openvswtich: start openvswitch at the boot time
2013-02-01 3:54 ` lei yang
@ 2013-02-01 14:10 ` Bruce Ashfield
0 siblings, 0 replies; 8+ messages in thread
From: Bruce Ashfield @ 2013-02-01 14:10 UTC (permalink / raw)
To: lei yang; +Cc: meta-virtualization@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 4007 bytes --]
On Thu, Jan 31, 2013 at 10:54 PM, lei yang <lei.yang@windriver.com> wrote:
> On 01/31/2013 09:39 PM, Bruce Ashfield wrote:
>
>>
>>
>>
>> On Thu, Jan 31, 2013 at 2:18 AM, <lei.yang@windriver.com <mailto:
>> lei.yang@windriver.com**>> wrote:
>>
>> From: Lei Yang <lei.yang@windriver.com
>> <mailto:lei.yang@windriver.com**>>
>>
>>
>> When I define the bridge, I get below error
>>
>> root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
>> Jan 25 17:34:07|00002|stream_unix|**ERR|/tmp/stream-unix.1487.0:
>> connection to /var/run/openvswitch/db.sock
>> failed: No such file or directory
>> ........
>> ........
>>
>> This is because openvswitch damon isn't running,so we'd better
>> start it at the boot time.
>> The split-pacakage ${PN}-switch ${PN}-controller provide us to
>> start nessary damon with update-rc.d
>>
>>
>> s/pacakage/package/
>> s/nessary/necessary/
>> s/damon/daemon/
>>
>> so we add them to the RDEPENDS.
>>
>> Since openvswtich depends on openvswtich kernel module,so auto
>> load the kernel module too.
>>
>>
>> s/openvswtich/openvswitch/
>>
>>
>> Signed-off-by: Lei Yang <lei.yang@windriver.com
>> <mailto:lei.yang@windriver.com**>>
>> Tested-by: Lei Yang <lei.yang@windriver.com
>> <mailto:lei.yang@windriver.com**>>
>>
>>
>>
>> We can drop the Tested-by: since it is your patch :)
>>
>> I can make the changes below if you agree with them, and save us another
>> revision
>> of the patch.
>>
>>
> Agree, I don't trust "set spell" again in vim :-( , it doesn't help me for
> this spelling error
>
>
No worries, we all make spelling mistakes and typos.
I wrapped the commit message to 72 chars, fixed the few issues and pushed it
to master.
Bruce
> Lei
>
>
>
> Cheers,
>>
>> Bruce
>>
>> ---
>> recipes-kernel/linux/linux-**yocto_3.4.bbappend | 2 +-
>> .../openvswitch/openvswitch_1.**4.3.bb <http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb> | 2 +-
>>
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> b/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> index 5b163ff..918302a 100644
>> --- a/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> +++ b/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> @@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
>> file://vswitch.scc \
>> file://lxc.scc \
>> "
>> -
>> +module_autoload_openvswitch = "openvswitch"
>> diff --git a/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>> b/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>>
>> index 88f0114..6764073 100644
>> --- a/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>> +++ b/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>>
>> @@ -7,7 +7,7 @@ LICENSE = "Apache-2"
>> DEPENDS += "bridge-utils openssl python perl"
>>
>> RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
>> - python perl perl-module-strict"
>> + python perl perl-module-strict ${PN}-switch
>> ${PN}-controller"
>> RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
>> RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
>> RDEPENDS_${PN}-pki = "${PN}"
>> --
>> 1.7.1
>>
>>
>>
>>
>> --
>> "Thou shalt not follow the NULL pointer, for chaos and madness await thee
>> at its end"
>>
>
>
--
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"
[-- Attachment #2: Type: text/html, Size: 6289 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] openvswtich: start openvswitch at the boot time
@ 2013-01-28 11:01 lei.yang
2013-01-29 13:46 ` Bruce Ashfield
0 siblings, 1 reply; 8+ messages in thread
From: lei.yang @ 2013-01-28 11:01 UTC (permalink / raw)
To: meta-virtualization, david.c.nystrom
From: Lei Yang <lei.yang@windriver.com>
When I define the bridge, I get below error
root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
Jan 25 17:34:07|00002|stream_unix|ERR|/tmp/stream-unix.1487.0: connection to /var/run/openvswitch/db.sock
failed: No such file or directory
........
........
This is becasue openvswitch isn't running,so we'd better start it at the boot time.
Since openvswtich depends on openvswtich module,so auto load the kernel module too.
Signed-off-by: Lei Yang <lei.yang@windriver.com>
Tested-by: Lei Yang <lei.yang@windriver.com>
---
recipes-kernel/linux/linux-yocto_3.4.bbappend | 2 +-
.../openvswitch/openvswitch_1.4.3.bb | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/recipes-kernel/linux/linux-yocto_3.4.bbappend b/recipes-kernel/linux/linux-yocto_3.4.bbappend
index 5b163ff..918302a 100644
--- a/recipes-kernel/linux/linux-yocto_3.4.bbappend
+++ b/recipes-kernel/linux/linux-yocto_3.4.bbappend
@@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
file://vswitch.scc \
file://lxc.scc \
"
-
+module_autoload_openvswitch = "openvswitch"
diff --git a/recipes-networking/openvswitch/openvswitch_1.4.3.bb b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
index 88f0114..6764073 100644
--- a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
+++ b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
@@ -7,7 +7,7 @@ LICENSE = "Apache-2"
DEPENDS += "bridge-utils openssl python perl"
RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
- python perl perl-module-strict"
+ python perl perl-module-strict ${PN}-switch ${PN}-controller"
RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
RDEPENDS_${PN}-pki = "${PN}"
--
1.7.1
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [PATCH] openvswtich: start openvswitch at the boot time
2013-01-28 11:01 lei.yang
@ 2013-01-29 13:46 ` Bruce Ashfield
2013-01-29 16:32 ` lei yang
0 siblings, 1 reply; 8+ messages in thread
From: Bruce Ashfield @ 2013-01-29 13:46 UTC (permalink / raw)
To: lei yang; +Cc: meta-virtualization@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 5290 bytes --]
On Mon, Jan 28, 2013 at 6:01 AM, <lei.yang@windriver.com> wrote:
> From: Lei Yang <lei.yang@windriver.com>
>
> When I define the bridge, I get below error
>
> root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
> Jan 25 17:34:07|00002|stream_unix|ERR|/tmp/stream-unix.1487.0: connection
> to /var/run/openvswitch/db.sock
> failed: No such file or directory
> ........
> ........
>
> This is becasue openvswitch isn't running,so we'd better start it at the
> boot time.
>
s/becasue/because/
Can you elaborate on this a bit more as well. What you are actually doing
in the
patch is adding to the RDEPENDS of openvswitch and it's the dependencies
that
you are adding, that once installed are starting the switch at runtime ..
right ? The
reason I'd like it clarified is that when I read your patch, I expected to
see an initscript
update, or inherit rc.d, but found the RDEPENDS update instead.
> Since openvswtich depends on openvswtich module,so auto load the kernel
> module too.
> Signed-off-by: Lei Yang <lei.yang@windriver.com>
> Tested-by: Lei Yang <lei.yang@windriver.com>
> ---
> recipes-kernel/linux/linux-yocto_3.4.bbappend | 2 +-
> .../openvswitch/openvswitch_1.4.3.bb | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> index 5b163ff..918302a 100644
> --- a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> +++ b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> @@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
> file://vswitch.scc \
> file://lxc.scc \
> "
> -
> +module_autoload_openvswitch = "openvswitch"
>
This looks fine. I double checked that the vswitch kernel fragment enables
the module,
so our autoload is fine.
Bruce
> diff --git a/recipes-networking/openvswitch/openvswitch_1.4.3.bbb/recipes-networking/openvswitch/
> openvswitch_1.4.3.bb
> index 88f0114..6764073 100644
> --- a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> +++ b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> @@ -7,7 +7,7 @@ LICENSE = "Apache-2"
> DEPENDS += "bridge-utils openssl python perl"
>
> RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
> - python perl perl-module-strict"
> + python perl perl-module-strict ${PN}-switch
> ${PN}-controller"
> RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
> RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
> RDEPENDS_${PN}-pki = "${PN}"
> --
> 1.7.1
>
> _______________________________________________
> meta-virtualization mailing list
> meta-virtualization@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-virtualization
>
--
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"
On Mon, Jan 28, 2013 at 6:01 AM, <lei.yang@windriver.com> wrote:
> From: Lei Yang <lei.yang@windriver.com>
>
> When I define the bridge, I get below error
>
> root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
> Jan 25 17:34:07|00002|stream_unix|ERR|/tmp/stream-unix.1487.0: connection
> to /var/run/openvswitch/db.sock
> failed: No such file or directory
> ........
> ........
>
> This is becasue openvswitch isn't running,so we'd better start it at the
> boot time.
> Since openvswtich depends on openvswtich module,so auto load the kernel
> module too.
>
> Signed-off-by: Lei Yang <lei.yang@windriver.com>
> Tested-by: Lei Yang <lei.yang@windriver.com>
> ---
> recipes-kernel/linux/linux-yocto_3.4.bbappend | 2 +-
> .../openvswitch/openvswitch_1.4.3.bb | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> index 5b163ff..918302a 100644
> --- a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> +++ b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> @@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
> file://vswitch.scc \
> file://lxc.scc \
> "
> -
> +module_autoload_openvswitch = "openvswitch"
> diff --git a/recipes-networking/openvswitch/openvswitch_1.4.3.bbb/recipes-networking/openvswitch/
> openvswitch_1.4.3.bb
> index 88f0114..6764073 100644
> --- a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> +++ b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> @@ -7,7 +7,7 @@ LICENSE = "Apache-2"
> DEPENDS += "bridge-utils openssl python perl"
>
> RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
> - python perl perl-module-strict"
> + python perl perl-module-strict ${PN}-switch
> ${PN}-controller"
> RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
> RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
> RDEPENDS_${PN}-pki = "${PN}"
> --
> 1.7.1
>
> _______________________________________________
> meta-virtualization mailing list
> meta-virtualization@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-virtualization
>
--
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"
[-- Attachment #2: Type: text/html, Size: 8377 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] openvswtich: start openvswitch at the boot time
2013-01-29 13:46 ` Bruce Ashfield
@ 2013-01-29 16:32 ` lei yang
2013-01-29 16:46 ` Bruce Ashfield
0 siblings, 1 reply; 8+ messages in thread
From: lei yang @ 2013-01-29 16:32 UTC (permalink / raw)
To: Bruce Ashfield; +Cc: meta-virtualization@yoctoproject.org
On 01/29/2013 09:46 PM, Bruce Ashfield wrote:
>
>
> On Mon, Jan 28, 2013 at 6:01 AM, <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>> wrote:
>
> From: Lei Yang <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>>
>
> When I define the bridge, I get below error
>
> root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
> Jan 25 17:34:07|00002|stream_unix|ERR|/tmp/stream-unix.1487.0:
> connection to /var/run/openvswitch/db.sock
> failed: No such file or directory
> ........
> ........
>
> This is becasue openvswitch isn't running,so we'd better start it
> at the boot time.
>
>
> s/becasue/because/
>
> Can you elaborate on this a bit more as well. What you are actually
> doing in the
> patch is adding to the RDEPENDS of openvswitch and it's the
> dependencies that
> you are adding, that once installed are starting the switch at runtime
> .. right ? The
> reason I'd like it clarified is that when I read your patch, I
> expected to see an initscript
> update, or inherit rc.d, but found the RDEPENDS update instead.
because : those tow packages used rc.d. so we only need to install them
inherit autotools update-rc.d
INITSCRIPT_PACKAGES = "${PN}-switch ${PN}-controller"
INITSCRIPT_NAME_${PN}-switch = "openvswitch-switch"
INITSCRIPT_PARAMS_${PN}-switch = "defaults 71"
INITSCRIPT_NAME_${PN}-controller = "openvswitch-controller"
INITSCRIPT_PARAMS_${PN}-controller = "defaults 72"
I will send V2 tomorrow
LeI
> Since openvswtich depends on openvswtich module,so auto load the
> kernel module too.
>
>
> Signed-off-by: Lei Yang <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>>
> Tested-by: Lei Yang <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>>
> ---
> recipes-kernel/linux/linux-yocto_3.4.bbappend | 2 +-
> .../openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb> | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> index 5b163ff..918302a 100644
> --- a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> +++ b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> @@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
> file://vswitch.scc \
> file://lxc.scc \
> "
> -
> +module_autoload_openvswitch = "openvswitch"
>
>
> This looks fine. I double checked that the vswitch kernel fragment
> enables the module,
> so our autoload is fine.
>
> Bruce
>
> diff --git a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> index 88f0114..6764073 100644
> --- a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> +++ b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> @@ -7,7 +7,7 @@ LICENSE = "Apache-2"
> DEPENDS += "bridge-utils openssl python perl"
>
> RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
> - python perl perl-module-strict"
> + python perl perl-module-strict ${PN}-switch
> ${PN}-controller"
> RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
> RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
> RDEPENDS_${PN}-pki = "${PN}"
> --
> 1.7.1
>
> _______________________________________________
> meta-virtualization mailing list
> meta-virtualization@yoctoproject.org
> <mailto:meta-virtualization@yoctoproject.org>
> https://lists.yoctoproject.org/listinfo/meta-virtualization
>
>
>
>
> --
> "Thou shalt not follow the NULL pointer, for chaos and madness await
> thee at its end"
>
>
> On Mon, Jan 28, 2013 at 6:01 AM, <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>> wrote:
>
> From: Lei Yang <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>>
>
> When I define the bridge, I get below error
>
> root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
> Jan 25 17:34:07|00002|stream_unix|ERR|/tmp/stream-unix.1487.0:
> connection to /var/run/openvswitch/db.sock
> failed: No such file or directory
> ........
> ........
>
> This is becasue openvswitch isn't running,so we'd better start it
> at the boot time.
> Since openvswtich depends on openvswtich module,so auto load the
> kernel module too.
>
> Signed-off-by: Lei Yang <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>>
> Tested-by: Lei Yang <lei.yang@windriver.com
> <mailto:lei.yang@windriver.com>>
> ---
> recipes-kernel/linux/linux-yocto_3.4.bbappend | 2 +-
> .../openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb> | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> index 5b163ff..918302a 100644
> --- a/recipes-kernel/linux/linux-yocto_3.4.bbappend
> +++ b/recipes-kernel/linux/linux-yocto_3.4.bbappend
> @@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
> file://vswitch.scc \
> file://lxc.scc \
> "
> -
> +module_autoload_openvswitch = "openvswitch"
> diff --git a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> index 88f0114..6764073 100644
> --- a/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> +++ b/recipes-networking/openvswitch/openvswitch_1.4.3.bb
> <http://openvswitch_1.4.3.bb>
> @@ -7,7 +7,7 @@ LICENSE = "Apache-2"
> DEPENDS += "bridge-utils openssl python perl"
>
> RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
> - python perl perl-module-strict"
> + python perl perl-module-strict ${PN}-switch
> ${PN}-controller"
> RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
> RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
> RDEPENDS_${PN}-pki = "${PN}"
> --
> 1.7.1
>
> _______________________________________________
> meta-virtualization mailing list
> meta-virtualization@yoctoproject.org
> <mailto:meta-virtualization@yoctoproject.org>
> https://lists.yoctoproject.org/listinfo/meta-virtualization
>
>
>
>
> --
> "Thou shalt not follow the NULL pointer, for chaos and madness await
> thee at its end"
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PATCH] openvswtich: start openvswitch at the boot time
2013-01-29 16:32 ` lei yang
@ 2013-01-29 16:46 ` Bruce Ashfield
0 siblings, 0 replies; 8+ messages in thread
From: Bruce Ashfield @ 2013-01-29 16:46 UTC (permalink / raw)
To: lei yang; +Cc: meta-virtualization@yoctoproject.org
[-- Attachment #1: Type: text/plain, Size: 8231 bytes --]
On Tue, Jan 29, 2013 at 11:32 AM, lei yang <lei.yang@windriver.com> wrote:
> On 01/29/2013 09:46 PM, Bruce Ashfield wrote:
>
>>
>>
>> On Mon, Jan 28, 2013 at 6:01 AM, <lei.yang@windriver.com <mailto:
>> lei.yang@windriver.com**>> wrote:
>>
>> From: Lei Yang <lei.yang@windriver.com
>> <mailto:lei.yang@windriver.com**>>
>>
>>
>> When I define the bridge, I get below error
>>
>> root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
>> Jan 25 17:34:07|00002|stream_unix|**ERR|/tmp/stream-unix.1487.0:
>> connection to /var/run/openvswitch/db.sock
>> failed: No such file or directory
>> ........
>> ........
>>
>> This is becasue openvswitch isn't running,so we'd better start it
>> at the boot time.
>>
>>
>> s/becasue/because/
>>
>> Can you elaborate on this a bit more as well. What you are actually doing
>> in the
>> patch is adding to the RDEPENDS of openvswitch and it's the dependencies
>> that
>> you are adding, that once installed are starting the switch at runtime ..
>> right ? The
>> reason I'd like it clarified is that when I read your patch, I expected
>> to see an initscript
>> update, or inherit rc.d, but found the RDEPENDS update instead.
>>
>
>
> because : those tow packages used rc.d. so we only need to install them
>
> inherit autotools update-rc.d
>
> INITSCRIPT_PACKAGES = "${PN}-switch ${PN}-controller"
> INITSCRIPT_NAME_${PN}-switch = "openvswitch-switch"
> INITSCRIPT_PARAMS_${PN}-switch = "defaults 71"
>
> INITSCRIPT_NAME_${PN}-**controller = "openvswitch-controller"
> INITSCRIPT_PARAMS_${PN}-**controller = "defaults 72"
>
>
>
Exactly, that's what I saw when I checked .. but it's worth being clear in
the
message.
Bruce
> I will send V2 tomorrow
>
> LeI
>
>
>
>
>
> Since openvswtich depends on openvswtich module,so auto load the
>> kernel module too.
>>
>>
>> Signed-off-by: Lei Yang <lei.yang@windriver.com
>> <mailto:lei.yang@windriver.com**>>
>> Tested-by: Lei Yang <lei.yang@windriver.com
>> <mailto:lei.yang@windriver.com**>>
>>
>> ---
>> recipes-kernel/linux/linux-**yocto_3.4.bbappend | 2 +-
>> .../openvswitch/openvswitch_1.**4.3.bb <http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb> | 2 +-
>>
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> b/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> index 5b163ff..918302a 100644
>> --- a/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> +++ b/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> @@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
>> file://vswitch.scc \
>> file://lxc.scc \
>> "
>> -
>> +module_autoload_openvswitch = "openvswitch"
>>
>>
>> This looks fine. I double checked that the vswitch kernel fragment
>> enables the module,
>> so our autoload is fine.
>>
>> Bruce
>>
>> diff --git a/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>> b/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>>
>> index 88f0114..6764073 100644
>> --- a/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>> +++ b/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>>
>> @@ -7,7 +7,7 @@ LICENSE = "Apache-2"
>> DEPENDS += "bridge-utils openssl python perl"
>>
>> RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
>> - python perl perl-module-strict"
>> + python perl perl-module-strict ${PN}-switch
>> ${PN}-controller"
>> RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
>> RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
>> RDEPENDS_${PN}-pki = "${PN}"
>> --
>> 1.7.1
>>
>> ______________________________**_________________
>> meta-virtualization mailing list
>> meta-virtualization@**yoctoproject.org<meta-virtualization@yoctoproject.org>
>> <mailto:meta-virtualization@**yoctoproject.org<meta-virtualization@yoctoproject.org>
>> >
>>
>> https://lists.yoctoproject.**org/listinfo/meta-**virtualization<https://lists.yoctoproject.org/listinfo/meta-virtualization>
>>
>>
>>
>>
>> --
>> "Thou shalt not follow the NULL pointer, for chaos and madness await thee
>> at its end"
>>
>>
>> On Mon, Jan 28, 2013 at 6:01 AM, <lei.yang@windriver.com <mailto:
>> lei.yang@windriver.com**>> wrote:
>>
>> From: Lei Yang <lei.yang@windriver.com
>> <mailto:lei.yang@windriver.com**>>
>>
>>
>> When I define the bridge, I get below error
>>
>> root@intel_5500_server:~# ovs-vsctl add-br ovsbr0
>> Jan 25 17:34:07|00002|stream_unix|**ERR|/tmp/stream-unix.1487.0:
>> connection to /var/run/openvswitch/db.sock
>> failed: No such file or directory
>> ........
>> ........
>>
>> This is becasue openvswitch isn't running,so we'd better start it
>> at the boot time.
>> Since openvswtich depends on openvswtich module,so auto load the
>> kernel module too.
>>
>> Signed-off-by: Lei Yang <lei.yang@windriver.com
>> <mailto:lei.yang@windriver.com**>>
>> Tested-by: Lei Yang <lei.yang@windriver.com
>> <mailto:lei.yang@windriver.com**>>
>>
>> ---
>> recipes-kernel/linux/linux-**yocto_3.4.bbappend | 2 +-
>> .../openvswitch/openvswitch_1.**4.3.bb <http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb> | 2 +-
>>
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> b/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> index 5b163ff..918302a 100644
>> --- a/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> +++ b/recipes-kernel/linux/linux-**yocto_3.4.bbappend
>> @@ -5,4 +5,4 @@ SRC_URI += "file://xt-checksum.scc \
>> file://vswitch.scc \
>> file://lxc.scc \
>> "
>> -
>> +module_autoload_openvswitch = "openvswitch"
>> diff --git a/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>> b/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>>
>> index 88f0114..6764073 100644
>> --- a/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>> +++ b/recipes-networking/**openvswitch/openvswitch_1.4.3.**bb<http://openvswitch_1.4.3.bb>
>> <http://openvswitch_1.4.3.bb>
>>
>> @@ -7,7 +7,7 @@ LICENSE = "Apache-2"
>> DEPENDS += "bridge-utils openssl python perl"
>>
>> RDEPENDS_${PN} += "util-linux-uuidgen util-linux-libuuid \
>> - python perl perl-module-strict"
>> + python perl perl-module-strict ${PN}-switch
>> ${PN}-controller"
>> RDEPENDS_${PN}-controller = "${PN} lsb ${PN}-pki"
>> RDEPENDS_${PN}-switch = "${PN} openssl procps util-linux-uuidgen"
>> RDEPENDS_${PN}-pki = "${PN}"
>> --
>> 1.7.1
>>
>> ______________________________**_________________
>> meta-virtualization mailing list
>> meta-virtualization@**yoctoproject.org<meta-virtualization@yoctoproject.org>
>> <mailto:meta-virtualization@**yoctoproject.org<meta-virtualization@yoctoproject.org>
>> >
>>
>> https://lists.yoctoproject.**org/listinfo/meta-**virtualization<https://lists.yoctoproject.org/listinfo/meta-virtualization>
>>
>>
>>
>>
>> --
>> "Thou shalt not follow the NULL pointer, for chaos and madness await thee
>> at its end"
>>
>
>
--
"Thou shalt not follow the NULL pointer, for chaos and madness await thee
at its end"
[-- Attachment #2: Type: text/html, Size: 12225 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2013-02-01 14:10 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-31 7:18 [PATCH] openvswtich: start openvswitch at the boot time lei.yang
2013-01-31 13:39 ` Bruce Ashfield
2013-02-01 3:54 ` lei yang
2013-02-01 14:10 ` Bruce Ashfield
-- strict thread matches above, loose matches on Subject: below --
2013-01-28 11:01 lei.yang
2013-01-29 13:46 ` Bruce Ashfield
2013-01-29 16:32 ` lei yang
2013-01-29 16:46 ` Bruce Ashfield
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.