stacks-blockchain-api icon indicating copy to clipboard operation
stacks-blockchain-api copied to clipboard

Rosetta cli construction fails while trying to run stacks-blockchain-api with mocknet

Open HassanAmed opened this issue 2 years ago • 5 comments
trafficstars

Describe the bug

I tried to run stacks-api-blockchain api as mocknet using this command npm run dev:integrated which failed I guess the problem is this, in docker/docker-compose.dev.rosetta.cli.yaml command: ${CMD} volume: ${OUTPUT} ${CMD} and ${OUTPUT} are undefined I get this output

WARN[0000] The "OUTPUT" variable is not set. Defaulting to a blank string. 
WARN[0000] The "CMD" variable is not set. Defaulting to a blank string. 
1 error(s) decoding:

* error decoding 'Volumes[0]': invalid empty volume spec

Steps To Reproduce

  1. git clone https://github.com/hirosystems/stacks-blockchain-api.git
  2. npm run dev:integrated

Expected behavior

should successfully run stacks-blockchain-api with mocknet

Environment

  • OS: macOS Monterey M1
  • Rust version: rustc 1.64.0

Additional context

When i set command: /bin/rosetta-cli --configuration-file /app/rosetta-config-docker.json check:construction volume: - ./rosetta-output-construction:/app/rosetta-output in docker/docker-compose.dev.rosetta.cli.yaml it runs successfully

HassanAmed avatar Feb 15 '23 23:02 HassanAmed

Hi @HassanAmed we will update documentation and tooling because this is a very old command that should be removed in favor of using Clarinet

rafaelcr avatar Mar 09 '23 17:03 rafaelcr

Hi @rafaelcr, I would like to fix this. Does changing the command and volume to command: /bin/rosetta-cli --configuration-file /app/rosetta-config-docker.json check:construction volume: - ./rosetta-output-construction:/app/rosetta-output fix the issue. I was able to run the stacks-blockchain-api successfully after the change. Let me know if anything else needs to be modified. I'll make a PR with the updates

neilmehta31 avatar May 07 '23 13:05 neilmehta31

Hi @neilmehta31 that's great! Please send the PR so we can review, thanks

rafaelcr avatar May 08 '23 02:05 rafaelcr

@rafaelcr Please review the PR : https://github.com/hirosystems/stacks-blockchain-api/pull/1647

neilmehta31 avatar May 09 '23 06:05 neilmehta31

@rafaelcr Please review the PR, I rebased with develop branch. Had problems in the earlier PR, hence created a new PR : https://github.com/hirosystems/stacks-blockchain-api/pull/1652. Let me know if any other changes need to be made. Thanks Sorry for the inconvenience

neilmehta31 avatar May 10 '23 08:05 neilmehta31

Looks like https://github.com/hirosystems/stacks-blockchain-api/issues/1666 closes this issue

zone117x avatar Apr 22 '24 10:04 zone117x