Skip to content

src/valgrind.supp: Adding know leaks unrelated to ceph#48641

Merged
yuriw merged 1 commit intoceph:mainfrom
NitzanMordhai:wip-nitzan-valgrind-supp-adding-rocksdb-dl-init
Dec 15, 2022
Merged

src/valgrind.supp: Adding know leaks unrelated to ceph#48641
yuriw merged 1 commit intoceph:mainfrom
NitzanMordhai:wip-nitzan-valgrind-supp-adding-rocksdb-dl-init

Conversation

@NitzanMordhai
Copy link
Contributor

@NitzanMordhai NitzanMordhai commented Oct 27, 2022 •
edited
Loading

During teuthology test we can hit few valgrind errors that stop the tests, one of them is Rocksdb version info and the other one is dl-init.
Adding them to the valgrind suppression file.

Fixes: https://tracker.ceph.com/issues/52136
https://tracker.ceph.com/issues/57751
https://tracker.ceph.com/issues/53575
Signed-off-by: Nitzan Mordechai nmordec@redhat.com

Contribution Guidelines

Checklist

  • Tracker (select at least one)
    • References tracker ticket
    • Very recent bug; references commit where it was introduced
    • New feature (ticket optional)
    • Doc update (no ticket needed)
    • Code cleanup (no ticket needed)
  • Component impact
    • Affects Dashboard, opened tracker ticket
    • Affects Orchestrator, opened tracker ticket
    • No impact that needs to be tracked
  • Documentation (select at least one)
    • Updates relevant documentation
    • No doc update is appropriate
  • Tests (select at least one)
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard cephadm
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox
  • jenkins test windows

@NitzanMordhai
src/valgrind.supp: Adding know leaks unrelated to ceph
43c94e1 
During teuthology test we can hit few valgrind errors that stop
the tests, one of them is Rocksdb version info and the other one
is dl-init.
Adding them to the valgrinf suppression file.

Fixes: https://tracker.ceph.com/issues/52136
       https://tracker.ceph.com/issues/57751
       https://tracker.ceph.com/issues/53575
Signed-off-by: Nitzan Mordechai <nmordec@redhat.com>
@ljflores ljflores self-requested a review November 1, 2022 16:10
@NitzanMordhai NitzanMordhai marked this pull request as ready for review November 1, 2022 16:10
ljflores
ljflores approved these changes Nov 1, 2022
View reviewed changes
Copy link
Member

@ljflores ljflores left a comment

Choose a reason for hiding this comment

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

We can perhaps run some of the tests recorded in the Trackers ~ 20-30 times, and if everything looks good, merge!

@ljflores
Copy link
Member

ljflores commented Nov 1, 2022

@NitzanMordhai perhaps another related tracker? https://tracker.ceph.com/issues/55809

@NitzanMordhai
Copy link
Contributor Author

NitzanMordhai commented Nov 1, 2022

@NitzanMordhai perhaps another related tracker? https://tracker.ceph.com/issues/55809

That one looks like a monitor leaking memory, need to fix that one

@NitzanMordhai
Copy link
Contributor Author

NitzanMordhai commented Nov 1, 2022

We can perhaps run some of the tests recorded in the Trackers ~ 20-30 times, and if everything looks good, merge!

Waiting for teuthology .. but yes, good idea!

@NitzanMordhai
Copy link
Contributor Author

NitzanMordhai commented Nov 28, 2022

Ran 50 jobs: http://pulpito.front.sepia.ceph.com/nmordech-2022-11-28_07:46:50-rados:verify-wip-nitzan-valgrind-supp-adding-rocksdb-dl-init-distro-default-smithi/

Not related failure:
7093112 - failed in Alarm for ceph_test_rados_api_aio
7093113 - Segmentation fault (core dumped) ceph_test_rados_api_watch_notify_pp
7093117 - api_watch_notify_pp: watches.size()

@NitzanMordhai NitzanMordhai mentioned this pull request Nov 29, 2022
Merged
14 tasks
@neha-ojha neha-ojha added the core label Dec 5, 2022
@ljflores
Copy link
Member

ljflores commented Dec 13, 2022

Rados suite review:

https://pulpito.ceph.com/?branch=wip-yuri-testing-2022-12-06-1204
https://pulpito.ceph.com/?branch=wip-yuri-testing-2022-12-12-1136

Failures, unrelated:
1. https://tracker.ceph.com/issues/58096
2. https://tracker.ceph.com/issues/52321
3. https://tracker.ceph.com/issues/58173
4. https://tracker.ceph.com/issues/52129
5. https://tracker.ceph.com/issues/58097
6. https://tracker.ceph.com/issues/57546
7. https://tracker.ceph.com/issues/58098
8. https://tracker.ceph.com/issues/57731
9. https://tracker.ceph.com/issues/55606
10. https://tracker.ceph.com/issues/58256
11. https://tracker.ceph.com/issues/58258

Details:
1. test_cluster_set_reset_user_config: NFS mount fails due to missing ceph directory - Ceph - Orchestrator
2. qa/tasks/rook times out: 'check osd count' reached maximum tries (90) after waiting for 900 seconds - Ceph - Orchestrator
3. api_aio_pp: failure on LibRadosAio.SimplePoolEIOFlag and LibRadosAio.PoolEIOFlag - Ceph - RADOS
4. LibRadosWatchNotify.AioWatchDelete failed - Ceph - RADOS
5. qa/workunits/post-file.sh: kex_exchange_identification: read: Connection reset by peer - Ceph - RADOS
6. rook: ensure CRDs are installed first - Ceph - Orchestrator
7. qa/workunits/rados/test_crash.sh: crashes are never posted - Ceph - RADOS
8. Problem: package container-selinux conflicts with udica < 0.2.6-1 provided by udica-0.2.4-1 - Infrastructure
9. [ERR] Unhandled exception from module ''devicehealth'' while running on mgr.y: unknown - Ceph - CephSqlite
10. ObjectStore/StoreTestSpecificAUSize.SpilloverTest/2: Expected: (logger->get(l_bluefs_slow_used_bytes)) >= (16 * 1024 * 1024), actual: 0 vs 16777216 - Ceph - Bluestore
11. rook: kubelet fails from connection refused - Ceph - Orchestrator

@yuriw yuriw merged commit a55987b into ceph:main Dec 15, 2022
This was referenced Dec 20, 2022
Merged
Merged
@NitzanMordhai NitzanMordhai mentioned this pull request May 4, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ljflores ljflores ljflores approved these changes

@rzarzynski rzarzynski Awaiting requested review from rzarzynski

@neha-ojha neha-ojha Awaiting requested review from neha-ojha

Assignees

No one assigned

Labels

core needs-qa wip-yuri-testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@NitzanMordhai @ljflores @rzarzynski @yuriw @neha-ojha