Page MenuHomeSoftware Heritage

opam: Directly use the --root flag instead of using an env variable
ClosedPublic

Authored by ardumont on Jul 20 2021, 4:56 PM.

Details

Summary

This aligns the behavior with the opam loader.

@aleo it's actually a good idea that I did this slight adaptations ;). Now I recall that
for the deployment [1], it would work but often passing through the opamroot
initialization step in the loader. Because, there is no guarantee that the actual worker
listing the package (with a given opamroot as /tmp/<something>) will be the same one
ingesting the package. Thus, the opamroot passed along to the loader may not exist, so
it would get initialized and possibly multiple times.

We should probably make that opamroot a configuration entry in the dedicated
configuration file targeting a fixed location in the workers to avoid the too many
opamroot locations in /tmp (for another time/diff).

[1] T3438

Related to T3358

Diff Detail

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

Event Timeline

Build is green

Patch application report for D6010 (id=21711)

Rebasing onto 6c12350863...

Current branch diff-target is up to date.
Changes applied before test
commit f00d41d0cdd05e2d696cd0a5dca001de1cf24d74
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Jul 20 16:46:10 2021 +0200

    opam: Directly use the --root flag instead of using an env variable
    
    This aligns the behavior with the opam loader
    
    Related to T3358

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

This revision was not accepted when it landed; it landed in state Needs Review.Jul 23 2021, 10:12 AM
This revision was automatically updated to reflect the committed changes.