Page MenuHomeSoftware Heritage
Differential D8529

RubyGems: List origins from https://rubygems.org
ClosedPublic
Actions

Authored by franckbret on Sep 23 2022, 8:05 PM.

Details

Reviewers
vlorentz
Group Reviewers
Reviewers
Maniphest Tasks
T1777: Rubygems Lister
Commits
rDLS52ccf49e1104: RubyGems: List origins from https://rubygems.org
Summary

Related T1777

Diff Detail

Repository
rDLS Listers
Branch
rubygems
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 31818
Build 49792: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 49791: arc lint + arc unit

Event Timeline

franckbret created this revision.Sep 23 2022, 8:05 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptSep 23 2022, 8:05 PM
swh-public-ci added a comment.Sep 23 2022, 8:10 PM

Build is green

Patch application report for D8529 (id=30736)

Rebasing onto 9b3e565cf7...

Current branch diff-target is up to date.
Changes applied before test
commit 1ede3f49f9c13e143969c63a417726832e16d6d1
Author: Franck Bret <franck.bret@octobus.net>
Date:   Fri Sep 23 20:01:59 2022 +0200

    [WIP] RubyGems: List origins from https://rubygems.org
    
    Related T1777

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

Harbormaster completed remote builds in B31713: Diff 30736.Sep 23 2022, 8:10 PM
franckbret requested review of this revision.Sep 23 2022, 8:10 PM
franckbret updated this revision to Diff 30749.Sep 26 2022, 12:10 PM

Make use of http_retry instead of throttling_retry decorator after D8519

franckbret retitled this revision from [WIP] RubyGems: List origins from https://rubygems.org to RubyGems: List origins from https://rubygems.org.Sep 26 2022, 12:10 PM
swh-public-ci added a comment.Sep 26 2022, 12:14 PM

Build is green

Patch application report for D8529 (id=30749)

Rebasing onto d5c30a3ce3...

Current branch diff-target is up to date.
Changes applied before test
commit 12a689d3264ac330ed1ee5073b75e98c4d7b82e3
Author: Franck Bret <franck.bret@octobus.net>
Date:   Fri Sep 23 20:01:59 2022 +0200

    RubyGems: List origins from https://rubygems.org
    
    Related T1777

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

Harbormaster completed remote builds in B31726: Diff 30749.Sep 26 2022, 12:14 PM
franckbret updated this revision to Diff 30844.Sep 27 2022, 3:23 PM

Make use of http_request after D8520, update documentation docker section.

swh-public-ci added a comment.Sep 27 2022, 3:28 PM

Build is green

Patch application report for D8529 (id=30844)

Rebasing onto cd596eb2b4...

First, rewinding head to replay your work on top of it...
Applying: RubyGems: List origins from https://rubygems.org
Using index info to reconstruct a base tree...
M	setup.py
Falling back to patching base and 3-way merge...
Auto-merging setup.py
Changes applied before test
commit 34d36c53864e3821cb27bf17ff52b3e63171dd0d
Author: Franck Bret <franck.bret@octobus.net>
Date:   Fri Sep 23 20:01:59 2022 +0200

    RubyGems: List origins from https://rubygems.org
    
    Related T1777

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

Harbormaster completed remote builds in B31818: Diff 30844.Sep 27 2022, 3:28 PM
vlorentz added a subscriber: vlorentz.Sep 27 2022, 3:29 PM

Is https://rubygems.org/versions documented somewhere?

franckbret added a comment.Sep 27 2022, 3:59 PM
In D8529#222126, @vlorentz wrote:

Is https://rubygems.org/versions documented somewhere?

No, not in the actual api documentation. As far as i can remember I think i've discovered it while reading the code here https://github.com/rubygems/rubygems.org/blob/4e11a3b28dfa359b807f7fe288e44a077250a4f8/config/routes.rb#L108
and it looks like it is used by official client https://github.com/rubygems/rubygems/blob/5922bdab10fbfe98b36e017cdd4eac51e9609e01/bundler/lib/bundler/compact_index_client.rb#L61

In D8529#222126, @vlorentz wrote:

Is https://rubygems.org/versions documented somewhere?

vlorentz accepted this revision.Sep 29 2022, 1:47 PM
This revision is now accepted and ready to land.Sep 29 2022, 1:47 PM
franckbret updated this revision to Diff 30947.Sep 29 2022, 2:20 PM

rebase

This revision was landed with ongoing or failed builds.Sep 29 2022, 2:20 PM
Closed by commit rDLS52ccf49e1104: RubyGems: List origins from https://rubygems.org (authored by franckbret). · Explain Why
This revision was automatically updated to reflect the committed changes.
franckbret added a commit: rDLS52ccf49e1104: RubyGems: List origins from https://rubygems.org.
swh-public-ci added a comment.Sep 29 2022, 2:25 PM

Build is green

Patch application report for D8529 (id=30947)

Rebasing onto dabb1a2ae5...

Current branch diff-target is up to date.
Changes applied before test
commit 52ccf49e1104faf9a90d753051e4f87ec4ed8335
Author: Franck Bret <franck.bret@octobus.net>
Date:   Fri Sep 23 20:01:59 2022 +0200

    RubyGems: List origins from https://rubygems.org
    
    Related T1777

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

Harbormaster completed remote builds in B31919: Diff 30947.Sep 29 2022, 2:25 PM
bchauvet added a task: T1777: Rubygems Lister.Sep 30 2022, 9:55 AM

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)
1 line--
swh/
lister/
rubygems/
66 lines100%
Loading...
75 lines100%
Loading...
19 lines100%
Loading...
tests/
80%
Loading...
data/
https_rubygems.org/
6 lines--
27 lines100%
Loading...
33 lines100%
Loading...

Diff 30844

View Options

setup.py

Loading...
View Options

swh/lister/rubygems/__init__.py

Loading...
View Options

swh/lister/rubygems/lister.py

Loading...
View Options

swh/lister/rubygems/tasks.py

Loading...
View Options

swh/lister/rubygems/tests/__init__.py

Loading...
View Options

swh/lister/rubygems/tests/data/https_rubygems.org/versions

Loading...
View Options

swh/lister/rubygems/tests/test_lister.py

Loading...
View Options

swh/lister/rubygems/tests/test_tasks.py

Loading...
About · Developer information · Legal