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

Added exception handler to trap the RuntimeError raised when #23906

Merged
merged 2 commits into from
May 20, 2015
Merged

Added exception handler to trap the RuntimeError raised when #23906

merged 2 commits into from
May 20, 2015

Conversation

gladiatr72
Copy link
Contributor

Depends.enforce_dependency() class method fires unsuccessfully. There appears
to be no synchronization within the Depends decorator class wrt the
class global dependency_dict which results in incomplete population of any
loader instantiation occuring at the time of one of these exceptions.

This would mitigate the immediate affects of #23839 and #23373.

Depends.enforce_dependency() class method fires unsuccessfully. There appears
to be no synchronization within the Depends decorator class wrt the
class global dependency_dict which results in incomplete population of any
loader instantiation occuring at the time of one of these exceptions.

This would mitigate the immediate affects of #23839 and #23373.
@jfindlay
Copy link
Contributor

@gladiatr72, there is a lint error.

thatch45 added a commit that referenced this pull request May 20, 2015
…ds_decorator

Added exception handler to trap the RuntimeError raised when
@thatch45 thatch45 merged commit 529e171 into saltstack:develop May 20, 2015
@thatch45 thatch45 added the bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch label May 20, 2015
@rallytime rallytime added ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch. and removed bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch labels May 20, 2015
jfindlay added a commit that referenced this pull request May 20, 2015
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.

4 participants