-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Issues deploying to staging #10077
Comments
Hmm. Not sure this is related to this package specifically. Just merged
tried bumping staging to 512Mb RAM still failing rolled back to the previous version. All is well. Currently stumped :/ |
Error is
The one thing I haven't tried is just yeeting one of the "bad" images to production. My instinct is they'd probably work because we're running multiple instance, but I'm reluctant to just try that without understanding wtf is going on first. Utterly baffling. |
Some things I haven't tried yet:
|
I think i can pitch in on this one 👍 |
Or could possibly be a platform level issue: https://community.fly.io/t/could-not-find-a-good-candidate-at-load-balancing-outage/19172/16 ? |
Thanks. When I circle back to this I will check that out. If I end up against a brick wall again, we do have email support with fly so raising a support ticket is another option once I have time to follow up. |
OK. So I just tried deploying I think I am going to assume that there is nothing wrong with any of the images and we were triggering some kind of issue on fly's side. Assuming I don't run into this again trying to deploy later I will assume this is fixed and close this. |
I've now successfully run staging and production deploys. |
Indeed! Glad it's sorted |
📋 Description
Basically, what happened here was:
sha256:e77392c2f4cfa1a9da6a7754fd26911f560bc522e3fc0d56ee7386b910b0c5b1
to staging. Staging fails to serve the app withcould not find a good candidate within 90 attempts at load balancing
sha256:e77392c2f4cfa1a9da6a7754fd26911f560bc522e3fc0d56ee7386b910b0c5b1
again. Back tocould not find a good candidate within 90 attempts at load balancing
I don't really have time to look into it right now, but we've got staging set to run a 256Mb VM. Prod uses 512Mb VMs. Review apps use whatever the default is (probably more than 256Mb).
My theory is that this update might now be using all the memory we have available on staging 😮 ? That would explain why it worked locally but not on staging. I'll circle back to this when I have a chance.
The text was updated successfully, but these errors were encountered: