aci-deploy icon indicating copy to clipboard operation
aci-deploy copied to clipboard

Setting `ip-address` to `Private` causes an error

Open cilerler opened this issue 3 years ago • 12 comments

Setting below

    - name: 'Deploy'
      uses: azure/aci-deploy@v1
      with:
        resource-group: ${{ secrets.AZURE_RESOURCE_GROUP }}
        registry-login-server: ${{ secrets.AZURE_REGISTRY_LOGIN_SERVER }}
        registry-username: ${{ secrets.AZURE_REGISTRY_USERNAME }}
        registry-password: ${{ secrets.AZURE_REGISTRY_PASSWORD }}
        name: $DEPLOYMENT_NAME
        image: $IMAGE:$GITHUB_SHA
        location: 'south central us'
        os-type: Windows
        cpu: 4
        memory: 16
        environment-variables: DOTNET_ENVIRONMENT=Production
        restart-policy: OnFailure
        dns-name-label: $DEPLOYMENT_NAME
        ip-address: Private

throws the error below

Error: DNS name label for container group is only supported when IP Address type is public.

and removing dns-name-label throws the error below

Error: Input required and not supplied: dns-name-label

cilerler avatar Jul 11 '21 19:07 cilerler

Hey @cilerler, Thanks for pointing this out. As of now, this action only supports Public IP scenario. Updated : https://github.com/Azure/aci-deploy/blob/master/action.yml#L69 Supporting Private IP will be taken up as an enhancement.

kanika1894 avatar Jul 22 '21 07:07 kanika1894

Are you sure that you don't want to mark it as "Bug" since it allows you to set a value, but then it throws an error? I mean, it literally throws an error if you set Private; to me, it means a bug and not an enhancement.

cilerler avatar Jul 22 '21 10:07 cilerler

Since we are calling out explicitly in the action ReadMe as well as action.yml that Private IP scenario is not supported, we'll be taking this as an enhancement.

kanika1894 avatar Jul 22 '21 11:07 kanika1894

This issue is idle because it has been open for 14 days with no activity.

github-actions[bot] avatar Aug 05 '21 12:08 github-actions[bot]

I am also having this issue. Is there some reason why Private IP addresses is not supported? Any idea on the timeline on this? Are y'all accepting PRs?

zmckinnon avatar Nov 22 '21 22:11 zmckinnon

This issue is idle because it has been open for 14 days with no activity.

github-actions[bot] avatar Dec 07 '21 00:12 github-actions[bot]

@zmckinnon We are accepting PRs. We will be glad to accept contributions.

BALAGA-GAYATRI avatar Dec 16 '21 12:12 BALAGA-GAYATRI

This issue is idle because it has been open for 14 days with no activity.

github-actions[bot] avatar Dec 30 '21 18:12 github-actions[bot]

Is there a reason for this restriction? In Azure Portal it's absolutely possible to create a container instance without a dns label name and a private IP address. I'm curious. :)

davemaul avatar Jan 12 '22 13:01 davemaul

Tested it. There's no reason. Sent a PR. :)

davemaul avatar Jan 13 '22 13:01 davemaul

This issue is idle because it has been open for 14 days with no activity.

github-actions[bot] avatar Jan 27 '22 15:01 github-actions[bot]

any updates on this issue?

XeniyaFesenko avatar Aug 09 '24 15:08 XeniyaFesenko