|Anonymous | Login | Signup for a new account||2020-01-26 00:11 PST|
|Main | My View | View Issues | Change Log | Roadmap | Summary | My Account|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0006389||opensim||[REGION] OpenSim Core||public||2012-10-30 18:01||2012-11-03 01:34|
|Product Version||master (dev code)|
|Target Version||Fixed in Version|
|Summary||0006389: save iar or oar, has no "overwrite" switch. Ommission ?|
|Description||In 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...
|Tags||No tags attached.|
|Git Revision or version number|
|Run Mode||Standalone (Multiple Regions)|
|Environment||Mono / Windows|
|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.|
|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.|
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 ! ;-)
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.
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.
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
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.
|don't update as root DORK... LOL :)|
|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|