R
Railway5mo ago
Joshie

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

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.
No description
No description
No description
31 Replies
Percy
Percy5mo ago
Project ID: 1b326884-0c17-43ed-9b52-f443662e8f50
Brody
Brody5mo ago
so for instance, if you push code to another folder in that repo other than proxy, will the proxy service redeploy?
Joshie
JoshieOP5mo ago
Yes, exactly
Brody
Brody5mo ago
don't you want an inverted rule then?
Joshie
JoshieOP5mo ago
Why? I want to only deploy proxy when code inside proxy is changed
Brody
Brody5mo ago
not sure, try it?
Joshie
JoshieOP5mo ago
I actually think the answer is that I need to do proxy/** vs /proxy/**
No description
matt
matt5mo ago
we should make that more explicit
Joshie
JoshieOP5mo ago
So was I right? Is that why it isn't working?
matt
matt5mo ago
oh, I haven't confirmed anything. I thought you had lol. sorry but I can help try
Joshie
JoshieOP5mo 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 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
matt
matt5mo ago
I'll def make a ticket for that.
Joshie
JoshieOP5mo ago
<#1250987443918995466> (it is what got me @Bug Basher) But idk since i have only ever used v2. I have never used anything else. So ,,, I it is just a speculation
matt
matt5mo ago
but root directory weirdness has been observed. best case scenario is docs are unclear root dir + monorepo
Joshie
JoshieOP5mo ago
Yes. Either the issue is the docs do a bad job explaining where root is; or this is just a bug. I will likely test it out tomorrow at some point. Just not going to bother making a new test repo right this second.
matt
matt5mo ago
for sure. i'll see if I can test something too
Joshie
JoshieOP5mo 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.
Joshie
JoshieOP5mo 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.
No description
No description
Joshie
JoshieOP5mo 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 😂 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
Joshie
JoshieOP5mo 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
No description
No description
No description
No description
matt
matt5mo ago
thank you so much. can confirm the watchpattern bug. thank you!
Joshie
JoshieOP5mo ago
I am going to delete all of the services and repo now. I can get it back up if need be Any insight / update on this? Coming back to this. This bug is still here 👀 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 ... 👀
Brody
Brody5mo ago
yes it is, just had my fellow conductor bring it up to me in DMs cc @Christian - watch path bug
angelo
angelo5mo ago
!t
Duchess
Duchess5mo ago
New reply sent from Help Station thread:
This thread has been escalated to the Railway team.
You're seeing this because this thread has been automatically linked to the Help Station thread.
christian
christian5mo ago
Thank you for flagging. I've created a bug report for this issue.
Joshie
JoshieOP4w ago
I could be delusional ... possible. But I think this bug is still live? Any update on its progress?
Brody
Brody4w 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 besides, the new legacy builder is faster, we now use buildkit and stream each layer compressed to the registry
Joshie
JoshieOP4w ago
Hmmm, so I should switch everything to the old builder. Just double checking
Brody
Brody4w ago
that's correct, the new old builder! it's faster, but it still doesn't support private networking though
Joshie
JoshieOP4w ago
👌 thank you
Want results from more Discord servers?
Add your server