-
Notifications
You must be signed in to change notification settings - Fork 58
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
Deployment doesn't detect new container image build #67
Comments
Figured out a workaround for now: public function package(): array
{
return [
'ImageUri' => config('services.pdf.image_uri'),
+ '__handler__' => substr(md5(file_get_contents(resource_path('lambda/main.py'))), 0, 8),
];
} Since Sidecar hashes the entire returned package object to determine if the function has changed, just including anything that changes when the underlying Python file changes seems to do the trick here and trigger Sidecar/Lambda to create a new version of the function. |
Thank you for the solution. I am using a different tag when pushing to ECR right now but seems your solution is better because I don't need to remember what tags I already used anymore. |
Hmm I need to fix this. @bakerkretzmar do you have a general solution that might work here? |
Short answer is no I don't sorry 😅 we eventually switched to using the Node runtime without a Docker image. It's quite possible that I was using Docker tags wrong and shouldn't have been targeting |
I'm running a Sidecar function that uses a container image on ECR, and Sidecar doesn't seem to be able to detect that I've built and pushed up a new version of the container.
php artisan sidecar:deploy --activate
tells meFunction code and configuration are unchanged. Not updating anything.
and then reuses the latest active version of my Lambda instead of creating/aliasing a new one.The
ImageUri
hasn't changed because I'm using thelatest
tag.Am I missing a step? Should I be tagging the new Docker image build with a date or something, instead of
latest
, so the Image URI changes?The text was updated successfully, but these errors were encountered: