Skip to content

[Draft] Test workflow - #381

Open
Laksh (lakshk98) wants to merge 2 commits into
microsoft:mainfrom
lakshk98:test-workflow
Open

[Draft] Test workflow#381
Laksh (lakshk98) wants to merge 2 commits into
microsoft:mainfrom
lakshk98:test-workflow

Conversation

@lakshk98

Copy link
Copy Markdown
Contributor

Description

Describe the purpose of and changes within this Pull Request.

Testing

Do any existing tests cover this change? Are new tests needed?

Documentation

Is there any documentation impact for this change?

Installation

Is there any installer impact for this change?

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a lightweight CI job to validate that the microsoft/ntosebpfext repository can successfully allocate a runner from the 1ES runner pool as part of the existing CI/CD workflow.

Changes:

  • Introduces a new test_1es_access job gated to the microsoft/ntosebpfext repo.
  • Prints basic host/OS info from the allocated runner to confirm runner selection.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +57 to +59
test_1es_access:
if: github.repository == 'microsoft/ntosebpfext'
runs-on:
Comment on lines +59 to +64
runs-on:
- self-hosted
- 1ES.Pool=ebpf-cicd-runner-pool-server-2019
- 1ES.ImageOverride=server2022_v2
- JobId=ntos-smoke-${{ github.run_id }}-${{ github.run_attempt }}
steps:
Comment on lines +65 to +66
- name: Print runner information
run: |
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants