-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
Bitbucket Server Catalog Plugin Event Support #19633
base: master
Are you sure you want to change the base?
Conversation
Changed Packages
|
Uffizzi Ephemeral Environment Deploying☁️ https://app.uffizzi.com/github.com/backstage/backstage/pull/19633 ⚙️ Updating now by workflow run 6502333863. What is Uffizzi? Learn more! |
This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution! |
Hi, would you please be re-open the PR? I still wish to get this merged. |
@davelil4 I am beginning to test this PR now, as I'm very much interested in using this. A quick clarification as I begin testing, I ended up following the README instructions to install the plugins (copied the src as local internals for now), and I followed the events backend instructions to create I then copied a sample paload from atlassian documentation for the curl request
When posting this from Postman, I get a successfull response from Postman but then the backstage processor fails with the following message:
Is this error expected? The error message doesn't seem to be too helpful, but I'm guessing it's possible this is because of the dummy data in the payload? I'm still working through some networking permissions to actually fully test this with bitbucket. |
After fussing with some WAF settings I was able to get this working in a full end-to-end test and happy to say it accepted the event for me with no errors! So this seems to work fine with legit payloads from bitbucket. |
I'm not sure why these other checks are failing but I adjusted the docs if you want to check @bforbis . |
@davelil4 any idea if the error message here could be cleaned up to be more human readable? |
@bforbis I'll look into soon as I get the chance. Just to speed up time, what are the "WAF" settings you mentioned that you changed to fix the error? I had never previously gotten this error so it would help me out to know. |
Ignore the part about the WAF. That was just something blocking me from testing end-to-end with Bitbucket directly (ie: I needed to update my firewall to allow bitbucket to send requests to backstage). The error that I saw before was from me testing the event webhook URL manually using Postman. Since I didn't yet have the bitbucket connection setup, I tried testing manually by using the sample payload provided on Bitbucket's doc site. I am not surprised that this caused an error within backstage, as that sample payload is all dummy data. However, the error message reported by Backstage was very confusing and did not actually clue me into what was going wrong. I would have expected some sort of message like "Could not find associated entity", but instead we got some crazy JS undefined behavior error. |
Sure. |
This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution! |
Hi @davelil4, this has 3 approvals, is there anything blocking this from being merged? Personally this seems good to 🚢 @Rugvip, look like for some reason there's an open change you requested but it looks resolved 🤔 That leads me here: #19633 (review) |
Hi @awanlin last I checked I was still looking at getting some requested changes resolved. If that got resolved, I will check my computer when I get home and getting the changes merged. |
Thanks for the update @davelil4, we are getting close to a year for this PR and there is a TON of value in here that IMO we should get out into the world. We can always iterate and make it better in smaller PRs from there. |
This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution! |
Bumping to unmark stale |
Hi @davelil4, were you able to follow up? We should really try and get this merged, I can't see any reason not too! |
75f1e17
to
5555609
Compare
5555609
to
270583e
Compare
Bumping to unmark stale, I will fix conflicts later. |
143fb3d
to
ac79dfb
Compare
@Rugvip can you please check the requested changes when you get the chance. |
Signed-off-by: David Lilienfeld <davidlilienfeld4@gmail.com>
Signed-off-by: David Lilienfeld <davidlilienfeld4@gmail.com>
Signed-off-by: David Lilienfeld <davidlilienfeld4@gmail.com>
Signed-off-by: David Lilienfeld <davidlilienfeld4@gmail.com>
Signed-off-by: David Lilienfeld <davidlilienfeld4@gmail.com>
ac79dfb
to
3bcec82
Compare
@awanlin Any chance you can help with getting these requested changes reviewed? If possible, thank you. |
Bitbucket Server Catalog Plugin Event Support
I added event support for the Bitbucket Server catalog backend plugin, as well as the event backend plugin that is for that event support. I follow much of the same logic as the Bitbucket Server Cloud plugin with its event support. If I need to be more rigorous or clear with my testing please let me know.
✔️ Checklist
Signed-off-by
line in the message. (more info)