On Wed, Feb 27, 2013 at 3:58 PM, Ross Brattain
<[email protected]> wrote:
>
> Hi,
>
> I ran into a problem with base_packages.GitFetcher.  It looks like 
> base_packages.BasePackageManager wants to grab a "packages.checksum" file to 
> check to see if a dependency needs to be updated when it has already been 
> installed.  BasePackageManager.parse_tarball_name fails when passed 
> "packages.checksum"
>
> I'm not sure of a good solution.  I will probably look into just disabling 
> the checksum feature for GitFetcher, since git archive --remote= probably 
> won't be able to fetch a packages.checksum directly anyway.

Hey Ross, sorry it took so long to work on this. I've opened issue

https://github.com/autotest/autotest/issues/603

And after investigating it, I came up with a simple fix and pushed it
to next. Please validate it in your test grid.

Cheers,

Lucas

commit 816c4ded25a4cc41bbf6eabd27786730f9e44510
Author: Lucas Meneghel Rodrigues <[email protected]>
Date:   Mon Mar 18 10:29:51 2013 -0300

    client.shared.base_packages: Override checksum feature for GitFetcher

    This is a solution for the issue #603.

    When adding a git repo as the package provider for autotest
    deps, the method BasePackageManager.install_pkg() will download
    and try to verify the checksum files for the given dependency.

    For example, if you use the following snippet:

    job.pkgmgr.add_repository("git://10.0.0.12/tools/iometer.git")
    job.setup_dep(['iometer'])

    This will fail because a GitFetcher doesn't have the ability
    to fetch .checksum files. So, to avoid the error, if the package
    repo is a git repo, let's skip checksum verification altogether,
    even overriding what the callers may have passed, since it
    doesn't make sense for the git fetcher anyway.

    Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>

_______________________________________________
Autotest-kernel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/autotest-kernel

Reply via email to