pantsbuild #general

Channels

welcome

github-notifications

announce

melodic-thailand-99227

12/23/2021, 5:06 PM

How can I tell pants to ignore directories when doing

fmt

bitter-ability-32190

12/23/2021, 5:44 PM

🤔 Why is yapf running on all my files when I'm using

--changed-since=origin/master

Copy code

(ve) joshuacannon@CEPHANDRIUS:~/work/techlabs$ ./pants --changed-since=origin/master fmt
⠄ 4.14s Run yapf on 1470 files.

Copy code

joshuacannon@CEPHANDRIUS:~/work/techlabs$ git diff origin/master --name-only | wc -l
67

👀 1

fresh-cat-90827

12/23/2021, 5:46 PM

I have noticed there is interest in programmatically modifying

BUILD

files (for instance, to add some boilerplate code across the monorepo or individual projects, such as disabling a formatter globally and then enabling it on the a project basis). I’ve explored this path and would love to share with the community. To learn more, see the thread.

ambitious-student-81104

12/23/2021, 10:03 PM

Hi Pants, I have a question about

./pants tailor

. We want to use it to check whether we have all the BUILD files and targets we need, and found 2 problems: 1. it always generates a

BUILD

file at the repo root, which we actually don't want; and 2. it generates a bunch of

pex_binary

targets which we don't need either. What are the rules that make

tailor

generate those

pex_binary

targets?

happy-kitchen-89482

12/24/2021, 3:50 PM

Official thread for discussing upstreaming your Sphinx plugin @fresh-account-42296 🙂

🤩 1

👀 1

curved-television-6568

12/24/2021, 5:46 PM

God Jul 🎄 Pants community! (Merry Christmas 🎅🏽)

🎄 7

bitter-ability-32190

12/27/2021, 3:56 PM

(Tangentially related to this discussion about programmatically modifying BUILD files by running Buildifier on them) Follow-up from this discussion about the lowest-bar way to introduce Pants for linting/formatting. 🧵

modern-leather-27612

12/27/2021, 8:01 PM

Happy belated holidays y’all. I come bearing bugs, possibly maybe. Sorry. I was debugging a resource dependency issue and thankfully remembered having read about the

--no-process-execution-local-cleanup

flag. When running

./pants test …

that worked like a charm and all files nicely stuck around, waiting to be inspected. But originally the resource issue only showed when running

./pants run …

. In that case, Pants happily kept around the PEX files for finding Python and also the cached PEX file for the dependencies. But it did not keep around the PEX file with the local Python code, which kind of defeated the purpose of the flag. Admittedly, that is with Pants 2.7. I’m good now, thanks to good old trusted

print()

plus some choice

importlib.resources

invocations. But that does look like a bug to me.

🎁 1

hundreds-father-404

12/27/2021, 8:06 PM

Hello!

./pants run

is interesting because it uses

InteractiveProcess

to run in the foreground, rather than running in a sandbox like normal. I don't think

--no-process-execution-local-cleanup

will do anything for the final run. Instead, you should see folders in the build root like

.pants.d/tmplpd86t9k

. Pants will write the PEX and relevant files to that folder and set

PYTHONPATH

appropriately

👍 1

happy-kitchen-89482

12/27/2021, 8:08 PM

There is an option to preserve those interactive process dirs: https://github.com/pantsbuild/pants/pull/13427

happy-kitchen-89482

12/27/2021, 8:08 PM

./pants run --no-cleanup ...

happy-kitchen-89482

12/27/2021, 8:10 PM

But this is only available in 2.9.x

❤️ 1

modern-leather-27612

12/27/2021, 8:55 PM

Ooh, now I have an excuse to upgrade… Alas, I’m guessing that 2.9 isn’t stable as of yet, right?

bitter-ability-32190

12/27/2021, 9:19 PM

Edit: Solved again by using my eyes How can I add VCS

extra_requirements

to a Pants thirdparty lockfile? I'm waiting on upstream PRs to merge, so in the meantime I'd like to use my branch so I'm not stuck. But setting

git+<https://github.com/>....

results in an error.

happy-kitchen-89482

12/27/2021, 9:58 PM

It's getting close - we're about to release the third release candidate, usually the 4th or 5th become the stable release

witty-family-13337

12/28/2021, 1:09 PM

hi all, not sure if this the right channel, but facing and odd behaviour in 2.9.0rc1 when using global scalac plugins, in particular with

kind-projector

. Seeing the error

Missing required plugin: kind-projector

when trying to compile some files, but adding the the

-Xplugin-list

argument to

scalac.args

pants.toml

seems to fix it (or at least makes it go away). Funny thing is that we also use the

better-monadic-for

scalac plugin and we never see any error related to it (even without the

scalac

global flag). Is this something you guys are aware or worth reporting it in Github?

rhythmic-battery-45198

12/28/2021, 5:33 PM

I am trying to debug some docker image builds which are taking longer than expected. Is there any option for logging the

docker build ...

output?

rhythmic-battery-45198

12/28/2021, 5:34 PM

Running the docker command manually is easy enough. It was actually paused waiting for user input. So, easy enough to debug outside of pants.

Copy code

Configuring tzdata
------------------

Please select the geographic area in which you live. Subsequent configuration
questions will narrow this down by presenting a list of cities, representing
the time zones in which they are located.

  1. Africa      4. Australia  7. Atlantic  10. Pacific  13. Etc
  2. America     5. Arctic     8. Europe    11. SystemV
  3. Antarctica  6. Asia       9. Indian    12. US
Geographic area:

hundreds-father-404

12/28/2021, 5:58 PM

Hi! I recommend using this flag and then running the

__run.sh

script to debug what's going on: https://www.pantsbuild.org/docs/troubleshooting#debug-tip-inspect-the-sandbox-with---no-process-execution-local-cleanup

✅ 1

curved-television-6568

12/28/2021, 6:08 PM

Agree that it would be preferable to be able to see the output from long running processes.. there’s been discussions regarding this for the dynamic UI.

➕ 2

rhythmic-battery-45198

12/28/2021, 6:25 PM

👍

silly-spring-87100

12/29/2021, 10:08 AM

Hi I have an error:

11:07:13.24 [ERROR] Invalid option 'pytest_plugins' under [pytest]

But according to the documentation, that how I should debug my tests. https://www.pantsbuild.org/docs/python-test-goal#running-tests-interactively

Copy code

[pytest]
pytest_plugins.add = ["ipdb"]

silly-spring-87100

12/29/2021, 10:27 AM

I also have a problem with including whole directory of files to test cases. Tests in

test_end_to_end.py

use files from

_input

directory I have it structured this way:

Copy code

tests
|---_input
   |---file1.csv
   |---file2.csv
|--- test_end_to_end.py
|--- BUILD

And BUILD file looks like this

Copy code

python_sources()

python_tests(
    name = "tests0",
)

files(
    name = "input",
    sources = [
        "_input/*",
    ],
)

And unfortunately this do not work. How should I modify my BUILD file?

silly-spring-87100

12/29/2021, 1:58 PM

~What approach would you recommend for this problem. I'm trying to prepare POC with pants, for our internal monorepo. Currently we produce multiple docker images in our CI, and all of them are tagged with current

commit-sha

I see that, in documentation I cannot pass

image_tag

for

docker_image(...)

as environment variable. Worst case is manual bash script, which iterates over all images and creates new tags for them. Is there a better solution with pants?~

curved-television-6568

12/29/2021, 3:59 PM

The Docker build context troubleshooter at work 😄 A little hint at what’s in the pipeline, for all Docker backend users out here.. 😉

🚀 1

witty-family-13337

12/30/2021, 10:39 AM

hi all again, just leaving this one here as a heads up. Found another issue when resolving jvm dependencies and this one is indeed in coursier. I reported it in https://github.com/coursier/coursier/issues/2316 with proposed fix in https://github.com/coursier/coursier/pull/2317. The consequence of the “bug” is that coursier hangs after blowing the stack, making pants hang too.

🙏 3

silly-spring-87100

12/30/2021, 1:06 PM

In my gitlab-ci I need to build docker images with

./pants pacakge

Unfortunately

docker:dind

image (we are using it curentlly) is not compatible with pants: • alpine linux • missing python What image would you use, to build and push docker images with

pants

inside CI container?

refined-addition-53644

12/30/2021, 6:12 PM

Hi, what's the correct way to ignore directory from

pants test ::

I added this to

pants.toml

Copy code

[pytest]
args = ["--ignore=./my_dir"]

Doesn't work :(

hundreds-father-404

12/30/2021, 6:14 PM

Hello! Do you still want Pants to generally operate on that code, like running linters and formatters? Just want to skip tests? If so, you can use the

skip_tests

field. This part of docs also gives a command to query which tests you use Pants for and which are your old test runner, which you can then pipe into that runner: https://www.pantsbuild.org/docs/existing-repositories#3-set-up-tests

refined-addition-53644

12/30/2021, 6:19 PM

I want to completely ignore the directory actually. Pants doesn't need to do anything with it