Discussion:
[Freeipa-devel] [freeipa PR#511][opened] Bump required version of gssproxy to 0.6.2
dkupka
2017-02-27 08:18:49 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Author: dkupka
Title: #511: Bump required version of gssproxy to 0.6.2
Action: opened

PR body:
"""
https://fedorahosted.org/freeipa/ticket/6698
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/511/head:pr511
git checkout pr511
stlaz
2017-02-27 08:43:45 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

Label: +ack
stlaz
2017-02-27 08:44:39 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

stlaz commented:
"""
Works for me
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-282659959
stlaz
2017-02-27 08:45:48 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

Label: -ack
stlaz
2017-02-27 08:46:15 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

stlaz commented:
"""
Works for me
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-282659959
stlaz
2017-02-27 08:48:03 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

stlaz commented:
"""
Unfortunately, we can't push this until we find a way to provide the rpm for Travis.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-282660623
HonzaCholasta
2017-02-27 09:07:52 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

HonzaCholasta commented:
"""
@stlaz, https://copr.fedorainfracloud.org/coprs/g/freeipa/freeipa-master/build/519196/
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-282664702
stlaz
2017-02-27 09:12:54 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

stlaz commented:
"""
@HonzaCholasta Thank you, please kick Travis once the build is done
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-282665763
stlaz
2017-02-27 12:11:34 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

Label: +ack
stlaz
2017-02-27 15:46:33 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

Label: -ack
stlaz
2017-02-27 15:46:52 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

stlaz commented:
"""
There's going to be 0.6.3 version fixing some more issues.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-282757858
puiterwijk
2017-02-28 16:59:45 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

puiterwijk commented:
"""
Perhaps it'd be an idea to update the ticket link in the code to https://pagure.io/freeipa/issue/6698 ?
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-283099335
MartinBasti
2017-02-28 17:27:03 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

MartinBasti commented:
"""
@puiterwijk It shouldn't be an issue with https://pagure.io/fedora-infrastructure/issue/5845 fixed :)
but yes since this is not acked yet commit should be updated
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-283107213
puiterwijk
2017-02-28 17:30:04 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

puiterwijk commented:
"""
@MartinBasti Yeah, I know. I just figured that since it's not merged yet, we might as well just change it :).
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-283108095
dkupka
2017-03-01 06:48:42 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

dkupka commented:
"""
@stlaz And do we need the fixes going to 0.6.3. for FreeIPA to work properly? Is there any fix that actually affects us planned for 0.6.3?
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-283259608
dkupka
2017-03-01 06:51:53 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Author: dkupka
Title: #511: Bump required version of gssproxy to 0.6.2
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/511/head:pr511
git checkout pr511
dkupka
2017-03-01 06:56:32 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

dkupka commented:
"""
@puiterwijk @MartinBasti with the redirection working it's not needed. But I should get used to paste links to pagure. Updated.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-283260762
stlaz
2017-03-01 06:58:46 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

stlaz commented:
"""
@dkupka Those fixes should allow us to setup trusts again (more or less).
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-283261093
dkupka
2017-03-01 07:22:00 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

dkupka commented:
"""
@stlaz Thanks. Then we really rather wait for 0.6.3.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-283264733
simo5
2017-03-07 14:29:20 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

simo5 commented:
"""
We are actually planning 0.7 at this point, due to the changes in the last few patchsets :-)
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-284736439
dkupka
2017-03-07 14:36:29 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

dkupka commented:
"""
Ok, please comment here once 0.7 is out and I will update the commit.

"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-284738537
MartinBasti
2017-03-09 17:02:07 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

MartinBasti commented:
"""
I see gssproxy 0.7 in koji, can we update this an test rather early by putting it into freeipa-master repo?
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-285413071
stlaz
2017-03-09 17:33:39 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

stlaz commented:
"""
+1
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-285422303
simo5
2017-03-09 22:43:10 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

simo5 commented:
"""
Can you prepare patch for spec file that requires gssproxy >= 0.7.0 and mod_auth_gssapi >= 1.5.0 ?
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-285507599
HonzaCholasta
2017-03-10 05:32:16 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.6.2

HonzaCholasta commented:
"""
FYI built both in the freeipa-master COPR.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-285582369
dkupka
2017-03-10 11:36:17 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Author: dkupka
Title: #511: Bump required version of gssproxy to 0.6.2
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/511/head:pr511
git checkout pr511
dkupka
2017-03-10 11:38:27 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Author: dkupka
Title: #511: Bump required version of gssproxy to 0.6.2
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/511/head:pr511
git checkout pr511
dkupka
2017-03-10 11:39:30 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Author: dkupka
Title: #511: Bump required version of gssproxy to 0.7.0
Action: edited

Changed field: title
Original value:
"""
Bump required version of gssproxy to 0.6.2
"""
dkupka
2017-03-10 11:41:09 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.7.0

dkupka commented:
"""
@simo5 We already require mod_auth_gssapi >= 1.5.0
https://github.com/freeipa/freeipa/blob/master/freeipa.spec.in#L255
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-285648584
stlaz
2017-03-10 12:20:27 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.7.0

stlaz commented:
"""
Is this in RHEL already? If not, it will break upstream-downstreamish tests 😞
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-285655609
MartinBasti
2017-03-10 12:25:59 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.7.0

MartinBasti commented:
"""
We need to have our upstream-upstream tests green first, then we can care about upstream-downstream
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-285656646
MartinBasti
2017-03-10 12:26:47 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.7.0

MartinBasti commented:
"""
and gssproxy 0.7 is not in RHEL yet
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-285656814
stlaz
2017-03-10 13:12:38 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.7.0

stlaz commented:
"""
Meh, I'll be damned to eternity then 👿
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-285665738
stlaz
2017-03-10 13:12:42 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.7.0

Label: +ack
MartinBasti
2017-03-10 13:18:02 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.7.0

MartinBasti commented:
"""
master:

* c37254e1b124c95d6ea874f6513979ca165fb31d Bump required version of gssproxy to 0.7.0
"""

See the full comment at https://github.com/freeipa/freeipa/pull/511#issuecomment-285666734
MartinBasti
2017-03-10 13:18:01 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Title: #511: Bump required version of gssproxy to 0.7.0

Label: +pushed
MartinBasti
2017-03-10 13:18:04 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/511
Author: dkupka
Title: #511: Bump required version of gssproxy to 0.7.0
Action: closed

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/511/head:pr511
git checkout pr511

Loading...