Skip to content

Conversation

@dzmitry-lahoda
Copy link
Contributor

@dzmitry-lahoda dzmitry-lahoda commented Aug 21, 2023

Reaching CVM to remote devnet (with configs).

And also docs/shells and usage on mainnet, with some data generated.

And docs updates, including returning docs to refactor (about cvm, I am updating it).

Required for merge:

  • pr-workflow-check / draft-release-check is ✅ success
  • Other rules GitHub shows you, or can be read in configuration

Makes review faster:

  • PR title is my best effort to provide summary of changes and has clear text to be part of release notes
  • I marked PR by misc label if it should not be in release notes
  • Linked Zenhub/Github/Slack/etc reference if one exists
  • I was clear on what type of deployment required to release my changes (node, runtime, contract, indexer, on chain operation, frontend, infrastructure) if any in PR title or description
  • Added reviewer into Reviewers
  • I tagged(@) or used other form of notification of one person who I think can handle best review of this PR
  • I have proved that PR has no general regressions of relevant features and processes required to release into production
  • Any dependency updates made, was done according guides from relevant dependency
  • Clicking all checkboxes
  • Adding detailed description of changes when it feels appropriate (for example when PR is big)

@dzmitry-lahoda dzmitry-lahoda added the Misc I marked PR by `misc` label if it should not be in release notes #owned:terraform label Aug 21, 2023
@github-actions
Copy link

Pull reviewers stats

Stats of the last 30 days for composable:

User Total reviews Time to review Total comments
dzmitry-lahoda 25 8h 12m 12
blasrodri 20 1h 58m 6
mina86 18 13h 19m 64
kkast 11 3h 17m 7
RustNinja 10 1h 24m 2
vmarkushin 5 7h 2
JafarAz 3 9h 25m 0
0xBrainjar2 2 3d 17h 35m 6

@github-actions
Copy link

# run Composable node
nix run "github:ComposableFi/composable/refs/pull/4091/merge" --allow-import-from-derivation --extra-experimental-features "flakes nix-command" --no-sandbox --accept-flake-config --option sandbox relaxed
# run local Picasso DevNet (for CosmWasm development)
nix run "github:ComposableFi/composable/refs/pull/4091/merge#devnet-picasso" --allow-import-from-derivation --extra-experimental-features "flakes nix-command" --no-sandbox --accept-flake-config --option sandbox relaxed 
# CosmWasm on Substrate CLI tool
nix run "github:ComposableFi/composable/refs/pull/4091/merge#ccw" --allow-import-from-derivation --extra-experimental-features "flakes nix-command" --no-sandbox --accept-flake-config --option sandbox relaxed 
# run cross chain devnet with Dotsama and Cosmos nodes 
nix run "github:ComposableFi/composable/refs/pull/4091/merge#devnet-xc-fresh" --allow-import-from-derivation --extra-experimental-features "flakes nix-command" --no-sandbox --accept-flake-config --option sandbox relaxed 
# or same with docker
nix build "github:ComposableFi/composable/refs/pull/4091/merge#devnet-xc-image" --allow-import-from-derivation --extra-experimental-features "flakes nix-command" --no-sandbox --accept-flake-config --option sandbox relaxed \
&& docker load --input result && docker run -it --entrypoint bash devnet-xc:latest -c /bin/devnet-xc-fresh 

About nix

@dzmitry-lahoda dzmitry-lahoda marked this pull request as ready for review August 21, 2023 12:51
@dzmitry-lahoda dzmitry-lahoda requested a review from kkast August 21, 2023 12:53
auto-merge was automatically disabled August 24, 2023 10:09

Merge queue setting changed

@github-actions
Copy link

stale-pr

@github-actions github-actions bot added the Stale-item bot: Stale PRs and issues handling #owned:terraform label Aug 26, 2023
@dzmitry-lahoda
Copy link
Contributor Author

needed

@github-actions github-actions bot added Stale-item bot: Stale PRs and issues handling #owned:terraform and removed Stale-item bot: Stale PRs and issues handling #owned:terraform labels Aug 27, 2023
@ComposableFi ComposableFi deleted a comment from github-actions bot Sep 2, 2023
@dzmitry-lahoda dzmitry-lahoda removed the Stale-item bot: Stale PRs and issues handling #owned:terraform label Sep 2, 2023
@dzmitry-lahoda dzmitry-lahoda changed the title ci(repositores): create repo to simplify nix usage ci(repositores): create repo to simplify nix usage for ETH Sep 6, 2023
@dzmitry-lahoda dzmitry-lahoda changed the title ci(repositores): create repo to simplify nix usage for ETH chore(nix): making nix more "usable" Sep 6, 2023
@dzmitry-lahoda dzmitry-lahoda marked this pull request as draft September 8, 2023 11:19
@github-actions
Copy link

stale-pr

@github-actions github-actions bot added the Stale-item bot: Stale PRs and issues handling #owned:terraform label Sep 13, 2023
@github-actions
Copy link

close-pr

@github-actions github-actions bot closed this Sep 17, 2023
@github-actions github-actions bot removed the Stale-item bot: Stale PRs and issues handling #owned:terraform label Sep 18, 2023
@dzmitry-lahoda dzmitry-lahoda marked this pull request as ready for review September 18, 2023 17:43
@dzmitry-lahoda dzmitry-lahoda changed the title chore(nix): making nix more "usable" chore(nix): cvm runs on devnet remote and examples/shells from mainnet Sep 18, 2023
@dzmitry-lahoda
Copy link
Contributor Author

merging as improve devnet stability

@dzmitry-lahoda dzmitry-lahoda merged commit 17f6f27 into main Sep 19, 2023
@dzmitry-lahoda dzmitry-lahoda deleted the dz/1 branch September 19, 2023 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Misc I marked PR by `misc` label if it should not be in release notes #owned:terraform

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant