[Freeipa-devel] [freeipa PR#184][comment] Minor install script fixes

2016-10-25 Thread pvoborni
URL: https://github.com/freeipa/freeipa/pull/184 Title: #184: Minor install script fixes pvoborni commented: """ The debug patch breaks test installation on RHEL. Following exception is printed only if install.py is adjusted to print exception on line ``` ipa : ERRORdebug

[Freeipa-devel] [freeipa PR#171][comment] Build system cleanup phase 2

2016-10-25 Thread lslebodn
URL: https://github.com/freeipa/freeipa/pull/171 Title: #171: Build system cleanup phase 2 lslebodn commented: """ On (20/10/16 10:29), Petr Špaček wrote: >@lslebodn I'm really trying to explain this but I'm still not able to get the >point across. >> My concerns are related purely to C-code.

[Freeipa-devel] [freeipa PR#145][comment] Refactoring: LDAP Connection Management

2016-10-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/145 Title: #145: Refactoring: LDAP Connection Management mbasti-rh commented: """ @rcritten Tomas removed timelimit that was used for repeated connections, it is not used for preventing hangs. (If we talk about the same commit 'ldap refactoring:

[Freeipa-devel] [freeipa PR#183][+pushed] Add __name__ == __main__ guards to setup.pys

2016-10-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/183 Title: #183: Add __name__ == __main__ guards to setup.pys Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#183][comment] Add __name__ == __main__ guards to setup.pys

2016-10-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/183 Title: #183: Add __name__ == __main__ guards to setup.pys mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/91920e7cb48cbf143ae281c9c073df14b2c2dddf """ See the full comment at

[Freeipa-devel] [freeipa PR#183][closed] Add __name__ == __main__ guards to setup.pys

2016-10-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/183 Author: tiran Title: #183: Add __name__ == __main__ guards to setup.pys Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/183/head:pr183 git checkout pr183 --

[Freeipa-devel] [freeipa PR#183][+ack] Add __name__ == __main__ guards to setup.pys

2016-10-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/183 Title: #183: Add __name__ == __main__ guards to setup.pys Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#185][opened] TESTS: Update group type name

2016-10-25 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/185 Author: pvomacka Title: #185: TESTS: Update group type name Action: opened PR body: """ As the group type has been changed from 'normal' to 'nonposix' we need to update this information also in tests.

[Freeipa-devel] [freeipa PR#139][comment] WebUI: Vault Management

2016-10-25 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/139 Title: #139: WebUI: Vault Management pvomacka commented: """ @mbasti-rh 2) fixed 3) I filled a ticket: https://fedorahosted.org/freeipa/ticket/6388 4) Tests added 5) Fixed 6) Fixed 7) Salt added 8) Field for public key added 9) Warning added 10)

[Freeipa-devel] [freeipa PR#139][synchronized] WebUI: Vault Management

2016-10-25 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/139 Author: pvomacka Title: #139: WebUI: Vault Management Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/139/head:pr139 git checkout pr139 From

[Freeipa-devel] [freeipa PR#182][comment] Use env var IPA_CONFDIR to get confdir for 'cli' context

2016-10-25 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/182 Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context tiran commented: """ Your proposals are hacks / workarounds, not a proper API to point ipa/ipalib to a custom configuration location. I'm proposing a proper API for

[Freeipa-devel] [freeipa PR#148][closed] Unaccessible variable self.attrs in Tracker

2016-10-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/148 Author: gkaihorodova Title: #148: Unaccessible variable self.attrs in Tracker Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/148/head:pr148 git checkout pr148 --

[Freeipa-devel] [freeipa PR#148][+pushed] Unaccessible variable self.attrs in Tracker

2016-10-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/148 Title: #148: Unaccessible variable self.attrs in Tracker Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

[Freeipa-devel] [freeipa PR#148][comment] Unaccessible variable self.attrs in Tracker

2016-10-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/148 Title: #148: Unaccessible variable self.attrs in Tracker mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/9b0b97073304ba6bfdd6292b07533ab3e7fe8bcb """ See the full comment at

Re: [Freeipa-devel] tomcat-8.0.37-3.fc24.noarch package from updates testing breaks CA instance spawn

2016-10-25 Thread Alexander Bokovoy
On ti, 25 loka 2016, Martin Babinsky wrote: An update for Apache Tmocat recently pushed into bodhi[1] seems to break CA instance spawning in a spectacular way.[2] It seems that the update once again breaks the loading of Java classes during Dogtag server initialization. I gave the package

[Freeipa-devel] [freeipa PR#184][+pushed] Minor install script fixes

2016-10-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/184 Title: #184: Minor install script fixes Label: +pushed -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#184][closed] Minor install script fixes

2016-10-25 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/184 Author: simo5 Title: #184: Minor install script fixes Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/184/head:pr184 git checkout pr184 -- Manage your

[Freeipa-devel] tomcat-8.0.37-3.fc24.noarch package from updates testing breaks CA instance spawn

2016-10-25 Thread Martin Babinsky
An update for Apache Tmocat recently pushed into bodhi[1] seems to break CA instance spawning in a spectacular way.[2] It seems that the update once again breaks the loading of Java classes during Dogtag server initialization. I gave the package negative karma and I suggest for you to do the

[Freeipa-devel] [freeipa PR#180][comment] Make api.env.nss_dir relative to api.env.confdir

2016-10-25 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/180 Title: #180: Make api.env.nss_dir relative to api.env.confdir tiran commented: """ The improvement depends on PR #143. """ See the full comment at https://github.com/freeipa/freeipa/pull/180#issuecomment-255978976 -- Manage your subscription

[Freeipa-devel] [freeipa PR#181][comment] Tests : User Tracker creation of user with minimal values

2016-10-25 Thread gkaihorodova
URL: https://github.com/freeipa/freeipa/pull/181 Title: #181: Tests : User Tracker creation of user with minimal values gkaihorodova commented: """ Yes, It's a valid point to add testcases for these changes . Will do. Thank you. """ See the full comment at

[Freeipa-devel] [freeipa PR#182][synchronized] Use env var IPA_CONFDIR to get confdir for 'cli' context

2016-10-25 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/182 Author: tiran Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/182/head:pr182 git

[Freeipa-devel] [freeipa PR#182][synchronized] Use env var IPA_CONFDIR to get confdir for 'cli' context

2016-10-25 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/182 Author: tiran Title: #182: Use env var IPA_CONFDIR to get confdir for 'cli' context Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/182/head:pr182 git

[Freeipa-devel] [freeipa PR#183][synchronized] Add __name__ == __main__ guards to setup.pys

2016-10-25 Thread tiran
URL: https://github.com/freeipa/freeipa/pull/183 Author: tiran Title: #183: Add __name__ == __main__ guards to setup.pys Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/183/head:pr183 git checkout pr183

Re: [Freeipa-devel] [Test][Patch-0049, 0050] Certs in ID overrides test

2016-10-25 Thread Oleg Fayans
Integration part of the tests is ready. 2 tests: 1. Adds a cert to idoverride of a windows user 2. sssd part - looks up user by his certificate using dbus-sssd Second and third dbus call are executed as a string insted of as array of strings because it just does not work otherwise. Some quote

[Freeipa-devel] [freeipa PR#181][comment] Tests : User Tracker creation of user with minimal values

2016-10-25 Thread mirielka
URL: https://github.com/freeipa/freeipa/pull/181 Title: #181: Tests : User Tracker creation of user with minimal values mirielka commented: """ The same minimal values apply for stageuser-add command, can you please modify the stageuser tracker as well? Also adding testcases for these changes