TTL value of analytics_data_ttl is being wrongly considered instead of analytics_config_audit_ttl

[Sandeep Sridhar]

Contrail Build 2.21.1 - Build 22

1. Define below parameters explicitly in contrail-collector.conf, and reload supervisor-analytics in every analytics nodes.

    analytics_data_ttl=1
    analytics_config_audit_ttl=2160
    analytics_statistics_ttl=168
    analytics_flow_ttl=2

2. Make configuration changes (create/delete virtual-networks) with Web UI once in every hour.

3. Three hours later, check logs with following commands.

    a. contrail-logs --object-type config --start-time now-3h --end-time now-1h
    b. contrail-logs --object-type config --start-time now-1h --end-time now

The result was, "a." shows nothing and "b." shows logs for last one hour.

So, I guess the TTL value of analytics_data_ttl is being considered here. Per my understanding, the TTL of analytics_config_audit_ttl should be ideally considered. Please fix this.

Greetings,
Sandeep.

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/18591
Submitter: Raj Reddy (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/18591
Committed: http://github.org/Juniper/contrail-controller/commit/3357a5be711bdfadaa20dfb5bb5a8e7acc411866
Submitter: Zuul
Branch: master

commit 3357a5be711bdfadaa20dfb5bb5a8e7acc411866
Author: Raj Reddy <email address hidden>
Date: Mon Mar 21 17:39:59 2016 -0700

config messages were supposed to be written with configaudit_ttl, but
they were still written w/ global_ttl [which messages data ttl].

Change-Id: Ifc9b847061b5ed785b43e24ca669d9e3eaffea0f
Closes-Bug: #1558875

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/18632
Submitter: Raj Reddy (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/18641
Submitter: Raj Reddy (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/18642
Submitter: Raj Reddy (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/18632
Committed: http://github.org/Juniper/contrail-controller/commit/5e72231fe2d9cc227c78795117149e9d937ebb40
Submitter: Zuul
Branch: R3.0

commit 5e72231fe2d9cc227c78795117149e9d937ebb40
Author: Raj Reddy <email address hidden>
Date: Mon Mar 21 17:39:59 2016 -0700

config messages were supposed to be written with configaudit_ttl, but
they were still written w/ global_ttl [which messages data ttl].

Change-Id: Ifc9b847061b5ed785b43e24ca669d9e3eaffea0f
Closes-Bug: #1558875
(cherry picked from commit 3357a5be711bdfadaa20dfb5bb5a8e7acc411866)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/18641
Committed: http://github.org/Juniper/contrail-controller/commit/f2e08974f7bb9e55d143ebb476dd5ca6dd01cbd9
Submitter: Zuul
Branch: R2.20

commit f2e08974f7bb9e55d143ebb476dd5ca6dd01cbd9
Author: Raj Reddy <email address hidden>
Date: Tue Mar 22 14:59:06 2016 -0700

config messages were supposed to be written with configaudit_ttl, but
they were still written w/ global_ttl [which messages data ttl].

Change-Id: Ib0e6db8e1f8e19afc7fe7a68f3f8c7fb56a89a71
Closes-Bug: #1558875
(cherry picked from commit efad385f9046f8ba95e8696da9d6d79488a14d97)

[OpenContrail Admin (ci-admin-f)]

Review in progress for https://review.opencontrail.org/18747
Submitter: Raj Reddy (<email address hidden>)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/18747
Committed: http://github.org/Juniper/contrail-controller/commit/d57b01dc160087a81bec795187c4544942a20a8e
Submitter: Zuul
Branch: R2.21.x

commit d57b01dc160087a81bec795187c4544942a20a8e
Author: Raj Reddy <email address hidden>
Date: Tue Mar 22 14:59:06 2016 -0700

config messages were supposed to be written with configaudit_ttl, but
they were still written w/ global_ttl [which messages data ttl].

Change-Id: Ib0e6db8e1f8e19afc7fe7a68f3f8c7fb56a89a71
Closes-Bug: #1558875
(cherry picked from commit efad385f9046f8ba95e8696da9d6d79488a14d97)
(cherry picked from commit f2e08974f7bb9e55d143ebb476dd5ca6dd01cbd9)

[OpenContrail Admin (ci-admin-f)]

Reviewed: https://review.opencontrail.org/18642
Committed: http://github.org/Juniper/contrail-controller/commit/d08a76f6436a1bdda3626a666cf731280c6e604f
Submitter: Zuul
Branch: R2.22.x

commit d08a76f6436a1bdda3626a666cf731280c6e604f
Author: Raj Reddy <email address hidden>
Date: Tue Mar 22 14:59:06 2016 -0700

config messages were supposed to be written with configaudit_ttl, but
they were still written w/ global_ttl [which messages data ttl].

Change-Id: Ib0e6db8e1f8e19afc7fe7a68f3f8c7fb56a89a71
Closes-Bug: #1558875
(cherry picked from commit efad385f9046f8ba95e8696da9d6d79488a14d97)