On the 1.30.x branch some of `test_export.py` fail...
# development
h
On the 1.30.x branch some of
test_export.py
fail, both in CI and locally: https://travis-ci.com/github/pantsbuild/pants/jobs/496026410
Full stack trace:
Copy code
E           pants.build_graph.address_lookup_error.AddressLookupError: Build graph construction failed: ExecutionError 1 Exception encountered:
E           
E           Traceback (most recent call last):
E             File "/tmp/process-executionSFpNNl/pants/engine/internals/build_files.py", line 277, in addresses_with_origins_from_address_families
E               address_family_by_directory
E             File "/tmp/process-executionSFpNNl/pants/base/specs.py", line 131, in matching_address_families
E               return self.address_families_for_dir(address_families_dict, self.directory)
E             File "/tmp/process-executionSFpNNl/pants/base/specs.py", line 75, in address_families_for_dir
E               'Path "{}" does not contain any BUILD files.'.format(spec_dir_path)
E           pants.base.specs.AddressSpec.AddressFamilyResolutionError: Path "" does not contain any BUILD files.
E           
E           The above exception was the direct cause of the following exception:
E           
E           Traceback (most recent call last):
E             File "/tmp/process-executionSFpNNl/pants/engine/internals/native.py", line 78, in generator_send
E               res = func.send(arg)
E             File "/tmp/process-executionSFpNNl/pants/engine/internals/build_files.py", line 280, in addresses_with_origins_from_address_families
E               raise ResolveError(e) from e
E           pants.base.exceptions.ResolveError: Path "" does not contain any BUILD files.
This happens even on current HEAD, which presumably passed when it was merged.
Familiar to anyone?
e
No, but it looks like it cropped up here and was ignored?: https://travis-ci.com/github/pantsbuild/pants/jobs/489544058
h
Hmm
The underlying cause seems to be that no actual BUILD file is found for targets created programmatically in the test context.
It seems to be confined to just
test_export.py
e
If I back up to before that commit it works.
So, with you on how the heck do those relate, but they seem to. We ignored CI at our own peril.
h
huh
I think I fixed it
e
Excellent.
h
Fixed in a second commit here : https://github.com/pantsbuild/pants/pull/11843
1.30.x branch is now stable again
e
Thanks Benjy.