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

deployment timeout #83

Closed
plmsprd opened this issue Apr 1, 2016 · 3 comments
Closed

deployment timeout #83

plmsprd opened this issue Apr 1, 2016 · 3 comments
Assignees
Labels

Comments

@plmsprd
Copy link

plmsprd commented Apr 1, 2016

We are using cron jobs, will disable crond during deployment and wait until active cron jobs are finished before activating the deployment. Because of this, one deployment process could take 5 minutes or more.
The deployment scripts on the server are fine (after adjusting Zend Server and PHP).
However, zs client throws a timeout exception after around 3 minutes regardless of the setting for HTTP timeout, e.g.:
--http=timeout=900
--http="timeout=560&sslverifypeer=0"
The deployment scripts on the server continue to run after this exception.

We call zs client with --wait --safe.

The application has thrown an exception!

Exception
The operation timed out!

@slaff
Copy link
Contributor

slaff commented Apr 4, 2016

Thank for pointing out this issue. The reason for it is in here: https://github.com/zend-patterns/ZendServerSDK/blob/master/module/Client/src/Client/Controller/Plugin/Repeater.php#L8.
What we can do is use the http timeout, if it is set and is bigger than 180 sec, so that deployment works also for long running tasks

@slaff slaff self-assigned this Apr 4, 2016
@slaff slaff added the bug label Apr 4, 2016
@slaff slaff closed this as completed in f6f0494 Apr 4, 2016
@plmsprd
Copy link
Author

plmsprd commented Apr 11, 2016

When will this be in regular zs-client.phar?


I created the phar myself and it worked, but I didn't have time to look into turning off those "deprecated" warnings:

Deprecated: ServiceLocatorAwareInterface is deprecated and will be removed in version 3.0, along with the ServiceLocatorAwareInitializer. Please update your class ZendServerWebApi\Model\ApiManager to remove the implementation, and start injecting your dependencies via factory instead. in phar:///usr/local/bin/zs-client/vendor/zendframework/zend-mvc/src/Service/ServiceManagerConfig.php on line 2

Deprecated: You are retrieving the service locator from within the class Client\Controller\ApiController. Please be aware that ServiceLocatorAwareInterface is deprecated and will be removed in version 3.0, along with the ServiceLocatorAwareInitializer. You will need to update your class to accept all dependencies at creation, either via constructor arguments or setters, and use a factory to perform the injections. in phar:///usr/local/bin/zs-client/vendor/zendframework/zend-mvc/src/Controller/AbstractController.php on line 2

@slaff
Copy link
Contributor

slaff commented Apr 12, 2016

When will this be in regular zs-client.phar?

@plmsprd It is already there. (Make sure to update your local repository or download the latest phar file). Give it a try and tell us if it works for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants