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.

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

Disable mmap based test for Xavier #2612

Merged
merged 2 commits into from Jan 14, 2021
Merged

Conversation

JanuszL
Copy link
Contributor

@JanuszL JanuszL commented Jan 13, 2021

  • some CI Xavier runners due to disc issue fails mmap based tests

Signed-off-by: Janusz Lisiecki jlisiecki@nvidia.com

Why we need this PR?

Pick one, remove the rest

  • It disables mmap based test for Xavier

What happened in this PR?

Fill relevant points, put NA otherwise. Replace anything inside []

  • What solution was applied:
    some CI Xavier runners due to disc issue fails mmap based tests so disable mmap based test for Xavier
  • Affected modules and functionalities:
    test is blacklisted
  • Key points relevant for the review:
    NA
  • Validation and testing:
    CI
  • Documentation (including examples):
    NA

JIRA TASK: [NA]

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1977181]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1977181]: BUILD FAILED

@JanuszL JanuszL force-pushed the fix_xavier_test branch 2 times, most recently from dcdd69f to c36fea8 Compare January 13, 2021 16:24
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1978029]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1978038]: BUILD STARTED

@@ -22,7 +22,9 @@ test_body() {
exit 1
fi

"$FULLPATH"
# for some reason mmap baed test tends to fail on some runners due to disc issue, so
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# for some reason mmap baed test tends to fail on some runners due to disc issue, so
# for some reason mmap based test tends to fail on some runners due to disc issue, so

doesn't matter much

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1978038]: BUILD FAILED

- some CI Xavier runners due to disc issue fails mmap based tests

Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1980774]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [1980774]: BUILD PASSED

@JanuszL JanuszL merged commit 2d44dcb into NVIDIA:master Jan 14, 2021
@JanuszL JanuszL deleted the fix_xavier_test branch January 14, 2021 12:20
klecki pushed a commit that referenced this pull request Jan 14, 2021
- some CI Xavier runners due to disc issue fails mmap based tests

Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants