Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0006389opensim[REGION] OpenSim Corepublic2012-10-30 18:012012-11-03 01:34
ReporterWhiteStar 
Assigned To 
PriorityhighSeveritytweakReproducibilityalways
StatusnewResolutionopen 
PlatformALLOSALLOS VersionALL
Product Versionmaster (dev code) 
Target VersionFixed in Version 
Summary0006389: save iar or oar, has no "overwrite" switch. Ommission ?
DescriptionIn Commit f76dceb90b5a76a7b6a5243c9032996c007c0cf5 R/20798
This change was done:
"Get "save oar" and "save iar" to tell you in a more friendly manner if the filename to save already exists, rather than exception throwing."

This is a much needed patch. Unfortunately no mechanism was provided to allow an overwrite if exists. Many of us have automated backups & archiving running and a variety of tools to do so but the lack of "overwrite" is causing no end of troubles.

Would it be possible to add a switch to save oar & save iar ?
--overwrite with the default being not to overwrite ? or
--preserve to preserve if already exists ? this would prevent breaking the various automated processes people are using...
TagsNo tags attached.
Git Revision or version number
Run Mode Standalone (Multiple Regions)
Physics EngineODE
Script Engine
EnvironmentMono / Windows
Mono VersionNone
Viewervarious
Attached Files

- Relationships

-  Notes
(0022961)
justincc (administrator)
2012-10-30 18:02

I don't really like this compilcated. I would rather people write their backup scripts to move the existing oar/iar. I appreciate that if this is being done via OpenSimulator's timed script commands option then this may be difficult.
(0022967)
justincc (administrator)
2012-10-30 19:31

So it may be easier just to roll this back for now, though that removes the protection of somebody accidentally executing an identical "save oar" or "save iar" command before the first one has finished.
(0022968)
WhiteStar (reporter)
2012-10-30 21:10

I use a combination of timer-script as do some commercial grids which I setup to be fully automated. I am also using in-world tools to generate on user request, either IAR or OAR as you know I was working on that several months ago when the serialization issue came to light.

An alternate solution may be to take an idea from a few Operating Systems and other software. If filename.ext exists then save_as filename_1.ext

Unfortunately I can see a major caveat with that if automated systems do not account for it... could fill up HDD's pretty quick if not guarded against.

Rolling back temporarily may be a solution till this is sorted out as I know for certain that 2 commercial grids right now are not going to be getting their daily OAR backups and their grandparenting of those archives will be certainly busted. Tiz Bad Mojo ! ;-)
(0022988)
justincc (administrator)
2012-10-31 20:46

Okay, I have reverted in 8dfe59d.

However, I would think any backup operation that doesn't copy the file to another location is extremely fragile, as something could easily go wrong with a particular OAR save or some kind of bad/griefing data creep into the backup. And if a file is being copied it can just as easily be moved away.

Anyway, need to think more about this.
(0022989)
WhiteStar (reporter)
2012-10-31 23:26

Thanks for the temp revert.

I do copy them out into a grandparenting system. This is also the way I deployed the mechanisms and the overwrites keep them current daily.
(0022993)
dutchglory (reporter)
2012-11-01 16:17
edited on: 2012-11-01 16:23

a --force or --overwrite switch would be Perfect..!!

Or a number method, like "region-backup(1).iar", "region-backup(2).iar" etc.
Or time/date, like "region-backup_2012-0011-0002.iar" (November 2nd 2012), 4 numbers for year, month and day. To make it consistent.

same go's for oar...

just some idea's
André

(0022994)
melanie (administrator)
2012-11-01 17:17

Well.....

Let's assume someone is a dork. He runs OpenSim as root.
Let's also assume he's a double dork and has enabled remote console because it's neat and didn't change his default password.

(Root) # save oar --force /etc/passwd

Good one, huh?

So, let's not have this.
(0023000)
dutchglory (reporter)
2012-11-03 01:34

don't update as root DORK... LOL :)

- Issue History
Date Modified Username Field Change
2012-10-30 18:01 WhiteStar New Issue
2012-10-30 18:02 justincc Note Added: 0022961
2012-10-30 19:31 justincc Note Added: 0022967
2012-10-30 21:10 WhiteStar Note Added: 0022968
2012-10-31 20:46 justincc Note Added: 0022988
2012-10-31 23:26 WhiteStar Note Added: 0022989
2012-11-01 16:17 dutchglory Note Added: 0022993
2012-11-01 16:19 dutchglory Note Edited: 0022993 View Revisions
2012-11-01 16:23 dutchglory Note Edited: 0022993 View Revisions
2012-11-01 17:17 melanie Note Added: 0022994
2012-11-03 01:34 dutchglory Note Added: 0023000


Copyright © 2000 - 2012 MantisBT Group
Powered by Mantis Bugtracker