Page MenuHomePhabricator

upgrade: add support for experimental safe share mode
Needs RevisionPublic

Authored by pulkit on Fri, Oct 2, 9:22 AM.

Details

Reviewers
marmoute
Group Reviewers
hg-reviewers
Summary

Recently we introduce the share-safe functionality which makes shares share
requirements and config of share source. This patch adds support to
debugupgraderepo command to upgrade repository to share-safe mode when
format.exp-share-safe config is enabled.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

pulkit created this revision.Fri, Oct 2, 9:22 AM
pulkit updated this revision to Diff 23061.Tue, Oct 6, 9:51 AM
pulkit updated this revision to Diff 23258.Fri, Oct 16, 4:02 AM
marmoute requested changes to this revision.Fri, Oct 16, 6:42 AM
marmoute added a subscriber: marmoute.
marmoute added inline comments.
mercurial/upgrade.py
355–357

Upgrades the shared source and current shared repository '

Can we upgrade share ? I though it wasn't possible ? And it is probably safer to deny any upgrade from a share.

to use safe method of sharing which will make requirements '
and configs being shared.'

Maybe clarify this using. It makes shares inherit repository requirement and configuration from its source.

What do you think ?

1463–1465

Could have have old share automatically pick up the source requirements and move to share safe mode ? That would seems safer.

tests/test-share-safe.t
298–299

We need to test that downgrading is working too.

This revision now requires changes to proceed.Fri, Oct 16, 6:42 AM
pulkit added inline comments.Fri, Oct 16, 9:16 AM
mercurial/upgrade.py
355–357

Oh, this message is wrong. Updating it in next version.

1463–1465

They will be required to re-shared. Also I am not sure if we have a way to list all the shares of the current repository.

tests/test-share-safe.t
298–299

Will add a patch after this one for that functionality.

marmoute added inline comments.Fri, Oct 16, 12:14 PM
mercurial/upgrade.py
1463–1465

We don't so we will not be perfect here.