feat: handled dns test with skip validation

[Hardikl]

No description provided.

[Copilot]

Pull request overview

Adds support for ONTAP DNS configuration creation with optional “skip config validation”, and updates REST/client + integration tests to cover the new behavior.

Changes:

• Extend DNS tool/API models with skip_config_validation and make DNS domains/servers optional in tool schema.
• Wire the new flag through server → ontap model mapping and adjust DNS REST calls’ response handling.
• Update integration tests for DNS/NFS error expectations and expand DNS test coverage (success with skip validation, failure without it).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
tool/tool.go	Adds SkipConfigValidation to DNS tool params; makes domains/servers optional via omitzero.
server/dns.go	Maps SkipConfigValidation into the ONTAP DNS config request.
rest/dns.go	Removes job-body buffering/handling; relies on status-code checking for DNS create/delete.
ontap/ontap.go	Extends DNSConfig with SkipConfigValidation and updates JSON/jsonschema tags.
integration/test/nfs_service_test.go	Updates expected error strings and adjusts “Clean SVM” expectation.
integration/test/dns_test.go	Expands DNS tests to cover skip-validation behavior; refactors DNS verification helper.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.