Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow specifying Gerrit review by full revision path #240

Open
marquiswang opened this issue Oct 22, 2021 · 0 comments
Open

Allow specifying Gerrit review by full revision path #240

marquiswang opened this issue Oct 22, 2021 · 0 comments

Comments

@marquiswang
Copy link
Contributor

Right now, Sputnik requires two parameters, the changeId and commitId to report to Gerrit:

sputnik --conf /path/to/conf.properties --changeId I0a2afb7ae4a94ab1ab473ba00e2ec7de381799a0 --revisionId 3f37692af2290e8e3fd16d2f43701c24346197f0

It turns out you can also use the change and revision's numeric ids

sputnik --conf /path/to/conf.properties --changeId 12345 --revisionId 5

However, with a lot of build setups, it is easiest to extract the identifier for the patchset as a single string "12345/5". You can get this out of the URI for a gerrit review, or from the ref in the Gerrit repo itself.

In my organization's build setup, the build configuration simply sees the branch as "12345/5".

Right now, we use a separate build step that splits this by the "/" and sets two build parameters for the changeId and revisionId. However, it would simplify the setup if we could just pass in --changeId 12345/5.

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

No branches or pull requests

1 participant