Docker resource waits only 4s for a container

Bug #1544506 reported by Łukasz Oleś
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Solar
Fix Released
Medium
Łukasz Oleś

Bug Description

2016-02-11 11:45:30,753 ERROR _async_event (zerorpc_executor.py::54)::
Traceback (most recent call last):
  File "/vagrant/solar/orchestration/executors/zerorpc_executor.py", line 47, in _async_event
    self._methods[event.name](*event.args)
  File "/vagrant/solar/orchestration/workers/subscription.py", line 62, in __call__
    rst = self.func(self.instance, ctxt, *args, **kwargs)
  File "/vagrant/solar/orchestration/workers/tasks.py", line 42, in solar_resource
    return actions.resource_action(res, action)
  File "/vagrant/solar/core/actions.py", line 35, in resource_action
    return h.action(resource, action)
  File "/vagrant/solar/core/handlers/ansible_template.py", line 128, in action
    self.verify_run_result(call_args, rst)
  File "/vagrant/solar/core/handlers/base.py", line 52, in verify_run_result
    raise errors.SolarError(result.output)
SolarError: [DEPRECATION WARNING]: Instead of sudo/sudo_user, use become/become_user and
make sure become_method is 'sudo' (default). This feature will be removed in a
future release. Deprecation warnings can be disabled by setting
deprecation_warnings=False in ansible.cfg.

Revision history for this message
Łukasz Oleś (loles) wrote :

I will disable deprecation_warnings for now

Changed in solar:
milestone: none → 0.2.0
assignee: nobody → Łukasz Oleś (loles)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to solar (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/279032

Revision history for this message
Jedrzej Nowak (jnowak) wrote : Re: Ansible depraction warrning makes Solar to fail

It was not because of deprecation warning. Ansible returns everything on stdout (even errors).

Revision history for this message
Jedrzej Nowak (jnowak) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to solar (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/279062

Revision history for this message
Łukasz Oleś (loles) wrote :

I updated title, the problem is in docker_container resource. Error is missleading

summary: - Ansible depraction warrning makes Solar to fail
+ Docker resource waits only 4s for a container
Revision history for this message
Łukasz Oleś (loles) wrote :
Łukasz Oleś (loles)
Changed in solar:
status: New → Fix Committed
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to solar (master)

Reviewed: https://review.openstack.org/279062
Committed: https://git.openstack.org/cgit/openstack/solar/commit/?id=3f75defff10c1351090be0724bfeee3bfed65a81
Submitter: Jenkins
Branch: master

commit 3f75defff10c1351090be0724bfeee3bfed65a81
Author: Jedrzej Nowak <email address hidden>
Date: Thu Feb 11 14:17:54 2016 +0100

    Fix handling solar resource action run errors

    Change-Id: I60975a50a370de0de408c1581dcd18b33bf325e0
    Closes-bug: #1544544
    Related-bug: #1544506

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/279032
Committed: https://git.openstack.org/cgit/openstack/solar/commit/?id=054e8d8d2820e63dc6be2d39bfaecde895acb1f7
Submitter: Jenkins
Branch: master

commit 054e8d8d2820e63dc6be2d39bfaecde895acb1f7
Author: Jedrzej Nowak <email address hidden>
Date: Thu Feb 11 13:48:31 2016 +0100

    Ansible returns error on stdout

    when error SolarTransportResult.output is stderr only if not empty, otherwise stdout

    Change-Id: Idc793a6a0c0de33295ef5a8f8c72cb4484b2d9bb
    Related-bug: #1544506

Łukasz Oleś (loles)
Changed in solar:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.