Skip to content

[Az.Network] Added a new command 'New-AzPacketCaptureSettingsConfig' for Network watcher Packet capture include ring buffer change with new updates#27869

Closed
v-sapanchal wants to merge 30 commits into
Azure:mainfrom
v-sapanchal:spanchal/pcap_ring_buffer
Closed

[Az.Network] Added a new command 'New-AzPacketCaptureSettingsConfig' for Network watcher Packet capture include ring buffer change with new updates#27869
v-sapanchal wants to merge 30 commits into
Azure:mainfrom
v-sapanchal:spanchal/pcap_ring_buffer

Conversation

@v-sapanchal
Copy link
Copy Markdown
Contributor

Modified the existing Packet capture command include ring buffer change, in order to that, created a new sub command for capture settings,
Example:
image
image

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/Network/Network/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

v-sapanchal and others added 29 commits May 16, 2025 18:22
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Yeming Liu <11371776+isra-fel@users.noreply.github.com>
@azure-client-tools-bot-prd
Copy link
Copy Markdown

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

@v-sapanchal v-sapanchal added this to the Az 14.1.0 (06/03/2025) milestone May 29, 2025
@v-sapanchal v-sapanchal marked this pull request as ready for review May 29, 2025 06:43
@vidai-msft vidai-msft removed this from the Az 14.1.0 (06/03/2025) milestone May 29, 2025
@vidai-msft
Copy link
Copy Markdown
Contributor

@v-sapanchal Please rebase your branch and resolve the conflict.

@vidai-msft
Copy link
Copy Markdown
Contributor

@v-sapanchal Is this PR a dup of #27880 ? Can we close it?

@v-sapanchal
Copy link
Copy Markdown
Contributor Author

v-sapanchal commented Jun 16, 2025

Closing this PR, not needed, raised a new PR includes these changes

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.

8 participants