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

S3 ext_pillar module has broken caching mechanism #26953

Merged
merged 2 commits into from Sep 10, 2015

Conversation

Projects
None yet
4 participants
@dsumsky
Copy link
Contributor

commented Sep 8, 2015

  • fixed broken caching in S3 ext_pillar module (file_md5 was a list)
  • added debugging messages
  • static parameters are available as module parameters now
- fixed broken caching in S3 ext_pillar module (file_md5 was a list)
- added debugging messages
- static parameters are available as module parameters now
@jfindlay

This comment has been minimized.

Copy link
Contributor

commented Sep 8, 2015

@dsumsky, thanks for the fix. Do you think this needs to be in 2015.5 or 2015.8?

@dsumsky

This comment has been minimized.

Copy link
Contributor Author

commented Sep 9, 2015

I also created pull request for 2015.5 version tree.

@rallytime

This comment has been minimized.

Copy link
Contributor

commented Sep 9, 2015

@dsumsky Thanks! It looks like you have some pylint errors. Would you mind grabbing those?

@rallytime rallytime added the Lint label Sep 9, 2015

@rallytime rallytime removed the Lint label Sep 10, 2015

@rallytime

This comment has been minimized.

Copy link
Contributor

commented Sep 10, 2015

Go Go Jenkins!

rallytime added a commit to rallytime/salt that referenced this pull request Sep 10, 2015

@rallytime rallytime merged commit 5299b96 into saltstack:develop Sep 10, 2015

1 of 5 checks passed

jenkins/salt-pr-linode-ubuntu14.04-n Salt PR - Linode Ubuntu 14.04 #497 — FAILURE
Details
jenkins/salt-pr-lint-n Salt PR - Code Lint #9142 — FAILURE
Details
jenkins/salt-pr-rs-cent7-n Salt PR - RS CentOS 7 #7971 — FAILURE
Details
default Merged build started.
Details
jenkins/salt-pr-clone Salt PR - Clone Repository #9421 — SUCCESS
Details

rallytime added a commit that referenced this pull request Sep 10, 2015

rallytime added a commit that referenced this pull request Sep 11, 2015

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