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

add ability to spin up a node based on a branch #118

Open
brunopgalvao opened this issue Apr 24, 2024 · 4 comments
Open

add ability to spin up a node based on a branch #118

brunopgalvao opened this issue Apr 24, 2024 · 4 comments
Labels
parachain Features related to parachain functionality

Comments

@brunopgalvao
Copy link
Collaborator

brunopgalvao commented Apr 24, 2024

pop up parachain -f ./network.toml -p https://github.com/r0gue-io/pop-node

As a user, I would like to run pop up parachain command specifying a specific branch so that I can test pop-node features that are on a branch and not yet on main.

Background: Right now I have made some changes to the Pop API and I want to test my ink! smart contract against it. How do I do this with pop up parachain command?

@brunopgalvao
Copy link
Collaborator Author

Apparently this exists with the -p flag:

  -p, --parachain <PARACHAIN>
    The url of the git repository of a parachain to be used, with branch/release tag specified as #fragment (e.g. 'https://github.com/org/repository#tag'). 
    A specific binary name can also be optionally specified via query string parameter (e.g. 'https://github.com/org/repository?binaryname#tag'), defaulting to the name of the repository when not specified

@brunopgalvao
Copy link
Collaborator Author

brunopgalvao commented Apr 26, 2024

Re-opening this as using -p specifying the branch did not work for me.

Can someone confirm that specifying https://github.com/org/repository#branch works and include the steps to reproduce.

self.parachain.as_ref(),

Seems like the logic is handled by zombienet-sdk

@brunopgalvao brunopgalvao reopened this Apr 26, 2024
@AlexD10S
Copy link
Collaborator

This functionality exists, code is here https://github.com/r0gue-io/pop-cli/blob/main/crates/pop-parachains/src/utils/git.rs#L21

However what is probably happening is that you have the pop-node cached already and don't clone it.

This issue raised by Frank letting user inspect and clean cache might be the best solution: #78. What do you think?

@AlexD10S AlexD10S added bug Something isn't working parachain Features related to parachain functionality and removed discussion labels May 22, 2024
@brunopgalvao
Copy link
Collaborator Author

cargo run up parachain -f ./tests/zombienet.toml -r v1.8.0 -p https://github.com/r0gue-io/pop-node#frank/feat-token

Running this command does not run the branch specified. Issue is still relevant.

@AlexD10S AlexD10S removed the bug Something isn't working label Jul 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
parachain Features related to parachain functionality
Projects
None yet
Development

No branches or pull requests

2 participants