Skip to content

feat: Restrict the self-hosted runners used by neug-test.#196

Merged
lnfjpt merged 1 commit into
alibaba:mainfrom
lnfjpt:main
Apr 9, 2026
Merged

feat: Restrict the self-hosted runners used by neug-test.#196
lnfjpt merged 1 commit into
alibaba:mainfrom
lnfjpt:main

Conversation

@lnfjpt
Copy link
Copy Markdown
Collaborator

@lnfjpt lnfjpt commented Apr 9, 2026

Fixes

@lnfjpt lnfjpt requested a review from zhanglei1949 April 9, 2026 08:08
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Restrict self-hosted runners to Linux platform

✨ Enhancement

Grey Divider

Walkthroughs

Description
• Add linux label to self-hosted runner specifications
• Restrict neug-test and neug-extension-test jobs to Linux runners
• Ensures CI/CD workflows run on appropriate Linux infrastructure
Diagram
flowchart LR
  A["CI/CD Workflows"] -->|"Updated runner spec"| B["self-hosted, linux"]
  B -->|"Applied to"| C["neug-test.yml"]
  B -->|"Applied to"| D["neug-extension-test.yml"]
Loading

Grey Divider

File Changes

1. .github/workflows/neug-extension-test.yml ⚙️ Configuration changes +1/-1

Add Linux label to extension tests runner

• Updated extension_tests_default job runner specification from [self-hosted] to `[self-hosted,
 linux]`
• Restricts job execution to self-hosted runners with Linux platform label

.github/workflows/neug-extension-test.yml


2. .github/workflows/neug-test.yml ⚙️ Configuration changes +1/-1

Add Linux label to build and test runner

• Updated build_and_test job runner specification from [self-hosted] to [self-hosted, linux]
• Restricts unified build and test job to self-hosted Linux runners

.github/workflows/neug-test.yml


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review Bot commented Apr 9, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

Copy link
Copy Markdown
Member

@zhanglei1949 zhanglei1949 left a comment

Choose a reason for hiding this comment

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

LGTM

@lnfjpt lnfjpt merged commit 5299279 into alibaba:main Apr 9, 2026
4 checks passed
@lnfjpt lnfjpt mentioned this pull request May 26, 2026
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.

2 participants