Ensure that epoptes-client is actually booted to ltsp before setting LTSP_FATCLIENT=True

Bug #1018498 reported by Alkis Georgopoulos
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Epoptes
Fix Released
Low
Unassigned
epoptes (Ubuntu)
Fix Released
Undecided
Unassigned
Precise
Fix Released
Undecided
Stéphane Graber
Quantal
Fix Released
Undecided
Unassigned

Bug Description

Epoptes version: 0.5.3-1

If one installs epoptes-client in a standalone (non-LTSP) PC which for some reason also has the ltsp-client package installed, then that PC is reported to be a fat client in the Epoptes UI, even if it was booted locally.

== SRU ==
[rational]
Epoptes reports regular workstations with ltsp-client installed as fat clients even though they didn't boot from LTSP.

[test case]
1) Install epoptes
2) Install ltsp-client
3) Restart epoptes
4) Start classroom manager on teacher computer

4) should report the machine as a workstation, not a fat client

[regression potential]
The fix just adds a check for /proc/cmdline, the check while not perfect should match 99% of the cases where ltsp-client and epoptes are installed on a non-ltsp machine.
If this still doesn't match, then the client will be reported as a fat client, but overall, the logic can't be any more wrong that it's today.

Revision history for this message
Alkis Georgopoulos (alkisg) wrote :
Changed in epoptes:
importance: Undecided → Low
status: New → Fix Released
Changed in epoptes (Ubuntu Quantal):
status: New → Fix Released
Changed in epoptes (Ubuntu Precise):
status: New → In Progress
assignee: nobody → Stéphane Graber (stgraber)
description: updated
Revision history for this message
Chris Halse Rogers (raof) wrote : Please test proposed package

Hello Alkis, or anyone else affected,

Accepted epoptes into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/epoptes/0.5.5-0ubuntu0.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please change the bug tag from verification-needed to verification-done. If it does not, change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in epoptes (Ubuntu Precise):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
Stéphane Graber (stgraber) wrote :

New check works fine on workstation and thin client.

tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package epoptes - 0.5.5-0ubuntu0.1

---------------
epoptes (0.5.5-0ubuntu0.1) precise-proposed; urgency=low

  * New upstream bugfix releases (0.5.4 and 0.5.5):
    - Split vncport to vncviewerport and vncserverport (LP: #1013808).
    - Remove workaround for bug fixed in twisted. (LP: #943598).
    - Ensure that epoptes-client is actually booted to LTSP before assuming it
      is operating as a Fat client. (LP: #1018498).
    - New translations: Arabic (ar), Catalan (ca), Czech (cs), Galician (gl),
      Lithuanian (lt), Somali (so), Turkish (tr), Chinese (Simplified) (zh_CN)
    - Updated translations: Afrikaans (af), Hebrew (he), Malay (ms),
      Norwegian Bokmal (nb), Dutch (nl), Portuguese (pt), Russian (ru)
 -- Stephane Graber <email address hidden> Wed, 27 Jun 2012 13:04:46 -0400

Changed in epoptes (Ubuntu Precise):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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