Skip to content

Skip plugin tests if not in amd64#28317

Merged
vdemeester merged 1 commit intomoby:masterfrom
tonistiigi:amd64-plugin-tests
Nov 12, 2016
Merged

Skip plugin tests if not in amd64#28317
vdemeester merged 1 commit intomoby:masterfrom
tonistiigi:amd64-plugin-tests

Conversation

@tonistiigi
Copy link
Copy Markdown
Member

Quick solution for #28305 until plugins are ported to other architectures.

fixes #28305

@vieux @tiborvass

Signed-off-by: Tonis Tiigi [email protected]

@tiborvass
Copy link
Copy Markdown
Contributor

LGTM, failure on Z is unrelated

@tophj-ibm
Copy link
Copy Markdown
Contributor

looks good to me too, thanks!

Copy link
Copy Markdown
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tophj-ibm
Copy link
Copy Markdown
Contributor

arm ci failed because node arm-7 ran out of space, do the hypriot guys still own those?

@tiborvass
Copy link
Copy Markdown
Contributor

@tophj-ibm no, we do.

@vdemeester vdemeester merged commit c8492eb into moby:master Nov 12, 2016
@GordonTheTurtle GordonTheTurtle added this to the 1.14.0 milestone Nov 12, 2016
@thaJeztah thaJeztah modified the milestones: 1.13.0, 1.14.0 Nov 13, 2016
@thaJeztah
Copy link
Copy Markdown
Member

@vieux added cherry pick to make the tests work on the release branch as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI fails on z and power due to newly enabled testcases

8 participants