Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tests/*: memory blacklisting for new Docker image [backport 2018.07] #9658

Merged
merged 1 commit into from
Jul 31, 2018
Merged

tests/*: memory blacklisting for new Docker image [backport 2018.07] #9658

merged 1 commit into from
Jul 31, 2018

Conversation

cladmi
Copy link
Contributor

@cladmi cladmi commented Jul 31, 2018

Backport of #9507

Contribution description

This adds boards to BOARD_INSUFFICIENT_MEMORY for tests that failed due to memory errors on the new Docker image.

EDIT from cladmi: They also fail on the old docker image

Issues/PRs references

RIOT-OS/riotdocker#42
#9405

@cladmi cladmi added Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Area: CI Area: Continuous Integration of RIOT components CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: tests Area: tests and testing framework labels Jul 31, 2018
@cladmi cladmi mentioned this pull request Jul 31, 2018
Merged
@miri64
Copy link
Member

miri64 commented Jul 31, 2018

Needs rebase

@cladmi cladmi force-pushed the 2018.07/RIOT_docker_compat branch from cc31bb7 to ea879f4 Compare July 31, 2018 18:44
@miri64
Copy link
Member

miri64 commented Jul 31, 2018

Needs rebase.

@cladmi cladmi force-pushed the 2018.07/RIOT_docker_compat branch from ea879f4 to dd3d278 Compare July 31, 2018 19:40
ZetaR60
ZetaR60 approved these changes Jul 31, 2018
View reviewed changes
Copy link
Contributor

@ZetaR60 ZetaR60 left a comment

Choose a reason for hiding this comment

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

ACK.

miri64
miri64 approved these changes Jul 31, 2018
View reviewed changes
Copy link
Member

@miri64 miri64 left a comment

Choose a reason for hiding this comment

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

ACK, looks alright and most tests I did worked out (before my disk ran out of space because of the insanely large u8g2 package ^^)

@miri64 miri64 merged commit 2ec6cce into RIOT-OS:2018.07-branch Jul 31, 2018
@cladmi cladmi deleted the 2018.07/RIOT_docker_compat branch August 1, 2018 13:31
@cladmi cladmi added this to the Release 2018.07 milestone Aug 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miri64 miri64 miri64 approved these changes

@ZetaR60 ZetaR60 ZetaR60 approved these changes

Assignees
No one assigned
Labels
Area: CI Area: Continuous Integration of RIOT components Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master
Projects
None yet
Milestone
Release 2018.07
Development

Successfully merging this pull request may close these issues.
3 participants
@cladmi @miri64 @ZetaR60