-
Notifications
You must be signed in to change notification settings - Fork 244
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
REQUEST: Repository maintenance on opentelemetry-collector-releases #2403
Comments
I support this |
sorry for the delay, I'll do this tomorrow if no one picks it up before then |
I've installed the app, and set up the branch protections https://developer.mend.io/github/open-telemetry reports that I think it's supposed to send a PR to onboard the repo, but it looks like there is an old PR in the repo from a prior onboarding: open-telemetry/opentelemetry-collector-releases#426 and that PR's branch is also still around based on https://github.com/renovatebot/renovate/blob/main/docs/usage/getting-started/installing-onboarding.md#nuke-config-and-re-onboard, I'm going to rename the old renovate onboarding PR and delete that PR's branch... |
I've renamed the old renovate onboarding PR: open-telemetry/opentelemetry-collector-releases#426 and deleted that PR's branch: based on https://github.com/renovatebot/renovate/blob/main/docs/usage/getting-started/installing-onboarding.md#nuke-config-and-re-onboard, it sounded like it could take a few hours for Renovate to notice and send a new onboarding PR so I uninstalled and re-installed the app to the so I manually triggered a renovate scan at https://developer.mend.io/github/open-telemetry/opentelemetry-collector-releases, and that did the trick and (re-)created the onboarding PR: open-telemetry/opentelemetry-collector-releases#709 |
thanks for going through that trouble 😅 |
@mowies let us know if everything is good, and if it's ok to close this issue, thanks! |
I would have closed this when the configuration PR is merged to make sure that it all works correctly. But we can close now as well and I'll reopen if there's any issues. |
As a follow-up to this, we'll need to also disable dependabot in the releases repo after we made sure that everything works with renovate. Should I create a separate issue for that or do we re-use this one here @trask ? |
sure, we can keep this issue open until you're able to confirm everything is working correctly IIRC, you disable dependabot by removing the |
Ok I can confirm that everything works as expected. |
Affected Repository
https://github.com/open-telemetry/opentelemetry-collector-releases
Requested changes
Purpose
The releases repo would like to start using renovate for dependency management.
Expected Duration
permanently
Repository Maintainers
The text was updated successfully, but these errors were encountered: