Ticket #2954 (closed defect: Fixed)

Opened 9 months ago

Last modified 4 months ago

New memory leak on dashboard

Reported by: ian Owned by: ian
Priority: 2 - High Milestone: zenoss-2.3
Component: User Interface Version: 2.1.91
Keywords: zenoss-2.2.4-verified Cc: ben.simmons@…, michael.sullivan@…
Community Patch Attached: Deployed @ Customer:
Installer: Maintenance Target:
Specific ZenPack: Maintenance Status:
Documentation Note?: Not required Regression:

Description

Probably related to the Google Maps portlet update.

Change History

Changed 9 months ago by ian

(In [8992]) * Refs #2954: Potential fix for dashboard memory leak.

Changed 8 months ago by ian

  • milestone changed from zenoss-2.2 to zenoss-2.3

Changed 6 months ago by cluther

  • keywords zenoss-2.2.4-proposed added
  • severity changed from low to high

Raising severity of this as it is severely impacting a customer.

Changed 6 months ago by cluther

  • cc ben.simmons@…, michael.sullivan@… added
  • reviewed set to 0

Changed 6 months ago by bwhichard

  • reviewed changed from 0 to 1

Changed 5 months ago by ian

(In [9767]) Refs #2954: Creating branch for work on ticket #2954

Changed 5 months ago by ian

  • status changed from new to assigned

Changed 5 months ago by ian

Bad news...this is actually a bug in Google Maps itself. We may be exacerbating it, though; I'm going to make sure we're recycling all the markers we can.

Changed 5 months ago by ian

  • status changed from assigned to closed
  • resolution set to fixed

(In [9819]) * Fixes #2954: Rejiggered to reuse markers when possible

Changed 5 months ago by ian

(In [9824]) Refs #2954: Merging branch changes in with trunk. Approved by Jason.
This change fixes the dashboard memory leak caused by the Google Maps portlet.The leak isn't fixed completely, because Google Maps has its own leak that we obviously can't fix, but it is slowed down as much as possible. Now, markers are reused whenever possible, instead of wiping and redrawing.

Changed 5 months ago by edahl

  • keywords zenoss-2.2.4-approved added; zenoss-2.2.4-proposed removed

Changed 5 months ago by jstevens

(In [9837]) refs #2954
merging r9824 into 2.2.x branch

Changed 5 months ago by jstevens

  • keywords zenoss-2.2.4-patched added; zenoss-2.2.4-approved removed

Changed 4 months ago by ecn

  • keywords zenoss-2.2.4-verified added; zenoss-2.2.4-patched removed

Verified the patch directly (no, I didn't wait 3 days to watch the memory usage).

Note: See TracTickets for help on using tickets.