Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[release/8.0] Change starter template to make frontend external endpoints #3571

Merged
merged 2 commits into from
Apr 11, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Apr 10, 2024

Backport of #3514 to release/8.0

/cc @eerhardt @timheuer

Customer Impact

When devs use the starter template and deploy it using azd, the frontend web app end isn't accessible. So they can't see their app running.

Testing

Manual validation of:

  • deployment manifest marks service as external
  • ACA deployment succeeds with appropriate ingress set on the service

Risk

Low. Just adds the WithExternalHttpEndpoints call to the template.

Regression?

Yes. Before the endpoint refactoring, azd would prompt which endpoints to make external. Now it doesn't prompt, and the web app isn't accessible after publishing.

Microsoft Reviewers: Open in CodeFlow

@dotnet-policy-service dotnet-policy-service bot added the Servicing-consider Issue for next servicing release review label Apr 10, 2024
@eerhardt eerhardt requested a review from davidfowl April 10, 2024 21:25
@danmoseley danmoseley added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Apr 10, 2024
@danmoseley danmoseley enabled auto-merge (squash) April 10, 2024 21:51
@danmoseley
Copy link
Member

@eerhardt @timheuer build break
src\Aspire.Hosting\Dcp\ApplicationExecutor.cs(1034,35): error CS0103: (NETCORE_ENGINEERING_TELEMETRY=Build) The name 'CommandLineArgsParser' does not exist in the current context

@eerhardt
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@eerhardt
Copy link
Member

eerhardt commented Apr 10, 2024

@eerhardt @timheuer build break src\Aspire.Hosting\Dcp\ApplicationExecutor.cs(1034,35): error CS0103: (NETCORE_ENGINEERING_TELEMETRY=Build) The name 'CommandLineArgsParser' does not exist in the current context

Fixed with #3575

@danmoseley danmoseley merged commit d7b8bed into release/8.0 Apr 11, 2024
7 checks passed
@danmoseley danmoseley deleted the backport/pr-3514-to-release/8.0 branch April 11, 2024 01:22
@danmoseley danmoseley mentioned this pull request Apr 12, 2024
@github-actions github-actions bot locked and limited conversation to collaborators May 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-templates Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants