Is the `test-release` post-release script known to...
# development
g
Is the
test-release
post-release script known to be broken? It doesn't seem to actually capture anything when I run it, despite the subprocess being set up to capture both stdout and stderr (my edit). Instead the version gets printed to the console:
before
is immediately before subprocess.run, and the other two are immediately after. The print can only come from the subprocess. I'm guessing this happens due to the rust client reopening the standard streams while setting up the tui?
s
I'm actually trying to fetch pants 2.19.0rc1 and it's failing
Copy code
Wasn't able to fetch the Pants PEX at <https://github.com/pantsbuild/pants/releases/download/release_2.19.0rc1/pants.cp39-linux_x86_64.pex>.
g
That is the wrong filename, hm
What are you running that errors? I don't think it's related to this failing test specifically
s
I've just switched the pants version, so every command is trying to download pants and is failing
c
which version of sice-pants are you using?
Copy code
PANTS_BOOTSTRAP_VERSION=report pants
s
0.10.0
c
there is a newer version, so try upgrading that first
s
yep, now it works, thanks!
👍 1
b
In answer to the original question, the release testing has worked for me previously
g
Ack, will dig into that more then.
Also doesn't work for me on 2.18, which I haven't touched... 😕
Copy code
git checkout 2.18.x && git pull upstream && ./pants run src/python/pants_release/release.py -- test-release
Filed https://github.com/pantsbuild/pants/issues/20283 about this, maybe someone else can try reproing if it's a setup issue? 🙂
b
Works for me on 2.19.x on (arm) mac, with scie-pants 0.10.3. What platform are you on? What version of scie-pants do you have?
g
Hmm. I tried on two WSL instances, not sure what version they're on. I'm travelling today but will check on laptop once I arrive.
Are you explicitly using
scie-pants
? I'm using
pants
from
get-pants.sh
-- which is the same binary, with maybe some slightly different semantics as far as I understand? Not sure. I just synced main +
SCIE_BOOT=update pants
on a fairly standard ubuntu, and still fail... Got 0.10.4 which should be latest.
c
when running on the pants repo, scie-pants shouldn’t matter, as it merely hands off to the
./pants
script..