ceilometer-polling agent can't start on Windows

Bug #1625602 reported by Claudiu Belu on 2016-09-20
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Undecided
Mehdi Abaakouk

Bug Description

The ceilometer-polling service cannot start on Windows due to the new usage of the cotyledon library, which tries to set and call Linux-specific attributes and functions, one of which is os.setsid().

LOG: http://paste.openstack.org/show/582226/

Julien Danjou (jdanjou) wrote :

Reported on Cotyledon side at: https://github.com/sileht/cotyledon/issues/6

Changed in ceilometer:
status: New → Invalid

Fix proposed to branch: master
Review: https://review.openstack.org/373242

Changed in ceilometer:
assignee: nobody → Claudiu Belu (cbelu)
status: Invalid → In Progress

Change abandoned by gordon chung (<email address hidden>) on branch: master
Review: https://review.openstack.org/373242

Mehdi Abaakouk (sileht) wrote :

@Claudiu Belu: I have released a new cotyledon (1.6.0) and it should be window compatible. Can you confirm ?

Changed in ceilometer:
status: In Progress → Fix Released
assignee: Claudiu Belu (cbelu) → Mehdi Abaakouk (sileht)
Lucian Petrut (petrutlucian94) wrote :

@Mehdi: Thanks for your interest in having Windows support. Using multiprocessing instead of forking doesn't help either as the service object that we're trying to pass to subprocesses isn't pickleable.

Reviewed: https://review.opendev.org/624380
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=28f0a70da99b2fde2479ce6a5c62128ab75554af
Submitter: Zuul
Branch: master

commit 28f0a70da99b2fde2479ce6a5c62128ab75554af
Author: Daniel Vincze <email address hidden>
Date: Thu Dec 6 22:19:16 2018 +0200

    Windows: avoid passing conf objects to subprocesses

    On Windows, cotyledon uses multiprocessing instead of forking. Meanwhile,
    config objects as well as service manager objects aren't pickleable.

    For this reason, we'll avoid passing unpickleable objects to subprocesses,
    making all the required initialization on the subprocess side.

    This change will allow ceilometer-polling to run on Windows.

    Change-Id: Id51db8f06f295bb7019915b53e560df52456cbda
    Closes-Bug: #1625602

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

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