URL: https://github.com/freeipa/freeipa/pull/809
Author: frasertweedale
Title: #809: Support specifying external CA template (preliminary refactors)
Action: opened
PR body:
"""
This PR contains a couple of refactorings, as groundwork for
upcoming support for specifying the external CA profile/template
during install / ca-install / cacert-manage renew / etc.
https://pagure.io/freeipa/issue/6858
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/809/head:pr809
git checkout pr809
URL: https://github.com/freeipa/freeipa/pull/802
Author: stlaz
Title: #802: Improve cert messages some more + do that for KDC certs as well
Action: opened
PR body:
"""
Some of the previous error message handling would not work in
a locale different from English so we just output the error message we get
from `certutil`.
Also, since this was previously done, there is now kdc cert validation that is
different from the others so make the error messages more verbose there.
OpenSSL prints quite verbose messages so you may want to grab only a
part of it, let me know what your opinion is.
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/802/head:pr802
git checkout pr802
URL: https://github.com/freeipa/freeipa/pull/833
Author: Tiboris
Title: #833: Fixes traceback in log and corrects console output
Action: opened
PR body:
"""
There was traceback in error_log, when running trustdomain-find
command for trusted child domain and it results into ERROR.
Correct behavior is to show 0 entries found.
Fixes https://pagure.io/freeipa/issue/6477
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/833/head:pr833
git checkout pr833
URL: https://github.com/freeipa/freeipa/pull/821
Author: martbab
Title: #821: fix incorrect suffix handling in topology checks
Action: opened
PR body:
"""
When trying to delete a partially removed master entry lacking
'iparepltopomanagedsuffix' attribute, the code that tries to retrieve
tha value for further computations passes None and causes unhandled
internal errors.
If the attribute is empty or not present, we should return empty list
instead as to not break calling cod attribute, the code that tries to
retrieve tha value for further computations passes None and causes
unhandled internal errors. We should return empty list instead.
https://pagure.io/freeipa/issue/6965
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/821/head:pr821
git checkout pr821
URL: https://github.com/freeipa/freeipa/pull/812
Author: felipevolpone
Title: #812: Refactoring cert-find to use API call directly instead of using
Action: opened
PR body:
"""
Refactoring cert-find to use API calls directly instead of using raw LDAP search.
Upstream ticket: https://pagure.io/freeipa/issue/6948
I removed the raw LDAP search and used the API directly. In the old code, the call ` self.obj._owners()` returns `service, hots and user`. However, when testing the code, only the service was being used, so I made it only use the service API.
If there another scenario where `user and host` are used, I thought to do something like:
```python
for owner in self.obj._owners():
api_name = owner.name
response = api.Command[api_name+'_find'](options[api_name])
... # continues
```
Is that correct?
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/812/head:pr812
git checkout pr812
URL: https://github.com/freeipa/freeipa/pull/818
Author: stlaz
Title: #818: Avoid possible recursion in RPC call from client
Action: opened
PR body:
"""
This commit removes recursion which may lack end condition.
https://pagure.io/freeipa/issue/6796
===============================
This is my try to fix ^--. The methods and their arguments in the module are very poorly documented (they are not documented), so I just hope I can initialize the variables used in logs prior to the cycle.
I also think this actually relates more to https://pagure.io/freeipa/issue/6775 than the ticket mentioned here and that the person "fixing" https://pagure.io/freeipa/issue/6796 messed up the tickets but it's hopefully OK to fix it this way.
Aside from just moving everything into a cycle, I also improved logging a bit and had same error handling for different errors merged into one `except` block (`SSLError`, `socket.error`).
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/818/head:pr818
git checkout pr818
URL: https://github.com/freeipa/freeipa/pull/803
Author: tomaskrizek
Title: #803: ipatests: add systemd journal collection for multihost tests
Action: opened
PR body:
"""
Some messages are only logged in journal. Collection of journal
makes debugging failed tests from logs easier.
Fixes: https://pagure.io/freeipa/issue/6971
Signed-off-by: Tomas Krizek <tkrizek(a)redhat.com>
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/803/head:pr803
git checkout pr803