Task: Remove functions used in errhelp.go across controllers

[melonrush13]

We are using functions from errhelp.go such as IsStatusCode204, and IsStatusCode404. We want to remove these function calls used throughout the controllers and have them use errors.go string checks instead (such as ParentNotFoundErrorCode) to be consistent across controllers.

In Addition, we would also like to ideally error check from HTTP status codes instead of strings.

Done criteria
Remove string functions and instead error check them from errors.go

Fix in the following controllers..

• Azure SQL Firewall Rule
• Azure DataLake
• Cosmos DB
• Storage Controller
• Azure SQL DB