r/aws_appmesh_route: Add support for HTTP-header-based routing and route priorities

[ewbankkit]

Community Note

• Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
• Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

Closes #9802.
Replaces #9803.

Relevant commits have been cherry picked from #9468.

Release note for CHANGELOG:

resource/aws_appmesh_route: Add `priority` and `header` attributes to support route priorities and HTTP-header-based routing

Output from acceptance testing:

$ make testacc TEST=./aws TESTARGS='-run=TestAccAWSAppmesh'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -parallel 20 -run=TestAccAWSAppmesh -timeout 120m
=== RUN   TestAccAWSAppmesh
=== RUN   TestAccAWSAppmesh/Mesh
=== RUN   TestAccAWSAppmesh/Mesh/basic
=== RUN   TestAccAWSAppmesh/Mesh/egressFilter
=== RUN   TestAccAWSAppmesh/Mesh/tags
=== RUN   TestAccAWSAppmesh/Route
=== RUN   TestAccAWSAppmesh/Route/httpHeader
=== RUN   TestAccAWSAppmesh/Route/httpRoute
=== RUN   TestAccAWSAppmesh/Route/tcpRoute
=== RUN   TestAccAWSAppmesh/Route/routePriority
=== RUN   TestAccAWSAppmesh/Route/tags
=== RUN   TestAccAWSAppmesh/VirtualNode
=== RUN   TestAccAWSAppmesh/VirtualNode/basic
=== RUN   TestAccAWSAppmesh/VirtualNode/cloudMapServiceDiscovery
=== RUN   TestAccAWSAppmesh/VirtualNode/listenerHealthChecks
=== RUN   TestAccAWSAppmesh/VirtualNode/logging
=== RUN   TestAccAWSAppmesh/VirtualNode/tags
=== RUN   TestAccAWSAppmesh/VirtualRouter
=== RUN   TestAccAWSAppmesh/VirtualRouter/basic
=== RUN   TestAccAWSAppmesh/VirtualRouter/tags
=== RUN   TestAccAWSAppmesh/VirtualService
=== RUN   TestAccAWSAppmesh/VirtualService/tags
=== RUN   TestAccAWSAppmesh/VirtualService/virtualNode
=== RUN   TestAccAWSAppmesh/VirtualService/virtualRouter
--- PASS: TestAccAWSAppmesh (811.85s)
    --- PASS: TestAccAWSAppmesh/Mesh (101.56s)
        --- PASS: TestAccAWSAppmesh/Mesh/basic (19.32s)
        --- PASS: TestAccAWSAppmesh/Mesh/egressFilter (38.74s)
        --- PASS: TestAccAWSAppmesh/Mesh/tags (43.51s)
    --- PASS: TestAccAWSAppmesh/Route (206.73s)
        --- PASS: TestAccAWSAppmesh/Route/httpHeader (41.00s)
        --- PASS: TestAccAWSAppmesh/Route/httpRoute (41.44s)
        --- PASS: TestAccAWSAppmesh/Route/tcpRoute (40.93s)
        --- PASS: TestAccAWSAppmesh/Route/routePriority (41.19s)
        --- PASS: TestAccAWSAppmesh/Route/tags (42.16s)
    --- PASS: TestAccAWSAppmesh/VirtualNode (280.35s)
        --- PASS: TestAccAWSAppmesh/VirtualNode/basic (22.52s)
        --- PASS: TestAccAWSAppmesh/VirtualNode/cloudMapServiceDiscovery (122.72s)
        --- PASS: TestAccAWSAppmesh/VirtualNode/listenerHealthChecks (49.25s)
        --- PASS: TestAccAWSAppmesh/VirtualNode/logging (35.40s)
        --- PASS: TestAccAWSAppmesh/VirtualNode/tags (50.45s)
    --- PASS: TestAccAWSAppmesh/VirtualRouter (86.63s)
        --- PASS: TestAccAWSAppmesh/VirtualRouter/basic (35.42s)
        --- PASS: TestAccAWSAppmesh/VirtualRouter/tags (51.21s)
    --- PASS: TestAccAWSAppmesh/VirtualService (136.58s)
        --- PASS: TestAccAWSAppmesh/VirtualService/tags (57.21s)
        --- PASS: TestAccAWSAppmesh/VirtualService/virtualNode (40.82s)
        --- PASS: TestAccAWSAppmesh/VirtualService/virtualRouter (38.54s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	811.930s

[ewbankkit]

The errors when running the labeler action

##[error]HttpError: Resource not accessible by integration
##[error]Resource not accessible by integration
##[error]Node run failed with exit code 1

are the same as seen in other PRs, e.g. #8259 (comment).
My guess is that since both AppMesh and RAM have entries in .github/PULL_REQUEST_LABELS.yml then the token used by the labeler action doesn't have permissions to call addLabels.

[aeschright]

@ewbankkit I think we can remove the GitHub action from our own repo as soon as someone copies the remaining items from https://github.com/terraform-providers/terraform-provider-aws/blob/master/.github/PULL_REQUEST_LABELS.yml to https://github.com/terraform-providers/terraform-provider-aws/blob/master/.hashibot.hcl#L431

[ewbankkit]

It seems that there is a known issue with the current GitHub labeler action and PRs on forks: actions/labeler#12.

[ewbankkit]

Closing in favor of #10075.

[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. Thanks!