Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Sep 12, 2025

This PR modernizes the azure.yaml configuration to be compatible with Azure Developer CLI (azd) version 1.17.0 and later by removing the deprecated pipeline: variables section.

Changes Made

  1. Updated template version from 0.2.0 to 0.4.0 to reflect the configuration changes
  2. Updated azd version requirement from ">= 1.10.0" to ">= 1.17.0"
  3. Removed the entire pipeline: variables section containing 14 environment variables
  4. Fixed YAML indentation to be consistent (2 spaces throughout)

Background

As of azd version 1.17.0, the pipeline: variables section in azure.yaml is no longer needed. The Azure Developer CLI now automatically detects and configures pipeline variables based on the parameters defined in infra/main.parameters.json.

All 14 variables that were previously listed in the pipeline section are already properly defined in main.parameters.json:

  • AZURE_OPENAI_CHAT_DEPLOYMENT, AZURE_OPENAI_CHAT_MODEL, AZURE_OPENAI_CHAT_MODEL_VERSION
  • AZURE_OPENAI_RESOURCE, AZURE_OPENAI_RESOURCE_GROUP, AZURE_OPENAI_RESOURCE_LOCATION
  • AZURE_OPENAI_DEPLOYMENT_CAPACITY, AZURE_OPENAI_DEPLOYMENT_SKU_NAME, AZURE_OPENAI_SKU_NAME
  • AZURE_OPENAI_API_VERSION, CREATE_AZURE_OPENAI, AZURE_OPENAI_ENDPOINT
  • SERVICE_ACA_RESOURCE_EXISTS, DISABLE_KEY_BASED_AUTH

This change removes redundant configuration while maintaining full functionality with modern azd versions.

Fixes #327.


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

Copy link
Contributor Author

Copilot AI commented Sep 12, 2025

@pamelafox 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

@Copilot Copilot AI changed the title [WIP] Remove the pipeline: variables: section from azure.yaml Remove pipeline variables section from azure.yaml for azd 1.17.0+ compatibility Sep 12, 2025
@Copilot Copilot AI requested a review from pamelafox September 12, 2025 05:49
Copilot finished work on behalf of pamelafox September 12, 2025 05:49
azure.yaml Outdated
name: azure-openai-chat-app-quickstart
metadata:
template: azure-openai-chat-app-quickstart@0.2.0
template: azure-openai-chat-app-quickstart@0.3.0
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot Change this to 0.4.0 please

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to 0.4.0 in commit 4027fef

@Copilot Copilot AI requested a review from pamelafox September 18, 2025 22:44
Copilot finished work on behalf of pamelafox September 18, 2025 22:44
@pamelafox pamelafox marked this pull request as ready for review September 18, 2025 23:38
@pamelafox pamelafox merged commit d2a9e25 into main Sep 18, 2025
5 checks passed
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.

Remove the pipeline: variables: section from azure.yaml
2 participants