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

Closed
wants to merge 9 commits into from
Closed

fix local docker build #1105

wants to merge 9 commits into from

Conversation

DanL0
Copy link
Contributor

@DanL0 DanL0 commented Dec 4, 2024

No description provided.

@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/[email protected]/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 Show resolved Hide resolved
@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
@ryandgoulding
Copy link
Contributor

Is this still relevant? Trying to determine what is stale and what is still applicable.

@DanL0
Copy link
Contributor Author

DanL0 commented Feb 6, 2025

I think we decided to go in a bit different direction. This might not be necessarily relevant anymore. Closing for now.

@DanL0 DanL0 closed this Feb 6, 2025
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