Skip to content

Jenkinsfile: set repo and branch for DCO check as well#40119

Merged
tiborvass merged 1 commit intomoby:masterfrom
thaJeztah:fix_dco_branch
Oct 21, 2019
Merged

Jenkinsfile: set repo and branch for DCO check as well#40119
tiborvass merged 1 commit intomoby:masterfrom
thaJeztah:fix_dco_branch

Conversation

@thaJeztah
Copy link
Member

Commit 7019b60 (#40035) added these
env-vars to other stages, but forgot to update the DCO stage,
which also does a diff to validate commits that are in a PR.

this should fix the issue we're seeing in docker-archive#408 (comment)

@thaJeztah
Copy link
Member Author

ping @tiborvass @kolyshkin @andrewhsu ptal

Copy link
Contributor

@kolyshkin kolyshkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Jenkinsfile Outdated
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

guess for our private repositories, I need to add openssh-client here

Commit 7019b60 added these
env-vars to other stages, but forgot to update the DCO stage,
which also does a diff to validate commits that are in a PR.

Also adding openssh-client, for situations where the upstream
needs to be accessed through an ssh connection.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Copy link
Contributor

@andrewhsu andrewhsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SGTM

@tiborvass tiborvass merged commit 5100013 into moby:master Oct 21, 2019
@thaJeztah thaJeztah deleted the fix_dco_branch branch October 21, 2019 23:35
@thaJeztah thaJeztah added this to the 20.03.0 milestone Apr 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants