Skip to content

[chore] Fix bot names#8094

Merged
yurishkuro merged 1 commit intojaegertracing:mainfrom
yurishkuro:fix-bot-names
Feb 25, 2026
Merged

[chore] Fix bot names#8094
yurishkuro merged 1 commit intojaegertracing:mainfrom
yurishkuro:fix-bot-names

Conversation

@yurishkuro
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Yuri Shkuro <github@ysh.us>
Copilot AI review requested due to automatic review settings February 25, 2026 14:38
@yurishkuro yurishkuro requested a review from a team as a code owner February 25, 2026 14:38
@yurishkuro yurishkuro added the changelog:ci Change related to continuous integration / testing label Feb 25, 2026
@yurishkuro yurishkuro merged commit 5074fc8 into jaegertracing:main Feb 25, 2026
63 checks passed
@yurishkuro yurishkuro deleted the fix-bot-names branch February 25, 2026 14:39
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates CI orchestrator logic intended to recognize dependency-update bots so their PRs can use the faster parallel CI path.

Changes:

  • Modifies the github.actor checks used to detect Dependabot/Renovate PRs in .github/workflows/ci-orchestrator.yml.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


# Parallel for known bots (dependency update automation)
if [[ "${{ github.actor }}" == "dependabot[bot]" || "${{ github.actor }}" == "renovate[bot]" ]]; then
if [[ "${{ github.actor }}" == "dependabot" || "${{ github.actor }}" == "renovate-bot" ]]; then
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The bot actor strings here look incorrect for GitHub App bots. Dependabot PRs typically run with github.actor == 'dependabot[bot]' (and the workflows README documents dependabot[bot]/renovate[bot]), so switching to dependabot will prevent bot PRs from using parallel mode. Consider matching the actual actor values (e.g., dependabot[bot] and either renovate[bot] and/or renovate-bot depending on which account is used in this repo) to avoid silently changing CI behavior.

Suggested change
if [[ "${{ github.actor }}" == "dependabot" || "${{ github.actor }}" == "renovate-bot" ]]; then
if [[ "${{ github.actor }}" == "dependabot[bot]" || \
"${{ github.actor }}" == "renovate-bot" || \
"${{ github.actor }}" == "renovate[bot]" ]]; then

Copilot uses AI. Check for mistakes.
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.69%. Comparing base (2cb4570) to head (4f7c549).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8094      +/-   ##
==========================================
+ Coverage   95.66%   95.69%   +0.02%     
==========================================
  Files         317      317              
  Lines       16668    16668              
==========================================
+ Hits        15946    15950       +4     
+ Misses        570      567       -3     
+ Partials      152      151       -1     
Flag Coverage Δ
badger_v1 9.00% <ø> (ø)
badger_v2 0.93% <ø> (ø)
cassandra-4.x-v1-manual 13.23% <ø> (ø)
cassandra-4.x-v2-auto 0.92% <ø> (ø)
cassandra-4.x-v2-manual 0.92% <ø> (ø)
cassandra-5.x-v1-manual 13.23% <ø> (ø)
cassandra-5.x-v2-auto 0.92% <ø> (ø)
cassandra-5.x-v2-manual 0.92% <ø> (ø)
clickhouse 1.01% <ø> (ø)
elasticsearch-6.x-v1 16.61% <ø> (ø)
elasticsearch-7.x-v1 16.64% <ø> (ø)
elasticsearch-8.x-v1 16.79% <ø> (ø)
elasticsearch-8.x-v2 0.93% <ø> (ø)
elasticsearch-9.x-v2 0.93% <ø> (ø)
grpc_v1 7.72% <ø> (ø)
grpc_v2 0.93% <ø> (ø)
kafka-3.x-v2 0.93% <ø> (ø)
memory_v2 0.93% <ø> (ø)
opensearch-1.x-v1 16.68% <ø> (ø)
opensearch-2.x-v1 16.68% <ø> (ø)
opensearch-2.x-v2 0.93% <ø> (ø)
opensearch-3.x-v2 0.93% <ø> (ø)
query 0.93% <ø> (ø)
tailsampling-processor 0.53% <ø> (ø)
unittests 94.37% <ø> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

OlegChumin pushed a commit to OlegChumin/jaeger that referenced this pull request Feb 28, 2026
Signed-off-by: Yuri Shkuro <github@ysh.us>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:ci Change related to continuous integration / testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants