Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

jenkins: force fetch on each PR. #895

Closed

Conversation

jcvenegas
Copy link
Contributor

If a PR was updated with a push force. We need to force
to update the brach to test this commit does:

  • clean current branch.
  • If testing a PR, force to fetch and pull a brach with latest changes.

Fixes: #894

Signed-off-by: Jose Carlos Venegas Munoz jose.carlos.venegas.munoz@intel.com

Copy link
Contributor

@grahamwhaley grahamwhaley left a comment

Choose a reason for hiding this comment

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

lgtm
and thanks for adding the git clean as well.

@clearcontainersbot
Copy link

kubernetes qa-failed 👎

@sboeuf
Copy link
Contributor

sboeuf commented Feb 6, 2018

the CI is not passing...

@jcvenegas
Copy link
Contributor Author

@chavafg @grahamwhaley any idea?

 # cd /srv/jenkins/workspace/workspace/clear-containers-tests-16.04-PR/go/src/github.com/clearcontainers/tests; git pull --ff-only
11:24:11 There is no tracking information for the current branch.
11:24:11 Please specify which branch you want to merge with.
11:24:11 See git-pull(1) for details.
11:24:11 
11:24:11     git pull <remote> <branch>
11:24:11 
11:24:11 If you wish to set tracking information for this branch you can do so with:
11:24:11 
11:24:11     git branch --set-upstream-to=origin/<branch> PR_886
11:24:11 
11:24:11 package github.com/clearcontainers/tests: exit status 1

@clearcontainersbot
Copy link

kubernetes qa-failed 👎

@clearcontainersbot
Copy link

kubernetes qa-failed 👎

@chavafg
Copy link
Contributor

chavafg commented Feb 6, 2018

lgtm

Approved with PullApprove Approved with PullApprove

If a PR was updated with a push force. We need to force
to update the brach to test this commit does:

- clean current branch.
- If testing a PR, force to fetch and pull a brach with latest changes.

Fixes: clearcontainers#894

Signed-off-by: Jose Carlos Venegas Munoz <jose.carlos.venegas.munoz@intel.com>
mcastelino pushed a commit to mcastelino/tests that referenced this pull request Jan 23, 2019
To get the proper docker version to install, we need to now use
`apt-cache madison` since `apt-cache show` is throwing an error.

Fixes: clearcontainers#895.

Signed-off-by: Salvador Fuentes <salvador.fuentes@intel.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants