Skip to content

Add --gerrit-unsafe-dont-check-for-dups.#349

Open
prbprbprb wants to merge 1 commit into
google:masterfrom
prbprbprb:unsafeDontCheckForDups
Open

Add --gerrit-unsafe-dont-check-for-dups.#349
prbprbprb wants to merge 1 commit into
google:masterfrom
prbprbprb:unsafeDontCheckForDups

Conversation

@prbprbprb

Copy link
Copy Markdown

Hope you can merge this - it is for a very local situation (Gerrit with no service account available and only SSH access feasible), but it's useful for anyone else in this situation and explicitly marked as unsafe to minimise footgun potential.

Option skips querying Gerrit for the internal copybara hashtag for an existing open change before creating one. Lets git.gerrit_destination run with no Gerrit REST calls needed.

Cost: each run mints a fresh Change-Id which are mostly stable but there's no belt and braces dup checking so you might get duplicate Gerrit CLs.

Hope you can merge this - it is for a very local situation (Gerrit
with no service account available and only SSH access feasible), but
it's useful for anyone else in this situation and explicitly marked as
unsafe to minimise footgun potential.

Option skips querying Gerrit for the internal copybara hashtag for an
existing open change before creating one. Lets git.gerrit_destination
run with no Gerrit REST calls needed.

Cost: each run mints a fresh Change-Id which are _mostly_ stable but
there's no belt and braces dup checking so you might get duplicate
Gerrit CLs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant