Landscape Client

upgrading landscape-client changes permissions of stored graph scripts

Bug #307321 reported by Christopher Armstrong
2
Affects Status Importance Assigned to Milestone
Landscape Client
Fix Released
High
Christopher Armstrong
Landscape Server
Fix Released
High
Christopher Armstrong
Landscape Server mountainview
landscape-client (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

There's a line in landscape-common.postinst

chown -R landscape /var/log/landscape /var/lib/landscape

This is bad because we've started storing stuff in /var/lib/landscape which needs to be chowned to particular (arbitrary) users in /var/lib/landscape/client/.

Christopher Armstrong (radix)
Changed in landscape:
milestone: none → mountainview-pre-3
Revision history for this message
Christopher Armstrong (radix) wrote :

the attached branch (~radix/landscape-client/keep-perms-on-upgrade) is ready for review against trunk.

Revision history for this message
Thomas Herve (therve) wrote :

+1.

Andreas Hasenack (ahasenack)
Changed in landscape-client:
importance: Undecided → High
status: New → In Progress
Changed in landscape:
importance: Undecided → High
Revision history for this message
Kevin McDermott (bigkevmcd) wrote :

+1 on the fix,

Revision history for this message
Christopher Armstrong (radix) wrote :

Merged to trunk.

Changed in landscape-client:
assignee: nobody → radix
status: In Progress → Fix Committed
Changed in landscape:
assignee: nobody → radix
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package landscape-client - 1.0.25-0ubuntu0.9.04

---------------
landscape-client (1.0.25-0ubuntu0.9.04) jaunty; urgency=low

  * New upstream release supporting custom graphs (LP: #306360)
    - Multiple custom graphs can be used at the same time (LP: #307314)
    - PATH is now set for scripts in script execution (LP: #257018)
  * debian/landscape-common.postinst: Only chown parts of /var/lib/landscape
    because we now store files in it that should maintain their ownership
    (LP: #307321).
  * debian/landscape-client.postinst: Work around chfn/system user problem
    by not specifying a --gecos (LP: #238755)
  * debian/landscape-client.logrotate: logrotate no longer reports spurious
    errors when the client isn't running (LP: #271767)

 -- Christopher Armstrong <email address hidden> Thu, 11 Dec 2008 17:11:08 -0800

Changed in landscape-client:
status: New → Fix Released
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

qa + 1

Andreas Hasenack (ahasenack)
Changed in landscape:
milestone: mountainview-pre-3 → mountainview-pre-4
Andreas Hasenack (ahasenack)
Changed in landscape:
milestone: mountainview-pre-4 → mountainview-pre-5
Andreas Hasenack (ahasenack)
Changed in landscape:
milestone: mountainview-pre-5 → mountainview-pre-6
Andreas Hasenack (ahasenack)
Changed in landscape:
milestone: mountainview-pre-6 → mountainview-pre-7
Andreas Hasenack (ahasenack)
Changed in landscape:
milestone: mountainview-pre-7 → mountainview-pre-8
Andreas Hasenack (ahasenack)
Changed in landscape:
status: Fix Committed → Fix Released
Andreas Hasenack (ahasenack)
Changed in landscape:
milestone: mountainview-pre-8 → mountainview
status: Fix Released → Fix Committed
tags: added: needs-package-release
Andreas Hasenack (ahasenack)
Changed in landscape:
status: Fix Committed → Fix Released
Andreas Hasenack (ahasenack)
Changed in landscape-client:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.