Add full import support for url map

[krotkiewicz]

Fixes part of #182

$ TF_ACC=1 go test ./google -v -run TestAccComputeUrlMap -timeout 120m
=== RUN   TestAccComputeUrlMap_import
=== RUN   TestAccComputeUrlMap_basic
=== RUN   TestAccComputeUrlMap_update_path_matcher
=== RUN   TestAccComputeUrlMap_advanced
=== RUN   TestAccComputeUrlMap_noPathRulesWithUpdate
--- PASS: TestAccComputeUrlMap_basic (69.27s)
--- PASS: TestAccComputeUrlMap_import (72.18s)
--- PASS: TestAccComputeUrlMap_noPathRulesWithUpdate (83.70s)
--- PASS: TestAccComputeUrlMap_update_path_matcher (85.37s)
--- PASS: TestAccComputeUrlMap_advanced (143.10s)
PASS
ok  	github.com/terraform-providers/terraform-provider-google/google	143.133s

[krotkiewicz]

@rosbo PR is rebased.
Does it mean that we have to wait for 2.0.0 version ? How long it could take ?

Thank you

[rosbo]

Thanks for rebasing!

Yes, we will wait for 2.0.0 before merging this breaking change. We already have a few breaking changes lined up but we don't have committed to any timeline yet.

[rileykarson]

Hey @krotkiewicz! This is awesome work, and you did a great job with it. I'm going to close this PR out in favour of #2245 though, as we've been able to move google_compute_url_map in to Magic Modules, our code generator.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!