Added aspire sample #2977

JerryNixon · 2025-11-15T05:50:13Z

This is a simple sample that demonstrates how to use DAB in Aspire.

I will add it to our docs/quick-starts when this is merged.

Please note there is no /src code change, just /samples.

Copilot

Pull Request Overview

This PR adds a sample demonstrating how to use Data API Builder (DAB) in .NET Aspire. The sample includes a Star Trek database schema with actors, characters, series, and species data, along with the necessary configuration files to run DAB within an Aspire orchestration environment.

Key changes:

Aspire orchestration setup with SQL Server, Data API Builder, SQL Commander, and MCP Inspector
Star Trek database schema with comprehensive seed data covering multiple series
DAB configuration with entities, relationships, views, and stored procedures

Reviewed Changes

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

Show a summary per file

File	Description
samples/aspire/readme.md	Documentation explaining prerequisites, resources, and data structures
samples/aspire/global.json	.NET SDK version configuration
samples/aspire/database.sql	SQL schema and seed data for Star Trek database
samples/aspire/dab-config.json	DAB configuration with entities and relationships
samples/aspire/dab-config.cmd	CLI commands to generate DAB configuration
samples/aspire/apphost.cs	Aspire orchestration configuration
samples/aspire/.gitignore	Git ignore configuration
samples/aspire/.aspire/settings.json	Aspire settings

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

samples/aspire/readme.md

samples/aspire/apphost.cs

JerryNixon · 2025-11-15T05:54:26Z

We really need to add this to our tests in order to avoid requiring the full suite of integration tests when /samples is updated. In fact, there are many paths in this repo that should be set up to allow pull requests to skip integration tests. Please consider.

on:
  pull_request:
    paths-ignore:
      - 'samples/**'

Co-authored-by: Copilot <[email protected]>

Aniruddh25 · 2025-11-17T05:34:15Z

/azp run

azure-pipelines · 2025-11-17T05:34:39Z

Azure Pipelines successfully started running 6 pipeline(s).

samples/aspire/apphost.cs

Aniruddh25 · 2025-11-20T06:46:30Z

/azp run

azure-pipelines · 2025-11-20T06:46:49Z

Azure Pipelines successfully started running 6 pipeline(s).

samples/aspire/apphost.cs

samples/aspire/global.json

aaronburtle

Looks good just a couple nits.

Copilot · 2025-11-22T15:30:16Z

@Aniruddh25 I've opened a new pull request, #2990, to work on those changes. Once the pull request is ready, I'll request review from you.

Addresses feedback on PR #2977 to add the standard Microsoft copyright header to the Aspire sample's C# file. ## Changes - Added copyright notice to `samples/aspire/apphost.cs`: ```csharp // Copyright (c) Microsoft Corporation. // Licensed under the MIT License. ```  --- ✨ Let Copilot coding agent [set things up for you](https://github.com/Azure/data-api-builder/issues/new?title=✨+Set+up+Copilot+instructions&body=Configure%20instructions%20for%20this%20repository%20as%20documented%20in%20%5BBest%20practices%20for%20Copilot%20coding%20agent%20in%20your%20repository%5D%28https://gh.io/copilot-coding-agent-tips%29%2E%0A%0A%3COnboard%20this%20repo%3E&assignees=copilot) — coding agent works faster and does higher quality work when set up for your repo. --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: Aniruddh25 <[email protected]>

Aniruddh25 · 2025-11-22T15:42:34Z

/azp run

azure-pipelines · 2025-11-22T15:42:56Z

Azure Pipelines successfully started running 6 pipeline(s).

@JerryNixon

…tes) (#2988) ## Why make this change? Closes #2977. Running full integration test suites on PRs that only modify samples, documentation, or configuration files wastes CI resources and slows down contribution velocity. ## What is this change? Added `pr:` trigger sections with path exclusions to all 7 Azure Pipeline definitions in `.pipelines/`: **Excluded paths:** - `samples/**` - Sample configurations and code - `docs/**` - Documentation - `*.md` - Markdown files (README, CONTRIBUTING, etc.) - `schemas/**` - JSON schema definitions - `templates/**` - Project templates **Modified pipelines:** - `azure-pipelines.yml` - Static analysis - `cosmos-pipelines.yml` - CosmosDB integration tests - `dwsql-pipelines.yml` - Data Warehouse SQL tests - `mssql-pipelines.yml` - SQL Server tests - `mysql-pipelines.yml` - MySQL tests - `pg-pipelines.yml` - PostgreSQL tests - `unittest-pipelines.yml` - Unit test suite PRs touching only excluded paths will skip pipeline execution. PRs with mixed changes (code + excluded paths) still run all validations. ## How was this tested? - [x] YAML syntax validation - [ ] Integration Tests - N/A (configuration change) - [ ] Unit Tests - N/A (configuration change) ## Sample Request(s) N/A - Pipeline configuration change only. Validation occurs automatically on PR creation.  <details> <summary>Original prompt</summary> > > ---- > > *This section details on the original issue you should resolve* > > <issue_title>Skip Pull Request validation for certain paths</issue_title> > <issue_description>We really need to add this to our tests in order to avoid requiring the full suite of integration tests when /samples is updated. In fact, there are many paths in this repo that should be set up to allow pull requests to skip integration tests. Please consider. > > ```yaml > on: > pull_request: > paths-ignore: > - 'samples/**' > ``` > > _Originally posted by @JerryNixon in #2977 (comment) > </issue_description> > > <agent_instructions>Modify the yml files in the .pipelines folder which do Pull Request validation to skip paths like samples. This will help checkin PRs on those paths quickly.</agent_instructions> > > ## Comments on the Issue (you are @copilot in this section) > > <comments> > </comments> > </details> - Fixes #2987  --- 💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more [Copilot coding agent tips](https://gh.io/copilot-coding-agent-tips) in the docs. --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: Aniruddh25 <[email protected]>

Aniruddh25 · 2025-11-25T02:18:45Z

/azp run

azure-pipelines · 2025-11-25T02:18:56Z

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

aaronburtle

Re-approving to merge

Added aspire sample

fd10804

Copilot AI review requested due to automatic review settings November 15, 2025 05:50

JerryNixon requested review from Alekhya-Polavarapu, Aniruddh25, RubenCerna2079, aaronburtle, anushakolan, neeraj-sharma2592, rusamant, sourabh1007, souvikghosh04 and vadeveka as code owners November 15, 2025 05:50

Copilot started reviewing on behalf of JerryNixon November 15, 2025 05:50 View session

Updated for typo

8dc08bf

Copilot finished reviewing on behalf of JerryNixon November 15, 2025 05:51

JerryNixon enabled auto-merge (squash) November 15, 2025 05:51

Copilot AI reviewed Nov 15, 2025

View reviewed changes

samples/aspire/readme.md Outdated Show resolved Hide resolved

samples/aspire/apphost.cs Outdated Show resolved Hide resolved

JerryNixon and others added 4 commits November 14, 2025 22:54

Update samples/aspire/apphost.cs

e3e05a1

Co-authored-by: Copilot <[email protected]>

Update readme.md

2993022

Updated with demo options for future changes

d32e3e3

Merge complete

be2b5d1

Aniruddh25 reviewed Nov 17, 2025

View reviewed changes

samples/aspire/apphost.cs Outdated Show resolved Hide resolved

Removed password

cdb5eeb

Aniruddh25 approved these changes Nov 20, 2025

View reviewed changes

Merge branch 'main' into jerry-sample-aspire

53fb46b

Aniruddh25 mentioned this pull request Nov 20, 2025

Skip Pull Request validation for certain paths #2987

Closed

Copilot AI mentioned this pull request Nov 20, 2025

Skip PR validation for non-code paths (samples, docs, templates) #2988

Merged

3 tasks

aaronburtle reviewed Nov 21, 2025

View reviewed changes

samples/aspire/apphost.cs Show resolved Hide resolved

aaronburtle reviewed Nov 21, 2025

View reviewed changes

samples/aspire/global.json Show resolved Hide resolved

aaronburtle approved these changes Nov 21, 2025

View reviewed changes

Copilot AI mentioned this pull request Nov 22, 2025

Add copyright header to apphost.cs #2990

Merged

Aniruddh25 approved these changes Nov 22, 2025

View reviewed changes

souvikghosh04 closed this in #2988 Nov 24, 2025

auto-merge was automatically disabled November 24, 2025 02:15
Pull request was closed

Aniruddh25 reopened this Nov 25, 2025

Merge branch 'main' into jerry-sample-aspire

d580f7b

Aniruddh25 enabled auto-merge (squash) November 25, 2025 02:18

aaronburtle approved these changes Nov 25, 2025

View reviewed changes

Aniruddh25 merged commit 08ca05f into main Nov 25, 2025
7 checks passed

Aniruddh25 deleted the jerry-sample-aspire branch November 25, 2025 03:16

Added aspire sample #2977

Added aspire sample #2977

Uh oh!

Conversation

JerryNixon commented Nov 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

JerryNixon commented Nov 15, 2025

Uh oh!

Aniruddh25 commented Nov 17, 2025

Uh oh!

azure-pipelines bot commented Nov 17, 2025

Uh oh!

Uh oh!

Aniruddh25 commented Nov 20, 2025

Uh oh!

azure-pipelines bot commented Nov 20, 2025

Uh oh!

Uh oh!

Uh oh!

aaronburtle left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Nov 22, 2025

Uh oh!

Aniruddh25 commented Nov 22, 2025

Uh oh!

azure-pipelines bot commented Nov 22, 2025

Uh oh!

Aniruddh25 commented Nov 25, 2025

Uh oh!

azure-pipelines bot commented Nov 25, 2025

Uh oh!

aaronburtle left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

JerryNixon commented Nov 15, 2025 •

edited

Loading