privacysandbox/aggregation-service

feedback on deploying aggregation service

Closed this issue · 6 comments

I got an api gateway error The API with ID my-api-id doesn’t include a route with path /* having an integration arn:aws:lambda:us-east-1:my-aws-account-id:function:stg-create-job. on aws console, after deploying aggregation service using terraform.

I changed Source ARN of lambda's permisson from arn:aws:execute-api:us-east-1:my-aws-account-id:my-api-id/*/** to arn:aws:execute-api:us-east-1:my-aws-account-id:my-api-id/*/*/v1alpha/getJob, and it solved the error.
https://github.com/privacysandbox/control-plane-shared-libraries/blob/9efe5591acc18e46263399d9785432a146d9675c/operator/terraform/aws/modules/frontend/api_gateway.tf#L62

Hi @k-o-ta,

thank you for reporting this, we will look into it. Can you share which Terraform version you used?

Thank you for reply.
My Terraform version was v1.2.3.

% terraform --version
Terraform v1.2.3
on darwin_amd64

I would like to correct some of the issue description.
Correct: arn:aws:execute-api:us-east-1:my-aws-account-id:my-api-id/*/*/v1alpha/createJob
Inorrect: arn:aws:execute-api:us-east-1:my-aws-account-id:my-api-id/*/*/v1alpha/getJob

Additionally, the API call succeeds, although an error is displayed on the console screen.

Hi @k-o-ta ,

thank you, can you test if you're still able to reproduce the issue on your end using release v0.5.0?

Thank you in advance.

k-o-ta commented

I'm sorry for late reply.

I retried using v0.8.0, but it reproduced the same error.

include-a-route-with-path

Hi @k-o-ta ,

thank you for the details. I was able to reproduce that on our end and we'll look into addressing this issue. This should not be interfering with normal operations of the service as you stated as well. We recommend to not change the path for the time being till we have a version that is tested fully during our releases.

Hi @k-o-ta, this issue has been addressed with our latest release version 0.10.0 that uses version 0.51.13 of our shared libraries.
Thank you for reporting this issue and for your patience.