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

[chore] Revert "[receiver/gitlab] add tracing via webhook skeleton (#36838)" #37218

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

TylerHelmuth
Copy link
Member

@TylerHelmuth TylerHelmuth commented Jan 14, 2025

This reverts commit 778d8f7.

I can't explain why, but it seems that after this commit the extension integration tests never complete. Reverting to test that theory.

@adrielp
Copy link
Contributor

adrielp commented Jan 14, 2025

This reverts commit 778d8f7.

I can't explain why, but it seems that after this commit the extension integration tests never complete. Reverting to test that theory.

There was a significant outage yesterday for GitHub. Not sure if that was the root cause. Are the jobs still "stuck" somewhere and can you link them? I approved anyway in case this reversion does fix it and things need to be unblocked.

@TylerHelmuth
Copy link
Member Author

@adrielp it seems persistent since the merge. Rerunning has not succeeded with the receiver's commit but passes after a revert.

@TylerHelmuth
Copy link
Member Author

See #37177

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants