Return metadata provider task whether it is successful or not

[liliankasem]

Pull request checklist

IMPORTANT: Currently, changes must be backported to the in-proc branch to be included in Core Tools and non-Flex deployments.

• Backporting to the in-proc branch is not required
  • Otherwise: Link to backporting PR
• My changes do not require documentation changes
  • Otherwise: Documentation issue linked to PR
• My changes should not be added to the release notes for the next release
  • Otherwise: I've added my notes to release_notes.md
• My changes do not need to be backported to a previous version
  • Otherwise: Backport tracked by issue/PR #issue_or_pr
• My changes do not require diagnostic events changes
  • Otherwise: I have added/updated all related diagnostic events and their documentation (Documentation issue linked to PR)
• I have added all required tests (Unit tests, E2E tests)

Additional information

With the current behaviour, even if the getFunctionMetadataFromProviderTask completes before the timeout period, we get a timeout exception if it is not successful. This is a bug and ignores potential failures from the task.

[liliankasem]

Pipeline passing here: https://azfunc.visualstudio.com/public/_build/results?buildId=188554&view=results#SourceCard