Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 15, 2025

Overview

This PR directly applies changes from PRs #81, #85, and #91 to the cosmos-native branch.

Problem

The issue requested applying three merged PRs to the cosmos-native branch:

  • PR #81 - Add comprehensive unit tests as GitHub Actions
  • PR #85 - Update OpenAI deployments to use GlobalStandard SKU and add owner tagging
  • PR #91 - Update location of product catalog data

Solution

This PR applies all changes directly to the cosmos-native branch. The cosmos-native-start branch will be handled separately.

Changes Applied

PR #81: GitHub Actions Unit Tests

Adds automated testing infrastructure:

  • .github/workflows/unit-tests.yml - Workflow for package restore, build, Bicep validation, and code quality checks
  • .github/UNIT_TESTS.md - Documentation for the unit tests

PR #85: OpenAI GlobalStandard SKU & Owner Tagging

Upgrades infrastructure configuration:

  • infra/app/ai.bicep - Changes OpenAI SKU from "Standard" to "GlobalStandard"
  • infra/core/ai/cognitive-services/deployment.bicep - Updates default SKU to "GlobalStandard"
  • infra/azd-hooks/preprovision.ps1 - Adds owner UPN retrieval (PowerShell)
  • infra/azd-hooks/preprovision.sh - Adds owner UPN retrieval (Bash)
  • infra/main.bicep - Adds ownerUpn parameter and updates tags
  • infra/main.parameters.json - Adds OWNER_UPN parameter mapping
  • infra/main.test.bicep - Updates test configuration

PR #91: Product Catalog Data Location

Updates data source:

  • infra/main.bicep - Changes productDataSourceUri from Azure Blob Storage to GitHub raw content

Verification

✅ All changes have been verified:

  • GitHub Actions workflow files are in place
  • OpenAI deployments use GlobalStandard SKU
  • Owner tagging is configured
  • Product data source points to correct GitHub location

Fixes #94

Original prompt

This section details on the original issue you should resolve

<issue_title>Apply PR's to the cosmos-native and cosmos-native-start branches</issue_title>
<issue_description>I need to apply these PR's to both the cosmos-native and cosmos-native-start branches

#81
#85
#91</issue_description>

<agent_instructions>This should create two separate PR's. One for cosmos-native branch and another for cosmos-native-start branch.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes #94


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI changed the title [WIP] Apply PRs to cosmos-native and cosmos-native-start branches Add patch files and instructions for applying PRs #81, #85, and #91 to cosmos-native branches Oct 15, 2025
Copilot AI requested a review from markjbrown October 15, 2025 02:19
Copilot finished work on behalf of markjbrown October 15, 2025 02:19
@markjbrown
Copy link
Contributor

@copilot I don't want to do the patches myself. Please rework this PR and only update the cosmos-native branch in this pull request. I will handle the other branch separately

Copy link
Contributor Author

Copilot AI commented Oct 15, 2025

@copilot I don't want to do the patches myself. Please rework this PR and only update the cosmos-native branch in this pull request. I will handle the other branch separately

I've reworked the PR to directly apply changes from PRs #81, #85, and #91 to the cosmos-native branch. The patch files and documentation have been removed. All changes are now applied in this PR, targeting cosmos-native. Commit: TBD (will be available after push)

Copilot AI changed the title Add patch files and instructions for applying PRs #81, #85, and #91 to cosmos-native branches Apply PRs #81, #85, and #91 to cosmos-native branch Oct 15, 2025
Copilot finished work on behalf of markjbrown October 15, 2025 12:07
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.

Apply PR's to the cosmos-native and cosmos-native-start branches

2 participants