Re: [Freeipa-devel] Travis CI unexpected PEP8 errors

2016-12-13 Thread Standa Laznicka
On 12/14/2016 02:53 AM, Ben Lipton wrote: Hi all, I'm pretty sure this is unrelated to the CI issues discussed in other threads recently, but they reminded me that I've been having this odd issue. https://travis-ci.org/freeipa/freeipa/jobs/183756995 is the most recent run on my pull

[Freeipa-devel] Travis CI unexpected PEP8 errors

2016-12-13 Thread Ben Lipton
Hi all, I'm pretty sure this is unrelated to the CI issues discussed in other threads recently, but they reminded me that I've been having this odd issue. https://travis-ci.org/freeipa/freeipa/jobs/183756995 is the most recent run on my pull request,

[Freeipa-devel] [freeipa PR#245][comment] Allow full customisability of IPA CA subject DN

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/245 Title: #245: Allow full customisability of IPA CA subject DN frasertweedale commented: """ @jcholast: new tickets pertaining to subject_base / certmap.conf config: - **do not update ipaCertificateSubjectBase and certmap.conf in CA-less mode**

Re: [Freeipa-devel] Travis CI broke after merging PR 177

2016-12-13 Thread Martin Babinsky
On 12/13/2016 09:41 AM, Martin Babinsky wrote: Hi list, https://github.com/freeipa/freeipa/pull/177 was recently merged despite causing nearly half of the tests in our Travis CI gating to fail. This broke Travis CI for all other PR that were rebased after this merge, causing false negative

[Freeipa-devel] [freeipa PR#332][closed] Fix regression in test suite

2016-12-13 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/332 Author: frasertweedale Title: #332: Fix regression in test suite Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/332/head:pr332 git checkout pr332 -- Manage your

[Freeipa-devel] [freeipa PR#332][+pushed] Fix regression in test suite

2016-12-13 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/332 Title: #332: Fix regression in test suite 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#332][comment] Fix regression in test suite

2016-12-13 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/332 Title: #332: Fix regression in test suite martbab commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/74b8cf2c4a8dd36577d76c35a9ef08352ef025b7 """ See the full comment at

[Freeipa-devel] [freeipa PR#332][+ack] Fix regression in test suite

2016-12-13 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/332 Title: #332: Fix regression in test suite Label: +ack -- 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#332][comment] Fix regression in test suite

2016-12-13 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/332 Title: #332: Fix regression in test suite martbab commented: """ Thanks for pinpointing the issue, looks like Travis is happy with your fix.  """ See the full comment at https://github.com/freeipa/freeipa/pull/332#issuecomment-266779010 --

[Freeipa-devel] [freeipa PR#333][opened] Remove named-pkcs11 workarounds from DNSSEC tests.

2016-12-13 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/333 Author: pspacek Title: #333: Remove named-pkcs11 workarounds from DNSSEC tests. Action: opened PR body: """ As far as I can tell the tests are passing for some time in Jenkins so maybe a bug in some underlying component was fixed. Let's remove

[Freeipa-devel] [freeipa PR#332][synchronized] Fix regression in test suite

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/332 Author: frasertweedale Title: #332: Fix regression in test suite Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/332/head:pr332 git checkout pr332 From

[Freeipa-devel] [freeipa PR#332][opened] Fix regression in test suite

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/332 Author: frasertweedale Title: #332: Fix regression in test suite Action: opened PR body: """ 32b1743e5fb318b226a602ec8d9a4b6ef2a25c9d introduced a regression in test_serverroles.py, caused by ca_find attempting to log into the Dogtag REST API.

[Freeipa-devel] [freeipa PR#329][closed] experiment: did pull/177 break ci?

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/329 Author: frasertweedale Title: #329: experiment: did pull/177 break ci? Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/329/head:pr329 git checkout pr329 --

[Freeipa-devel] [freeipa PR#272][closed] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Author: pspacek Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/272/head:pr272

[Freeipa-devel] [freeipa PR#272][comment] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/b7d70baee73c64898de91e2fa59b3f9f417c8e01

[Freeipa-devel] [freeipa PR#272][+pushed] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time 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#272][+ack] Build: makerpms.sh generates Python 2 & 3 packages at the same time

2016-12-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/272 Title: #272: Build: makerpms.sh generates Python 2 & 3 packages at the same time 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#328][comment] fix: regression in API version comparison

2016-12-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/328 Title: #328: fix: regression in API version comparison mbasti-rh commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/0663faf2585be4af3501965934703da0ede41610 """ See the full comment at

[Freeipa-devel] [freeipa PR#328][+pushed] fix: regression in API version comparison

2016-12-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/328 Title: #328: fix: regression in API version comparison 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#328][closed] fix: regression in API version comparison

2016-12-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/328 Author: mbasti-rh Title: #328: fix: regression in API version comparison Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/328/head:pr328 git checkout pr328 --

[Freeipa-devel] [freeipa PR#329][synchronized] experiment: did pull/177 break ci?

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/329 Author: frasertweedale Title: #329: experiment: did pull/177 break ci? Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/329/head:pr329 git checkout pr329

[Freeipa-devel] [freeipa PR#328][+ack] fix: regression in API version comparison

2016-12-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/328 Title: #328: fix: regression in API version comparison Label: +ack -- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA:

Re: [Freeipa-devel] Travis CI broke after merging PR 177

2016-12-13 Thread Martin Babinsky
On 12/13/2016 01:41 PM, Fraser Tweedale wrote: On Tue, Dec 13, 2016 at 01:11:37PM +0100, Martin Babinsky wrote: On 12/13/2016 01:07 PM, Fraser Tweedale wrote: On Tue, Dec 13, 2016 at 09:41:40AM +0100, Martin Babinsky wrote: Hi list, https://github.com/freeipa/freeipa/pull/177 was recently

Re: [Freeipa-devel] Travis CI broke after merging PR 177

2016-12-13 Thread Fraser Tweedale
On Tue, Dec 13, 2016 at 01:11:37PM +0100, Martin Babinsky wrote: > On 12/13/2016 01:07 PM, Fraser Tweedale wrote: > > On Tue, Dec 13, 2016 at 09:41:40AM +0100, Martin Babinsky wrote: > > > Hi list, > > > > > > https://github.com/freeipa/freeipa/pull/177 was recently merged despite > > > causing

[Freeipa-devel] [freeipa PR#331][opened] WebUI: don't change casing of Auth Indicators values

2016-12-13 Thread pvomacka
URL: https://github.com/freeipa/freeipa/pull/331 Author: pvomacka Title: #331: WebUI: don't change casing of Auth Indicators values Action: opened PR body: """ All values were previously converted to lowercase which was not coresponding with CLI behaviour. Now they stay as they are inserted.

Re: [Freeipa-devel] CI failures - I need your help

2016-12-13 Thread Martin Babinsky
On 12/13/2016 12:20 PM, Fraser Tweedale wrote: Hi all, The CI failures caused by one of my recent commits have me baffled. It is exactly this commit[1] at which the problems begin. I cannot see anything in the commit to point a finger at. In-tree tests run fine. [1]

Re: [Freeipa-devel] Travis CI broke after merging PR 177

2016-12-13 Thread Martin Babinsky
On 12/13/2016 01:07 PM, Fraser Tweedale wrote: On Tue, Dec 13, 2016 at 09:41:40AM +0100, Martin Babinsky wrote: Hi list, https://github.com/freeipa/freeipa/pull/177 was recently merged despite causing nearly half of the tests in our Travis CI gating to fail. This broke Travis CI for all other

[Freeipa-devel] [freeipa PR#329][synchronized] experiment: did pull/177 break ci?

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/329 Author: frasertweedale Title: #329: experiment: did pull/177 break ci? Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/329/head:pr329 git checkout pr329

Re: [Freeipa-devel] Travis CI broke after merging PR 177

2016-12-13 Thread Fraser Tweedale
On Tue, Dec 13, 2016 at 09:41:40AM +0100, Martin Babinsky wrote: > Hi list, > > https://github.com/freeipa/freeipa/pull/177 was recently merged despite > causing nearly half of the tests in our Travis CI gating to fail. This broke > Travis CI for all other PR that were rebased after this merge,

[Freeipa-devel] [freeipa PR#329][synchronized] experiment: did pull/177 break ci?

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/329 Author: frasertweedale Title: #329: experiment: did pull/177 break ci? Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/329/head:pr329 git checkout pr329

[Freeipa-devel] [freeipa PR#324][comment] Check for conflict entries before raising domain level

2016-12-13 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/324 Title: #324: Check for conflict entries before raising domain level martbab commented: """ There were some PEP8 errors but I fixed them and pushed to: ipa-4-4: https://fedorahosted.org/freeipa/changeset/d028d23c5f0c3e1b18c15fad67a0893870f5d27c

[Freeipa-devel] [freeipa PR#324][closed] Check for conflict entries before raising domain level

2016-12-13 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/324 Author: tbordaz Title: #324: Check for conflict entries before raising domain level Action: closed To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/324/head:pr324 git checkout

[Freeipa-devel] CI failures - I need your help

2016-12-13 Thread Fraser Tweedale
Hi all, The CI failures caused by one of my recent commits have me baffled. It is exactly this commit[1] at which the problems begin. I cannot see anything in the commit to point a finger at. In-tree tests run fine. [1]

[Freeipa-devel] [freeipa PR#324][synchronized] Check for conflict entries before raising domain level

2016-12-13 Thread tbordaz
URL: https://github.com/freeipa/freeipa/pull/324 Author: tbordaz Title: #324: Check for conflict entries before raising domain level Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/324/head:pr324 git

[Freeipa-devel] [freeipa PR#329][reopened] experiment: did pull/177 break ci?

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/329 Author: frasertweedale Title: #329: experiment: did pull/177 break ci? Action: reopened To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/329/head:pr329 git checkout pr329 --

[Freeipa-devel] [freeipa PR#329][synchronized] experiment: did pull/177 break ci?

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/329 Author: frasertweedale Title: #329: experiment: did pull/177 break ci? Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/329/head:pr329 git checkout pr329 --

[Freeipa-devel] [freeipa PR#329][synchronized] experiment: did pull/177 break ci?

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/329 Author: frasertweedale Title: #329: experiment: did pull/177 break ci? Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/329/head:pr329 git checkout pr329 --

[Freeipa-devel] [bind-dyndb-ldap PR#5][opened] Add GDB pretty-printers for plugin data structures to contrib.

2016-12-13 Thread pspacek
URL: https://github.com/freeipa/bind-dyndb-ldap/pull/5 Author: pspacek Title: #5: Add GDB pretty-printers for plugin data structures to contrib. Action: opened PR body: """ These are convenience scripts I created over time to ease digging in bind-dyndb-ldap data structures. It might make

[Freeipa-devel] [freeipa PR#330][opened] Build: forbid builds in working directories containing white spaces

2016-12-13 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/330 Author: pspacek Title: #330: Build: forbid builds in working directories containing white spaces Action: opened PR body: """ Spaces are causing problems in libtool, makefiles, autoconf itself, gettextize framework etc. so this issue cannot be

[Freeipa-devel] [freeipa PR#329][synchronized] experiment: did pull/177 break ci?

2016-12-13 Thread frasertweedale
URL: https://github.com/freeipa/freeipa/pull/329 Author: frasertweedale Title: #329: experiment: did pull/177 break ci? Action: synchronized To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/329/head:pr329 git checkout pr329

[Freeipa-devel] [freeipa PR#301][comment] scripts, tests: explicitly set confdir in the rest of server code

2016-12-13 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/301 Title: #301: scripts, tests: explicitly set confdir in the rest of server code stlaz commented: """ @tiran I find all the changes actually required. I think ACK is in order unless you spell out those which you think are not necessary. """ See

[Freeipa-devel] Travis CI broke after merging PR 177

2016-12-13 Thread Martin Babinsky
Hi list, https://github.com/freeipa/freeipa/pull/177 was recently merged despite causing nearly half of the tests in our Travis CI gating to fail. This broke Travis CI for all other PR that were rebased after this merge, causing false negative errors everywhere. Fraser reverted the