-
Notifications
You must be signed in to change notification settings - Fork 8
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
Resource does not work with Bitbucket >7.0 #19
Comments
The problem might be related with the following line in the
Listing all refs in the given repository via
Please note that its suffixed with The commit SHA of the ref matched the given SHA found by the check script: |
I did some further investigation and it looks like the whole world was beaten by this one. The relating information can be found here:
It turns out that the internal Git refs everyone is relying on is only internal API of Bitbucket which has been removed in Bitbucket 7.0. As stated by Atlassian, this is not going to come back and the REST API should be used for that matter. I tried to work around by using the following Bitbucket plugin for eagerly updating the refs: https://github.com/Cyanoth/Bitbucket-EagerPR-Updates So it looks like the only chance we have is to modify the In the end, what is used is the If there are any other solutions or workarounds to this one, please share 'em. |
Any updates on this one? I'm trying to setup a pipeline in combination with this resource but I'm getting blocked by this issue. |
We found another fork of this resource that seems to be maintained lately. It still has the same problems. We did a initial PR on that that fixes some of the problems and is working for us, but is not final:
We also had to install the plugin because in the I´ll try to fix that also in the resource, but cannot give any timeline by now. |
Hi @holgerstolzenberg, thanks for replying. This is the first time I'm checking again. Did you manage to fix the resource in the fork by any chance? |
No not finally. At emeraldsquad, there is a pending code review for our PR that is missing a final comment. |
We have recently run into similar issues with the pullrequest refs. There are race conditions where sometimes the ref isn't there by the time Concourse checks. We also have issues with bitbucket mirrors not having all the refs. Bitbucket admins and Atlassian claim this is an internal API and it sounds like they've finally removed it. I also started working on some changes to use the API instead of ls-remote with pullrequest refs in this branch: https://github.com/mmb/concourse-bitbucket-pullrequest-resource/tree/list-prs-api The main issue I'm seeing is a lot of this resource is based on the concept of a "merge" ref and that doesn't seem to exist in the API. From the API you can only get a "from" commit and a "to" commit for a PR. We set "rebuild_when_target_changed" to false so we don't need the merge refs but other use cases might. I'll work on it but I wanted to get some community feedback about making this resource use only what is available in the REST API, which could mean removal of some features. |
This might be of interest for you if you want to continue this effort: https://www.atlassian.com/blog/announcements/journey-to-cloud i.e. Atlassian pulls the plug on server licences, drags customers to the cloud |
@mmb Is this fixed or is there a workaround for this ? |
@holgerstolzenberg @sjoerdschouten how did you all fix this issue ? |
Today our pipelines broke without any change to our PR builds. It turns out that our on-prem Bitbucket Server was updated to 7.2.x. It looks like this resource is not compatible with Bitbucket 7.2+.
The following error occurs during the GET step:
Screenshot:
![Screenshot 2020-06-26 at 16 42 41](https://user-images.githubusercontent.com/282736/85869807-7f967680-b7cc-11ea-95f4-1be580016719.png)
Googling around for the error, the assumption arises that this might also effect Bitbucket Server 7.0/7.1, but I am not sure about this.
This is a nasty one, as this affects a lot of pipelines…
The text was updated successfully, but these errors were encountered: