This is an archive of the discontinued Mercurial Phabricator instance.

Differential D9692

sharesafe: introduce config to disallow outdated shares if upgrade fails
ClosedPublic

Authored by pulkit on Jan 8 2021, 11:24 AM.

Details

Reviewers
marmoute
Alphare
Group Reviewers
hg-reviewers
Commits
rHGd159d0fafa78: sharesafe: introduce config to disallow outdated shares if upgrade fails
Summary

After this patch, we have config option to control all aspects of shares
when share source is upgraded or downgraded.

Diff Detail

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

Event Timeline

pulkit created this revision.Jan 8 2021, 11:24 AM
marmoute requested changes to this revision.Jan 8 2021, 9:46 PM
marmoute added a subscriber: marmoute.
marmoute added inline comments.
mercurial/localrepo.py
611–618

We will want the abort to get into play when the upgrade was tried, but failed.

This revision now requires changes to proceed.Jan 8 2021, 9:46 PM
baymax retitled this revision from sharesafe: introduce config to disallow outdated shares from working to sharesafe: introduce config to disallow outdated shares if upgrade fails.Jan 9 2021, 8:03 AM
baymax updated this revision to Diff 24697.

✅ refresh by Heptapod after a successful CI run (🐙 💚)

pulkit added a comment.Jan 9 2021, 2:10 PM

Incorporated the suggestion.

Alphare accepted this revision.Jan 12 2021, 11:42 AM
Alphare added a subscriber: Alphare.

This looks good on its own, but I haven't seen any user-facing documentation/help text for the config options. Please send a follow-up.

pulkit updated this revision to Diff 24779.Jan 13 2021, 6:30 AM
pulkit added a commit: rHGd159d0fafa78: sharesafe: introduce config to disallow outdated shares if upgrade fails.Jan 13 2021, 2:40 PM
This revision was not accepted when it landed; it landed in state Needs Review.
Closed by commit rHGd159d0fafa78: sharesafe: introduce config to disallow outdated shares if upgrade fails (authored by pulkit). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 24646

mercurial/configitems.py

) )
coreconfigitem( coreconfigitem(
b'experimental', b'experimental',
b'sharesafe-auto-upgrade-shares', b'sharesafe-auto-upgrade-shares',
default=False, default=False,
) )
coreconfigitem( coreconfigitem(
b'experimental', b'experimental',
b'sharesafe-disallow-outdated-shares',
default=False,
)
coreconfigitem(
b'experimental',
b'sharesafe-warn-outdated-shares', b'sharesafe-warn-outdated-shares',
default=True, default=True,
) )
coreconfigitem( coreconfigitem(
b'experimental', b'experimental',
b'single-head-per-branch', b'single-head-per-branch',
default=False, default=False,
) )

mercurial/localrepo.py

requirements = upgrade.upgrade_share_to_safe( requirements = upgrade.upgrade_share_to_safe(
ui, ui,
hgvfs, hgvfs,
storevfs, storevfs,
requirements, requirements,
) )
elif ui.configbool( elif ui.configbool(
b'experimental', b'sharesafe-disallow-outdated-shares'
):
raise error.Abort(
_(
b'source repository uses share-safe while'
b' current share does not'
)
)
marmouteUnsubmitted
Not Done

We will want the abort to get into play when the upgrade was tried, but failed.

marmoute: We will want the abort to get into play when the upgrade was tried, but failed.
elif ui.configbool(
b'experimental', b'sharesafe-warn-outdated-shares' b'experimental', b'sharesafe-warn-outdated-shares'
): ):
ui.warn( ui.warn(
_( _(
b'warning: source repository supports share-safe functionality.' b'warning: source repository supports share-safe functionality.'
b' Reshare to upgrade.\n' b' Reshare to upgrade.\n'
) )
) )

tests/test-share-safe.t

Make sure existing shares still works Make sure existing shares still works
$ hg log -GT "{node}: {desc}\n" -R ../nss-share --config experimental.sharesafe-warn-outdated-shares=false $ hg log -GT "{node}: {desc}\n" -R ../nss-share --config experimental.sharesafe-warn-outdated-shares=false
@ f63db81e6dde1d9c78814167f77fb1fb49283f4f: added bar @ f63db81e6dde1d9c78814167f77fb1fb49283f4f: added bar
| |
o f3ba8b99bb6f897c87bbc1c07b75c6ddf43a4f77: added foo o f3ba8b99bb6f897c87bbc1c07b75c6ddf43a4f77: added foo
$ hg log -GT "{node}: {desc}\n" -R ../nss-share --config experimental.sharesafe-disallow-outdated-shares=true
abort: source repository uses share-safe while current share does not
[255]
$ hg log -GT "{node}: {desc}\n" -R ../nss-share $ hg log -GT "{node}: {desc}\n" -R ../nss-share
warning: source repository supports share-safe functionality. Reshare to upgrade. warning: source repository supports share-safe functionality. Reshare to upgrade.
@ f63db81e6dde1d9c78814167f77fb1fb49283f4f: added bar @ f63db81e6dde1d9c78814167f77fb1fb49283f4f: added bar
| |
o f3ba8b99bb6f897c87bbc1c07b75c6ddf43a4f77: added foo o f3ba8b99bb6f897c87bbc1c07b75c6ddf43a4f77: added foo