logrotate.d does not have correct info

Bug #1505860 reported by Daisuke Nakajima
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
Medium
Megh Bhatt
R2.21.x
Fix Committed
Medium
Megh Bhatt
R3.0
Fix Committed
Medium
Megh Bhatt
R3.1
Fix Committed
Medium
Megh Bhatt
Trunk
Fix Committed
Medium
Megh Bhatt

Bug Description

Following logs have logrotate issue. (I think /var/log/contrail/*.log is easy to resolve.)

#### /etc/logrotate.d/contrail-analytics
Typo
/var/log/contrail/contrail-analytic-api.log <<< contrail-analytics-api.log
/var/log/contrail/query-engine.log <<< contrail-query-engine.log

Need following logs?
contrail-analytics-api.log

anything else?

#### /etc/logrotate.d/contrail-config
Typo
/var/log/contrail/api.log <<<< contrail-api.log

Need following logs?
contrail-discovery-0-stdout.log
contrail-api-0-stdout.log
schema-zk.log
api-0-zk.log
contrail-device-manager.log
device-manager-zk.log
svc-monitor-zk.log
contrail-snmp-collector.log
contrail-snmp-collector-stdout.log

anything else?

#### /etc/logrotate.d/contrail-control
Need following logs?
contrail-control-nodemgr-stderr.log
contrail-dns.log
contrail-dns-stdout.log
contrail-named.log
contrail-named-stdout.log

anything else?

#### /etc/logrotate.d/ifmap-server
Not used
ifmap-server-raw.log

### TSN / Compute node do not have /etc/logroted.d

information type: Proprietary → Public
tags: added: customer
tags: added: bms
Revision history for this message
Megh Bhatt (meghb) wrote :

By default, the daemons (c++ and python) are using internal loggers that do the rotation of the files themselves and hence we do not really need the log rotate entries. However, we do need log rotate for the -stdout/stderr.log files and as part of this bug, we will cleanup the log rotate to just include those files. We will need corresponding change in supervisor conf files to disable supervisord doing the rotation.

Revision history for this message
Megh Bhatt (meghb) wrote :

Correction to the above comment, since supervisord also does the rotation for the -stdout/stderr.log files, instead of disabling supervisord to do the rotation and enabling it via log rotate, we will just remove the log rotate files.

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/14940
Submitter: Megh Bhatt (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.21.x

Review in progress for https://review.opencontrail.org/14942
Submitter: Megh Bhatt (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/14940
Committed: http://github.org/Juniper/contrail-packages/commit/0c249411e7e7ffd26955ba3eaa68073cfcc34ed3
Submitter: Zuul
Branch: R2.20

commit 0c249411e7e7ffd26955ba3eaa68073cfcc34ed3
Author: Megh Bhatt <email address hidden>
Date: Fri Nov 6 10:28:07 2015 -0800

Remove logrotate files since the log rotation is already
handled by the daemons
Closes-Bug: #1505860

Change-Id: If113c020a98aa307d665355053eeb1cdc5cae9ea

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/14942
Committed: http://github.org/Juniper/contrail-packages/commit/568828f170806a00de18d07a9075a4559d3f6ad2
Submitter: Zuul
Branch: R2.21.x

commit 568828f170806a00de18d07a9075a4559d3f6ad2
Author: Megh Bhatt <email address hidden>
Date: Fri Nov 6 10:55:06 2015 -0800

Remove logrotate files since the log rotation is already
handled by the daemons
Closes-Bug: #1505860

Change-Id: Ia4814e3e9581effdf96bd83fbe78e146578dff8c

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Bug update]

bug update...

Raj Reddy (rajreddy)
tags: added: analytics
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.1

Review in progress for https://review.opencontrail.org/22860
Submitter: Megh Bhatt (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/22861
Submitter: Megh Bhatt (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.0

Review in progress for https://review.opencontrail.org/22862
Submitter: Megh Bhatt (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/22860
Committed: http://github.org/Juniper/contrail-packages/commit/8227d37354149d2ff2d03fc69d758bc3c890c1fa
Submitter: Zuul
Branch: R3.1

commit 8227d37354149d2ff2d03fc69d758bc3c890c1fa
Author: Megh Bhatt <email address hidden>
Date: Fri Nov 6 10:28:07 2015 -0800

Remove logrotate files since the log rotation is already
handled by the daemons
Closes-Bug: #1505860

Change-Id: If113c020a98aa307d665355053eeb1cdc5cae9ea
(cherry picked from commit 0c249411e7e7ffd26955ba3eaa68073cfcc34ed3)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/22862
Committed: http://github.org/Juniper/contrail-packages/commit/09bc12cc29ff2d6c216d0aa87ddbc2af9ee1197b
Submitter: Zuul
Branch: R3.0

commit 09bc12cc29ff2d6c216d0aa87ddbc2af9ee1197b
Author: Megh Bhatt <email address hidden>
Date: Fri Nov 6 10:28:07 2015 -0800

Remove logrotate files since the log rotation is already
handled by the daemons
Closes-Bug: #1505860

Change-Id: If113c020a98aa307d665355053eeb1cdc5cae9ea
(cherry picked from commit 0c249411e7e7ffd26955ba3eaa68073cfcc34ed3)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/22861
Committed: http://github.org/Juniper/contrail-packages/commit/7afadeb9992276443d1b41a41a187b7da1b5ca8e
Submitter: Zuul
Branch: master

commit 7afadeb9992276443d1b41a41a187b7da1b5ca8e
Author: Megh Bhatt <email address hidden>
Date: Fri Nov 6 10:28:07 2015 -0800

Remove logrotate files since the log rotation is already
handled by the daemons
Closes-Bug: #1505860

Change-Id: If113c020a98aa307d665355053eeb1cdc5cae9ea
(cherry picked from commit 0c249411e7e7ffd26955ba3eaa68073cfcc34ed3)

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.