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

Resource routeros_ip_dns missing address_list_extra_time fields #270

Closed
jlpedrosa opened this issue Oct 1, 2023 · 3 comments · Fixed by #303
Closed

Resource routeros_ip_dns missing address_list_extra_time fields #270

jlpedrosa opened this issue Oct 1, 2023 · 3 comments · Fixed by #303
Labels
bug Something isn't working duplicate This issue or pull request already exists released

Comments

@jlpedrosa
Copy link
Contributor

Describe the bug
resource routeros_ip_dns is missing address_list_extra_time field, resulting in warning:

Terraform has compared your real infrastructure against your configuration and found no differences, so no changes are needed.
╷
│ Warning: Field 'address_list_extra_time' not found in the schema
│ 
│   with routeros_ip_dns.dns_settings,
│   on dhcp.tf line 167, in resource "routeros_ip_dns" "dns_settings":
│  167: resource "routeros_ip_dns" "dns_settings" {
│ 
│ [MikrotikResourceDataToTerraform] The field was lost during the Schema development: ▷ 'address_list_extra_time': '0s' ◁

To Reproduce
Apply and re-apply this TF

resource "routeros_ip_dns" "dns_settings" {
  servers               = "8.8.8.8"
  allow_remote_requests = true
}

Expected behavior
No warnings.

Stack Trace
If applicable, add the stack trace the crash produced.

Additional context
Add any other context about the problem here.

@jlpedrosa jlpedrosa added the bug Something isn't working label Oct 1, 2023
@jlpedrosa
Copy link
Contributor Author

@vaerh I can create a PR to fix it if you're ok with it.

@vaerh
Copy link
Collaborator

vaerh commented Oct 1, 2023

I think this problem is due to a field change in the current version of ROS. We leave this query open for now, as I have started to prepare the provider for schema versioning, but this process will be long.
#230

@vaerh vaerh added the duplicate This issue or pull request already exists label Oct 1, 2023
@vaerh
Copy link
Collaborator

vaerh commented Nov 27, 2023

🎉 This issue has been resolved in version 1.25.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@vaerh vaerh added the released label Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants