Page MenuHomeSoftware Heritage

Allow vault cooking from the UI
Needs RevisionPublic

Authored by vlorentz on Feb 8 2022, 11:56 AM.

Details

Summary

The vault is now about as stable as it can be in the short term,
so I think it is time to finally make this available to everyone,
without having to go throught the API manually.

Closes T3504.
Depends on T3918.

Diff Detail

Repository
rDWAPPS Web applications
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 26670
Build 41705: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 41704: arc lint + arc unit

Event Timeline

Build was aborted

Patch application report for D7120 (id=25817)

Rebasing onto b0bba742ec...

Current branch diff-target is up to date.
Changes applied before test
commit 03c66c7acedcbc354c9a83f67c71132e960c9e37
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 8 11:55:15 2022 +0100

    Allow vault cooking from the UI
    
    The vault is now about as stable as it can be in the short term,
    so I think it is time to finally make this available to everyone,
    without having to go throught the API manually.

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

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 8 2022, 12:17 PM
Harbormaster failed remote builds in B26661: Diff 25817!

Build was aborted

Patch application report for D7120 (id=25820)

Rebasing onto b0bba742ec...

Current branch diff-target is up to date.
Changes applied before test
commit 177eb6b56be826a8e1f5e6ed2b862be2cbb463b3
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 8 11:55:15 2022 +0100

    Allow vault cooking from the UI
    
    The vault is now about as stable as it can be in the short term,
    so I think it is time to finally make this available to everyone,
    without having to go throught the API manually.

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

Harbormaster returned this revision to the author for changes because remote builds failed.Feb 8 2022, 1:06 PM
Harbormaster failed remote builds in B26664: Diff 25820!

Build is green

Patch application report for D7120 (id=25826)

Rebasing onto b0bba742ec...

Current branch diff-target is up to date.
Changes applied before test
commit d891b0465a7cae907e721307fed044f7e7a918b7
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 8 11:55:15 2022 +0100

    Allow vault cooking from the UI
    
    The vault is now about as stable as it can be in the short term,
    so I think it is time to finally make this available to everyone,
    without having to go throught the API manually.

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

anlambert added a subscriber: anlambert.

Cypress tests can be simplified before landing this.

cypress/integration/vault.spec.js
132–136

You can replace these two instructions by:

checkVaultCookingTask('as tarball');

Same thing for all your changes below.

442

You need to remove that line once using chackVaultCookingTask above as that call is already wrapped in it.

This revision now requires changes to proceed.Feb 8 2022, 2:02 PM

simplify a couple of tests

oops, included wrong commit

anlambert added inline comments.
cypress/integration/vault.spec.js
160–166

You can also replace this one and the others below, I have tested it works.

This revision now requires changes to proceed.Feb 8 2022, 3:55 PM

Build has FAILED

Patch application report for D7120 (id=25830)

Rebasing onto b0bba742ec...

Current branch diff-target is up to date.
Changes applied before test
commit 57298fe585f7936024022167a03cc106b6903d4e
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 8 14:02:04 2022 +0100

    Allow cooking release objects via the API
    
    No snapshots yet; they are very costly to cook when swh-graph is
    outdated; and currently it is outdated for many objects.

commit d891b0465a7cae907e721307fed044f7e7a918b7
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 8 11:55:15 2022 +0100

    Allow vault cooking from the UI
    
    The vault is now about as stable as it can be in the short term,
    so I think it is time to finally make this available to everyone,
    without having to go throught the API manually.

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

Build is green

Patch application report for D7120 (id=25831)

Rebasing onto b0bba742ec...

Current branch diff-target is up to date.
Changes applied before test
commit 5b517e6e4d55c74d508d39f850cbe4b6d68c1e20
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Feb 8 11:55:15 2022 +0100

    Allow vault cooking from the UI
    
    The vault is now about as stable as it can be in the short term,
    so I think it is time to finally make this available to everyone,
    without having to go throught the API manually.

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

cypress/integration/vault.spec.js
160–166

How come? Isn't the test supposed to click the Ok button inbetween?

We are going to postpone this diff until T3924 is resolved one way or another.