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

[core] [windows] Fixes windows agent for windows server 2008 and earlier #2486

Merged
merged 1 commit into from
May 13, 2016

Conversation

gmmeyer
Copy link
Contributor

@gmmeyer gmmeyer commented May 10, 2016

Summary

A client contacted us telling us that the agent had stopped working on Windows Server 2008. There were a few things obscuring what was going on, I won't go into detail about how I figured it out. But, after digging through their logs for a while, the root of it was the following error: WindowsError: [Error -2146893795] Provider DLL failed to initialize correctly.

This is a known py2exe issue in Windows. Amazon had the same issue last year with their AWS cli not working on Windows Server 2008 and earlier, if it had been built on a later system.

What's happening is that py2exe is bundling all the necessary .dll's into the installer. However, in this case, the .dll is an Operating System .dll. It's not appropriate to bundle the OS specific .dll into the installer. It will cause an error that looks exactly like this.

Our error was propagating from the python random number generators, just like Amazon's. And, like Amazon's, excluding crypt32.dll from py2exe was sufficient to fix it.

I have tested this on a Windows 2008 system. And, the client has also confirmed to me that an installer bundled with this specific change fixes their issues.

Open Questions

  1. Are there other dll's that we inadvertently bundle which we should leave out? (Are there ones more specific to this problem?)
  2. How do we best prevent this from happening in the future?

@degemer
Copy link
Member

degemer commented May 13, 2016

Did you try this change on WS 2012 ? (just to be sure)

I don't think we can really do anything about it (more than what you did), because it's more a py2exe issue. (so maybe file an issue ?)

@gmmeyer
Copy link
Contributor Author

gmmeyer commented May 13, 2016

Yea, I think this is the best solution. It's the one Amazon uses, so I'm fine using it as well.

And yes, I did. It works fine there.

@yannmh yannmh added this to the 5.8.0 milestone May 13, 2016
@degemer
Copy link
Member

degemer commented May 13, 2016

Ok, then :shipit:

@yannmh yannmh merged commit c14cf6b into master May 13, 2016
@yannmh yannmh deleted the greg/fix-py2exe-windows2008 branch May 13, 2016 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants