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.

Sign up for GitHub

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

Jump to bottom

Fix many flaky tests on macOs #281

Closed
wants to merge 10 commits into from
Closed

Fix many flaky tests on macOs #281

wants to merge 10 commits into from

Commits on May 15, 2023

  1. bugfix: Connect() panics on schema update 

    Part of #278
    @oleg-jukovec
    oleg-jukovec committed May 15, 2023
    Configuration menu
    Copy the full SHA
    fe2cf85 View commit details
    Browse the repository at this point in the history

  2. queue: bump queue module to 1.3.0 

    Part of #278
    @oleg-jukovec
    oleg-jukovec committed May 15, 2023
    Configuration menu
    Copy the full SHA
    ff58114 View commit details
    Browse the repository at this point in the history

  3. bugfix: set wrong Ttr by Queue.Cfg() 

    Ttr should be in seconds.

Part of #278
    @oleg-jukovec
    oleg-jukovec committed May 15, 2023
    Configuration menu
    Copy the full SHA
    94f3b72 View commit details
    Browse the repository at this point in the history

  4. bugfix: call queue.cfg() before box.cfg() 

    We don't have stable results if queue.cfg({in_replicaset = true})
is called after box.cfg(). Probably it is expected [1].

1. tarantool/queue#206

Part of #278
    @oleg-jukovec
    oleg-jukovec committed May 15, 2023
    Configuration menu
    Copy the full SHA
    80bf863 View commit details
    Browse the repository at this point in the history

  5. test: fix flaky queue/Example_connectionPool 

    We need to wait for an additional events to make the test stable:

1. A ready state for a queue.
2. A success queue configuration on all instances.
3. An available RW instance.
4. A success role switch.

Closes #278
    @oleg-jukovec
    oleg-jukovec committed May 15, 2023
    Configuration menu
    Copy the full SHA
    ac5be0d View commit details
    Browse the repository at this point in the history

  6. crud: bump crud module to 1.1.1 

    Part of #288
    @oleg-jukovec
    oleg-jukovec committed May 15, 2023
    Configuration menu
    Copy the full SHA
    75bff3f View commit details
    Browse the repository at this point in the history

  7. test: fix flaky crud tests 

    An instance was listening on a testing port until the configuration
was complete. At the end of the configuration, the port was
reopened.

As a result, we saw connection loss in tests.

Closes #288
    @oleg-jukovec
    oleg-jukovec committed May 15, 2023
    Configuration menu
    Copy the full SHA
    bd29b39 View commit details
    Browse the repository at this point in the history

  8. test: wait for queue cluster 

    We need to wait for a success role cluster configuration because an
instance bootstrap may hasn't finished yet.

Part of #291
    @oleg-jukovec
    oleg-jukovec committed May 15, 2023
    Configuration menu
    Copy the full SHA
    a215108 View commit details
    Browse the repository at this point in the history

  9. test: fix TestGracefulShutdownCloseConcurrent hang 

    We should release the lock even if connect fails.

Part of #282
    @oleg-jukovec
    oleg-jukovec committed May 15, 2023
    Configuration menu
    Copy the full SHA
    893ccf8 View commit details
    Browse the repository at this point in the history

  10. test: increase timeout for all tests 

    Increased timeout for all test's server options.
Have done it to solve the problem with macOs flaky tests.

Closes #277
Closes #282
    @better0fdead @oleg-jukovec
    better0fdead authored and oleg-jukovec committed May 15, 2023
    Configuration menu
    Copy the full SHA
    1ce5c9e View commit details
    Browse the repository at this point in the history