[BUG] Watchpath is ignored for v2 builder + mono repo
coderjoshdk
TRIALOP

2 years ago

I have a mono repo project
project id: 1b326884-0c17-43ed-9b52-f443662e8f50
And I have multiple services with a root (like /proxy) and then a watch path that is just /proxy/**. But all of the services are being re-deployed, when I push any change.

This should be valid. At least, if I am reading the docs right. For example:

Note, if a Root Directory is provided, patterns still operate from /. For a root directory of /app, /app/**.js would be used as a pattern to match files in the new root.

40 Replies

2 years ago

so for instance, if you push code to another folder in that repo other than proxy, will the proxy service redeploy?


coderjoshdk
TRIALOP

2 years ago

Yes, exactly


2 years ago

don't you want an inverted rule then?


coderjoshdk
TRIALOP

2 years ago

Why? I want to only deploy proxy when code inside proxy is changed


2 years ago

not sure, try it?


coderjoshdk
TRIALOP

2 years ago

I actually think the answer is that I need to do proxy/** vs /proxy/**

1255635020450893800


2 years ago

we should make that more explicit


coderjoshdk
TRIALOP

2 years ago

So was I right? Is that why it isn't working?


2 years ago

oh, I haven't confirmed anything. I thought you had lol. sorry


2 years ago

but I can help try


coderjoshdk
TRIALOP

2 years ago

I have not had a chance to open a test repo. I might be able to get to it tomorrow. If you are able to test before that, by all means


coderjoshdk
TRIALOP

2 years ago

But I don't think think that is the issue. Or at least, it shouldn't be the issue. I wonder if the new build system is part of it? Idk. Becuase the site / docs shows using /path/**. And this isn't the first thing that the new builder messes up on when it comes to mono repos. So that is also worth looking at


2 years ago

I'll def make a ticket for that.


coderjoshdk
TRIALOP

2 years ago

<#1250987443918995466> (it is what got me <@&875753109501141003>)

But idk since i have only ever used v2. I have never used anything else. So ,,, I it is just a speculation


2 years ago

but root directory weirdness has been observed. best case scenario is docs are unclear


2 years ago

root dir + monorepo


coderjoshdk
TRIALOP

2 years ago

Yes. Either the issue is the docs do a bad job explaining where root is; or this is just a bug.


coderjoshdk
TRIALOP

2 years ago

I will likely test it out tomorrow at some point. Just not going to bother making a new test repo right this second.


2 years ago

for sure. i'll see if I can test something too


coderjoshdk
TRIALOP

2 years ago

I was able to get a push to the main project. Removing the leading /, didn't do anything. So ,,,, going to need to make a test repo if I want to actually find the cause.


coderjoshdk
TRIALOP

2 years ago

Testing a few things at once. But yea. I found it out. It is the new builder. @matt 👍
Idk who to raise this to that can fix it.

1255891569849139200
1255891570113384400


coderjoshdk
TRIALOP

2 years ago

The project id for the above : cd724b67-a692-4005-812c-617f8b9238bf
Lmk when I can delete it. Lol, I am still doing all of this on a trial plan account 😂


coderjoshdk
TRIALOP

2 years ago

oh, extra info about the "few things" I was testing at the same time.
Service A has a watch path of /serviceA/** and service B has a watch path of serviceB/**
ServiceA is a nixpacks default deploy, and serviceB is a docker file.
https://github.com/CoderJoshDK/railway-mono-repo-test
I like keeping my repos "clean". So just lmk when I am free to delete everything


coderjoshdk
TRIALOP

2 years ago

Hmmmm I guess this is also a different bug, but railway is claiming that one of the deploys failed. However, clicking on details, takes me to an older dummy build. bcddd97e-3c35-44a5-9221-3b7cb1b507c1. Not really sure what it is from
It is the removed one. But even if that one was a failed deploy, it shouldn't matter. There is a more recent successful build.
No idea what went on here. And I won't be exploring the cause of this bug. Just letting you know about it

1255893767894995000
1255893768247181300
1255893768532398300
1255893768779993300


2 years ago

thank you so much. can confirm the watchpattern bug. thank you!


coderjoshdk
TRIALOP

2 years ago

I am going to delete all of the services and repo now. I can get it back up if need be


coderjoshdk
TRIALOP

2 years ago

[BUG] Watchpath is ignored for v2 builder + mono repo


coderjoshdk
TRIALOP

2 years ago

Any insight / update on this?


coderjoshdk
TRIALOP

2 years ago

Coming back to this. This bug is still here 👀


coderjoshdk
TRIALOP

2 years ago

Totally understand that it is a low priority bug. But it should help reduce the redundant builds. And if there are a bunch of build issues going on … 👀


2 years ago

yes it is, just had my fellow conductor bring it up to me in DMs


2 years ago

cc @christian - watch path bug



This thread has been escalated to the Railway team.

Status changed to Awaiting Railway Response angelo-railway over 1 year ago


Thank you for flagging. I've created a bug report for this issue.


coderjoshdk
TRIALOP

a year ago

I could be delusional … possible. But I think this bug is still live? Any update on its progress?


a year ago

yep bug is still there, the v2 builder is currently in an unmaintained state and we do not recommend it's use right now, we are currently focusing on stateful workloads on metal and thus have put the v2 builder on the backburner


a year ago

besides, the new legacy builder is faster, we now use buildkit and stream each layer compressed to the registry


coderjoshdk
TRIALOP

a year ago

Hmmm, so I should switch everything to the old builder. Just double checking


a year ago

that's correct, the new old builder! it's faster, but it still doesn't support private networking though


coderjoshdk
TRIALOP

a year ago

👌 thank you


Loading...