* [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable
@ 2013-12-05 12:55 Ross Burton
2013-12-05 12:56 ` [PATCH 2/3] lib/oeqa/runtime: print connman status if connman failed to start Ross Burton
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Ross Burton @ 2013-12-05 12:55 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/conf/documentation.conf | 1 -
1 file changed, 1 deletion(-)
diff --git a/meta/conf/documentation.conf b/meta/conf/documentation.conf
index 6666648..db3701b 100644
--- a/meta/conf/documentation.conf
+++ b/meta/conf/documentation.conf
@@ -118,7 +118,6 @@ IMAGE_LINGUAS[doc] = "Specifies the list of locales to install into the image du
IMAGE_OVERHEAD_FACTOR[doc] = "Defines a multiplier that the build system applies to the initial image size for cases when the multiplier times the returned disk usage value for the image is greater than the sum of IMAGE_ROOTFS_SIZE and IMAGE_ROOTFS_EXTRA_SPACE."
IMAGE_ROOTFS_EXTRA_SPACE[doc] = "Defines additional free disk space created in the image in Kbytes. By default, this variable is set to '0'."
IMAGE_ROOTFS_SIZE[doc] = "Defines the size in Kbytes for the generated image."
-IMAGETEST[doc] = "Enable test booting of virtual machine images under the qemu emulator after any root filesystems are created and run tests against those images."
INC_PR[doc] = "Helps define the recipe revision for recipes that share a common include file."
INHIBIT_PACKAGE_STRIP[doc] = "If set to "1", causes the build to not strip binaries in resulting packages."
INHERIT[doc] = "Causes the named class to be inherited at this point during parsing. The variable is only valid in configuration files."
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/3] lib/oeqa/runtime: print connman status if connman failed to start
2013-12-05 12:55 [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable Ross Burton
@ 2013-12-05 12:56 ` Ross Burton
2013-12-05 12:56 ` [PATCH 3/3] lib/oeqa/runtime: output more logging from systemd when services have failed Ross Burton
2013-12-05 13:32 ` [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable Richard Purdie
2 siblings, 0 replies; 6+ messages in thread
From: Ross Burton @ 2013-12-05 12:56 UTC (permalink / raw)
To: openembedded-core
If connman isn't running and we're running under systemd, use systemctl to get
the state according to systemd and the end of the connman log.
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/lib/oeqa/runtime/connman.py | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/meta/lib/oeqa/runtime/connman.py b/meta/lib/oeqa/runtime/connman.py
index b592ecc..c036882 100644
--- a/meta/lib/oeqa/runtime/connman.py
+++ b/meta/lib/oeqa/runtime/connman.py
@@ -9,6 +9,13 @@ def setUpModule():
class ConnmanTest(oeRuntimeTest):
+ def service_status(self, service):
+ if oeRuntimeTest.hasFeature("systemd"):
+ (status, output) = self.target.run('systemctl status -l %s' % service)
+ return output
+ else:
+ return "Unable to get status or logs for %s" % service
+
@skipUnlessPassed('test_ssh')
def test_connmand_help(self):
(status, output) = self.target.run('/usr/sbin/connmand --help')
@@ -18,4 +25,6 @@ class ConnmanTest(oeRuntimeTest):
@skipUnlessPassed('test_connmand_help')
def test_connmand_running(self):
(status, output) = self.target.run(oeRuntimeTest.pscmd + ' | grep [c]onnmand')
- self.assertEqual(status, 0, msg="no connmand process, ps output: %s" % self.target.run(oeRuntimeTest.pscmd)[1])
+ if status != 0:
+ print self.service_status("connman")
+ self.fail("No connmand process running")
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 3/3] lib/oeqa/runtime: output more logging from systemd when services have failed
2013-12-05 12:55 [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable Ross Burton
2013-12-05 12:56 ` [PATCH 2/3] lib/oeqa/runtime: print connman status if connman failed to start Ross Burton
@ 2013-12-05 12:56 ` Ross Burton
2013-12-05 13:32 ` [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable Richard Purdie
2 siblings, 0 replies; 6+ messages in thread
From: Ross Burton @ 2013-12-05 12:56 UTC (permalink / raw)
To: openembedded-core
If some services have failed to start, get the status of them and some of their
log to help debug the problem.
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
meta/lib/oeqa/runtime/systemd.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/meta/lib/oeqa/runtime/systemd.py b/meta/lib/oeqa/runtime/systemd.py
index e4f4336..17cc19f 100644
--- a/meta/lib/oeqa/runtime/systemd.py
+++ b/meta/lib/oeqa/runtime/systemd.py
@@ -21,7 +21,9 @@ class SystemdTests(oeRuntimeTest):
@skipUnlessPassed('test_systemd_version')
def test_systemd_failed(self):
(status, output) = self.target.run('systemctl --failed | grep "0 loaded units listed"')
- self.assertEqual(status, 0, msg="Failed systemd services: %s" % self.target.run('systemctl --failed')[1])
+ if status != 0:
+ print self.target.run('systemctl status --failed -l')[1]
+ self.fail("Some systemd units failed.")
@skipUnlessPassed('test_systemd_version')
def test_systemd_service(self):
--
1.7.10.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable
2013-12-05 12:55 [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable Ross Burton
2013-12-05 12:56 ` [PATCH 2/3] lib/oeqa/runtime: print connman status if connman failed to start Ross Burton
2013-12-05 12:56 ` [PATCH 3/3] lib/oeqa/runtime: output more logging from systemd when services have failed Ross Burton
@ 2013-12-05 13:32 ` Richard Purdie
2013-12-05 13:42 ` Paul Eggleton
2 siblings, 1 reply; 6+ messages in thread
From: Richard Purdie @ 2013-12-05 13:32 UTC (permalink / raw)
To: Ross Burton; +Cc: openembedded-core
On Thu, 2013-12-05 at 12:55 +0000, Ross Burton wrote:
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
> meta/conf/documentation.conf | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/meta/conf/documentation.conf b/meta/conf/documentation.conf
> index 6666648..db3701b 100644
> --- a/meta/conf/documentation.conf
> +++ b/meta/conf/documentation.conf
> @@ -118,7 +118,6 @@ IMAGE_LINGUAS[doc] = "Specifies the list of locales to install into the image du
> IMAGE_OVERHEAD_FACTOR[doc] = "Defines a multiplier that the build system applies to the initial image size for cases when the multiplier times the returned disk usage value for the image is greater than the sum of IMAGE_ROOTFS_SIZE and IMAGE_ROOTFS_EXTRA_SPACE."
> IMAGE_ROOTFS_EXTRA_SPACE[doc] = "Defines additional free disk space created in the image in Kbytes. By default, this variable is set to '0'."
> IMAGE_ROOTFS_SIZE[doc] = "Defines the size in Kbytes for the generated image."
> -IMAGETEST[doc] = "Enable test booting of virtual machine images under the qemu emulator after any root filesystems are created and run tests against those images."
> INC_PR[doc] = "Helps define the recipe revision for recipes that share a common include file."
> INHIBIT_PACKAGE_STRIP[doc] = "If set to "1", causes the build to not strip binaries in resulting packages."
> INHERIT[doc] = "Causes the named class to be inherited at this point during parsing. The variable is only valid in configuration files."
Isn't it actually IMAGE_TEST ?
Cheers,
Richard
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable
2013-12-05 13:32 ` [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable Richard Purdie
@ 2013-12-05 13:42 ` Paul Eggleton
2013-12-05 13:44 ` Paul Eggleton
0 siblings, 1 reply; 6+ messages in thread
From: Paul Eggleton @ 2013-12-05 13:42 UTC (permalink / raw)
To: Richard Purdie, Ross Burton; +Cc: openembedded-core
On Thursday 05 December 2013 13:32:54 Richard Purdie wrote:
> On Thu, 2013-12-05 at 12:55 +0000, Ross Burton wrote:
> > Signed-off-by: Ross Burton <ross.burton@intel.com>
> > ---
> >
> > meta/conf/documentation.conf | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/meta/conf/documentation.conf b/meta/conf/documentation.conf
> > index 6666648..db3701b 100644
> > --- a/meta/conf/documentation.conf
> > +++ b/meta/conf/documentation.conf
> > @@ -118,7 +118,6 @@ IMAGE_LINGUAS[doc] = "Specifies the list of locales to
> > install into the image du>
> > IMAGE_OVERHEAD_FACTOR[doc] = "Defines a multiplier that the build system
> > applies to the initial image size for cases when the multiplier times
> > the returned disk usage value for the image is greater than the sum of
> > IMAGE_ROOTFS_SIZE and IMAGE_ROOTFS_EXTRA_SPACE."
> > IMAGE_ROOTFS_EXTRA_SPACE[doc] = "Defines additional free disk space
> > created in the image in Kbytes. By default, this variable is set to
> > '0'." IMAGE_ROOTFS_SIZE[doc] = "Defines the size in Kbytes for the
> > generated image.">
> > -IMAGETEST[doc] = "Enable test booting of virtual machine images under the
> > qemu emulator after any root filesystems are created and run tests
> > against those images.">
> > INC_PR[doc] = "Helps define the recipe revision for recipes that share a
> > common include file." INHIBIT_PACKAGE_STRIP[doc] = "If set to "1",
> > causes the build to not strip binaries in resulting packages."
> > INHERIT[doc] = "Causes the named class to be inherited at this point
> > during parsing. The variable is only valid in configuration files."
>
> Isn't it actually IMAGE_TEST ?
If you mean what it is now, it's TEST_IMAGE.
If we're removing the old IMAGETEST variable (and we should) we should also
remove TEST_SCEN at the same time.
Cheers,
Paul
--
Paul Eggleton
Intel Open Source Technology Centre
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable
2013-12-05 13:42 ` Paul Eggleton
@ 2013-12-05 13:44 ` Paul Eggleton
0 siblings, 0 replies; 6+ messages in thread
From: Paul Eggleton @ 2013-12-05 13:44 UTC (permalink / raw)
To: Richard Purdie, Ross Burton; +Cc: openembedded-core
On Thursday 05 December 2013 13:42:17 Paul Eggleton wrote:
> On Thursday 05 December 2013 13:32:54 Richard Purdie wrote:
> > On Thu, 2013-12-05 at 12:55 +0000, Ross Burton wrote:
> > > Signed-off-by: Ross Burton <ross.burton@intel.com>
> > > ---
> > >
> > > meta/conf/documentation.conf | 1 -
> > > 1 file changed, 1 deletion(-)
> > >
> > > diff --git a/meta/conf/documentation.conf b/meta/conf/documentation.conf
> > > index 6666648..db3701b 100644
> > > --- a/meta/conf/documentation.conf
> > > +++ b/meta/conf/documentation.conf
> > > @@ -118,7 +118,6 @@ IMAGE_LINGUAS[doc] = "Specifies the list of locales
> > > to
> > > install into the image du>
> > >
> > > IMAGE_OVERHEAD_FACTOR[doc] = "Defines a multiplier that the build
> > > system
> > > applies to the initial image size for cases when the multiplier times
> > > the returned disk usage value for the image is greater than the sum of
> > > IMAGE_ROOTFS_SIZE and IMAGE_ROOTFS_EXTRA_SPACE."
> > > IMAGE_ROOTFS_EXTRA_SPACE[doc] = "Defines additional free disk space
> > > created in the image in Kbytes. By default, this variable is set to
> > > '0'." IMAGE_ROOTFS_SIZE[doc] = "Defines the size in Kbytes for the
> > > generated image.">
> > >
> > > -IMAGETEST[doc] = "Enable test booting of virtual machine images under
> > > the
> > > qemu emulator after any root filesystems are created and run tests
> > > against those images.">
> > >
> > > INC_PR[doc] = "Helps define the recipe revision for recipes that share
> > > a
> > > common include file." INHIBIT_PACKAGE_STRIP[doc] = "If set to "1",
> > > causes the build to not strip binaries in resulting packages."
> > > INHERIT[doc] = "Causes the named class to be inherited at this point
> > > during parsing. The variable is only valid in configuration files."
> >
> > Isn't it actually IMAGE_TEST ?
>
> If you mean what it is now, it's TEST_IMAGE.
>
> If we're removing the old IMAGETEST variable (and we should) we should also
> remove TEST_SCEN at the same time.
... and also TEST_SERIALIZE.
Cheers,
Paul
--
Paul Eggleton
Intel Open Source Technology Centre
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-12-05 13:44 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-05 12:55 [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable Ross Burton
2013-12-05 12:56 ` [PATCH 2/3] lib/oeqa/runtime: print connman status if connman failed to start Ross Burton
2013-12-05 12:56 ` [PATCH 3/3] lib/oeqa/runtime: output more logging from systemd when services have failed Ross Burton
2013-12-05 13:32 ` [PATCH 1/3] documentation.conf: remove unused IMAGETEST variable Richard Purdie
2013-12-05 13:42 ` Paul Eggleton
2013-12-05 13:44 ` Paul Eggleton
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.