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.

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

Remote labels #696

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Commits on Jul 19, 2022

  1. Replace address resolve with snapshot labels

    Previously, the demux-snapshotter had to make an http request to the
    http-address-resolver to find the vsock address and metrics
    configuration for a remote snapshotter. With this change, the
    http-address-resolver is removed and this information is sent to the
    demux-snapshotter via snapshot labels directly from the client.
    
    The labels are sent to the demux-snapshotter in the first pepare call
    at which point the demux-snapshotter will establish connection with the
    remote snapshotter. The connection is then cached via the namespace key
    as it was in the previous implementation for subsequent calls.
    
    Since each microVM houses its own remote snapshotter in a clean slate on
    boot, any snapshot API call before a prepare will be operating on an empty
    snapshotter and therefore will be a noop/not found. Therefore, if we
    don't have a connection for an API, we can treat it as the noop/not found
    case.
    
    Signed-off-by: Kern Walster <walster@amazon.com>
    Kern-- committed Jul 19, 2022
    Configuration menu
    Copy the full SHA
    cdb75d2 View commit details
    Browse the repository at this point in the history