Skip to content

Add Azure Linux 4 to test run#3636

Open
narrieta wants to merge 5 commits into
Azure:developfrom
narrieta:azl4
Open

Add Azure Linux 4 to test run#3636
narrieta wants to merge 5 commits into
Azure:developfrom
narrieta:azl4

Conversation

@narrieta

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds Azure Linux 4 to the E2E image catalog and updates several extension-related test suites to skip Azure Linux 4 (and Debian 11 in China cloud for one suite) due to known extension incompatibilities.

Changes:

  • Added azure-linux_4 and azure-linux_4_arm64 image definitions and included them in relevant image sets.
  • Skipped Azure Linux 4 for extension sequencing/policy suites due to Azure Monitor Linux Agent support gap.
  • Fixed YAML structure/indentation for skip_on_clouds in one suite and adjusted skip ordering in another.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
tests_e2e/test_suites/images.yml Adds Azure Linux 4 (x64/arm64) URNs and includes them in image sets.
tests_e2e/test_suites/ext_sequencing.yml Excludes Azure Linux 4 from the suite with a TODO note about agent support.
tests_e2e/test_suites/ext_policy_with_dependencies.yml Excludes Azure Linux 4; fixes skip_on_clouds list indentation.
tests_e2e/test_suites/ext_policy.yml Adds targeted skip for Debian 11 in China cloud and excludes Azure Linux 4.

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

Comment thread tests_e2e/test_suites/ext_sequencing.yml Outdated
Comment thread tests_e2e/test_suites/ext_policy_with_dependencies.yml Outdated
Comment thread tests_e2e/test_suites/ext_policy.yml Outdated
Comment thread tests_e2e/test_suites/ext_policy.yml
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.

3 participants