On 05/19/2017 07:47 AM, Ryan Goulding wrote: > Right, but that really has nothing to do with abandoned patches, lest > you are proposing to not migrate them. > > > I would say we should prefer a strategy of keeping patches over shedding > them. As someone who has worked on this project for several years, > sometimes you try something only to realize it is going to be a *LOT* > harder than you would expect. You have still done a good amount of > work, and don't want to lose what you have done, even if it is no longer > a priority. Even if you aren't going to directly use that patch again, > you want it to reference when you pick up the particular activity > again. Merge Conflicts look ugly but they really aren't a big deal. So > I am on board with keeping things forever. I don't think we should ever > get rid of work. It really shouldn't be a big deal to keep things > kicking around.
I'm _not_ suggesting we don't migrate the things. AFIAK _every_ change will migrate _as_ a change. It's more a matter of what changes are still open and visible with the default queries. Abandoning a change doesn't mean it's destroyed, it just makes it no longer part of the status:open query and moves it to status:abandoned The _only_ time Gerrit destroys a change is if it's a draft change and then the change is deleted. It's the only time Gerrit even offers the option for delete. Even then, all it does is remove all records of it from the database and then tombstone the ref in the git repo. It only actually dies out of the repo when the automatic garbage collector reaps the tombstones. It's more a matter of our account re-connections I'm anticipating. I would far rather deal with a much smaller subset of our 5000+ Gerrit accounts if possible and the problem set is far better reduced to active people when we start making it obvious. -Andy- -- Andrew J Grimberg Lead, IT Release Engineering The Linux Foundation
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Discuss mailing list [email protected] https://lists.opendaylight.org/mailman/listinfo/discuss
