Differential D8611

nixguix: Deal with connection error with server
ClosedPublic
Actions

Authored by ardumont on Oct 4 2022, 2:57 PM.

Details

Reviewers

anlambert

Group Reviewers

Reviewers

Maniphest Tasks

T3781: Replace the Nixguix loader with a lister

Commits

rDLS0f8f293f968e: nixguix: Deal with connection error with server

Summary

When that arises, we skip the origins.

Related to T3781

Diff Detail

Repository

rDLS Listers

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Oct 4 2022, 2:57 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptOct 4 2022, 2:57 PM

Build is green

Patch application report for D8611 (id=31105)

Rebasing onto d92474bbda...

Current branch diff-target is up to date.

Changes applied before test

commit 0f8f293f968ee1efc8b269f4073bbac99a4f3615
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Oct 4 14:57:01 2022 +0200

    nixguix: Deal with connection error with server
    
    When that arises, we skip the origins.
    
    Related to T3781

See https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/744/ for more details.

Harbormaster completed remote builds in B32075: Diff 31105.Oct 4 2022, 3:02 PM

ardumont requested review of this revision.Oct 4 2022, 3:02 PM

anlambert accepted this revision.Oct 4 2022, 3:48 PM

This revision is now accepted and ready to land.Oct 4 2022, 3:48 PM

ardumont added a child revision: D8612: nixguix: Add support for pseudo url with missing schema.Oct 4 2022, 4:01 PM

Closed by commit rDLS0f8f293f968e: nixguix: Deal with connection error with server (authored by ardumont). · Explain WhyOct 4 2022, 4:02 PM

This revision was automatically updated to reflect the committed changes.

ardumont added a commit: rDLS0f8f293f968e: nixguix: Deal with connection error with server.

ardumont mentioned this in T3781: Replace the Nixguix loader with a lister.Oct 4 2022, 4:05 PM