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

modules/rabbitmq.py version checking had a logical error #33734

Merged
merged 1 commit into from
Jun 3, 2016
Merged

modules/rabbitmq.py version checking had a logical error #33734

merged 1 commit into from
Jun 3, 2016

Conversation

glomium
Copy link
Contributor

@glomium glomium commented Jun 2, 2016

What does this PR do?

I recently contributed to an issue and created a PR. The code I committed had a logical error when checking the version number - version 3.6.Xwith X<7 were not matched (and so on).

This PR solves this.

Related issue #33588 and PR #33641

The current version of rabbitmq, 3.6.2 was not matched
@cachedout cachedout merged commit 3f4d4de into saltstack:develop Jun 3, 2016
@tiramiseb
Copy link

Hello,

I think this needs to be backported to 2016.3.

I get the error mentioned in #33588 with RabbitMQ 3.6.1 on Ubuntu 14.04.
I'm using Salt 2106.3.1.

@rallytime rallytime added the ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch. label Jun 27, 2016
@rallytime
Copy link
Contributor

@tiramiseb Thank you for letting us know. I have back-ported this to 2016.3 in #34304.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants