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

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 32075
Build 50219: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 50218: arc lint + arc unit

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