Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump required version of gssproxy to 0.7.0 #511

Closed
wants to merge 1 commit into from
Closed

Bump required version of gssproxy to 0.7.0 #511

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 27, 2017

@stlaz stlaz added the ack Pull Request approved, can be merged label Feb 27, 2017
@stlaz
Copy link
Contributor

stlaz commented Feb 27, 2017

Works for me
edit: but apparently not for Travis, need to investigate that first

@stlaz stlaz removed the ack Pull Request approved, can be merged label Feb 27, 2017
@stlaz
Copy link
Contributor

stlaz commented Feb 27, 2017

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

@HonzaCholasta
Copy link
Contributor

@stlaz
Copy link
Contributor

stlaz commented Feb 27, 2017

@HonzaCholasta Thank you, please kick Travis once the build is done

@stlaz stlaz added ack Pull Request approved, can be merged and removed ack Pull Request approved, can be merged labels Feb 27, 2017
@stlaz
Copy link
Contributor

stlaz commented Feb 27, 2017

There's going to be 0.6.3 version fixing some more issues.

@puiterwijk
Copy link

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

@MartinBasti
Copy link
Contributor

@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

@puiterwijk
Copy link

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

@ghost
Copy link
Author

ghost commented Mar 1, 2017

@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?

@ghost
Copy link
Author

ghost commented Mar 1, 2017

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

@stlaz
Copy link
Contributor

stlaz commented Mar 1, 2017

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

@ghost
Copy link
Author

ghost commented Mar 1, 2017

@stlaz Thanks. Then we really rather wait for 0.6.3.

@simo5
Copy link
Contributor

simo5 commented Mar 7, 2017

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

@ghost
Copy link
Author

ghost commented Mar 7, 2017 via email

@MartinBasti
Copy link
Contributor

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

@stlaz
Copy link
Contributor

stlaz commented Mar 9, 2017

+1

@simo5
Copy link
Contributor

simo5 commented Mar 9, 2017

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

@HonzaCholasta
Copy link
Contributor

FYI built both in the freeipa-master COPR.

@ghost ghost changed the title Bump required version of gssproxy to 0.6.2 Bump required version of gssproxy to 0.7.0 Mar 10, 2017
@ghost
Copy link
Author

ghost commented Mar 10, 2017

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

@stlaz
Copy link
Contributor

stlaz commented Mar 10, 2017

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

@MartinBasti
Copy link
Contributor

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

@MartinBasti
Copy link
Contributor

and gssproxy 0.7 is not in RHEL yet

@stlaz
Copy link
Contributor

stlaz commented Mar 10, 2017

Meh, I'll be damned to eternity then 👿

@stlaz stlaz added the ack Pull Request approved, can be merged label Mar 10, 2017
@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Mar 10, 2017
@MartinBasti
Copy link
Contributor

master:

  • c37254e Bump required version of gssproxy to 0.7.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
5 participants