#2869 cache_req tests don't use leak_check_push/leak_check_pop in fixtures
Closed: Fixed None Opened 5 years ago by jhrozek.

During review of patches for #2730, we noticed the tests should use check_leaks_push/pop in the test fixtures to catch issues where memory was allocated on long-living contexts (like responder or event context).

Triage notes - this should be a prerequisite for using cache_req in responders (#1126) which is currently 1.14.

For the time being, filing into 1.14 beta, we will decide if there is still time to use cache_req in 1.14 or whether we need to move that enhancement to 1.15..

milestone: NEEDS_TRIAGE => SSSD 1.14 beta

Fields changed

rhbz: => 0

Pavel agreed on IRC to fix this ticket.

owner: somebody => pbrezina

Fields changed

status: new => assigned

Fields changed

patch: 0 => 1

Fields changed

milestone: SSSD 1.14 beta => SSSD 1.14 alpha
resolution: => fixed
status: assigned => closed

Metadata Update from @jhrozek:
- Issue assigned to pbrezina
- Issue marked as blocked by: #1126
- Issue set to the milestone: SSSD 1.14 alpha

4 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/3910

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.