Bug 6102 - DC and ICT need to use pkg set-property
: DC and ICT need to use pkg set-property
Status: RESOLVED FIXINSOURCE
Product: distro-constructor
dc-cli
: unspecified
: i86pc/amd64 OpenSolaris
: P3 major (vote)
: 2009.04
Assigned To: Jean McCormack
:
:
:
:
:
:
  Show dependency treegraph
 
Reported: 2009-01-14 08:07 UTC by Dave Miner
Modified: 2009-01-29 10:30 UTC (History)
3 users (show)

See Also:


Attachments


Note

You need to log in before you can comment on or make changes to this bug.


Description Dave Miner 2009-01-14 08:07:05 UTC
Both DC_tm.py and ict.py directly manipulate /var/pkg/cfg_cache to control the
content cache flush setting.  This was done because at the time there was no
interface from IPS to control the setting, and bug 2266 was filed to request an
interface.  The introduction of "pkg set-property" has provided the needed
interface, so these modules should be changed to use it instead.  This has some
urgency, as Shawn has initial proposals out on redesigning cfg_cache's format
to meet additional requirements, and the current code will almost certainly be
broken should the proposed changes occur.  Filed against DC, but both modules
should be fixed at once.
Comment 1 Jean McCormack 2009-01-15 11:37:27 UTC
I've talked with Joe about the ict.py changes. They should be fairly
straightforward, remove the code to manually change cfg_cache and put in the
code to use IPS.
For DC_tm.py, same as above but also add the ips set-property functionality to
the transfer module.
Comment 2 Jean McCormack 2009-01-29 10:30:03 UTC
Fixed in changeset: 6b7db52d6b63214257cb4ea567591bb1ae6cd379
Comment 3 Karen Tung 2009-02-02 16:38:39 UTC
*** Bug 5791 has been marked as a duplicate of this bug. ***