restart-policy only accepts "Always" value about aci-deploy HOT 12 CLOSED

@iluxa86 certainly pointed in the right direct - pull request #38 fixes this, which on our fork is working fine.

from aci-deploy.

this line should be changed to
if(restartPolicy not in ("Always", "OnFailure", "Never")) {

But I do not have permissions to submit PR

from aci-deploy.

I did not realize it is TS code.
The code should be:
if(!["Always", "OnFailure", "Never"].includes(restartPolicy)) {

from aci-deploy.

@narula0781

from aci-deploy.

We've also met this - a real pain. Seems @iluxa86 has given the fix three months ago...

from aci-deploy.

when is it resolved?
restart policy bug is still alive.

from aci-deploy.

@huinalam I've given up on this. I met another, similar bug, and concluded there must be a better way. And there is - just use https://github.com/azure/cli directly, and then you can do whatever the CLI allows you too, and you end up with a better understanding of what's going on, much more power over it - and, it works.

from aci-deploy.

If I am not wrong, #43 this solves the issue, right?
@iluxa86 @alexlydiate

from aci-deploy.

It appears this has been fixed in master, but we do not have a new release cut. What's the ETA on getting a new tag with this change?

from aci-deploy.

This issue is marked need-to-triage for generating issues report.

from aci-deploy.

@jseadragon @iluxa86 @alexlydiate
This has been fixed in the patch version. We'll soon update the major version as well.
Can you please verify if using [email protected] solves the issue and works well?

from aci-deploy.

Fixed in v1 as well.

from aci-deploy.