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

DAOS-13490 test: Update valgrind suppressions. #13142

Merged
merged 2 commits into from
Oct 17, 2023
Merged

Conversation

ashleypittman
Copy link
Contributor

A go change has introduced some new failure traces, suppress them.

Required-githooks: true

Signed-off-by: Ashley Pittman [email protected]

A go change has introduced some new failure traces, suppress them.

Required-githooks: true

Signed-off-by: Ashley Pittman <[email protected]>
@github-actions
Copy link

github-actions bot commented Oct 9, 2023

Bug-tracker data:
Ticket title is 'NLT valgrind errors - NLT.core.container, NLT.core.Errors, Valgrind.Valgrind_Memcheck'
Status is 'In Review'
Labels: 'ci_impact,daily_test,triaged'
https://daosio.atlassian.net/browse/DAOS-13490

Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

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

LGTM. No errors found by checkpatch.

@daosbuild1
Copy link
Collaborator

Test stage Build RPM on Leap 15.4 completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13142/1/execution/node/393/log

@daosbuild1
Copy link
Collaborator

Test stage Build DEB on Ubuntu 20.04 completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13142/1/execution/node/374/log

Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

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

LGTM. No errors found by checkpatch.

@ashleypittman ashleypittman marked this pull request as ready for review October 9, 2023 17:10
Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

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

LGTM. No errors found by checkpatch.

@mchaarawi mchaarawi merged commit 105fbb0 into master Oct 17, 2023
15 checks passed
@mchaarawi mchaarawi deleted the amd/valgrind-update branch October 17, 2023 12:46
jolivier23 pushed a commit that referenced this pull request Apr 16, 2024
A go change has introduced some new failure traces, suppress them.

Signed-off-by: Ashley Pittman <[email protected]>
jolivier23 added a commit that referenced this pull request Apr 17, 2024
disable CODEOWNERS for google branch
disable upstream hardware tests on branch by default
remove bad merge block
fix ordering of imports
Rename google-changeId.py
set option for dynamic fuse

Backports included here for test fixes
DAOS-15429 test: Fix Go unit tests (#13981)
DAOS-13490 test: Update valgrind suppressions. (#13142)
DAOS-15159 test: add a supression for new valgrind warning in NLT (#13782)
DAOS-14669 test: switch tcp;ofi_rxm testing to tcp (#13365)
DAOS-15548 test: add new valgrind suppression for daos tool (#14081)

Signed-off-by: Jeff Olivier <[email protected]>
Signed-off-by: Michael MacDonald <[email protected]>
Signed-off-by: Ashley Pittman <[email protected]>
Signed-off-by: Mohamad Chaarawi <[email protected]>
Signed-off-by: Jerome Soumagne <[email protected]>
jolivier23 added a commit that referenced this pull request May 21, 2024
disable CODEOWNERS for google branch
disable upstream hardware tests on branch by default
remove bad merge block
fix ordering of imports
Rename google-changeId.py
set option for dynamic fuse

Backports included here for test fixes
DAOS-15429 test: Fix Go unit tests (#13981)
DAOS-13490 test: Update valgrind suppressions. (#13142)
DAOS-15159 test: add a supression for new valgrind warning in NLT (#13782)
DAOS-14669 test: switch tcp;ofi_rxm testing to tcp (#13365)
DAOS-15548 test: add new valgrind suppression for daos tool (#14081)

Required-githooks: true

Change-Id: Ifc50889fd7aada1ae1666ed928b7edc9293da5b7
Signed-off-by: Jeff Olivier <[email protected]>
Signed-off-by: Michael MacDonald <[email protected]>
Signed-off-by: Ashley Pittman <[email protected]>
Signed-off-by: Mohamad Chaarawi <[email protected]>
Signed-off-by: Jerome Soumagne <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants