Re: [OE-core] [PATCH 2/2] go: Use dl.google.com for SRC_URI

["Khem Raj" <raj.khem@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 4899 bytes --]

On Fri, Apr 30, 2021 at 9:50 AM Michael Halstead <
mhalstead@linuxfoundation.org> wrote:

> I've removed the following:
> go1.16.3.src.tar.gz ( 0 bytes)
> go1.16.3.src.tar.gz_bad-checksum_b298d29de9236ca47a023e382313bcc2d2eed31dfa706b60a04103ce83a71a25
> ( 20912861 bytes)
>
> I can add a job to check for 0 byte tarballs. Do you think that would be
> helpful?
>

Yes it will be very helpful and also remove the bad-checksum files too they
are bad downloads of corrupt tarballs

>
>
>
> On Thu, Apr 29, 2021 at 4:26 PM Khem Raj <raj.khem@gmail.com> wrote:
>
>> you also want to get rid of *.lock files as well.
>>
>> On Thu, Apr 29, 2021 at 4:24 PM Khem Raj <raj.khem@gmail.com> wrote:
>> >
>> > Adding Miachel as well.
>> >
>> > this seems to be running
>> > buildoptions.SourceMirroring.test_yocto_source_mirror selftest, which
>> > AFAICT should force downloads from yp mirror only
>> > when I check the YP mirror then this tarball exists there
>> >
>> > http://downloads.yoctoproject.org/mirror/sources/go1.16.3.src.tar.gz
>> >
>> > but problem is that its 0 sized.
>> >
>> > Secondly I also see a bad checksum file there
>> >
>> >
>> http://downloads.yoctoproject.org/mirror/sources/go1.16.3.src.tar.gz_bad-checksum_b298d29de9236ca47a023e382313bcc2d2eed31dfa706b60a04103ce83a71a25
>> >
>> > I am seeing that there are lot of *_bad-checksum_* files in YP src
>> > mirror http://downloads.yoctoproject.org/mirror/sources/
>> >
>> > Michael,
>> >
>> > Can we delete these *_bad-checksum_* files as well as zero-sized
>> > tarballs please?
>> >
>> > On Thu, Apr 29, 2021 at 3:57 PM Alexandre Belloni
>> > <alexandre.belloni@bootlin.com> wrote:
>> > >
>> > > Hello,
>> > >
>> > > On 20/04/2021 11:28:38-0700, Khem Raj wrote:
>> > > > golang.org/dl is resolving to this anyway
>> > > >
>> > >
>> > > Unfortunately, this is still an issue on the autobuilders:
>> > >
>> > > ERROR: go-cross-core2-32-1.16.3-r0 do_fetch: The URL: '
>> https://dl.google.com/go/go1.16.3.src.tar.gz;name=main' is not trusted
>> and cannot be used
>> > > ERROR: go-cross-core2-32-1.16.3-r0 do_fetch: Fetcher failure for URL:
>> 'https://dl.google.com/go/go1.16.3.src.tar.gz;name=main'. Unable to
>> fetch URL from any source.
>> > > ERROR: Logfile of failure stored in:
>> /home/pokybuild/yocto-worker/a-quick/build/build-st/tmp/work/x86_64-linux/go-cross-core2-32/1.16.3-r0/temp/log.do_fetch.525
>> > > ERROR: go-runtime-1.16.3-r0 do_fetch: The URL: '
>> https://dl.google.com/go/go1.16.3.src.tar.gz;name=main' is not trusted
>> and cannot be used
>> > > ERROR: go-runtime-1.16.3-r0 do_fetch: Fetcher failure for URL: '
>> https://dl.google.com/go/go1.16.3.src.tar.gz;name=main'. Unable to fetch
>> URL from any source.
>> > > ERROR: Logfile of failure stored in:
>> /home/pokybuild/yocto-worker/a-quick/build/build-st/tmp/work/core2-32-poky-linux/go-runtime/1.16.3-r0/temp/log.do_fetch.544
>> > >
>> > > This is fairly reproducible:
>> > >
>> > >
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/85/builds/1404/steps/23/logs/stdio
>> > >
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/85/builds/1403/steps/23/logs/stdio
>> > >
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/85/builds/1402/steps/23/logs/stdio
>> > >
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/85/builds/1401/steps/23/logs/stdio
>> > >
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/2076/steps/23/logs/stdio
>> > >
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/2074/steps/23/logs/stdio
>> > >
>> https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/2070/steps/23/logs/stdio
>> > >
>> > >
>> > > > Signed-off-by: Khem Raj <raj.khem@gmail.com>
>> > > > ---
>> > > >  meta/recipes-devtools/go/go-common.inc | 2 +-
>> > > >  1 file changed, 1 insertion(+), 1 deletion(-)
>> > > >
>> > > > diff --git a/meta/recipes-devtools/go/go-common.inc
>> b/meta/recipes-devtools/go/go-common.inc
>> > > > index 39a681a712..c368b95b69 100644
>> > > > --- a/meta/recipes-devtools/go/go-common.inc
>> > > > +++ b/meta/recipes-devtools/go/go-common.inc
>> > > > @@ -14,7 +14,7 @@ LICENSE = "BSD-3-Clause"
>> > > >
>> > > >  inherit goarch
>> > > >
>> > > > -SRC_URI = "https://golang.org/dl/go${PV}.src.tar.gz;name=main"
>> > > > +SRC_URI = "https://dl.google.com/go/go${PV}.src.tar.gz;name=main"
>> > > >  S = "${WORKDIR}/go"
>> > > >  B = "${S}"
>> > > >  UPSTREAM_CHECK_REGEX = "(?P<pver>\d+(\.\d+)+)\.src\.tar"
>> > > > --
>> > > > 2.31.1
>> > > >
>> > >
>> > > >
>> > > > 
>> > > >
>> > >
>> > >
>> > > --
>> > > Alexandre Belloni, co-owner and COO, Bootlin
>> > > Embedded Linux and Kernel engineering
>> > > https://bootlin.com
>>
>
>
> --
> Michael Halstead
> Linux Foundation / Yocto Project
> Systems Operations Engineer
>

[-- Attachment #2: Type: text/html, Size: 8700 bytes --]