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

[2017.7.9] Lock PyYAML to < 5.1 #52208

Merged
merged 2 commits into from Mar 15, 2019

Conversation

@s0undt3ch
Copy link
Member

commented Mar 15, 2019

The reason being that at least one unit test case will leak memory until exhaustion.

FIY, PyYAML 5.1 was released on 13th March 2019

s0undt3ch added 2 commits Mar 15, 2019
The reason being that at least one unit test case will leak memory until
exhaustion.
@s0undt3ch s0undt3ch requested a review from saltstack/team-core as a code owner Mar 15, 2019
@garethgreenaway garethgreenaway merged commit 3cb5c82 into saltstack:2017.7.9 Mar 15, 2019
3 of 9 checks passed
3 of 9 checks passed
jenkins/pr/py2-centos-7 running py2-centos-7...
Details
jenkins/pr/py2-ubuntu-1604 running py2-ubuntu-1604...
Details
jenkins/pr/py2-windows-2016 running py2-windows-2016...
Details
jenkins/pr/py3-centos-7 running py3-centos-7...
Details
jenkins/pr/py3-ubuntu-1604 running py3-ubuntu-1604...
Details
jenkins/pr/py3-windows-2016 running py3-windows-2016...
Details
WIP Ready for review
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details
jenkins/pr/lint Python lint test has passed
Details
@prometheanfire

This comment has been minimized.

Copy link

commented on cee9480 Oct 7, 2019

uh can we get more info here, I can't package this as we've already moved to newer pyyaml

This comment has been minimized.

Copy link

replied Oct 7, 2019

by more info I mean a link to an upstream bug so we can know when to uncap

This comment has been minimized.

Copy link
Contributor

replied Oct 7, 2019

@cmcmarrow can you weigh in on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.