4:23 p.m.
People,
We have 27 PRs opened by the moment I'm writing this email and I'd
like to have a clear idea which ones are the *must* have for our next
release.
- Fix group renaming issue when "id_provider = ldap" is set
(https://github.com/SSSD/sssd/pull/128)
We have a bugzilla for this one. Code has been reviewed and last
comments addressed.
- Add "Wants=" to sssd unit (https://github.com/SSSD/sssd/pull/132)
We don't have a bugzilla for this one. IMO, this can be postponed
for the next release
- Initial revision of sssd pytest framework
(https://github.com/SSSD/sssd/pull/139)
This PR will be reviewed on a phone session. IMO, this can be
postponed for the next release.
- Add module for starting services (https://github.com/SSSD/sssd/pull/175)
This PR has been stalled for a quite long time. Although the
idea/work seems quite nice, I don't see this one as something that we
should prioritize. So, IMO, this can the postponed for the next
release.
- TEST: Adding krb5-libs to dependencies (https://github.com/SSSD/sssd/pull/218)
This PR has been stalled since celestian left the project. It's
something good to have but far from having high priority. IMO, this
can be postponed to the next release.
- changing all talloc_get_type() with talloc_get_type_abort()
(https://github.com/SSSD/sssd/pull/231)
This PR has been stalled for a quite long time. Can easily be
postponed to the next release.
- provider: Move hostid from ipa to sdap (https://github.com/SSSD/sssd/pull/237)
This PR comes from an external contributor and as far as I
understood they changed whatever has been requested. IMO this should
be part of this release.
- Subdomain inherit (https://github.com/SSSD/sssd/pull/247):
This PR has been stalled for a long time and according to the
dicussion in our phone meeting Today, it can be postponed to the next
release.
- Update sss_override.c (https://github.com/SSSD/sssd/pull/260)
This PR has been stalled for some time and, IMO, can be postponed to
the next release.
- Add support for ActiveDirectory's logonHorous restrictions
(https://github.com/SSSD/sssd/pull/269)
This PR comes from an external contributor and as far as I
understood there's still some work to be done. So, should be postponed
to the next release
- Merge sss_cache and sss_debuglevel into sssctl
(https://github.com/SSSD/sssd/pull/274)
This PR seems to be in a good shape and justin addressed the last
comments. IMO, should be part of this release.
- Implement access verification by rhost using ldap_access_order rhost
option (https://github.com/SSSD/sssd/pull/275)
By the comments I'm not sure whether we may or may not should have
it in the next release. In any case, we must provide a feedback to the
external contributor.
- Print a warning when enumeration is requested but disabled
(https://github.com/SSSD/sssd/pull/334)
Already acked.
- Fix for few el6 gcc warnings (https://github.com/SSSD/sssd/pull/371)
- This patch set fixes a bunch warnings (https://github.com/SSSD/sssd/pull/377)
PR 377 has all the patches contained in PR 371. Any of those have
high priority IMO and both could just be postponed to the next
release.
- ldap: Change ldap_user_certificate to userCertificate;binary
(https://github.com/SSSD/sssd/pull/372)
Already reviewed. Whoever pushes the patches could squash the
patches instead of keep waiting for the contributor. In any case,
should be part of this release.
- intg: Add sanity tests for pysss_nss_idmap
(https://github.com/SSSD/sssd/pull/373)
Already acked.
- IPA: Add threshold for sudo command and command group searches
(https://github.com/SSSD/sssd/pull/374)
Does it have some bugzilla linked? I guess it would be nice to be
reviewed and pushed for this release.
- sssd-1.13 Backported patches for ticket 3505
(https://github.com/SSSD/sssd/pull/375)
Doesn't affect our release at all
- [RFC] Use GNULIB's compiler warning code
(https://github.com/SSSD/sssd/pull/378)
IMO this one can be postponed to the next release
- CI: Enable pep8 check (https://github.com/SSSD/sssd/pull/379)
IMO this one can be postponed to the next release
- intg: prevent "TypeError: mustbe type, not classobj"
(https://github.com/SSSD/sssd/pull/386)
There's a small change requested that can be done by whoever pushes
the patches instead of keep waiting for the contributor
- Setting ldap_sudo_include_regexp to false
(https://github.com/SSSD/sssd/pull/387)
Should be part of 2.0 relase in the future
- sssd_client: add mutex protected call to the PAC responder
(https://github.com/SSSD/sssd/pull/389)
Should be part of this release
- NSS: Add option to disabled memcache (https://github.com/SSSD/sssd/pull/390)
IMO this one be postponed to the next release
- Use dbus-daemon in cwrap environment for test
(https://github.com/SSSD/sssd/pull/391)
IMO this one can be postponed to the next release
- GPO: Don't use freed LDAPURLDesc if domain for AD DC cannot be found
IMO this one should be part of this release
So, can we have an agreement that we're going to focus on reviewing:
- Fix group renaming issue when "id_provider = ldap" is set
(https://github.com/SSSD/sssd/pull/128)
- provider: Move hostid from ipa to sdap (https://github.com/SSSD/sssd/pull/237)
- Add support for ActiveDirectory's logonHorous restrictions
(https://github.com/SSSD/sssd/pull/269)
- Merge sss_cache and sss_debuglevel into sssctl
(https://github.com/SSSD/sssd/pull/274)
- Implement access verification by rhost using ldap_access_order rhost
option (https://github.com/SSSD/sssd/pull/275)
- IPA: Add threshold for sudo command and command group searches
(https://github.com/SSSD/sssd/pull/374)
- sssd_client: add mutex protected call to the PAC responder
(https://github.com/SSSD/sssd/pull/389)
- GPO: Don't use freed LDAPURLDesc if domain for AD DC cannot be found
And also, do we agree that the bugs mentioned above are the material
for this release and pretty much anything else that is already opened?
Best Regards,
--
Fabiano Fidêncio