Repository navigation
Add private Splunk MCP deployment templates - #349
Merged
Dalibor Kovacevic (RobiladK) merged 10 commits intoSep 24, 2026
Merged
Dalibor Kovacevic (RobiladK) merged 10 commits into
Dalibor Kovacevic (RobiladK) merged 10 commits into
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Integrate review fixes, harden package transfer, and add runtime regression coverage for the PowerShell deployment paths. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 85074578-9d57-4bab-934c-5ec8888ceeac
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 85074578-9d57-4bab-934c-5ec8888ceeac
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 85074578-9d57-4bab-934c-5ec8888ceeac
Scrub and verify deletion of token-bearing Managed Run Commands, preserve primary failures, add cleanup regression coverage, and clarify trusted HTTPS versus lab-only HTTP guidance. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 85074578-9d57-4bab-934c-5ec8888ceeac
Keep the public examples focused on reproducible deployment and testing guidance without dated environment-specific results or capacity rationale. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 85074578-9d57-4bab-934c-5ec8888ceeac
Replace the ASCII topology sketches with the Mermaid diagrams used in the pull request description. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 85074578-9d57-4bab-934c-5ec8888ceeac
Collaborator
Author
|
The failing check is repository-level, not caused by the Splunk templates. The default-branch workflow passes about 216 KB of changed-file JSON as one jq argument, which exceeds Linux's per-argument limit; the configured webhook also targets a direct SRE Agent endpoint instead of the required Logic App callback. I opened #350 with the tested file-backed payload fix and a fail-fast webhook diagnostic. Because this workflow uses pull_request_target, #350 must land on main and the repository secret must be updated to the Logic App callback before #349 can pass; then synchronizing this branch will rerun the corrected check. |
Dalibor Kovacevic (RobiladK)
approved these changes
Sep 24, 2026
Dalibor Kovacevic (RobiladK)
merged commit Sep 24, 2026
25a4230
into
microsoft:main
2 of 5 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Architecture
Cross-region
flowchart LR subgraph AgentRegion[Agent region] Agent[SRE Agent] AgentSubnet[Delegated agent subnet<br/>Microsoft.App/environments] AgentVNet[Agent VNet] Agent --> AgentSubnet AgentSubnet --- AgentVNet end subgraph SplunkRegion[Splunk region] SplunkVNet[Splunk VNet] SplunkSubnet[Private Splunk subnet] SplunkVM[Splunk VM<br/>No public IP] NAT[NAT Gateway<br/>Outbound bootstrap only] SplunkVNet --- SplunkSubnet SplunkSubnet --> SplunkVM SplunkSubnet -->|Outbound package and image access| NAT end AgentVNet <-->|Global VNet Peering<br/>Private MCP traffic| SplunkVNet DNS[Private DNS zone<br/>Splunk hostname to private IP] DNS -. VNet link .-> AgentVNet DNS -. VNet link .-> SplunkVNetThe SRE Agent reaches the private Splunk MCP endpoint across Global VNet Peering; private DNS is linked to both VNets.
Same-region
flowchart LR subgraph Region[Shared Azure region] subgraph SharedVNet[Shared VNet] Agent[SRE Agent] AgentSubnet[Delegated agent subnet<br/>Microsoft.App/environments] SplunkSubnet[Private Splunk subnet] SplunkVM[Splunk VM<br/>No public IP] Agent --> AgentSubnet AgentSubnet -->|Private MCP traffic| SplunkSubnet SplunkSubnet --> SplunkVM end NAT[NAT Gateway<br/>Outbound bootstrap only] SplunkSubnet -->|Outbound package and image access| NAT end DNS[Private DNS zone<br/>Splunk hostname to private IP] DNS -. VNet link .-> SharedVNetThe SRE Agent and Splunk use isolated subnets in one VNet, so no VNet peering is required.
Validation
git diff --checkpassedThe aggregate repository test runner was also attempted; unrelated existing tests require
jqand have Windows runner failures outside these examples. The two new targeted suites pass.