OK. Any suggestions on where my rule is failing? M...
# pluginsp
proud-dentist-22844
03/31/2023, 5:24 PMOK. Any suggestions on where my rule is failing? Maybe something to do with environments that I didn't plumb? Or is pex trying to use the wrong interpreter for some reason (and why would that have changed between 2.16.0a0 and 2.16.0a1)?
Here's the test error: https://github.com/StackStorm/st2/actions/runs/4577090796/jobs/8082076631?pr=5950#step:7:229
Here's the test: https://github.com/StackStorm/st2/blob/master/pants-plugins/uses_services/platform_rules_test.py
Here's the rule under test: https://github.com/StackStorm/st2/blob/master/pants-plugins/uses_services/platform_rules.py
Here's the script that rule runs via `VenvPex`: https://github.com/StackStorm/st2/blob/master/pants-plugins/uses_services/scripts/inspect_platform.py
✅ 1
b
bitter-ability-32190
03/31/2023, 5:48 PMI can't see the logs, but https://github.com/StackStorm/st2/actions/runs/4577090796/jobs/8082076631?pr=5950#step:7:229 might be relevant
p
proud-dentist-22844
03/31/2023, 5:49 PMThe only way that would be invalid syntax is if it is trying to use a python2 interpreter
b
bitter-ability-32190
03/31/2023, 5:50 PMThat was the issue that sparked the PR. PEX choosing 2.7 🙂
p
proud-dentist-22844
03/31/2023, 5:50 PMsorry did you mean to link back to my GHA logs? or something else?
proud-dentist-22844
03/31/2023, 5:51 PMThis is the relevant bit of the logs
Copy code
E File "/tmp/pants-sandbox-yKexC5/.cache/pex_root/venvs/bbd16d7f110473a5a1fb4d5f57d0d53fcfdca204/d17f107617a7c22c1510ce981ad56ef6420683d2/pex", line 258, in <module>
224
E ast = compile(content, filename, "exec", flags=0, dont_inherit=1)
225
E File "./inspect_platform.py", line 23
226
E arch: str
227
E ^
228
E SyntaxError: invalid syntax
229b
bitter-ability-32190
03/31/2023, 5:51 PMYeah I'm feeling more likely it's the linked issue.
p
proud-dentist-22844
03/31/2023, 5:52 PMyou didn't link to an issue - got a link?
b
bitter-ability-32190
03/31/2023, 5:56 PMOh wow my link was not what I thought it was lol
bitter-ability-32190
03/31/2023, 5:56 PMp
proud-dentist-22844
03/31/2023, 6:02 PMoh. Ok. That sounds like my issue.
proud-dentist-22844
03/31/2023, 6:03 PMHas it been cherry-picked to 2.16 yet?
proud-dentist-22844
03/31/2023, 6:03 PMWould you like me to cherry-pick it?
b
bitter-ability-32190
03/31/2023, 6:07 PMBe my guest!
bitter-ability-32190
03/31/2023, 6:31 PMSounds like John is going to do it 🙏
p
proud-dentist-22844
03/31/2023, 6:31 PMI cherry-picked - he just approved. 🙂
proud-dentist-22844
04/01/2023, 7:10 PM@bitter-ability-32190 that PexPEX fix (landed in 2.16.0rc0) resolved my issue. Thanks!
🙌 1
2 Views