Not able to use local valueFiles when using reposi...
# 🌱|help-and-getting-started
b
Hey guys! Just creating this one for visibility, hopefully somebody can take a look at this one, also if there’s a possibility of getting some context on the Code that handles this part I might be able to try to implement a solution for it! https://github.com/garden-io/garden/issues/3367
Hey guys, just looking if there's anything I can help to further triage this issue? ❤️ @swift-garage-61180 @curved-intern-91221
c
Hi @bright-policeman-43626! Sorry for the late reply. We have a draft PR to address the issue above: https://github.com/garden-io/garden/pull/3445 It looks like it's enough to weaken the path validation conditions. We'll double check to make sure there is no regression.
b
You guys are amazing thank you! Is there anything that I can do to help testing this? Should I build this PR locally to test?
c
@bright-policeman-43626 it would be great if you can build that PR and test it locally. By the way, the PR can have some side effects. We have been allowing only sub-paths as valid
valueFiles
to make sure that the hashes of the value files are factored into the version hash. With this PR, it should be possible to use arbitrary paths as
valueFiles
. But, if you try to modify files that are located outside the module working dir, then the
garden build
command might not rebuild the image, because the version hash won't change. Please let us know if you get any unexpected behaviour while testing the PR.
q
@bright-policeman-43626 have you had a chance to build and test the PR?
a
Sorry for the delay and thanks for your patience! This should be fixed in #3445 and available in a fresh edge pre-release of garden, as well as coming to the stable release in the next few weeks.
Caveat: this might introduce some issues for the project hash and "has anything changed since last deployment" detection - if you run into any, please report here https://github.com/garden-io/garden/issues/3487 thank you! 🙇‍♂️
b
Thanks a lot! ❤️ this will be shipped in 0.13 right?
a
earlier - it will be in the next 0.12.xxx patch release, likely some time next week
b
Thanks a lot!
a
thanks a lot for reporting and helping make garden better ❤️