Skip to content

fix(iroha_config): broken trusted peers check#5121

Merged
0x009922 merged 1 commit intohyperledger-iroha:mainfrom
0x009922:fix/config-trusted-peers-check
Oct 8, 2024
Merged

fix(iroha_config): broken trusted peers check#5121
0x009922 merged 1 commit intohyperledger-iroha:mainfrom
0x009922:fix/config-trusted-peers-check

Conversation

@0x009922
Copy link
Copy Markdown
Contributor

@0x009922 0x009922 commented Oct 4, 2024

Context

Split from #5087

When there was one trusted peer, it decided that there are no.

Solution

Fix the check

Migration Guide (optional)

None


Review notes (optional)

None

Checklist

  • I've read CONTRIBUTING.md.
  • (optional) I've written unit tests for the code changes.
  • All review comments have been resolved.
  • All CI checks pass.

@0x009922 0x009922 added Bug Something isn't working Chore This is a small task that can be done at any point in time and is easier than others labels Oct 4, 2024
@0x009922 0x009922 self-assigned this Oct 4, 2024
Comment thread crates/iroha_config/Cargo.toml
@s8sato s8sato self-assigned this Oct 7, 2024
@mversic
Copy link
Copy Markdown
Contributor

mversic commented Oct 7, 2024

there seem to be a test failing

@0x009922
Copy link
Copy Markdown
Contributor Author

0x009922 commented Oct 8, 2024

there seem to be a test failing

Yup, flaky unrelated test (integration::triggers::by_call_trigger::unregistering_one_of_two_triggers_with_identical_wasm_should_not_cause_original_wasm_loss)

Signed-off-by: 0x009922 <43530070+0x009922@users.noreply.github.com>
@0x009922 0x009922 force-pushed the fix/config-trusted-peers-check branch from 41c155d to e43db16 Compare October 8, 2024 05:12
@0x009922 0x009922 enabled auto-merge (squash) October 8, 2024 05:12
@0x009922 0x009922 merged commit 0059c39 into hyperledger-iroha:main Oct 8, 2024
@alexstroke1 alexstroke1 self-assigned this Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working Chore This is a small task that can be done at any point in time and is easier than others

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants