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

github pr plugin: fork creation is hanging when creating a pull request #18664

Closed
5 of 9 tasks
sunix opened this issue Dec 18, 2020 · 14 comments · Fixed by eclipse-che/che-plugin-registry#798
Closed
5 of 9 tasks
Assignees
Labels
area/plugins kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system. sprint/next

Comments

@sunix
Copy link
Contributor

sunix commented Dec 18, 2020

Describe the bug

Using the Github PR plugin to create a new PR, it is proposing to create a fork but is hanging at that point

Selection_111

Che version

  • 7.22.1

Steps to reproduce

  • Create a new workspace from a github project.
  • add the github pr plugin in the devfile
  • start/restart the workspace
  • allow/signin github
  • checkout a new branch
  • make changes, commit
  • from the github pr panel, create a new PR
  • user is invited to create a fork. Accept.
  • the popup hangs with the message: Creating Pull Request: Start publishing branch ...

Runtime

  • kubernetes

Installation method

  • chectl

Environment

  • Cloud
    • Amazon
    • Azure
    • GCE
    • other (please specify)
  • other: please specify
@sunix sunix added the kind/bug Outline of a bug - must adhere to the bug report template. label Dec 18, 2020
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Dec 18, 2020
@yhontyk yhontyk added area/plugins severity/P1 Has a major impact to usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. labels Dec 18, 2020
@ericwill ericwill added this to the 7.26 milestone Jan 12, 2021
@vinokurig vinokurig self-assigned this Jan 14, 2021
@ericwill ericwill mentioned this issue Jan 14, 2021
33 tasks
@vinokurig
Copy link
Contributor

Couldn't reproduce the issue:
video-convert (2)

@sunix
Copy link
Contributor Author

sunix commented Jan 20, 2021

@vinokurig try to create a PR where you are not the owner :)

@sunix
Copy link
Contributor Author

sunix commented Jan 29, 2021

@vinokurig I am wondering if the github app we create have enough permissions:

Selection_181

@vinokurig
Copy link
Contributor

@sunix
I don't think that the problem is in permissions, I've authenticated the plugin with hardcoded token which had repo permissions and the issue is still reproducible.

@vinokurig
Copy link
Contributor

vinokurig commented Feb 3, 2021

depends on #18933
I can't build my custom che-theia image with additional logging to catch the error.

@vinokurig vinokurig added the status/blocked Issue that can’t be moved forward. Must include a comment on the reason for the blockage. label Feb 3, 2021
@ericwill ericwill mentioned this issue Feb 4, 2021
35 tasks
@ericwill ericwill removed the status/blocked Issue that can’t be moved forward. Must include a comment on the reason for the blockage. label Feb 8, 2021
@ericwill ericwill modified the milestones: 7.26, 7.27 Feb 8, 2021
@ericwill
Copy link
Contributor

ericwill commented Feb 8, 2021

Removed blocked status as the blocker issue has been resolved.

@sunix
Copy link
Contributor Author

sunix commented Feb 9, 2021

I don't have this problem anymore !!!!
Don't know what did the fix but, this is what I have now:
Selection_202

And then when using the Github PR:

Selection_201

@vinokurig
Copy link
Contributor

@sunix

I don't have this problem anymore !!!!

Ok, I'll check it on my side.

@sunix
Copy link
Contributor Author

sunix commented Feb 9, 2021

To test it I created that PR eclipse-che/che-theia#988

@vinokurig
Copy link
Contributor

I still can reproduce it by creating a PR to https://github.com/testmagic-123/test.git

@sunix
Copy link
Contributor Author

sunix commented Feb 10, 2021

I have this now if I don't have permission, not sure why it is not creating a fork:
Selection_215

Extension-Host:Promise rejection not handled in one second: Failed to execute git {
  "exitCode": 128,
  "gitErrorCode": "PermissionDenied",
  "gitCommand": "push",
  "stdout": "",
  "stderr": "remote: Permission to testmagic-123/test.git denied to sunix.\nfatal: unable to access 'https://github.com/testmagic-123/test/': The requested URL returned error: 403\n"
} , reason: Failed to execute git {
  "exitCode": 128,
  "gitErrorCode": "PermissionDenied",
  "gitCommand": "push",
  "stdout": "",
  "stderr": "remote: Permission to testmagic-123/test.git denied to sunix.\nfatal: unable to access 'https://github.com/testmagic-123/test/': The requested URL returned error: 403\n"
}

@ericwill ericwill mentioned this issue Feb 25, 2021
46 tasks
@vinokurig
Copy link
Contributor

I've tried this on the latest version of the GitHub PR plugin an the issue is NOT reproducible there. So updating to a newer version should fix the problem: #18661

@ericwill ericwill modified the milestones: 7.27, 7.28 Mar 2, 2021
@ericwill ericwill mentioned this issue Mar 18, 2021
46 tasks
@ericwill ericwill removed this from the 7.28 milestone Mar 24, 2021
@che-bot
Copy link
Contributor

che-bot commented Sep 20, 2021

Issues go stale after 180 days of inactivity. lifecycle/stale issues rot after an additional 7 days of inactivity and eventually close.

Mark the issue as fresh with /remove-lifecycle stale in a new comment.

If this issue is safe to close now please do so.

Moderators: Add lifecycle/frozen label to avoid stale mode.

@che-bot che-bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Sep 20, 2021
@svor svor added sprint/next team/plugins and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Sep 20, 2021
@vinokurig
Copy link
Contributor

/remove-lifecycle stale

@sunix sunix closed this as completed Oct 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/plugins kind/bug Outline of a bug - must adhere to the bug report template. severity/P1 Has a major impact to usage or development of the system. sprint/next
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants