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

UPSTREAM: 30510: Endpoint controller logs errors during behavior #10415

Merged
merged 1 commit into from
Aug 15, 2016

Conversation

derekwaynecarr
Copy link
Member

@derekwaynecarr
Copy link
Member Author

/cc @ncdc @smarterclayton

I was unable to use hack/cherry-pick to do this pick, it looks like @smarterclayton last bumped the Godeps, but I am unable to find the referenced revision.

[decarr@localhost origin (cherry-pick-30510)]$ UPSTREAM_REMOTE=upstream hack/cherry-pick.sh 30510
Auto packing the repository in background for optimum performance.
See "git help gc" for manual housekeeping.
++ pr/30510 appears to have merged at f98de240614410b5c719bf4210a990c6835fcab5
++ Generating patch for 9fe15e7376a6ba8774fc3b584214d0eb841553c6..038c754b7fe8238e40aa8e5c15db9002e9d6254a onto 507d3a7 ...
fatal: Cannot update paths and switch to branch 'last_upstream_branch' at the same time.
Did you intend to checkout '507d3a7' which can not be resolved as commit?
[ERROR] PID 25481: hack/cherry-pick.sh:44: `git checkout -b last_upstream_branch "${lastrev}"` exited with status 128.
[INFO]      Stack Trace: 
[INFO]        1: hack/cherry-pick.sh:44: `git checkout -b last_upstream_branch "${lastrev}"`
[INFO]   Exiting with code 128.

@smarterclayton
Copy link
Contributor

Git fetch openshift?

On Aug 15, 2016, at 11:06 AM, Derek Carr [email protected] wrote:

/cc @ncdc https://github.com/ncdc @smarterclayton
https://github.com/smarterclayton

I was unable to use hack/cherry-pick to do this pick, it looks like
@smarterclayton https://github.com/smarterclayton last bumped the Godeps,
but I am unable to find the referenced revision.

[decarr@localhost origin (cherry-pick-30510)]$
UPSTREAM_REMOTE=upstream hack/cherry-pick.sh 30510
Auto packing the repository in background for optimum performance.
See "git help gc" for manual housekeeping.
++ pr/30510 appears to have merged at f98de24
++ Generating patch for
9fe15e7..038c754
onto 507d3a7 ...
fatal: Cannot update paths and switch to branch 'last_upstream_branch'
at the same time.
Did you intend to checkout '507d3a7' which can not be resolved as commit?
[ERROR] PID 25481: hack/cherry-pick.sh:44: git checkout -b last_upstream_branch "${lastrev}" exited with status 128.
[INFO] Stack Trace:
[INFO] 1: hack/cherry-pick.sh:44: git checkout -b last_upstream_branch "${lastrev}"
[INFO] Exiting with code 128.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#10415 (comment),
or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABG_p6_uZtJv78fUiyi06OpDtS6HI-vmks5qgIBggaJpZM4JkdZ5
.

@ncdc
Copy link
Contributor

ncdc commented Aug 15, 2016

[merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented Aug 15, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/7922/) (Image: devenv-rhel7_4831)

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 48a0c5c

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 48a0c5c

@derekwaynecarr
Copy link
Member Author

@smarterclayton - i forgot git fetch openshift. Ideally, cherry-pick should do that for me, no?

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/7922/)

openshift-bot pushed a commit that referenced this pull request Aug 15, 2016
@openshift-bot openshift-bot merged commit 070f847 into openshift:master Aug 15, 2016
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.

4 participants