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

1027500 - init python-gofer before agent and tasking services started. #707

Merged
merged 1 commit into from Nov 7, 2013

Conversation

Projects
None yet
2 participants
@jortel
Member

jortel commented Nov 7, 2013

https://bugzilla.redhat.com/show_bug.cgi?id=1027500

Also moving back to gofer 0.77 for now. 0.77.1 uses python-qpid builtin SSL instead of very old work around for when python-qpid SSL did not support client certificates. However, on f19, this work around is broken because the transport API changed in python-qpid-0.24. Long story short, the builtin SSL does validates the hostname in the server cert matches. For MDP2, it is always localhost. Rather than introducing installer changes, we can just relax the requirement on gofer so that 0.77 will suffice.

@skarmark

This comment has been minimized.

Show comment
Hide comment
@skarmark

skarmark Nov 7, 2013

Contributor

I can't say I completely follow the reasoning of the gofer version change, but the PR looks good according to intended change described in the summary above. Merge away.

Contributor

skarmark commented Nov 7, 2013

I can't say I completely follow the reasoning of the gofer version change, but the PR looks good according to intended change described in the summary above. Merge away.

jortel added a commit that referenced this pull request Nov 7, 2013

Merge pull request #707 from pulp/jortel-1027500
1027500 - init python-gofer before agent and tasking services started.

@jortel jortel merged commit 427048f into master Nov 7, 2013

@jortel jortel deleted the jortel-1027500 branch Nov 7, 2013

asmacdo added a commit to asmacdo/pulp that referenced this pull request Mar 24, 2015

asmacdo added a commit to asmacdo/pulp that referenced this pull request Mar 24, 2015

asmacdo added a commit to asmacdo/pulp that referenced this pull request Mar 24, 2015

asmacdo added a commit to asmacdo/pulp that referenced this pull request Mar 24, 2015

asmacdo added a commit to asmacdo/pulp that referenced this pull request Mar 25, 2015

asmacdo added a commit to asmacdo/pulp that referenced this pull request Mar 26, 2015

asmacdo added a commit to asmacdo/pulp that referenced this pull request Mar 27, 2015

asmacdo added a commit to asmacdo/pulp that referenced this pull request Mar 27, 2015

asmacdo added a commit to asmacdo/pulp that referenced this pull request Mar 27, 2015

asmacdo added a commit to asmacdo/pulp that referenced this pull request Mar 30, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment