Skip to content

Try to seek for race conditions occurring in Actions only #445

Description

@bertllll

I've experienced them more than little times in: Node/masq/communication/communication_manager
at more places in there.

Activity

  1. dnwiebe commented on Feb 6, 2021

    @dnwiebe
    Collaborator

    We can move this out of Backlog when we get some specific details about particular tests that fail sometimes but not always in Actions. When you experience that, post a description in a comment here and attach the downloaded console log from Actions.

  2. changed the title [-]Try to seek for Race Conditions occurring in Actions only: masq/commu...[/-] [+]Try to seek for Race Conditions occurring in Actions only[/+] on Feb 12, 2021
  3. changed the title [-]Try to seek for Race Conditions occurring in Actions only[/-] [+]Try to seek for race conditions occurring in Actions only[/+] on Feb 12, 2021
  4. bertllll commented on Feb 12, 2021

    @bertllll
    ContributorAuthor

    I got this despite this area of our code could not be affected by the worked card (GH-381). Happened after a break from seeing any strangely failing tests. When a new Rust version came.

    thread 'main' panicked at 'ClientListenerThread could not be restarted: Timeout', D:\a\Node\Node\masq\src\communications\connection_manager.rs:455:23
    stack backtrace:
    0: 0x7ff74c24187e - std::backtrace_rs::backtrace::dbghelp::trace
    test communications::connection_manager::tests::handles_outgoing_fire_and_forget_messages_to_dead_server ... FAILED
    at /rustc/cb75ad5db02783e8b0222fee363c5f63f7e2cf5b/library\std\src....\backtrace\src\backtrace\dbghelp.rs:98
    1: 0x7ff74c24187e - std::backtrace_rs::backtrace::trace_unsynchronized
    at /rustc/cb75ad5db02783e8b0222fee363c5f63f7e2cf5b/library\std\src....\backtrace\src\backtrace\mod.rs:66
    2: 0x7ff74c24187e - std::sys_common::backtrace::_print_fmt
    at /rustc/cb75ad5db02783e8b0222fee363c5f63f7e2cf5b/library\std\src\sys_common\backtrace.rs:67

    communications::connection_manager::tests::handles_outgoing_fire_and_forget_messages_to_dead_server

  5. bertllll commented on Feb 15, 2021

    @bertllll
    ContributorAuthor

    This problem occurred at the same day as the previous but I didn't have time to report it. It showed itself once and in the next run Actions did not complain though I did no changes in the code.

    log multinode test

    multinode_integration_tests/tests/connection_termination_test

  6. dnwiebe commented on Feb 19, 2021

    @dnwiebe
    Collaborator

    One of the multinode tests failed: provided_and_consumed_services_are_recorded_in_databases.
    logs_473.zip

    The same test...Bert.
    2_Build node on ubuntu-latest.txt

    Again
    2_Build node on ubuntu-latest.txt

  7. bertllll commented on Mar 10, 2021

    @bertllll
    ContributorAuthor

    Windows ...one of node unit tests. No relation to the card being coped (GH-373).

    logs_509.zip

  8. bertllll commented on Mar 14, 2021

    @bertllll
    ContributorAuthor

    configurator_standard ... setting the gas price

    logs_519.zip

  9. bertllll commented on Mar 17, 2021

    @bertllll
    ContributorAuthor

    windows: tls_through_node_test (integration); I know this test because that's the one which permanently fails on my Win machine.

    logs_530.zip

  10. bertllll commented on Apr 6, 2021

    @bertllll
    ContributorAuthor

    MacOS: Node: unit_tests:
    initialize_with_clandestine_port_produces_expected_clandestine_discriminator_factories_vector

    logs_599.zip

  11. self-assigned this
    on Apr 6, 2021
  12. bertllll commented on Apr 10, 2021

    @bertllll
    ContributorAuthor

    This is unfortunate. By all evidences, GitHub stopped to support zipped packages and these ours which are attached to the comments here are inaccessible by now. I'd be happy if I was wrong.

    Now it accept zipped folders again...so...I don't know. But we lost the old ones.

  13. bertllll commented on Apr 15, 2021

    @bertllll
    ContributorAuthor

    handle_startup_and_shutdown_integration

    This test was renovated for this card, MASQ-Project/MASQ-Node-issues#386, however, usually it passes, this time it didn't. Maybe RC.

    3_Build node on windows-latest.txt
    ...
    3_Build node on windows-latest.txt

    And another one:
    22.6.21
    3_Build node on windows-latest.txt

    Hmmm...this is growing up fast
    24.6.2021
    3_Build node on windows-latest.txt
    26.6.2021 twice
    3_Build node on windows-latest.txt
    3_Build node on windows-latest.txt

    In MASQ-Project/MASQ-Node-issues#451, I was trying to fix this by adding a piece of logic to the code. It may happen that this error will evaporate then.

    Nope, without a relief, the frequency of this failure may even have increased.

    3_Build node on windows-latest.txt

    27th of September
    I tried to prevent this issue more reliably with a modification in fallback - connection manager. Card MASQ-Project/MASQ-Node-issues#305.

    4th October
    Not fixed yet

  14. 12 remaining items

  15. bertllll commented on Sep 14, 2021

    @bertllll
    ContributorAuthor

    blockchain::blockchain_interface::tests::blockchain_interface_non_clandestine_can_retrieve_eth_balance_of_a_wallet
    2_Build node on ubuntu-latest.txt

  16. FinsaasGH commented on Oct 21, 2021

    @FinsaasGH
    Collaborator

    multinode
    multiple_stream_zero_hop_test
    2_Build node on ubuntu-latest.txt

  17. bertllll commented on Oct 31, 2021

    @bertllll
    ContributorAuthor

    pass_args_or_print_messages_work_under_fine_lock_for_eof

    I guess this is a rare one. I suggest just to increase the multiplier in the assertion like from 3 to 4 or something. I know there is a couple of tests like this and this was the latest I added, maybe the chosen multiplier doesn't fit as well as for the other tests, I think its value could be a bit test specific.

    1_Build node on macOS-latest.txt

  18. bertllll commented on Feb 9, 2022

    @bertllll
    ContributorAuthor

    tls_through_node_integration

    This one is really rare in Actions I think, on the other hand we come to see it quite often on dev machines. Speaking mainly for myself (maybe occurred almost all the time) but I also saw it at Utkarsh.

    3_Build node on windows-latest.txt

  19. dnwiebe commented on Feb 14, 2022

    @dnwiebe
    Collaborator

    The multinode test tls_end_to_end_routing_test hung for most of five hours. We don't know why, but here's the log.

    logs_830.zip
    2_Build node on ubuntu-latest.txt

  20. utkarshg6 commented on Mar 3, 2022

    @utkarshg6
    Collaborator

    graph_connects_but_does_not_over_connect
    1_build.txt
    2_Build node on ubuntu-latest.txt

  21. bertllll commented on Apr 16, 2022

    @bertllll
    ContributorAuthor

    http_through_node_integration

    1_Build node on macOS-latest.txt

  22. bertllll commented on Apr 27, 2022

    @bertllll
    ContributorAuthor

    masq_erc20_contract_exists_on_polygon_mumbai_integration

    2_Build node on ubuntu-latest.txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions