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

azd infra synth should put yaml files for container apps under infra folder #3409

Closed
davidfowl opened this issue Feb 21, 2024 · 7 comments
Closed
Assignees
Labels
Milestone

Comments

@davidfowl
Copy link
Member

All of the infrastructure should go into a single root folder instead of each project folder. Make it easier to look at and delete.

@ellismg
Copy link
Member

ellismg commented Feb 22, 2024

I think that if we model the AppHost as the thing that provides the infrastructure (in a more first class way than we do today, where the ImportManager basically removes the app host from the project graph and replaces it with the individual services and the infra folder this will more or less all fall out.

@davidfowl
Copy link
Member Author

Agreed, that would be ideal. Are there any side effects of doing that? We still want to deploy individual services.

@rajeshkamal5050 rajeshkamal5050 modified the milestones: March 2024, April 2024 Mar 4, 2024
@rajeshkamal5050
Copy link

I think that if we model the AppHost as the thing that provides the infrastructure (in a more first class way than we do today, where the ImportManager basically removes the app host from the project graph and replaces it with the individual services and the infra folder this will more or less all fall out.

@davidfowl @vhvb1989 @mitchdenny @ellismg moving it as a fit-n-finish item post Preview 5. Please let me know if you think otherwise.

@davidfowl
Copy link
Member Author

Can we get ths for P5?

@rajeshkamal5050
Copy link

@ellismg can we get this in for Preview 5 i.e., Apr 09 azd release?

@rajeshkamal5050
Copy link

rajeshkamal5050 commented Apr 2, 2024

Not a blocker for Preview 5 moving to next milestones.

cc @davidfowl @mitchdenny @ellismg @vhvb1989

@weikanglim
Copy link
Contributor

Added with #3800, closing out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants