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
yumpkg.installed broken in salt v2015.8.1 on CentOS 6 minions #28526
Comments
@clarkperkins, thanks for the report. Related to #26129. |
@clarkperkins I cannot reproduce this but I am wondering if this might be related: https://github.com/saltstack/salt/pull/27049/files Can you revert that change on your minion that's exhibiting this behavior and let me know if it makes a difference? |
@clarkperkins Before you do that, please login to the minion and run the following command, and post the output:
I'd like to see the command being run which triggers the traceback. |
Recreated the issue on my CentOS 6.5 box deploying redis. I changed the:
to
as a work around. Here is the output of
Here is the info when running wit salt-call on the minion (
|
+1 |
I was able to replicate the issue since yesterday.. :-( |
|
salt --versions-reportSalt Version: Dependency Versions: System Versions: |
If you do a salt-call and repoquery returns a -9 look at your messages log. It may be because the linux oom-killer striked and killed your process. |
@RuriRyan - when I do the salt-call command, that is exactly what happens actually. It says there's a -9 retcode, and the messages log reports that it killed repoquery. The box I'm running on only has 600 some MB of ram, so that might be a part of it. But that doesn't really explain why this wasn't a problem with salt 2015.8.0, and only occurs in 2015.8.1 |
@grep4linux You have removed all the debug logging that I was hoping to see from the salt-call output. Can you please re-attempt and post the full output, not just the state return? |
Here is the debug logging: for 2015.8.1:
for 2015.8.0:
|
Ah I think I see the problem. From that PR that was mentioned earlier by @cachedout (https://github.com/saltstack/salt/pull/27049/files) it looks like it's blindly running that second repoquery command, even if |
#28526 fixed yumpkg module issue with pkg.installed
@clarkperkins, thanks for fixing this. @grep4linux, @ankitforcode, can you confirm that this fixes the issue for you? |
I was experiencing this bug on CentOS 7. The patch fixes it for me. How is there not test coverage on this? |
@jfindlay Please close this as has been resolved. |
The code that runs the |
Hi,
I'm running almost exclusively CentOS 6 minions, and there seems to be a bug whenever you run
pkg.installed
more than once, but ONLY on 2015.8.1. 2015.8.0 is fine.Here's the stack trace I get from the command line:
And here's the state definition in the sls file:
To clarify, both master and minion are running salt 2015.8.1, but the master is on 14.04.3 LTS, while my minions are on CentOS 6.5.
The text was updated successfully, but these errors were encountered: