-
Notifications
You must be signed in to change notification settings - Fork 15
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
[Bug][Compose] #149
Comments
Hey @darylteo - thanks a lot for reporting and sorry you've run into trouble. That's true, currently default fallback is not supported for Compose params - they work a bit differently than params in individual services. It's definitely something to consider moving forward cc @mnapoli I'm going to transfer the issue to |
👍 that sounds like a reasonable improvement indeed. @pgrzesik do you think that would be easy to implement? Also I'm curious:
Compose is no longer in beta, would you be able to tell us what made you think that? Maybe we have some piece of documentation that we need to update. |
I think that should be possible to implement without too much effort |
Hi Team, wondering what happened with this? Is this still something that could be added in the future? 🙏 |
Are you certain it's a bug?
Is the issue caused by a plugin?
Are you using the latest v3 release?
Is there an existing issue for this?
Issue description
In
serverless-compose.yml
, param placeholders do not have the same "default value" functionality that is available in the generalserverless.yml
https://github.com/darylteo/serverless-compose-bug-repro
In this reproduction repo, I have 2 services - 1 for the api functions, another for Cloudformation StackSet. I need to pass the RoleARN from the schema service over to the api service. This works once running
sls deploy
but in environments where I cannot do this (developer environments without access keys to remote AWS accounts), it fails to retrieve the ARN as the ARN does not exist yet.I definitely do not want to make it a requirement for every developer to have to do a deploy before they can perform any changes. So I expected to be able to use the default value template pattern to solve this but it seems that
serverless-compose.yml
does not support this, and insteads just passes the entire placeholder as is.As is, I cannot run the service offline using
serverless-offline
, and I cannot run any offline unit tests, without actually doing a deploy first.I understand this is a beta, and I may have missed something perhaps. If so please let me know!
Appreciate the package.
Service configuration (serverless.yml) content
Command name and used flags
sls api:invoke local --function hello
Command output
Environment information
The text was updated successfully, but these errors were encountered: