Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parachains zombienet fails on local machines #1765

Closed
alexggh opened this issue Sep 30, 2023 · 1 comment · Fixed by #1780
Closed

Parachains zombienet fails on local machines #1765

alexggh opened this issue Sep 30, 2023 · 1 comment · Fixed by #1780

Comments

@alexggh
Copy link
Contributor

alexggh commented Sep 30, 2023

The error I'm getting when running zombienet locally is:

WARN tokio-runtime-worker parachain::pvf: cannot bind unix socket: Custom { kind: InvalidInput, 
error: "path must be shorter than libc::sockaddr_un.sun_path" } 
debug_id=prepare program_path="/home/alexggh/repos/polkadot-sdk/target/testnet/polkadot-prepare-worker" 
extra_args=["prepare-worker", "--node-impl-version", "1.1.0"] 
worker_dir=WorkerDir { path: "/tmp/zombie-a609ef68a12d8517c8174b62ee3a0899_-484651-6nFd48QbCHfO/alice/data/chains/rococo_local_testnet/db/full/pvf-artifacts/worker-dir-prepare-2Q4B1v7qio" } 
socket_path="/tmp/zombie-a609ef68a12d8517c8174b62ee3a0899_-484651-6nFd48QbCHfO/alice/data/chains/rococo_local_testnet/db/full/pvf-artifacts/worker-dir-prepare-2Q4B1v7qio/socket"

This is because by default zombienet decides to prefix things with /tmp/zombie-a609ef68a12d8517c8174b62ee3a0899_-484651-6nFd48QbCHfO and that gets us past the socket limitation size.

Workaround

This can be easily worked around by just specifying explicitly a shorter prefix on zombienet with:

zombienet -d /tmp/x/ <your_zombienet_params>

This was introduced with #1373 fyi @mrcnski and it wasn't caught by the CI because it has a shorter prefix.

Given that we can't really control what prefix paths our users decide to use or already have configured on their validators I think we should try to either prevent this limitation or expose(if it doesn't exist already) some cli parameter or environment variable for users to be able to manually decide this path.

@mrcnski
Copy link
Contributor

mrcnski commented Oct 1, 2023

Ooh, looks like the limit is 108 bytes. This should definitely be fixed, because like you say, users are allowed to use an arbitrary cache path. There may be no need for the socket to be under the cache path though, I think we can revert to using a tmp location like we were before.

mrcnski added a commit that referenced this issue Oct 3, 2023
Looks like [the socket path limit is 108
bytes](https://man7.org/linux/man-pages/man7/unix.7.html). This should
definitely be fixed, users are allowed to use an arbitrary cache path. There may
be no need for the socket to be under the cache path though, I think we can
revert to using a tmp location like we were before.

Closes #1765
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Mar 26, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Mar 27, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 8, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 9, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 10, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
serban300 pushed a commit to serban300/polkadot-sdk that referenced this issue Apr 10, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (paritytech#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
bkchr pushed a commit that referenced this issue Apr 10, 2024
…ED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH (#1765)

* increase MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX for RBH/WBH

* moved MAX_UNREWARDED_RELAYERS_IN_CONFIRMATION_TX and MAX_UNCONFIRMED_MESSAGES_IN_CONFIRMATION_TX to cumulus bridge primitives

* typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants