R
Railway2mo ago
rob

New deployment overlapping with old one when RAILWAY_DEPLOYMENT_OVERLAP_SECONDS is set to 0

Hello all, I've been working with RAILWAY_DEPLOYMENT_OVERLAP_SECONDS set to 0 for quite some time and I noticed that there is still some overlapping I can't get rid off. Is this a bug on Railway side or is there a proper way I can configureit to be sure the old deploy gets shut down before starting the newest one? The project affected by this isssue is f6d25e17-9bb2-457f-8ce2-e55b5ce1dcd8
No description
17 Replies
Percy
Percy2mo ago
Project ID: f6d25e17-9bb2-457f-8ce2-e55b5ce1dcd8
Adam
Adam2mo ago
This'll be something for @Brody to look into when he gets back
rob
rob2mo ago
Is there anything I can do to help you triage this behaviour? I just deployed and it happened again
Brody
Brody2mo ago
hey there, Adam tagged me when I was on PTO, and we had this monday off as it's a holiday. I will be trying to reproduce this.
rob
rob2mo ago
Hi Brody! Sorry, I had no idea there was a holiday I'm under the impression Railway does something of this sort when a new deploy is triggered In my case, I'd expect it to check if RAILWAY_DEPLOYMENT_OVERLAP_SECONDS = 0 and in that case the start container action should be triggered only once the stop container one completes
No description
Brody
Brody2mo ago
for fun, try setting it to a negative value
rob
rob2mo ago
I thought about that but I didn't dare, let me try right away
Brody
Brody2mo ago
the worst that can happen is that the variable would be ignored
rob
rob2mo ago
Unfortunately the behaviour persists
No description
Brody
Brody2mo ago
how urget is this, aka is this blocking something you are doing? or is it just an annoyance?
rob
rob2mo ago
No urgency at all, it was just something I noticed very long ago and was worth reporting
Brody
Brody2mo ago
ah gotcha, this isnt affecting you at all?
rob
rob2mo ago
Nope, I only face a few seconds of overlap which results in a couple of error logs
Brody
Brody2mo ago
well we dont want error logs hello, we won't be able to prioritize this issue at the moment, for now please add a sleep to the start of your app if its needed.
rob
rob2mo ago
Hey Brody! Somehow I missed your reply so apologies for that It really is fine if the team can't act on it for the time being, do you know / are you allowed to share if it will be added to the backlog as a future activity?
Adam
Adam2mo ago
@Brody pinging for visibility
Brody
Brody2mo ago
yep i can share the procedures. we do have this added to our backlog since you reported this, at this time you are the only one to report this issue, so as more people report this the backlogged ticket will get more traffic and we will pick it back up and move it into triage when there is a sufficient need to.
Want results from more Discord servers?
Add your server