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

fix local docker build #1105

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

fix local docker build #1105

wants to merge 3 commits into from

Conversation

DanL0
Copy link
Contributor

@DanL0 DanL0 commented Dec 4, 2024

should fix:

pnpm test:local

@ryandgoulding
Copy link
Contributor

Pulling and testing locally now.

@ryandgoulding
Copy link
Contributor

pnpm test:local does not work locally for me:

@layerzerolabs/test-devtools-ton:test:   ● TON local node › should be available
@layerzerolabs/test-devtools-ton:test: 
@layerzerolabs/test-devtools-ton:test:     expect(received).resolves.toEqual()
@layerzerolabs/test-devtools-ton:test: 
@layerzerolabs/test-devtools-ton:test:     Received promise rejected instead of resolved
@layerzerolabs/test-devtools-ton:test:     Rejected to value: [Error: connect ECONNREFUSED ::1:8081]
@layerzerolabs/test-devtools-ton:test: 
@layerzerolabs/test-devtools-ton:test:        7 |         const client = new TonClient({ endpoint })
@layerzerolabs/test-devtools-ton:test:        8 |
@layerzerolabs/test-devtools-ton:test:     >  9 |         await expect(client.getMasterchainInfo()).resolves.toEqual({
@layerzerolabs/test-devtools-ton:test:          |               ^
@layerzerolabs/test-devtools-ton:test:       10 |             workchain: -1,
@layerzerolabs/test-devtools-ton:test:       11 |             initSeqno: 0,
@layerzerolabs/test-devtools-ton:test:       12 |             latestSeqno: expect.any(Number),
@layerzerolabs/test-devtools-ton:test: 
@layerzerolabs/test-devtools-ton:test:       at expect (../../node_modules/.pnpm/expect@29.7.0/node_modules/expect/build/index.js:113:15)
@layerzerolabs/test-devtools-ton:test:       at Object.expect (test/node.spec.ts:9:15)

The ton node is started in a separate terminal using docker-compose up. Any ideas?

Dockerfile Outdated
@@ -153,7 +153,7 @@ ARG CARGO_BUILD_JOBS=default
ENV CARGO_BUILD_JOBS=$CARGO_BUILD_JOBS

# Install AVM - Anchor version manager for Solana
RUN cargo install --git https://github.com/coral-xyz/anchor avm
RUN cargo install --git https://github.com/coral-xyz/anchor avm --tag v0.30.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the version tag should be made 0.29.0 to align with the needed Anchor version

@nazreen
Copy link
Contributor

nazreen commented Dec 9, 2024

I faced a different error

 => ERROR [network-vengaboys aptos 4/7] RUN cargo build --package aptos --profile cli 
...
132.0     ... which satisfies git dependency `server-framework` (locked to 1.0.0) of package `aptos v4.2.3 (/app/aptos/src/crates/aptos)`
132.0     ... which satisfies path dependency `aptos` (locked to 4.2.3) of package `aptos-release-builder v0.1.0 (/app/aptos/src/aptos-move/aptos-release-builder)`
132.0 
132.0 Caused by:
132.0   failed to load source for dependency `aptos-system-utils`
132.0 
132.0 Caused by:
132.0   Unable to update https://github.com/aptos-labs/aptos-core.git?rev=202bdccff2b2d333a385ae86a4fcf23e89da9f62#202bdccf
132.0 
132.0 Caused by:
132.0   revspec '202bdccff2b2d333a385ae86a4fcf23e89da9f62' not found; class=Reference (4); code=NotFound (-3)
------
failed to solve: process "/bin/sh -c cargo build --package aptos --profile cli" did not complete successfully: exit code: 101
 ELIFECYCLE  Command failed with exit code 17.
 ELIFECYCLE  Command failed with exit code 17.

@DanL0 DanL0 marked this pull request as draft January 10, 2025 07:35
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 this pull request may close these issues.

3 participants