Page MenuHomeSoftware Heritage

swh.core.config: Drop support for ini configuration file
ClosedPublic

Authored by ardumont on Wed, Sep 16, 2:32 PM.

Details

Summary

This is no longer used [1]

This also adds some types on impacted changed functions.

[1] Historically, we started using .ini file as configuration file for swh.
We moved to yaml a while ago. Only one last instance of it is currently
getting rid of in D3963.

Related to T1532
Related to D3963

Test Plan

tox

Diff Detail

Repository
rDCORE Foundations and core functionalities
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ardumont created this revision.Wed, Sep 16, 2:32 PM

Build is green

Patch application report for D3964 (id=13953)

Rebasing onto 60c535ddee...

Current branch diff-target is up to date.
Changes applied before test
commit a7dec76d7139ee6c3398c47d4215b2cbbd19d07f
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Sep 16 14:32:19 2020 +0200

    swh.core.config: Drop support for ini configuration file
    
    This is no longer used.
    
    This also adds some types on impacted changed functions.
    
    Related to T1532

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

ardumont edited the summary of this revision. (Show Details)Wed, Sep 16, 2:33 PM
ardumont added a project: Core & foundations.
ardumont added inline comments.Wed, Sep 16, 2:36 PM
swh/core/config.py
130

Should be return conf.
Fixing it and adding a test for it.

ardumont edited the summary of this revision. (Show Details)Wed, Sep 16, 2:36 PM
ardumont updated this revision to Diff 13955.Wed, Sep 16, 2:45 PM

Fix and add tests about edge case (no default config provided during read call)

Build is green

Patch application report for D3964 (id=13955)

Rebasing onto 60c535ddee...

Current branch diff-target is up to date.
Changes applied before test
commit 5efe22dec6debdcd27fe10abaef3094b9fbdf69a
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Sep 16 14:32:19 2020 +0200

    swh.core.config: Drop support for ini configuration file
    
    This is no longer used.
    
    This also adds some types on impacted changed functions.
    
    Related to T1532

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

ardumont added inline comments.Wed, Sep 16, 2:48 PM
swh/core/config.py
27–28

This needs to get dropped as well indeed!

[1] D3963#97708

ardumont updated this revision to Diff 13956.Wed, Sep 16, 2:49 PM

Drop unnecessary default configuration about log_db

Build is green

Patch application report for D3964 (id=13956)

Rebasing onto 60c535ddee...

Current branch diff-target is up to date.
Changes applied before test
commit 66ac6ed3aa31a63da3226f2857755f7398721d31
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Wed Sep 16 14:32:19 2020 +0200

    swh.core.config: Drop support for ini configuration file
    
    This is no longer used.
    
    This also adds some types on impacted changed functions.
    
    Related to T1532

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

vlorentz accepted this revision.Wed, Sep 16, 4:23 PM
This revision is now accepted and ready to land.Wed, Sep 16, 4:23 PM