Thanks Daniel.

Regards

Liang

From: Badea, Daniel [mailto:Daniel.Badea@windriver.com]
Sent: Tuesday, November 6, 2018 10:03 PM
To: Fang, Liang A <liang.a.fang@intel.com>; Poncea, Ovidiu <Ovidiu.Poncea@windriver.com>
Cc: starlingx-discuss@lists.starlingx.io
Subject: RE: [Starlingx-discuss] Analysis of patch 294a63c for StarlingX Upstreaming

Hi Liang,

No objections here, mark this patch as abandoned.

Regards,

Daniel

---

From: Fang, Liang A [liang.a.fang@intel.com]
Sent: Tuesday, November 06, 2018 10:55
To: Badea, Daniel; Poncea, Ovidiu
Cc: starlingx-discuss@lists.starlingx.io
Subject: [Starlingx-discuss] Analysis of patch 294a63c for StarlingX Upstreaming

Hi Daniel and Ovidiu

Repo: cinder

Commit: 294a63c821233e3307f142965f366e8eb3d0c1de

Title: Pike Rebase: Fix Cinder memory leak

This patch is to call novaclient’s API “set_timings_max_len”, to limit the max timing logs, so that the log list will not be to huge. But novaclient(both the upstream version and stx- version) don’t contain the API set_timings_max_len, I also cannot find in repo history. It seems this patch may co-worked with some private version of novaclient previously, but not the public version. In fact current novaclient contains an API “reset_timings” which can be called to purge the timing logs. I want to mark this patch as abandoned, do you have any comments?

Regards

Liang