Support for App Insights workspaces

[isaacabraham]

This PR closes #770

The changes in this PR are as follows:

• Adds an extra keyword for linking a Log Analytics Workspace to an App Instance instance.

I have read the contributing guidelines and have completed the following:

• Tested my code end-to-end against a live Azure subscription.
• Updated the documentation in the docs folder for the affected changes.
• Written unit tests against the modified code that I have made.
• Updated the release notes with a new entry for this PR.
• Checked the coding standards outlined in the contributions guide and ensured my code adheres to them.

Below is a minimal example configuration that includes the new features, which can be used to deploy to Azure:

let workspace = logAnalytics {
    name "loganalytics-workspace"
}

let myAppInsights = appInsights {
    name "appInsights"
    log_analytics_workspace workspace
}

[isaacabraham]

@xdaDaveShaw can you have a look at this - I'm hopeful it does what you're after, too. Let me know.

[ninjarobot]

Looks great, thank you for adding test coverage over the classic and new resource types.

[isaacabraham]

@ninjarobot do you know if we need to keep the old API version - can we just start using the new ARM schema version for both classic and workspace AI? That would simplify some code.

[ninjarobot]

@ninjarobot do you know if we need to keep the old API version - can we just start using the new ARM schema version for both classic and workspace AI? That would simplify some code.

The old API version will make a classic instance, so best not to use that. We may want to consider dropping it from Farmer, though, just because people probably should not be deploying those anymore.

[ninjarobot]

Thanks for adding this, it's now released in farmer 1.6.30.

[xdaDaveShaw]

Thanks @isaacabraham - this looks great.

Sorry I couldn't reply sooner, had a busy week.