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

azureedge.net sample return values #1796

Open
felixfontein opened this issue Jan 3, 2025 · 5 comments
Open

azureedge.net sample return values #1796

felixfontein opened this issue Jan 3, 2025 · 5 comments
Labels
hold The problem holds for a particular reason medium_priority Medium priority question Further information is requested work in In trying to solve, or in working with contributors

Comments

@felixfontein
Copy link
Contributor

SUMMARY

The azure_rm_cdnendpoint and azure_rm_afdendpoint_info modules mention the azureedge.net domain in return value samples.

A company called Edgio have gone bankrupt and the azureedge.net CDN will be going away on 15th January 2025. I don't know how these sample return values are related to that company, but it might make sense to update them.

Ref: ansible-community/ansible-build-data#512

ISSUE TYPE
  • Bug Report
COMPONENT NAME

azure_rm_cdnendpoint
azure_rm_afdendpoint_info

@Fred-sun
Copy link
Collaborator

Fred-sun commented Jan 6, 2025

@felixfontein Thank you for submitting the problems encountered. I don't quite understand your idea. Do you want to update host_name(azureedge.net) through this module? Thank you!

@Fred-sun Fred-sun added question Further information is requested medium_priority Medium priority work in In trying to solve, or in working with contributors labels Jan 6, 2025
@felixfontein
Copy link
Contributor Author

I mainly forwarded the issue from ansible-community/ansible-build-data#512.

Check out the links posted there, like:

Keeping references to a domain that could suddenly contain things totally unrelated to Azure sounds like a bad idea to me.

@Fred-sun
Copy link
Collaborator

@felixfontein I am very sorry for replying you so late. I have checked the latest version of the azure-mgmt-cdn package, and the parameter host_name is the default, which cannot be configured (azureedge.net domain name is the default), thank you!

@felixfontein
Copy link
Contributor Author

If you think this should be kept then feel free to close the issue. I'm not using Azure at all (except AZP for some collections), I was just passing the warning on.

@Fred-sun
Copy link
Collaborator

@felixfontein Yes, the question you raised is necessary, and I will continue to follow up on this issue. Once the API update, I will immediately use azure.azcollection. Thank you!

@Fred-sun Fred-sun added the hold The problem holds for a particular reason label Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold The problem holds for a particular reason medium_priority Medium priority question Further information is requested work in In trying to solve, or in working with contributors
Projects
None yet
Development

No branches or pull requests

2 participants