[26229] in Source-Commits
Re: /svn/athena r25462 - in trunk/debathena/debathena/pyhesiodfs/debian: . patches
daemon@ATHENA.MIT.EDU (Jonathan Reed)
Thu Apr 19 17:05:41 2012
Mime-Version: 1.0 (Apple Message framework v1084)
Content-Type: text/plain; charset=us-ascii
From: Jonathan Reed <jdreed@MIT.EDU>
In-Reply-To: <alpine.DEB.2.02.1204191354020.24758@tyger.mit.edu>
Date: Thu, 19 Apr 2012 17:05:38 -0400
Cc: Alex Dehnert <adehnert@MIT.EDU>, source-commits@MIT.EDU
Message-Id: <6F87D3F7-C52A-412D-A75D-675D219489FB@mit.edu>
To: Geoffrey Thomas <geofft@MIT.EDU>
Content-Transfer-Encoding: 8bit
On Apr 19, 2012, at 4:55 PM, Geoffrey Thomas wrote:
> I don't think we have a plausible way of ever getting those messages to the end user, and I also disagree that the messages are ever helpful -- knowing that a locker was deactivated with a generic message isn't too different from just knowing it doesn't exist.
>
> As an end user, if I get an ENOENT on a directory in /mit, I'll run hesinfo or attach or add or something to see what's up long before I ask my sysadmin to go look in /var/log/syslog.
>
> In any case, the message that ERR is "unknown" is wrong, so we should change that.
I agree. I think it can be as simple as:
> ++ if pointer['type'] == 'ERR':
> ++ syslog(LOG_INFO, "Locker "+name+" has been deactivated. Contact accounts@mit.edu with questions.")
> ++ return None
