pad.net/bugs/1386968>
<waigani> ugh
<menn0> ericsnow: ping
<ericsnow> menn0: hey
<menn0> ericsnow: so doesn't option 2 involve using --oplog
<menn0> ericsnow: ?
<ericsnow> menn0: yeah
<menn0> ericsnow: but it looks like it's not being used when specific dbs are backed up
<ericsnow> menn0: correct
<menn0> ericsnow: so what's the plan?
<ericsnow> menn0: with option 2 we don't dump specific DBs
<menn0> ericsnow: ok. so why leave the support in there for backing up individual dbs?
<ericsnow> menn0: I'm hopeful that we will be able to stick to option 2
<ericsnow> menn0: in case option 2 doesn't pan out :)
<menn0> menn0: right. got it.
<waigani> menn0: started looking but Bella will be here in 10min, I'll be doing the dad thing
<menn0> waigani: np
<menn0> waigani: i have an errand to run shortly myself
<waigani> menn0: btw cleanup docs has an id type of bson.ObjectId - which screws up our upgrade step assumtions
<waigani> menn0: not a huge problem, but I'll have to do some rejigging tomorrow
<thumper> owwie...
 * thumper goes to lie down
<thumper> shoulder is really starting to ache now
<menn0> waigani: yeah, I've seen that. let's discuss tomorrow. there's a few ways we could tackle that.
<thumper> go an injection in it this afternoon
<thumper> hopefully should settle down in the next two weeks
<thumper> here's hoping
<waigani> menn0: sounds like a plan
<menn0> ericsnow: are you sure this branch even compiles?
<menn0> ericsnow: mongoDumper.Dump looks incomplete with both unused and undeclared variables
<menn0> ericsnow: and mongoDumper.strip() has no code
<ericsnow> menn0: yeah, I created the review request thinking I'd wrapped things up
<ericsnow> menn0: I have it fixed up and will be updating momentarily
<menn0> ericsnow: ok
<waigani> menn0: shouldn't we always use strictLocalID unless we have a reason not to?
<davecheney> tada, data race http://paste.ubuntu.com/8727559/
<davecheney> func (s *MockCharmStore) WithTestMode(testMode bool) charm.Repository { s.TestMode = testMode return s
<davecheney> }
<davecheney> man
