Skip to content

chore(consensus): [CON-1341] inline node_registration_test and ssh_access_to_nodes_test system tests#481

Merged
kpop-dfinity merged 32 commits into
masterfrom
kpop/CON-1341/inline_ssh_access_to_nodes_test_and_node_registration_test
Jul 22, 2024
Merged

chore(consensus): [CON-1341] inline node_registration_test and ssh_access_to_nodes_test system tests#481
kpop-dfinity merged 32 commits into
masterfrom
kpop/CON-1341/inline_ssh_access_to_nodes_test_and_node_registration_test

Conversation

@kpop-dfinity
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the chore label Jul 19, 2024
@kpop-dfinity kpop-dfinity marked this pull request as ready for review July 19, 2024 15:21
@kpop-dfinity kpop-dfinity requested review from a team as code owners July 19, 2024 15:21
Comment thread rs/tests/consensus/orchestrator/node_registration_test.rs Outdated
Comment thread rs/tests/consensus/orchestrator/ssh_access_to_nodes_test.rs Outdated
Copy link
Copy Markdown
Collaborator

@basvandijk basvandijk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome job inlining these tests. I'll use you as a shining example that other teams should follow!

@kpop-dfinity kpop-dfinity enabled auto-merge July 22, 2024 07:11
@kpop-dfinity kpop-dfinity added this pull request to the merge queue Jul 22, 2024
Merged via the queue into master with commit 311f6a7 Jul 22, 2024
@kpop-dfinity kpop-dfinity deleted the kpop/CON-1341/inline_ssh_access_to_nodes_test_and_node_registration_test branch July 22, 2024 08:20
tmu0 added a commit that referenced this pull request May 11, 2026
ic-bn-lib 0.1.15 pinned `rand = "=0.8.5"`; 0.1.19 relaxes that to allow
0.8.6, which closes the remaining rand alerts (#476, #479, #481).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants