Azure / aci-deploy

Enable GitHub developers to deploy to Azure Container Instances using GitHub Actions
MIT License
63 stars 60 forks source link

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

Open cilerler opened 3 years ago

cilerler commented 3 years ago

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
kanika1894 commented 3 years ago

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.

cilerler commented 3 years ago

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.

kanika1894 commented 3 years ago

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.

github-actions[bot] commented 3 years ago

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

zmckinnon commented 2 years ago

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?

github-actions[bot] commented 2 years ago

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

BALAGA-GAYATRI commented 2 years ago

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

github-actions[bot] commented 2 years ago

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

davemaul commented 2 years ago

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 commented 2 years ago

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

github-actions[bot] commented 2 years ago

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

XeniyaFesenko commented 2 months ago

any updates on this issue?