pr-1358/derrickstolee/maintenance-unregister-v2
tagged this
22 Sep 13:37
I noticed this while we were updating the microsoft/git fork to include v2.38.0-rc0. I don't think 'git maintenance unregister' was idempotent before, but instead some change in 'scalar unregister' led to it relying on the return code of 'git maintenance unregister'. Our functional tests expect 'scalar unregister' to be idempotent, and I think that's a good pattern for 'git maintenance unregister', so I'm fixing it at that layer. Despite finding this during the 2.38.0-rc0 integration, this isn't critical to the release. Perhaps an argument could be made that "failure means it wasn't registered before", but I think that isn't terribly helpful. Our functional tests are running the unregister subcommand to disable maintenance in order to run tests on the object store (such as running maintenance commands in the foreground and checking the object store afterwards). This is a form of automation using 'unregister' as a check that maintenance will not run at the same time, and it doesn't care if maintenance was already disabled. I can imagine other scripting scenarios wanting that kind of guarantee. Updates in v2 ============= * This is now a two-patch series. * I rebased onto v2.38.0-rc1 for two reasons: Scalar is now merged, and the usage for 'git maintenance unregister' removed its translation markers. * Instead of making git maintenance unregister idempotent, add a --force option for those who do not want to require that the repository is already registered. * Make scalar unregister idempotent, with reasons argued in patch 2. Thanks, -Stolee Derrick Stolee (2): maintenance: add 'unregister --force' scalar: make 'unregister' idempotent Documentation/git-maintenance.txt | 6 +++++- builtin/gc.c | 31 +++++++++++++++++++++++++------ scalar.c | 5 ++++- t/t7900-maintenance.sh | 6 +++++- t/t9210-scalar.sh | 5 ++++- 5 files changed, 43 insertions(+), 10 deletions(-) base-commit: 1b3d6e17fe83eb6f79ffbac2f2c61bbf1eaef5f8 Submitted-As: https://lore.kernel.org/git/pull.1358.v2.git.1663853837.gitgitgadget@gmail.com In-Reply-To: https://lore.kernel.org/git/pull.1358.git.1663635732095.gitgitgadget@gmail.com
Assets 2
-
2022-09-22T13:37:17Z -
2022-09-22T13:37:17Z - Loading