[v4,4/4] MAINTAINERS: merge Build and test automation with Docker tests

Message ID 20170216123456.28621-5-alex.bennee@linaro.org
State Superseded
Headers show
Series
  • Docker cross-compile targets and user build support
Related show

Commit Message

Alex Bennée Feb. 16, 2017, 12:34 p.m.
The docker framework is really just another piece in the build
automation puzzle so lets merge it together. For added bonus I've also
included the Travis and Patchew status links. The Shippable links will
be added later once mainline tests have been configured and setup.

Suggested-by: Fam Zheng <famz@redhat.com>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

---
 MAINTAINERS | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

-- 
2.11.0

Comments

Fam Zheng Feb. 16, 2017, 1:23 p.m. | #1
On Thu, 02/16 12:34, Alex Bennée wrote:
> The docker framework is really just another piece in the build

> automation puzzle so lets merge it together. For added bonus I've also

> included the Travis and Patchew status links. The Shippable links will

> be added later once mainline tests have been configured and setup.

> 

> Suggested-by: Fam Zheng <famz@redhat.com>

> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

> ---

>  MAINTAINERS | 13 ++++++-------

>  1 file changed, 6 insertions(+), 7 deletions(-)

> 

> diff --git a/MAINTAINERS b/MAINTAINERS

> index 7afbadaa15..4e4a3fb83c 100644

> --- a/MAINTAINERS

> +++ b/MAINTAINERS

> @@ -1802,9 +1802,14 @@ F: docs/block-replication.txt

>  Build and test automation

>  -------------------------

>  M: Alex Bennée <alex.bennee@linaro.org>

> +M: Fam Zheng <famz@redhat.com>

>  L: qemu-devel@nongnu.org

> -S: Supported

> +S: Maintained


Yes:

> Supported:	Someone is actually paid to look after this.

> Maintained:	Someone actually looks after it.


While I'm paid to work on QEMU upstream, docker testing is not stricly my job
responsibility. So as for patchew. That's probably why I put 'maintained'
before.

>  F: .travis.yml

> +F: .shippable.yml


This line could be pulled into patch 3.

> +F: tests/docker/

> +W: https://travis-ci.org/qemu/qemu

> +W: http://patchew.org/QEMU/


Thanks for the link!

If you move the 'shippable.yml' line:

Reviewed-by: Fam Zheng <famz@redhat.com>

Patch hide | download patch | download mbox

diff --git a/MAINTAINERS b/MAINTAINERS
index 7afbadaa15..4e4a3fb83c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1802,9 +1802,14 @@  F: docs/block-replication.txt
 Build and test automation
 -------------------------
 M: Alex Bennée <alex.bennee@linaro.org>
+M: Fam Zheng <famz@redhat.com>
 L: qemu-devel@nongnu.org
-S: Supported
+S: Maintained
 F: .travis.yml
+F: .shippable.yml
+F: tests/docker/
+W: https://travis-ci.org/qemu/qemu
+W: http://patchew.org/QEMU/
 
 Documentation
 -------------
@@ -1813,9 +1818,3 @@  M: Daniel P. Berrange <berrange@redhat.com>
 S: Odd Fixes
 F: docs/build-system.txt
 
-Docker testing
---------------
-Docker based testing framework and cases
-M: Fam Zheng <famz@redhat.com>
-S: Maintained
-F: tests/docker/