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

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 request

[Freeipa-devel] Travis CI unexpected PEP8 errors

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, https://github.com/freeipa/freeipa/pull/10

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

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

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 error

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

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

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

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 https://github.com/freeipa/freeipa/pull/332#i

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

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

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 -- Ma

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

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 w

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

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 caf18

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

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?

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 -- Manage

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

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 gi

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

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 https://fedorahosted.org/freeipa/c

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

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: http://ww

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

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: http://www.f

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

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 https://github.com/freeipa/fre

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

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: http://www.freeipa.org/page/Contrib

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

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 -- Mana

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

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 From

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

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: http://www.freeipa.org/page/Contribute

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

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 mer

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

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 ne

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

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. I

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

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] https://github.com/freeip

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

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 P

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

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 From

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

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, ca

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

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 From

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

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 ma

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

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 pr3

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

URL: https://github.com/freeipa/freeipa/pull/324 Title: #324: Check for conflict entries before raising domain level 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/pa

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

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] https://github.com/freeipa/freeipa/commit/32b1743e5fb318b226a602ec8d9a4b6e

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

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 checko

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

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 -- Mana

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

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?

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][closed] experiment: did pull/177 break ci?

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 -- Manage

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

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 sens

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

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 e

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

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 er

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

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 From

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

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 th

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

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 offending