Skip to content

Conversation

@yagreut
Copy link
Contributor

@yagreut yagreut commented Oct 16, 2025

Update Azure Compute schema

Add networkInterfaceIDs

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've followed the conventions in the PR title.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).
  • I've added usage information (if the PR introduces new options)
  • I've included a "before" and "after" example to the description (if the PR is a user interface change).

@github-actions github-actions bot added the apidiff Indicates Go API changes relevant to library consumers (CLI compatibility may be unaffected) label Oct 16, 2025
@github-actions
Copy link

github-actions bot commented Oct 16, 2025

📊 API Changes Detected

Semver impact: major

github.com/aquasecurity/trivy/pkg/iac/providers/azure/compute
  Incompatible changes:
  - LinuxVirtualMachine: old is comparable, new is not
  - VirtualMachine: old is comparable, new is not
  - WindowsVirtualMachine: old is comparable, new is not
  Compatible changes:
  - LinuxVirtualMachine.NetworkInterfaces: added
  - NetworkInterface: added
  - VirtualMachine.NetworkInterfaces: added
  - WindowsVirtualMachine.NetworkInterfaces: added

@aqua-bot aqua-bot requested a review from a team October 16, 2025 11:54
@yagreut yagreut force-pushed the reut-azure-compute branch from 42aa18c to 723edbd Compare October 16, 2025 11:55
@nikpivkin
Copy link
Contributor

Why is support for networkInterfaceIDs needed? This field stores the IDs of other resources to establish a connection. Usually, we find the related resources and perform checks on them directly. In this case, networkInterfaceIDs specifies network interfaces with subnet configurations on which we can run misconfiguration checks.

@yagreut yagreut force-pushed the reut-azure-compute branch from 681115a to 96a01b9 Compare October 27, 2025 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

apidiff Indicates Go API changes relevant to library consumers (CLI compatibility may be unaffected)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants