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

experimental client: Remove mirrors #1373

Merged
merged 10 commits into from
May 12, 2021

Commits on Apr 29, 2021

  1. Download metadata from a single mirror

    Keep the current API and mirrors configuration but
    use only the first mirror from the list for metadata
    download.
    
    Target files download remains unchanged.
    
    Signed-off-by: Teodora Sechkova <tsechkova@vmware.com>
    sechkova committed Apr 29, 2021
    Configuration menu
    Copy the full SHA
    80ff532 View commit details
    Browse the repository at this point in the history
  2. Drop mirrors support

    Updater now uses only a single url for metadata download.
    Target files download use either a default url or an
    optional one for each file passed by the caller.
    
    Signed-off-by: Teodora Sechkova <tsechkova@vmware.com>
    sechkova committed Apr 29, 2021
    Configuration menu
    Copy the full SHA
    576d055 View commit details
    Browse the repository at this point in the history

Commits on May 4, 2021

  1. new updater: Seek to beginning of file after length check

    Signed-off-by: Jussi Kukkonen <jkukkonen@vmware.com>
    Jussi Kukkonen committed May 4, 2021
    Configuration menu
    Copy the full SHA
    45259cf View commit details
    Browse the repository at this point in the history
  2. new updater: remove unnecessary file objects

    Removing mirrors means we no longer need to do file object handling
    manually.
    
    Note that this means we're now exposing the Updater caller to all kinds
    of new exceptions (as NoWorkingMirrorError is no longer an excuse we can
    use).
    
    Signed-off-by: Jussi Kukkonen <jkukkonen@vmware.com>
    Jussi Kukkonen committed May 4, 2021
    Configuration menu
    Copy the full SHA
    5732163 View commit details
    Browse the repository at this point in the history
  3. new updater: Rename _get_target_hash()

    The function actually hashes the target filepath.
    
    Signed-off-by: Jussi Kukkonen <jkukkonen@vmware.com>
    Jussi Kukkonen committed May 4, 2021
    Configuration menu
    Copy the full SHA
    9fae500 View commit details
    Browse the repository at this point in the history
  4. new updater: Remove misleading comment

    Signed-off-by: Jussi Kukkonen <jkukkonen@vmware.com>
    Jussi Kukkonen committed May 4, 2021
    Configuration menu
    Copy the full SHA
    888f022 View commit details
    Browse the repository at this point in the history
  5. Rename url prefixes so they are consistent

    Signed-off-by: Jussi Kukkonen <jkukkonen@vmware.com>
    Jussi Kukkonen committed May 4, 2021
    Configuration menu
    Copy the full SHA
    3a02583 View commit details
    Browse the repository at this point in the history

Commits on May 5, 2021

  1. new updater: Clean up url handling

    * Make sure all base urls (prefixes) end in a slash
    * Add documentation to get_one_valid_targetinfo(): That is the one place
      where the API accepts ill-defined "paths" from the caller
    * Remove checks from download url handling: we control both the base url
      and the relative path so there should be no surprises here.
    
    Signed-off-by: Jussi Kukkonen <jkukkonen@vmware.com>
    Jussi Kukkonen committed May 5, 2021
    Configuration menu
    Copy the full SHA
    ab210b4 View commit details
    Browse the repository at this point in the history

Commits on May 7, 2021

  1. new updater: Improve docstrings

    Signed-off-by: Jussi Kukkonen <jkukkonen@vmware.com>
    Jussi Kukkonen committed May 7, 2021
    Configuration menu
    Copy the full SHA
    9605e19 View commit details
    Browse the repository at this point in the history

Commits on May 12, 2021

  1. tests: Don't use os.path.join() for URLS

    The test has issues like this alsready but let's not add more...
    
    Signed-off-by: Jussi Kukkonen <jkukkonen@vmware.com>
    Jussi Kukkonen committed May 12, 2021
    Configuration menu
    Copy the full SHA
    ec4c5ce View commit details
    Browse the repository at this point in the history