#1991 Make return codes of basic sysdb operations consistent
Closed: Fixed Opened by jhrozek.

The return codes of various sysdb operations differ. Some search operations would return ENOENT if they don't find a matching object some would return EOK but an empty result list. This leads to bugs like #1989 where the programmer is not sure what the particular search function returns.

We should make sure that all functions of the same family (all delete functions, all search functions etc) behave consistently.


Fields changed

changelog: => N/A internal task only

Fields changed

milestone: NEEDS_TRIAGE => SSSD 1.12 beta
priority: minor => critical
rhbz: => 0

Fields changed

milestone: SSSD 1.12 beta => Interim Bucket
priority: critical => major

Fields changed

milestone: Interim Bucket => SSSD 1.12 beta

When implementing this ticket, please check our usage of tevent_req_error(req, ENOENT). The tevent requests should be consistent as well. We might not catch all cases, but checking the explicit use of ENOENT to terminate a request would be a good start.

Fields changed

owner: somebody => preichl

Fields changed

milestone: SSSD 1.12 beta => SSSD 1.12.1

netgroup related changes landed upstream:
- 8ca1875209325ddeb28fccb421457b76b4b0c7db
- 50c009a3d51521880ddf568e7173d1ed9d5c2685
- 9436c86caf9b2f7ec33c3022dfed2a653d3ec965
- 940dd08102ac8527ef9e367dc5d6fb88cd53a8a2
- 5153e8b9793dea1e212ca08af0f77ea1d023cbb7

Fields changed

design: => N/A not needed

Fields changed

review: 0 => 1

another change in master - 61602026ed8c91efd166000562899670449f1b50

Mass-moving all tickets that didn't make 1.12.1 into 1.12.2

milestone: SSSD 1.12.1 => SSSD 1.12.2

We need to do a release as requested by downstream. Moving tickets that are not fixed already or very close to acking to 1.12.3

milestone: SSSD 1.12.2 => SSSD 1.12.3

This is an ongoing task. We're accepting patches as they are reviewed on sssd-devel.

mark: => 0
milestone: SSSD 1.12.3 => SSSD 1.12.4

another patch landed in master - a5b55bdfcda8bfce8cb2ced981773998093d7857

Fields changed

patch: 0 => 1

More patches landed in the master:
13c0cf829eca7891ad9d0087e91c72650f990149
d7b90921c1a404f0d9fb8384a8fd55fd15b86916
4bbcc2d6d3f16b015796818746a45134861c93a4

I think that this ticket might be considered as done. Related work will be done in #2479.

Fields changed

resolution: => fixed
status: new => closed

Metadata Update from @jhrozek:
- Issue assigned to preichl
- Issue marked as depending on: #2381
- Issue set to the milestone: SSSD 1.12.4

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/3033

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.

Metadata