jimzah/openafs
1
0
Fork 0
mirror of https://git.openafs.org/openafs.git synced 2025-01-19 07:20:11 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

vos: noise messages when attempting to delete non-existent volume

Browse Source 
With vos refactor commit f4e73067cd,
some formerly conditional volume deletes are now unconditional.
This regresses 'vos move' output with harmless "error" messages
when AFSVolTransCreate() returns VNOVOL:
  "Failed to start transaction on <volume>"
  "Volume not attached, does not exist, or not online"

Modify DoVolDelete() to return early (and silently) with VNOVOL
in this case, allowing the caller to handle this appropriately.

Change-Id: I21a72e38f330335adf97ac50a7ac7d0fb97c1a9c
Reviewed-on: http://gerrit.openafs.org/9596
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
This commit is contained in:
Mark Vitale 2013-03-07 09:34:55 -05:00 committed by Derrick Brashear
parent bb42c02ce5
commit 1092cbe34f
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/volser/vsprocs.c
View File

@ -1171,6 +1171,12 @@ DoVolDelete(struct rx_connection *aconn, afs_uint32 avolid,
code =
AFSVolTransCreate_retry(aconn, avolid, apart, ITOffline, &ttid);
/* return early and quietly for VNOVOL; don't continue the attempt to delete. */
if (code == VNOVOL) {
error = code;
goto dfail;
}
EGOTO2(dfail, code, "%sFailed to start transaction on %u\n",
prefix, avolid);