Daniel Goldman

Results 95 comments of Daniel Goldman

> Ah, hm, another question: if we could hypothetically `pants export` tools like this, would that solve the problem too? I've thought about implementing this, and it shouldn't be that...

This is just a file ownership problem. `RUN mkdir /home/myuser/app` and `COPY . .` are run as root, so is created as root. You never chown it. Simply move the...

Also do [Environments](https://www.pantsbuild.org/2.21/docs/using-pants/environments) cover your usecase of running integration tests in a docker container?

ah, I see it now. `COPY . .` is copying the local context in. `COPY` operates as root unless `--chown` is passed. This includes the `.pants.d` directory, so the user...

I'm a bit surprised that `__package__` doesn't work. For assets we have inference tested for `pkgutil.get_data(__file__, "foo/bar.txt")`. We compute the expected path by taking the dir of the current python...

It looks like `__package__` works for me in [this test](https://github.com/lilatomic/pants/blob/c0fb87c1b0385f2a01e9a5dee6f7f9b309a10ed3/src/python/pants/backend/python/dependency_inference/rules_test.py#L235), although it's pretty basic. Can you outline a scenario where it fails?

~Seems we don't use `file://...` for dependency inference, we just use the chart name and glue it together [ref](https://www.pantsbuild.org/docs/helm-overview#managing-chart-dependencies). I think it makes sense to also resolve those.~ Seems I...

Hm, I'm not able to get a repro. I've committed the files I'm using to a subtree [here](https://github.com/lilatomic/pants/tree/helm/19966-file-dep/testprojects/src/helm). you can try with ```shell cp -r testprojects/src/helm/ /tmp/helm19966/ pushd /tmp/helm19966/ pants...

Thanks for that sample. I'm not sure what to do, though, as I'm not really a Helm power user. The error `pants lint` gives is the same as the error...

Oh neat, I didn't realise that you could pass vars files to `helm lint`. I think we could implement that. I tried fiddling with `exports` and `import-values`, but couldn't get...