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

Add support for exposed_port and additional ports in ingress. #71

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

lonegunmanb
Copy link
Member

Describe your changes

Issue number

#69 #57

Checklist before requesting a review

  • The pr title can be used to describe what this pr did in CHANGELOG.md file
  • I have executed pre-commit on my machine
  • I have passed pr-check on my machine

Thanks for your cooperation!

@lonegunmanb lonegunmanb changed the title Add support for target_port and additional ports in ingress. Add support for exposed_port and additional ports in ingress. Nov 18, 2024
@lonegunmanb
Copy link
Member Author

This pr must be halt since we'll have issue if we use azapi_update_resource to manage additional ports, if we have secret defined in azurerm resource and we don't include them in azapi_update_resource, API would return error, complaining that we have passed incorrect values to secrets.

I'll open some pull requests to azurerm provider to bump API version, then bring these arguments into the resource schema.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

1 participant