On 14.09.2016 18:53, Sumit Bose wrote:
On Wed, Sep 14, 2016 at 06:03:37PM +0200, Martin Basti wrote:
On 14.09.2016 17:53, Alexander Bokovoy wrote:
On Wed, 14 Sep 2016, Martin Basti wrote:
On 14.09.2016 17:41, Alexander Bokovoy wrote:
On Wed, 14 Sep 2016, Martin Basti wrote:
1)
I still do
Hi Martin,
The file was renamed. Did I understand correctly that for now we are
leaving the test as is and are planning to extend it later?
On 09/15/2016 09:49 AM, Martin Basti wrote:
On 14.09.2016 18:53, Sumit Bose wrote:
On Wed, Sep 14, 2016 at 06:03:37PM +0200, Martin Basti wrote:
On
On 14.09.2016 10:43, Oleg Fayans wrote:
Again ping for review, please it completely blocks the whole job.
On 09/07/2016 03:27 PM, Oleg Fayans wrote:
ping for review
On 08/24/2016 01:58 PM, Oleg Fayans wrote:
And here is how the run looks like:
$ ipa-run-tests test_integration/test_topology
On 15.09.2016 10:10, Oleg Fayans wrote:
Hi Martin,
The file was renamed. Did I understand correctly that for now we are
leaving the test as is and are planning to extend it later?
I would like to have there SSSD check involved, please use what Summit
recommends. No new test cases.
And th
stlaz's pull request #84: "Removed update_from_dict function from ldapupdate"
was opened
PR body:
"""
update_from_dict was basically dead code as it's used nowhere in
the project.
https://fedorahosted.org/freeipa/ticket/6311
"""
See the full pull-request at https://github.com/freeipa/freeipa/pu
abbra commented on a pull request
"""
NACK. Please instead fix update_from_dict() to follow _run_updates()
expectations.
update_from_dict() is a handy function for externally provided FreeIPA modules.
They will need to implement the same functionality if they would want to do
dynamic updates t
Akasurde's pull request #83: "Added a fix for setting Priority as required
field " was synchronize
See the full pull-request at https://github.com/freeipa/freeipa/pull/83
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/83/head
pvomacka's pull request #85: "WebUI: Change group name from 'normal' to
'Non-POSIX'" was opened
PR body:
"""
It will correspond with CLI and will be more self-explanatory.
https://fedorahosted.org/freeipa/ticket/6334
"""
See the full pull-request at https://github.com/freeipa/freeipa/pull/85
..
pvomacka's pull request #72: "WebUI: Add handling for HTTP error 404" label
*ack* has been added
See the full pull-request at https://github.com/freeipa/freeipa/pull/72
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribut
jcholast commented on a pull request
"""
Functional ACK, but please don't use newlines in exception messages. If you
want the original error on a separate line, you can use the logger to log it,
but I think it would be preferable to use this format:
```
Peer's certificate issuer is not trusted
mbasti-rh commented on a pull request
"""
Shouldn't external plugins use update files or update plugins as IPA does?
We don't have any guaranteed internal API for anything, we don't have any
document about external plugins, we don't have prepared any API for 3rd party
plugins. We just have your
abbra commented on a pull request
"""
Update plugins are higher level of abstraction. They use
ipaserver.install.ldapupdate.LDAPUpdate which provides both .update() and
.update_from_dict() methods. Update plugins can produce dictionaries. With the
change in this pull request they will have to a
stlaz's pull request #84: "Removed update_from_dict function from ldapupdate"
was synchronize
See the full pull-request at https://github.com/freeipa/freeipa/pull/84
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/84/head:pr84
stlaz commented on a pull request
"""
Well, I did fix the test, then. I can imagine the function being pretty handy
as a library function although it'd better be used in the future.
What infuriates me is the fact that the test might have never worked (well, at
least year and a half, but my guess
Akasurde's pull request #83: "Added a fix for setting Priority as required
field " label *ack* has been added
See the full pull-request at https://github.com/freeipa/freeipa/pull/83
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-de
stlaz commented on a pull request
"""
Looks OK to me, thanks.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/83#issuecomment-247310347
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA
ofayans's pull request #86: "Made sssd restart a non-raising opration" was
opened
PR body:
"""
Uninstallation of ipa-server usually removes sssd configuration file,
/etc/sssd/sssd.conf
If we then issue syustemctl restart sssd.service, the command fails because is
unable to find the config file. W
jcholast's pull request #87: "dns: re-introduce --raw in dnsrecord-del" was
opened
PR body:
"""
The flag was removed in commit ff52c25ae299abba8bed653fe324951979a41293
because it is unused. Add it back for compatibility with old clients.
https://fedorahosted.org/freeipa/ticket/5644
"""
See the
Akasurde commented on a pull request
"""
@stlaz Thanks
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/83#issuecomment-247312920
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http:
Akasurde commented on a pull request
"""
@pspacek Should I close this PR then?
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/25#issuecomment-247315968
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Con
jcholast's pull request #88: "test_plugable: update the rest of test_init" was
opened
PR body:
"""
In commit ed4c2d9252a995d01dc098e5b761ded8cd9373d8, changes to the Plugin
class were made, but the test was updated only partially.
Update the rest to fix the failing test.
https://fedorahosted.or
jcholast's pull request #89: "client: remove hard dependency on pam_krb5" was
opened
PR body:
"""
If ipa-client-install is executed with --no-sssd, check if pam_krb5 is
available before proceeding with the install.
https://fedorahosted.org/freeipa/ticket/5557
"""
See the full pull-request at ht
jcholast's pull request #89: "client: remove hard dependency on pam_krb5" was
synchronize
See the full pull-request at https://github.com/freeipa/freeipa/pull/89
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/89/head:pr89
git
flo-renaud's pull request #50: "Add cert checks in ipa-server-certinstall" was
synchronize
See the full pull-request at https://github.com/freeipa/freeipa/pull/50
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/50/head:pr50
gi
mbasti-rh commented on a pull request
"""
I was thinking hard about it, if there should be SSSD restart, but I cannot
find any case where it is needed after server uninstall
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/86#issuecomment-247322795
--
Manage your subscriptio
mbasti-rh commented on a pull request
"""
Please split this into 2 commits, it contains independent issues
* wrong path to IPA join
* extra attribute
It is not related to this PR, but for me it looks that this test is focused to
ipa-join command and I don't see how this is related to API tests.
ofayans commented on a pull request
"""
Probably Lenka has some use-case for this. I am pretty sure these lines weren't
added just for fun :)
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/86#issuecomment-247328115
--
Manage your subscription for the Freeipa-devel mailing
rcritten commented on a pull request
"""
For the record this test used to pass. Don't blame the test when the code it is
testing was changed.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/84#issuecomment-247329152
--
Manage your subscription for the Freeipa-devel mailing
mbasti-rh commented on a pull request
"""
Can wait until Lenka give us the reason?
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/86#issuecomment-247344542
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
ofayans commented on a pull request
"""
sure
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/86#issuecomment-247344841
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.free
mbasti-rh commented on a pull request
"""
Sorry guys, but ticket is in closed milestone you need a new one.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/83#issuecomment-247348762
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailma
Akasurde's pull request #83: "Added a fix for setting Priority as required
field " label *ack* has been removed
See the full pull-request at https://github.com/freeipa/freeipa/pull/83
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-
mbasti-rh commented on a pull request
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/b18a35145df92522ae990e020513d1a77e311493
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/72#issuecomment-247349710
--
Manage your subscription for the Freeipa-devel ma
pvomacka's pull request #72: "WebUI: Add handling for HTTP error 404" was closed
See the full pull-request at https://github.com/freeipa/freeipa/pull/72
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/72/head:pr72
git checkout
pvomacka's pull request #72: "WebUI: Add handling for HTTP error 404" label
*pushed* has been added
See the full pull-request at https://github.com/freeipa/freeipa/pull/72
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contri
LiptonB's pull request #10: "Client-side CSR autogeneration" was synchronized
See the full pull-request at https://github.com/freeipa/freeipa/pull/10
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/10/head:pr10
git checkout pr1
On 09/15/2016 02:12 AM, jcholast wrote:
jcholast commented on a pull request
"""
In addition to my inline comments above:
1. "Certificate mapping" does not really evoke "certificate request templating"
to me, and is also used in the context of mapping identities to certificates. Could we use a
Akasurde commented on a pull request
"""
@mbasti-rh I re-opened the ticket, Could you please provide ack ?
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/83#issuecomment-247519895
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailma
mbasti-rh commented on a pull request
"""
@Akasurde I said we need a *new* ticket for it. I opened one for you
https://fedorahosted.org/freeipa/ticket/6335
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/83#issuecomment-247530662
--
Manage your subscription for the Freeipa-
Akasurde's pull request #83: "Added a fix for setting Priority as required
field " was edited
See the full pull-request at https://github.com/freeipa/freeipa/pull/83
... or pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/83/head:pr83
40 matches
Mail list logo