Padraig, +1

Shawn we will be removing the old get_inventory_list check in both UMN and PM, but not as part of this bug.

9852 UMN and PM check for updates should use plan_update_all

JR

Padraig O'Briain wrote:


Shawn Walker wrote:
On Jun 24, 2009, at 4:57 AM, Padraig O'Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-8631-v4/ fixes
8631 abnormal heavy memory consumption of updatemanagernotifier

The fix is to move the call to misc.get_inventory_list to a separate process.


A few things:

src/checkforupdates.py:
lines 55-60: I had thought based on last conversation with John that this was going to be switching to using a plan_update_all, and then checking the response to see if an update was available.

Eventually it will be.


src/updatemanagernotifierpy:
line 94: this doesn't seem useful now that the bulk of the work is done in checkforupdates.py.

It cannot hurt.

Padraig


Cheers,
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to