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

Setting ip-address to Private causes an error #53

Open
cilerler opened this issue Jul 11, 2021 · 11 comments · May be fixed by #69
Open

Setting ip-address to Private causes an error #53

cilerler opened this issue Jul 11, 2021 · 11 comments · May be fixed by #69
Labels
enhancement New feature or request idle

Comments

@cilerler
Copy link

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
@BALAGA-GAYATRI BALAGA-GAYATRI added question Further information is requested and removed need-to-triage labels Jul 13, 2021
@kanika1894 kanika1894 added bug Something isn't working P0 Action not working and removed question Further information is requested labels Jul 19, 2021
@kanika1894
Copy link
Contributor

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 kanika1894 added enhancement New feature or request and removed P0 Action not working bug Something isn't working labels Jul 22, 2021
@cilerler
Copy link
Author

cilerler commented Jul 22, 2021

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
Copy link
Contributor

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
Copy link

github-actions bot commented Aug 5, 2021

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

@github-actions github-actions bot added the idle label Aug 5, 2021
@zmckinnon
Copy link

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 github-actions bot removed the idle label Nov 23, 2021
@github-actions
Copy link

github-actions bot commented Dec 7, 2021

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

@github-actions github-actions bot added the idle label Dec 7, 2021
@BALAGA-GAYATRI
Copy link
Contributor

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

@github-actions github-actions bot removed the idle label Dec 16, 2021
@github-actions
Copy link

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

@github-actions github-actions bot added the idle label Dec 30, 2021
@davemaul
Copy link

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
Copy link

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

@github-actions
Copy link

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

@github-actions github-actions bot added the idle label Jan 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request idle
Projects
None yet
5 participants