Page MenuHomeSoftware Heritage
Differential D7025

maven: Fix logging instruction and add more debug ones
ClosedPublic
Actions

Authored by ardumont on Jan 24 2022, 2:42 PM.

Details

Reviewers
vlorentz
Group Reviewers
Reviewers
Commits
rDLSa599493b4846: maven: Let logging instruction do the formatting
rDLS8667b04abcbb: maven: Add more debug logging instruction
Summary

And log the metadata dictionary.

This allows easier debugging:

swh-lister_1                        | [2022-01-24 13:31:06,174: INFO/ForkPoolWorker-1] Found 2 poms.
swh-lister_1                        | [2022-01-24 13:31:06,174: INFO/ForkPoolWorker-1] Fetching poms..
swh-lister_1                        | [2022-01-24 13:31:06,174: DEBUG/ForkPoolWorker-1] Fetching URL https://repo1.maven.org/maven2/al/aldi/sprova4j/0.1.0/sprova4j-0.1.0.pom with params {}
swh-lister_1                        | [2022-01-24 13:31:06,210: DEBUG/ForkPoolWorker-1] * Yielding pom https://repo1.maven.org/maven2/al/aldi/sprova4j/0.1.0/sprova4j-0.1.0.pom: {'type': 'scm', 'doc': 1, 'url': 'scm:https://github.com/aldialimucaj/sprova4j.git', 'project': 'al.aldi.sprova4j'}
swh-lister_1                        | [2022-01-24 13:31:06,215: INFO/ForkPoolWorker-1] Fetching URL https://repo1.maven.org/maven2/al/aldi/sprova4j/0.1.1/sprova4j-0.1.1.pom with params {}
swh-lister_1                        | [2022-01-24 13:31:06,222: INFO/ForkPoolWorker-1] * Yielding pom https://repo1.maven.org/maven2/al/aldi/sprova4j/0.1.1/sprova4j-0.1.1.pom: {'type': 'scm', 'doc': 3, 'url': 'scm:https://github.com/aldialimucaj/sprova4j.git', 'project': 'al.aldi.sprova4j'},
Test Plan

tox still happy

Diff Detail

Repository
rDLS Listers
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 26322
Build 41158: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 41157: arc lint + arc unit

Event Timeline

ardumont created this revision.Jan 24 2022, 2:42 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptJan 24 2022, 2:42 PM
ardumont mentioned this in T3746: staging: Deploy maven indexer/lister/loader.Jan 24 2022, 2:44 PM
swh-public-ci added a comment.Jan 24 2022, 2:44 PM

Build is green

Patch application report for D7025 (id=25459)

Rebasing onto 771c406710...

Current branch diff-target is up to date.
Changes applied before test
commit b2a23f755b73026592befa623a174758f99ac754
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Jan 24 14:39:28 2022 +0100

    maven: Add more debug logging instruction
    
    And log the metadata dictionary.

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

Harbormaster completed remote builds in B26296: Diff 25459.Jan 24 2022, 2:44 PM
ardumont requested review of this revision.Jan 24 2022, 2:44 PM
ardumont updated this revision to Diff 25460.Jan 24 2022, 2:49 PM

Use correct logging instructions

swh-public-ci added a comment.Jan 24 2022, 2:52 PM

Build is green

Patch application report for D7025 (id=25460)

Rebasing onto 771c406710...

Current branch diff-target is up to date.
Changes applied before test
commit 98810bb90ba117f599820fa204c27f19a9e303b5
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Jan 24 14:39:28 2022 +0100

    maven: Add more debug logging instruction
    
    And log the metadata dictionary.

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

Harbormaster completed remote builds in B26297: Diff 25460.Jan 24 2022, 2:52 PM
ardumont updated this revision to Diff 25475.Jan 24 2022, 5:49 PM
  • maven: Let logging instruction do the formatting
swh-public-ci added a comment.Jan 24 2022, 5:52 PM

Build is green

Patch application report for D7025 (id=25475)

Rebasing onto f6ca1dc3dc...

First, rewinding head to replay your work on top of it...
Applying: maven: Add more debug logging instruction
Applying: maven: Let logging instruction do the formatting
Changes applied before test
commit dcb58dfa638e3b9a9e8ca0acd77f5c34f02d863b
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Jan 24 17:48:42 2022 +0100

    maven: Let logging instruction do the formatting

commit 18e582794b05676a4f022094100f9021b3e29ec1
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Jan 24 14:39:28 2022 +0100

    maven: Add more debug logging instruction
    
    And log the metadata dictionary.

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

Harbormaster completed remote builds in B26310: Diff 25475.Jan 24 2022, 5:52 PM
ardumont retitled this revision from maven: Add more debug logging instruction to maven: Fix logging instruction and add more debug ones.Jan 24 2022, 5:53 PM
ardumont edited the summary of this revision. (Show Details)
vlorentz accepted this revision.Jan 25 2022, 11:09 AM
This revision is now accepted and ready to land.Jan 25 2022, 11:09 AM
ardumont updated this revision to Diff 25488.Jan 25 2022, 11:32 AM

Rebase

swh-public-ci added a comment.Jan 25 2022, 11:35 AM

Build is green

Patch application report for D7025 (id=25488)

Rebasing onto f6ca1dc3dc...

Current branch diff-target is up to date.
Changes applied before test
commit a599493b48461a7e3378a5ba0ab1fd69f3573338
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Jan 24 17:48:42 2022 +0100

    maven: Let logging instruction do the formatting

commit 8667b04abcbb646f4da81f9166925cfbdf41da0b
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Mon Jan 24 14:39:28 2022 +0100

    maven: Add more debug logging instruction
    
    And log the metadata dictionary.

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

Harbormaster completed remote builds in B26322: Diff 25488.Jan 25 2022, 11:35 AM
Closed by commit rDLS8667b04abcbb: maven: Add more debug logging instruction (authored by ardumont). · Explain WhyJan 25 2022, 2:18 PM
This revision was automatically updated to reflect the committed changes.
ardumont added a commit: rDLS8667b04abcbb: maven: Add more debug logging instruction.
ardumont added a commit: rDLSa599493b4846: maven: Let logging instruction do the formatting.

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)
swh/
lister/
maven/
21 lines96%
Loading...

Diff 25488

View Options

swh/lister/maven/lister.py

Loading...
About · Developer information · Legal