Climate should have different configfiles for API and Manager

Bug #1284716 reported by Sylvain Bauza
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Blazar
Fix Released
Medium
Sylvain Bauza

Bug Description

There is need for separate logging in between API and Manager.
At least, API should also load config from climate-api.conf (but leave climate.conf as default) and Manager should also load from climate-manager.conf

That should allow to have distinct values from either Manager or API and for instance allow logging to separate files if needed.

Changed in climate:
assignee: nobody → Sylvain Bauza (sylvain-bauza)
Revision history for this message
Dina Belova (dbelova) wrote :

Don't think there is some need in that. Or I may not understand your idea.
May you explain your idea a little bit more?

Revision history for this message
Sylvain Bauza (sylvain-bauza) wrote :

The issue is quite simple : how do you define separate log files and log levels for API and Manager ?

As there is only one climate.conf file, there is no way to get specific values for either one or other. By the way, this is not as other projects do.

Revision history for this message
Dina Belova (dbelova) wrote :

Sylvain, as I remember that's solved by other projects in the following way:

1) all projects have default config files in /etc/<project>/<project_conf>
2) all projects read that by default as config file
3) but if there is need to have some other options (like default levels) - simply you may give other file to that service via --config-file flag while service (like climate-api) starting

so you may simply set other config file for climate-a service.

Moreover, I'm pretty sure oslo.config cannot work with multiple config files.

Revision history for this message
Dina Belova (dbelova) wrote :

Moreover, you may set log level in CLI options.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to climate (master)

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

Changed in climate:
status: New → In Progress
Dina Belova (dbelova)
Changed in climate:
importance: Undecided → Medium
milestone: none → 0.1.1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to climate (master)

Reviewed: https://review.openstack.org/76525
Committed: https://git.openstack.org/cgit/stackforge/climate/commit/?id=c219fb69034ec2c2c72d2a3b3ae05143e35883ce
Submitter: Jenkins
Branch: master

commit c219fb69034ec2c2c72d2a3b3ae05143e35883ce
Author: Sylvain Bauza <email address hidden>
Date: Wed Feb 26 12:00:03 2014 +0100

    Clean-up config files for API and Manager

    Moves some config files for better clarity, and adds the possibility to
    define climate-api.conf and climate-manager.conf

    Change-Id: I8b628d051a563bec9f644710e44b6d87a1f7de88
    Closes-Bug: #1284716

Changed in climate:
status: In Progress → Fix Committed
Changed in blazar:
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.