Skip to content

[chore] Let renovatebot run actions#30112

Merged
bogdandrutu merged 1 commit intoopen-telemetry:mainfrom
TylerHelmuth:let-renovatebot-run-actions
Dec 19, 2023
Merged

[chore] Let renovatebot run actions#30112
bogdandrutu merged 1 commit intoopen-telemetry:mainfrom
TylerHelmuth:let-renovatebot-run-actions

Conversation

@TylerHelmuth
Copy link
Copy Markdown
Member

Description:
We ran into an issue today where #30091 was mergable because skipped tests count as success. If the CI had been allowed to run, the PR would not have been mergable.

So, despite this meaning we'll be running a bunch of useless actions for most renovatebot PRs, we are re-enabling actions for renovatebot for now to protect ourselves.

@bogdandrutu bogdandrutu merged commit 1b56c29 into open-telemetry:main Dec 19, 2023
@github-actions github-actions bot added this to the next release milestone Dec 19, 2023
@TylerHelmuth TylerHelmuth deleted the let-renovatebot-run-actions branch December 19, 2023 21:50
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Jan 10, 2024
**Description:**
We ran into an issue today where
open-telemetry#30091
was mergable because [skipped tests count as
success](https://docs.github.com/en/actions/using-jobs/using-conditions-to-control-job-execution#overview).
If the CI had been allowed to run, the PR would not have been mergable.

So, despite this meaning we'll be running a bunch of useless actions for
most renovatebot PRs, we are re-enabling actions for renovatebot for now
to protect ourselves.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants