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