Skip to content

[WIP] Convert the bicep code to use Azure Verified Modules and Function code to .NET 6.0 Isolated#6

Draft
Copilot wants to merge 10 commits into
mainfrom
copilot/fix-5
Draft

[WIP] Convert the bicep code to use Azure Verified Modules and Function code to .NET 6.0 Isolated#6
Copilot wants to merge 10 commits into
mainfrom
copilot/fix-5

Conversation

Copy link
Copy Markdown

Copilot AI commented May 22, 2025

  • Research Azure Verified Modules compatible with our infrastructure
  • Convert bicep files to use Azure Verified Modules
    • Update appserviceplan.bicep
    • Update storage-account.bicep
    • Update functions.bicep
    • Update loganalytics.bicep
    • Update applicationinsights.bicep
  • Convert .NET project to use isolated process model
    • Update project file dependencies
    • Create Program.cs to replace Startup.cs
    • Update function signatures to use isolated model
    • Update PowerAppsFunction.cs to use isolated model
    • Update WebAPIGetExample.cs to use isolated model namespaces
  • Update documentation to reflect new technologies
    • Add .NET 6.0 Isolated process model information to README
    • Add Azure Verified Modules information to README

Fixes #5.


💡 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 in the docs.

Copilot AI and others added 3 commits May 22, 2025 07:46
Co-authored-by: rajyraman <5035266+rajyraman@users.noreply.github.com>
Co-authored-by: rajyraman <5035266+rajyraman@users.noreply.github.com>
Co-authored-by: rajyraman <5035266+rajyraman@users.noreply.github.com>
Copy link
Copy Markdown
Owner

@rajyraman rajyraman left a comment

Choose a reason for hiding this comment

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

Update to .NET 8

Copilot AI and others added 2 commits May 22, 2025 08:28
Co-authored-by: rajyraman <5035266+rajyraman@users.noreply.github.com>
Co-authored-by: rajyraman <5035266+rajyraman@users.noreply.github.com>
Copy link
Copy Markdown
Author

Copilot AI commented May 22, 2025

Update to .NET 8

Updated the project to .NET 8.0. Changes include updating the target framework, package references, removing Startup.cs (not needed in isolated model), updating OpenAPI namespaces, and updating the README with .NET 8.0 information. Build is now successful with no errors. Commit 415dbbc

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • cdn.fwupd.org
    • Triggering command: /usr/bin/fwupdmgr refresh (dns block)
  • crl3.digicert.com
    • Triggering command: dotnet build (dns block)
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)
  • crl4.digicert.com
    • Triggering command: dotnet build (dns block)
  • ocsp.digicert.com
    • Triggering command: dotnet build (dns block)
  • s.symcb.com
    • Triggering command: dotnet build (dns block)
  • s.symcd.com
    • Triggering command: dotnet build (dns block)
  • ts-crl.ws.symantec.com
    • Triggering command: dotnet build (dns block)
  • ts-ocsp.ws.symantec.com
    • Triggering command: dotnet build (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Comment thread infra/core/host/functions.bicep Outdated
use32BitWorkerProcess: use32BitWorkerProcess
subnetId: subnetId
serverFarmId: appServicePlanId
systemAssignedIdentity: true
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Use user assigned managed identity

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Implemented user-assigned managed identity for the function app. Created a new managed-identity.bicep module, updated functions.bicep to replace system-assigned identity with user-assigned identity, and modified main.bicep to reference and output the identity. Commit ade46ee

Co-authored-by: rajyraman <5035266+rajyraman@users.noreply.github.com>
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.

Convert the bicep code to use Azure Verified Modules and Function code to .NET 6.0 Isolated

2 participants