Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: functions name when basePath is setup #345

Open
wants to merge 33 commits into
base: v0.x
Choose a base branch
from

Conversation

framled
Copy link

@framled framled commented Jul 14, 2022

Terraform failed when basePath is set, this is because lambda functions name can not have a special character.

issues related to this are #288 #310

@framled framled changed the title Fix: rename basePath for lambda funcitons name Fix: functions name when basePath is setup Jul 14, 2022
@ofhouse ofhouse changed the base branch from release-v0.x to v0.x July 25, 2022 10:27
@ofhouse
Copy link
Member

ofhouse commented Jul 25, 2022

Hey, thanks for submitting this PR.
Unfortunately the target branch you selected (release-v0.x) is managed by our CI system and not meant for opening PRs against it.
You need to rebase your changes to the v0.x branch instead.
I've already changed the target branch, you need to rebase/cherry-pick and force push your commits to make it work.

If it's too confusing, please allow me to make the necessary changes, by Enabling repository maintainer permissions on existing pull requests.

@framled
Copy link
Author

framled commented Jul 25, 2022

Hi @ofhouse I've already rebase and cherry pick... reigncl/v0.x branch is already up to date...
But I didn´t find the option in the PR... :/

I've just create a new PR #351, that is ok?

@sumantapakira
Copy link

Hi @framled I think this is not about only the fixing the Lambda function name, it also requires the change in proxy. Have you got success to test end to end flow only by changing Lambda function name?

@framled
Copy link
Author

framled commented Sep 26, 2022

@sumantapakira Yes it works

@sumantapakira
Copy link

ok, but with your change when I tried to create new infra then it fails -

Error: error putting S3 Bucket Notification Configuration: InvalidArgument: Unable to validate the following destination configurations
status code: 400, request id: 0Y96VW1JYZT35NNG, host id: 6LM/nRmfUTxNLAMrJAqjHEjkPUhtD0FxIthh5qkFcwWgGTAQ2EJ/6k1oAq2RdKfGWtnGJgq354o=

with module.tf_next.module.statics_deploy.aws_s3_bucket_notification.on_create,
on ../original_source/modules/statics-deploy/main.tf line 31, in resource "aws_s3_bucket_notification" "on_create":
31: resource "aws_s3_bucket_notification" "on_create" {

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants