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

containerapp compose create is not respecting the --location argument #5252

Closed
charris-msft opened this issue Aug 18, 2022 · 11 comments
Closed
Assignees
Labels
Auto-Assign Auto assign by bot ContainerApp CXP Attention This issue is handled by CXP team. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that

Comments

@charris-msft
Copy link

Related command

az containerapp compose create

Extension name (the extension in question)

containerapp-compose

Description of issue (in as much detail as possible)

When running the command and specifying a location via -l or --location it ignores the specified location and defaults to 'westus3' which "is not currently supported".

image

That value is not coming from my param-persist either:
image


@ghost ghost added Auto-Assign Auto assign by bot ContainerApp CXP Attention This issue is handled by CXP team. labels Aug 18, 2022
@yonzhan
Copy link
Collaborator

yonzhan commented Aug 18, 2022

route to CXP team

@smurawski
Copy link
Contributor

@charris-msft What versions of the extensions (containerapp and containerapp-compose) do you have installed?

@charris-msft
Copy link
Author

@smurawski

Name                  Path                                                        Preview    Version
--------------------  ----------------------------------------------------------  ---------  ---------
containerapp          C:\Users\charris\.azure\cliextensions\containerapp          True       0.3.9
containerapp-compose  C:\Users\charris\.azure\cliextensions\containerapp-compose  True       0.2.0

@smurawski
Copy link
Contributor

@charris-msft

I'm finding another bug in the 0.3.9 that has blocked me from testing so far ( #5253 ) , but I was able to deploy successfully to Canada Central with the containerapp extension at 0.3.8 and 0.3.7.

I'll continue to check with 0.3.9 with a private image shortly.

@smurawski
Copy link
Contributor

@charris-msft

I've confirmed your problem - it appears that the resource group location is overriding command line provided location arguments - which appears to be happening regardless of containerapp extension version. I'll investigate further.

@charris-msft
Copy link
Author

@smurawski I tested by moving my existing resources to a new rg in the location where I wanted to create the new containerapp, ran the command and it didn't error out this time. (still running)

@RakeshMohanMSFT RakeshMohanMSFT self-assigned this Aug 19, 2022
@smurawski
Copy link
Contributor

This is fixed in #5262 (version 0.2.1) which is available now.

@charris-msft
Copy link
Author

Looks like it's working - it created the ACA Environment in the correct location.

Though because I'm still getting prompted my registry credentials (which I don't know), I can't confirm that the ACA gets created in the correct location as well.

@smurawski
Copy link
Contributor

Thanks @charris-msft ! The registry credentials prompting should be fixed once v0.3.10 of the containerapp extension ships.

The PR for that is here - #5241

@RakeshMohanMSFT
Copy link
Contributor

@charris-msft and @smurawski can we close this issue, i see that the PR is merged.

@charris-msft
Copy link
Author

I did some more testing and I'm comfortable closing it.

@RakeshMohanMSFT RakeshMohanMSFT added the question The issue doesn't require a change to the product in order to be resolved. Most issues start as that label Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Auto-Assign Auto assign by bot ContainerApp CXP Attention This issue is handled by CXP team. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Projects
None yet
Development

No branches or pull requests

4 participants