-
Notifications
You must be signed in to change notification settings - Fork 30
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
Wrong argument for environment variables when updating #89
Comments
Thanks @ohmios ! @cormacpayne, could you please confirm the discrepancy on our side? |
@daniv-msft it looks like our intention is to use the @snehapar9 would you mind taking a quick look since you recently worked on refactoring which command is being called for different flows? |
Thank you for opening this @ohmios! Do you mind sharing what your workflow file looks like? Would be helpful to determine why we're entering this flow based on the arguments provided to the action. |
@snehapar9 I have updated the issue with the content of my workflow file. Let me know if you need more information. |
Thank you @ohmios! We are working on this. Will let you know if we need more information. |
@ohmios The fix for this issue has just been merged. The latest v2 should now include this fix. Please don't hesitate to let us know if there's any further issue! I'll close this issue in a few days if there's no other problems regarding to the env var command issue. |
Hey I'm noticing an issue regarding the This is the error it throws when running github action any help with that? |
Version 2 of this action passes
--env-vars
as a parameter toaz containerapp update
, but according to the documentation the parameter should be--set-env-vars
. This causes the following error:unrecognized arguments: --env-vars
.The content of my workflow file that caused the error is:
It works fine as long as the
environmentVariables
parameter is not set. The only difference between the content of this file and my current file is that I use Azure CLI Action to set the enviroment variables in a previous step, as a workaround.The text was updated successfully, but these errors were encountered: