Page MenuHomeSoftware Heritage

Puppet: Lister for Puppet modules
ClosedPublic

Authored by franckbret on Sep 26 2022, 11:29 AM.

Details

Summary

The puppet lister retrieves origins from from https://forge.puppet.com/modules

Related T4519

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

Build has FAILED

Patch application report for D8531 (id=30742)

Rebasing onto 9b3e565cf7...

Current branch diff-target is up to date.
Changes applied before test
commit 83d6cd041cfe094b8116296ca7352b98edddd255
Author: Franck Bret <franck.bret@octobus.net>
Date:   Mon Sep 26 11:25:23 2022 +0200

    Puppet: Lister for Puppet modules
    
    The puppet lister retrieves origins from from https://forge.puppet.com/modules
    
    Related T4519

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/687/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/687/console

Harbormaster returned this revision to the author for changes because remote builds failed.Sep 26 2022, 11:33 AM
Harbormaster failed remote builds in B31719: Diff 30742!

Build has FAILED

Patch application report for D8531 (id=30747)

Rebasing onto d5c30a3ce3...

First, rewinding head to replay your work on top of it...
Applying: Puppet: Lister for Puppet modules
Changes applied before test
commit 713e20276d36a48bf8d6cbabb2d6d4267568b11f
Author: Franck Bret <franck.bret@octobus.net>
Date:   Mon Sep 26 11:25:23 2022 +0200

    Puppet: Lister for Puppet modules
    
    The puppet lister retrieves origins from from https://forge.puppet.com/modules
    
    Related T4519

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/688/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/688/console

Harbormaster returned this revision to the author for changes because remote builds failed.Sep 26 2022, 11:40 AM
Harbormaster failed remote builds in B31724: Diff 30747!

Make use of http_retry instead of throttling_retry decorator after D8519

Build is green

Patch application report for D8531 (id=30748)

Rebasing onto d5c30a3ce3...

Current branch diff-target is up to date.
Changes applied before test
commit 5dcdcb7969e0de717d5792620143e35da4cc78d2
Author: Franck Bret <franck.bret@octobus.net>
Date:   Mon Sep 26 11:25:23 2022 +0200

    Puppet: Lister for Puppet modules
    
    The puppet lister retrieves origins from from https://forge.puppet.com/modules
    
    Related T4519

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

vlorentz added a subscriber: vlorentz.

Thanks, looks good.

A few requests below, though:

Please trim the large text blobs from the test files, it's not useful to keep them verbatim

swh/lister/puppet/__init__.py
12–13

Let's keep it simple :)

29–30
47–48
swh/lister/puppet/lister.py
23–27

reuse the value defined in in pattern.py

34
84–86

to make it clickable

104

lol, the Puppet Forge documentation advertises this format as ISO 8601

*head desk*

This revision now requires changes to proceed.Sep 26 2022, 2:26 PM
franckbret marked 7 inline comments as done.

Make use of self.http_request as introduced by D8520

Update documentation (typos, rephrase)

Build is green

Patch application report for D8531 (id=30791)

Rebasing onto fd1a4244a0...

Current branch diff-target is up to date.
Changes applied before test
commit f242ff0ead88c1854f0b803466ec08777220ab8c
Author: Franck Bret <franck.bret@octobus.net>
Date:   Mon Sep 26 11:25:23 2022 +0200

    Puppet: Lister for Puppet modules
    
    The puppet lister retrieves origins from from https://forge.puppet.com/modules
    
    Related T4519

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

swh/lister/puppet/lister.py
23–27

I've change code to use self.http_request instead.

104

lol, the Puppet Forge documentation advertises this format as ISO 8601

*head desk*

yup

Build has FAILED

Patch application report for D8531 (id=30792)

Rebasing onto fd1a4244a0...

Current branch diff-target is up to date.
Changes applied before test
commit 366430878fdd7f34794279e9be2522ed24fe89ae
Author: Franck Bret <franck.bret@octobus.net>
Date:   Mon Sep 26 11:25:23 2022 +0200

    Puppet: Lister for Puppet modules
    
    The puppet lister retrieves origins from from https://forge.puppet.com/modules
    
    Related T4519

Link to build: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/702/
See console output for more information: https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/702/console

Build is green

Patch application report for D8531 (id=30794)

Rebasing onto fd1a4244a0...

Current branch diff-target is up to date.
Changes applied before test
commit b0c63d133ab034ace9057ed0c7b001736a835c9c
Author: Franck Bret <franck.bret@octobus.net>
Date:   Mon Sep 26 11:25:23 2022 +0200

    Puppet: Lister for Puppet modules
    
    The puppet lister retrieves origins from from https://forge.puppet.com/modules
    
    Related T4519

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

This revision is now accepted and ready to land.Sep 27 2022, 11:08 AM
This revision was landed with ongoing or failed builds.Sep 27 2022, 2:46 PM
This revision was automatically updated to reflect the committed changes.

Build is green

Patch application report for D8531 (id=30829)

Rebasing onto a4aec3894e...

Current branch diff-target is up to date.
Changes applied before test
commit cd596eb2b46ddf21482f6b85b0b92b84b9988774
Author: Franck Bret <franck.bret@octobus.net>
Date:   Mon Sep 26 11:25:23 2022 +0200

    Puppet: Lister for Puppet modules
    
    The puppet lister retrieves origins from from https://forge.puppet.com/modules
    
    Related T4519

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