Page MenuHomeSoftware Heritage
Differential D6004 Diff 21698 swh/web/misc/coverage.py

Changeset View

Standalone View

View Options

swh/web/misc/coverage.py

# Copyright (C) 2018-2019 The Software Heritage developers # Copyright (C) 2018-2021 The Software Heritage developers
# See the AUTHORS file at the top-level directory of this distribution # See the AUTHORS file at the top-level directory of this distribution
# License: GNU Affero General Public License version 3, or any later version # License: GNU Affero General Public License version 3, or any later version
# See top-level LICENSE file for more information # See top-level LICENSE file for more information
from collections import Counter, defaultdict
from functools import lru_cache
from typing import Dict, List, Tuple
from urllib.parse import urlparse
from django.conf.urls import url from django.conf.urls import url
from django.shortcuts import render from django.shortcuts import render
from django.views.decorators.clickjacking import xframe_options_exempt from django.views.decorators.clickjacking import xframe_options_exempt
from swh.web.config import get_config from swh.scheduler.model import SchedulerMetrics
from swh.web.common.utils import get_deposits_list
from swh.web.config import scheduler
_swh_arch_overview_doc = (
"https://docs.softwareheritage.org/devel/architecture/overview.html"
)
# Current coverage list of the archive # Current coverage list of the archive in a high level overview fashion,
# categorized as follow:
# - listed origins: origins discovered using a swh lister
# - legacy: origins where public hosting service has closed
# - deposited: origins coming from swh-deposit
# - miscellaneous: other origin types
# TODO: Retrieve that list dynamically instead of hardcoding it # TODO: Retrieve that list dynamically instead of hardcoding it
_code_providers = [ _listed_origins = {
"info": (
"These software origins get continuously discovered and archived thanks to "
f'the <a href="{_swh_arch_overview_doc}#listers" target="_blank" '
'rel="noopener noreferrer">listers</a> implemented by Software Heritage.'
),
"origins": [
{
"type": "bitbucket",
"info_url": "https://bitbucket.org",
"info": "public repositories from Bitbucket",
},
{
"type": "cgit",
"info_url": "https://git.zx2c4.com/cgit/about",
ardumontUnsubmitted
Not Done
Inline Actions

or swh-web's?

ardumont: or swh-web's?
anlambertAuthorUnsubmitted
Done
Inline Actions

Those info could be needed in other swh components so I would not put that in swh-web db.

anlambert: Those info could be needed in other swh components so I would not put that in swh-web db.
ardumontUnsubmitted
Not Done
Inline Actions

sounds fair.

ardumont: sounds fair.
"info": "public repositories from cgit instances",
},
{
"type": "CRAN",
"info_url": "https://cran.r-project.org",
"info": "source packages from The Comprehensive R Archive Network",
},
{
"type": "debian",
"info_url": "https://www.debian.org",
"info": "source packages from the Debian distribution",
},
{
"type": "gitea",
"info_url": "https://gitea.io",
"info": "public repositories from Gitea instances",
},
{
"type": "github",
"info_url": "https://github.com",
"info": "public repositories from GitHub",
},
{
"type": "gitlab",
"info_url": "https://gitlab.com",
"info": "public repositories from multiple GitLab instances",
},
{
"type": "GNU",
"info_url": "https://www.gnu.org",
"info": "releases from the GNU project (as of August 2015)",
},
{
"type": "launchpad",
"info_url": "https://launchpad.net",
"logo": "img/logos/launchpad.png",
"info": "public repositories from Launchpad",
},
{
"type": "npm",
"info_url": "https://www.npmjs.com",
"info": "public packages from the package registry for javascript",
},
{
"type": "phabricator",
"info_url": "https://www.phacility.com/phabricator",
"info": "public repositories from multiple Phabricator instances",
},
{
"type": "pypi",
"info_url": "https://pypi.org",
"info": "source packages from the Python Package Index",
},
{ {
"provider_id": "bitbucket", "type": "sourceforge",
ardumontUnsubmitted
Not Done
Inline Actions

should be like the other nixguix loader visit type is nixguix [1]
But those are complicated, they are dealing with other types like 'tar' for example.

That probably explains why we got so few in your screenshot (26) is too few to my taste ;)

[1] https://forge.softwareheritage.org/source/swh-loader-core/browse/master/swh/loader/package/nixguix/loader.py$68

ardumont: should be like the other nixguix loader visit type is nixguix [1] But those are complicated…
anlambertAuthorUnsubmitted
Done
Inline Actions

I missed the nixguix loader configuartion for guix, correct origin counts is the number of branches in that origin, will adapt to retrieve it.

anlambert: I missed the nixguix loader configuartion for guix, correct origin counts is the number of…
ardumontUnsubmitted
Not Done
Inline Actions

right!

ardumont: right!
"provider_url": "https://bitbucket.org/", "info_url": "https://sourceforge.net",
"provider_logo": "img/logos/bitbucket.png", "info": "public repositories from SourceForge",
"provider_info": "public repositories from Bitbucket "
"(continuously archived)",
"origin_url_regexp": "^https://bitbucket.org/",
"origin_types": "repositories",
},
{
"provider_id": "cran",
"provider_url": "https://cran.r-project.org/",
"provider_logo": "img/logos/cran.svg",
"provider_info": "source packages from The Comprehensive R Archive "
"Network (continuously archived)",
"origin_url_regexp": "^https://cran.r-project.org/",
"origin_types": "packages",
},
{
"provider_id": "debian",
"provider_url": "https://www.debian.org/",
"provider_logo": "img/logos/debian.png",
"provider_info": "source packages from the Debian distribution "
"(continuously archived)",
"origin_url_regexp": "^deb://",
"origin_types": "packages",
},
{
"provider_id": "framagit",
"provider_url": "https://framagit.org/",
"provider_logo": "img/logos/framagit.png",
"provider_info": "public repositories from Framagit " "(continuously archived)",
"origin_url_regexp": "^https://framagit.org/",
"origin_types": "repositories",
},
{
"provider_id": "github",
"provider_url": "https://github.com",
"provider_logo": "img/logos/github.png",
"provider_info": "public repositories from GitHub " "(continuously archived)",
"origin_url_regexp": "^https://github.com/",
"origin_types": "repositories",
},
{
"provider_id": "gitlab",
"provider_url": "https://gitlab.com",
"provider_logo": "img/logos/gitlab.svg",
"provider_info": "public repositories from GitLab " "(continuously archived)",
"origin_url_regexp": "^https://gitlab.com/",
"origin_types": "repositories",
},
{
"provider_id": "gitorious",
"provider_url": "https://gitorious.org/",
"provider_logo": "img/logos/gitorious.png",
"provider_info": "public repositories from the former Gitorious code "
"hosting service",
"origin_url_regexp": "^https://gitorious.org/",
"origin_types": "repositories",
},
{
"provider_id": "googlecode",
"provider_url": "https://code.google.com/archive/",
"provider_logo": "img/logos/googlecode.png",
"provider_info": "public repositories from the former Google Code "
"project hosting service",
"origin_url_regexp": "^http.*.googlecode.com/",
"origin_types": "repositories",
},
{
"provider_id": "gnu",
"provider_url": "https://www.gnu.org",
"provider_logo": "img/logos/gnu.png",
"provider_info": "releases from the GNU project (as of August 2015)",
"origin_url_regexp": "^rsync://ftp.gnu.org/",
"origin_types": "releases",
},
{
"provider_id": "guix",
"provider_url": "https://guix.gnu.org/",
"provider_logo": "img/logos/guix.svg",
"provider_info": "source code tarballs used to build the Guix package "
"collection",
"origin_url_regexp": "^https://guix.gnu.org/",
"origin_types": "tarballs",
},
{
"provider_id": "hal",
"provider_url": "https://hal.archives-ouvertes.fr/",
"provider_logo": "img/logos/hal.png",
"provider_info": "scientific software source code deposited in the "
"open archive HAL",
"origin_url_regexp": "^https://hal.archives-ouvertes.fr/",
"origin_types": "deposits",
},
{
"provider_id": "inria",
"provider_url": "https://gitlab.inria.fr",
"provider_logo": "img/logos/inria.jpg",
"provider_info": "public repositories from Inria GitLab "
"(continuously archived)",
"origin_url_regexp": "^https://gitlab.inria.fr/",
"origin_types": "repositories",
},
{
"provider_id": "ipol",
"provider_url": "https://www.ipol.im/",
"provider_logo": "img/logos/ipol.png",
"provider_info": "software artifacts associated to the articles "
"IPOL publishes",
"origin_url_regexp": "^https://doi.org/10.5201/ipol",
"origin_types": "tarballs",
},
{
"provider_id": "npm",
"provider_url": "https://www.npmjs.com/",
"provider_logo": "img/logos/npm.png",
"provider_info": "public packages from the package registry for "
"javascript (continuously archived)",
"origin_url_regexp": "^https://www.npmjs.com/",
"origin_types": "packages",
},
{
"provider_id": "nixos",
"provider_url": "https://nixos.org/",
"provider_logo": "img/logos/nixos.png",
"provider_info": "source code tarballs used to build the Nix package "
"collection",
"origin_url_regexp": "^https://nix-community.github.io/nixpkgs-swh",
"origin_types": "tarballs",
},
{
"provider_id": "pypi",
"provider_url": "https://pypi.org",
"provider_logo": "img/logos/pypi.svg",
"provider_info": "source packages from the Python Packaging Index "
"(continuously archived)",
"origin_url_regexp": "^https://pypi.org/",
"origin_types": "packages",
}, },
],
}
_legacy_origins = {
"info": (
"The hosting services for these origins have been discontinued but hopefully "
"Software Heritage managed to archive them."
ardumontUnsubmitted
Not Done
Inline Actions
"info": (
- "The hosting services for these origins have been discontinued but hopefully "
- "Software Heritage managed to archive them."
+ "Discontinued hosting services. Those origins are now also archived "
+ "in Software Heritage."
),
"origins": [

(proposal)

ardumont: (proposal)
),
"origins": [
{
"type": "gitorious",
"info_url": "https://en.wikipedia.org/wiki/Gitorious",
"info": (
"public repositories from the former Gitorious code hosting service"
ardumontUnsubmitted
Not Done
Inline Actions

i'd say nixpkgs here.

ardumont: i'd say nixpkgs here.
anlambertAuthorUnsubmitted
Done
Inline Actions

I use the type value to get associated png logo in static assets so I will keep nixos value here.

anlambert: I use the type value to get associated png logo in static assets so I will keep nixos value…
ardumontUnsubmitted
Not Done
Inline Actions

I use the type value to get associated png logo in static assets so I will keep nixos value here.

ack

ardumont: > I use the type value to get associated png logo in static assets so I will keep nixos value…
ardumontUnsubmitted
Not Done
Inline Actions

NixOS is the linux distribution (that uses nix and nix's dsl for everything).
Nixpkgs is the set of packages proposed by the nix package manager. That's what we are ingesting with the nixguix loader implementation (well a derivative of it through the json file just below).

I understood nixpkgs as the superset of all packages (including the one we can install for nixos).
Hence why i suggest we use nixpkgs here ;)

ardumont: NixOS is the linux distribution (that uses nix and nix's dsl for everything). Nixpkgs is the…
),
"count": "122,014",
},
{
"type": "googlecode",
"info_url": "https://code.google.com/archive",
"info": (
"public repositories from the former Google Code project "
"hosting service"
),
"count": "790,026",
},
],
}
_deposited_origins = {
"info": (
"These origins are directly pushed into the archive by trusted partners "
f'using the <a href="{_swh_arch_overview_doc}#deposit" target="_blank" '
'rel="noopener noreferrer">deposit</a> service of Software Heritage.'
),
"origins": [
{
"type": "elife",
"info_url": "https://elifesciences.org",
"info": (
"research software source code associated to the articles "
"eLife publishes"
),
"netloc": "elife.stencila.io",
},
{
"type": "hal",
"info_url": "https://hal.archives-ouvertes.fr",
"info": "scientific software source code deposited in the open archive HAL",
"netloc": "hal.archives-ouvertes.fr",
},
{
"type": "ipol",
"info_url": "https://www.ipol.im",
"info": "software artifacts associated to the articles IPOL publishes",
"netloc": "doi.org/10.5201",
},
],
}
_miscellaneous_origins = {
"info": (
"These origins do not fall into the above categories but are important "
"to mention."
),
"origins": [
{
"type": "guix",
"info_url": "https://guix.gnu.org",
"info": "source code tarballs used to build the Guix package collection",
},
{
"type": "nixos",
"info_url": "https://nixos.org",
"info": "source code tarballs used to build the Nix package collection",
},
],
}
@lru_cache(maxsize=32)
def _get_listers_metrics() -> Dict[str, List[Tuple[str, SchedulerMetrics]]]:
listers_metrics = defaultdict(list)
listers = scheduler().get_listers()
scheduler_metrics = {m.lister_id: m for m in scheduler().get_metrics()}
for lister in listers:
if lister.id in scheduler_metrics:
listers_metrics[lister.name].append(
(lister.instance_name, scheduler_metrics[lister.id])
)
return listers_metrics
@lru_cache(maxsize=32)
def _get_deposits_netloc_counts() -> Counter:
def _process_origin_url(origin_url):
parsed_url = urlparse(origin_url)
netloc = parsed_url.netloc
# special treatment for doi.org netloc as it is not specific enough
# for origins mapping
if parsed_url.netloc == "doi.org":
netloc += "/" + parsed_url.path.split("/")[1]
return netloc
deposits = get_deposits_list()
netlocs = [
_process_origin_url(d["origin_url"]) for d in deposits if d["status"] == "done"
] ]
return Counter(netlocs)
@xframe_options_exempt @xframe_options_exempt
ardumontUnsubmitted
Not Done
Inline Actions

til [1]

[1] https://docs.djangoproject.com/en/3.2/ref/clickjacking/#setting-x-frame-options-for-all-responses

ardumont: til [1] [1] https://docs.djangoproject.com/en/3.2/ref/clickjacking/#setting-x-frame-options…
anlambertAuthorUnsubmitted
Done
Inline Actions

Originally that view was also integrated in an iframe on softwareheritage.org but this is no longer the case.
I kept that particular view implementation in case we want to integrate it again.

anlambert: Originally that view was also integrated in an iframe on softwareheritage.org but this is no…
def _swh_coverage(request): def _swh_coverage(request):
count_origins = get_config()["coverage_count_origins"] listers_metrics = _get_listers_metrics()
for origins in _listed_origins["origins"]:
origins_type = origins["type"]
if origins_type not in listers_metrics:
vlorentzUnsubmitted
Done
Inline Actions

There should be a fallback when this fails (eg. because the scheduler isn't available, or not configured in the testing env)

vlorentz: There should be a fallback when this fails (eg. because the scheduler isn't available, or not…
anlambertAuthorUnsubmitted
Done
Inline Actions

I ensured that widget will still be displayed when metrics are not available, only counters info will be missing in that case.

anlambert: I ensured that widget will still be displayed when metrics are not available, only counters…
continue
count = sum([d[1].origins_known for d in listers_metrics[origins_type]])
count_never_visited = sum(
[d[1].origins_never_visited for d in listers_metrics[origins_type]]
)
ardumontUnsubmitted
Not Done
Inline Actions

how weird!?

ardumont: how weird!?
anlambertAuthorUnsubmitted
Done
Inline Actions

Yes, do not know what is wrong here but this what we currently have in scheduler_metrics table:

softwareheritage-scheduler=> select name, instance_name, last_update, origins_known, origins_enabled, origins_never_visited, origins_with_pending_changes from listers inner join scheduler_metrics on id = lister_id order by name;
     name      |        instance_name         |          last_update          | origins_known | origins_enabled | origins_never_visited | origins_with_pending_changes 
---------------+------------------------------+-------------------------------+---------------+-----------------+-----------------------+------------------------------
 CRAN          | cran                         | 2021-07-13 12:14:07.919027+00 |         18292 |           18292 |                 18292 |                            0
anlambert: Yes, do not know what is wrong here but this what we currently have in scheduler_metrics table…
# CRAN origins are currently marked as not visited while they have been
if origins_type != "CRAN":
count -= count_never_visited
origins["count"] = f"{count:,}"
deposits_counts = _get_deposits_netloc_counts()
for origins in _deposited_origins["origins"]:
if origins["netloc"] in deposits_counts:
origins["count"] = f"{deposits_counts[origins['netloc']]:,}"
ardumontUnsubmitted
Not Done
Inline Actions
"info": "software artifacts associated to the articles IPOL publishes",
- "netloc": "doi.org",
+ "netloc": "doi.org/10.5201",
},
]

That might pose an issue because long term, other deposit clients could also reference their origin through the doi site...
I don't know how deep that breaks the current assumpion on the netloc...

The ipol specific doi is https://doi.org/10.5201/ so might be change the netloc entry to what the changes suggest?

Or maybe it's yagni, let's consider this when the time comes?

At least now you know ^ ;)

ardumont: That might pose an issue because long term, other deposit clients could also reference their…
anlambertAuthorUnsubmitted
Done
Inline Actions

I think the best way is to add a special processing for doi.org netloc and add the first path to it.

anlambert: I think the best way is to add a special processing for doi.org netloc and add the first path…
return render( return render(
request, request,
"misc/coverage.html", "misc/coverage.html",
{"providers": _code_providers, "count_origins": count_origins}, {
"origins": {
"Listed origins": _listed_origins,
"Legacy origins": _legacy_origins,
"Deposited origins": _deposited_origins,
"Miscellaneous origins": _miscellaneous_origins,
}
},
) )
urlpatterns = [ urlpatterns = [
url(r"^coverage/$", _swh_coverage, name="swh-coverage"), url(r"^coverage/$", _swh_coverage, name="swh-coverage"),
] ]
vlorentzUnsubmitted
Not Done
Inline Actions

crashes with KeyError: 'instances' in an empty docker instance.

vlorentz: crashes with `KeyError: 'instances'` in an empty docker instance.
ardumontUnsubmitted
Not Done
Inline Actions
@lru_cache()
- def _get_nixos_origins_count(origin_url: str) -> int:
+ def _get_nixpkgs_origins_count(origin_url: str) -> int:
"""Returns number of archived tarballs for NixOS, aka the number
ardumont:
ardumontUnsubmitted
Not Done
Inline Actions
@lru_cache()
- def _get_nixos_origins_count(origin_url: str) -> int:
+ def _get_nixguix_origins_count(origin_url: str) -> int:
"""Returns number of archived tarballs for NixOS, aka the number

Scratch my previous suggestion change then ;)
Given one of your last comment, then this method should be named _nixguix_.
nixpkgs and guix origins counts should be computed as this function does (counting the number of branches in the snapshots).

ardumont: Scratch my previous suggestion change then ;) Given one of your last comment, then this method…
anlambertAuthorUnsubmitted
Done
Inline Actions

already done ;-) update incoming

anlambert: already done ;-) update incoming
ardumontUnsubmitted
Not Done
Inline Actions

*thumbs up*

ardumont: *thumbs up*
About · Developer information · Legal