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

🐛 Start the Cache if the Manager has already started #1681

Merged
merged 3 commits into from
Oct 8, 2021
Merged

🐛 Start the Cache if the Manager has already started #1681

merged 3 commits into from
Oct 8, 2021

Commits on Oct 4, 2021

  1. Start the Cache if the Manager has already started

    @jsanda
    jsanda committed Oct 4, 2021
    Configuration menu
    Copy the full SHA
    8c25b12 View commit details
    Browse the repository at this point in the history

Commits on Oct 8, 2021

  1. verify adding cluster after mgr started results in working cache 

    Add a test that adds a cluster to the manager after the manager has
already started. Verify that the cluster is started and its cache
is sycned.

Added the startClusterAfterManager struct which is a basically just a
hook to verify that the cluster is started.
    @jsanda
    jsanda committed Oct 8, 2021
    Configuration menu
    Copy the full SHA
    6ea9679 View commit details
    Browse the repository at this point in the history

  2. remove unnecessary methods 

    Only GetCache and Start methods are neeed for new test that adds a
cluster to the manage after the manager has already started.
    @jsanda
    jsanda committed Oct 8, 2021
    Configuration menu
    Copy the full SHA
    10aed67 View commit details
    Browse the repository at this point in the history