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

Cosmovisor v1.2 can't find the upgrade folder if plan name contains capital letters #12915

Closed
RaulBernal opened this issue Aug 12, 2022 · 1 comment · Fixed by #12918
Closed
Assignees
Labels
C:Cosmovisor Issues and PR related to Cosmovisor T:Bug

Comments

@RaulBernal
Copy link

Summary of Bug

When a new upgrade happens Cosmovisor can't find the proper folder path if plan name (upgrade name) contains capital letters.
In example, we perform the upgrade with plan name "strangeBuddheads" and Cosmovisor was looking for "strangebuddheads".

Aug 11 19:10:31 seed1.bitcanna.io cosmovisor[109539]: Error: binary not present, downloading disabled: cannot stat dir /home/raul/.bcna/cosmovisor/upgrades/strangebuddheads/bin/bcnad: stat /home/raul/.bcna/cosmovisor/upgrades/strangebuddheads/bin/bcnad: no such file or directory
Aug 11 19:10:31 seed1.bitcanna.io cosmovisor[109539]: 7:10PM ERR error="binary not present, downloading disabled: cannot stat dir /home/raul/.bcna/cosmovisor/upgrades/strangebuddheads/bin/bcnad: stat /home/raul/.bcna/cosmovisor/upgrades/strangebuddheads/bin/bcnad: no such file or directory" module=cosmovisor 

While the plan name was "strangeBuddheads"

2:24PM INF ABCI Replay Blocks appHeight=4490419 module=consensus stateHeight=4490419 storeHeight=4490420
2:24PM INF Replay last block using real app module=consensus
2:24PM INF applying upgrade "strangeBuddheads" at height: 4490420
2:24PM INF adding a new module: authz
2:24PM INF minted coins from module account amount=4465309ubcna from=mint module=x/bank

Version

v1.2

Steps to Reproduce

  1. Send a proposal for to upgrade the network. Name the plan "strangeBuddheads" or any other that contains capital letters.
  2. Setup the folders for cosmovisor and the binary
  3. Wait for the upgrade and check if Cosmovisor looks in the proper folder path.
RaulBernal added a commit to BitCannaGlobal/bcna that referenced this issue Aug 12, 2022
RaulBernal added a commit to BitCannaGlobal/bcna that referenced this issue Aug 12, 2022
@julienrbrt julienrbrt added C:Cosmovisor Issues and PR related to Cosmovisor T:Bug T:Sprint labels Aug 12, 2022
@julienrbrt julienrbrt self-assigned this Aug 13, 2022
@julienrbrt
Copy link
Member

As per comment in #12918, this is an expected behavior, #12921 will add documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:Cosmovisor Issues and PR related to Cosmovisor T:Bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants