diff mbox

[OSSTEST,v13,24/24] openstack tests: Don't run them on arm*

Message ID 20170725115759.21895-25-anthony.perard@citrix.com (mailing list archive)
State New, archived
Headers show

Commit Message

Anthony PERARD July 25, 2017, 11:57 a.m. UTC 
Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
---
 make-flight | 1 +
 1 file changed, 1 insertion(+)

Comments

Ian Jackson July 25, 2017, 6:18 p.m. UTC | #1 
Anthony PERARD writes ("[OSSTEST PATCH v13 24/24] openstack tests: Don't run them on arm*"):
> Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

You should probably mention the existence of this patch in the other
one's commit message...

Ian.
Anthony PERARD July 26, 2017, 3:19 p.m. UTC | #2 
On Tue, Jul 25, 2017 at 07:18:21PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[OSSTEST PATCH v13 24/24] openstack tests: Don't run them on arm*"):
> > Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
> 
> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
> 
> You should probably mention the existence of this patch in the other
> one's commit message...

The goal of this patch was to not use ARM capacity while there is not
enough. And also to be reverted once there there is plenty of capacity.

So I did not think it was usefull to say anything in the other patch.
Or maybe under the ---.

Thanks,
Ian Jackson July 26, 2017, 3:45 p.m. UTC | #3 
Anthony PERARD writes ("Re: [OSSTEST PATCH v13 24/24] openstack tests: Don't run them on arm*"):
> On Tue, Jul 25, 2017 at 07:18:21PM +0100, Ian Jackson wrote:
> > Anthony PERARD writes ("[OSSTEST PATCH v13 24/24] openstack tests: Don't run them on arm*"):
> > > Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
> > 
> > Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
> > 
> > You should probably mention the existence of this patch in the other
> > one's commit message...
> 
> The goal of this patch was to not use ARM capacity while there is not
> enough. And also to be reverted once there there is plenty of capacity.
> 
> So I did not think it was usefull to say anything in the other patch.
> Or maybe under the ---.

Either way is fine.  I don't have a strong opinion.

Thanks,
Ian.
diff mbox

Patch

diff --git a/make-flight b/make-flight
index cc95529..f615155 100755
--- a/make-flight
+++ b/make-flight
@@ -224,6 +224,7 @@  arch_branch_filter_callback () {
         rumprun) return 1;;
         seabios) return 1;;
         ovmf) return 1;;
+        openstack*) return 1;;
         esac
         ;;
   i386)