Page MenuHomeSoftware Heritage
Differential D1284

content_find in in_memory returns list of copies
AbandonedPublic
Actions

Authored by vlorentz on Mar 21 2019, 2:49 PM.
This revision can not be accepted until the required legal agreements have been signed.

Details

Reviewers
ashryaagr
Group Reviewers
Reviewers
Required Signatures
L3 Software Heritage Contributor License Agreement, version 1.0
Summary

content_find in in_memory returns list of copies

Diff Detail

Repository
rDSTO Storage manager
Branch
content_find
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 4765
Build 6348: tox-on-jenkinsJenkins
Build 6347: arc lint + arc unit

Event Timeline

ashryaagr created this revision.Mar 21 2019, 2:49 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptMar 21 2019, 2:49 PM
Herald added a required legal document: L3 Software Heritage Contributor License Agreement, version 1.0. · View Herald Transcript
swh-public-ci added a subscriber: swh-public-ci.Mar 21 2019, 2:51 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tox/216/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tox/216/console

Harbormaster failed remote builds in B4765: Diff 4044!Mar 21 2019, 2:51 PM
vlorentz added a subscriber: vlorentz.Mar 25 2019, 10:38 AM

Hi, could you sign the CLA so we can see your diff?

ashryaagr added a comment.Mar 25 2019, 12:49 PM

@vlorentz I have signed the CLA now . You can see the diff now .

swh-public-ci added a comment.Mar 25 2019, 3:01 PM

Build has FAILED

Link to build: https://jenkins.softwareheritage.org/job/DSTO/job/tox/218/
See console output for more information: https://jenkins.softwareheritage.org/job/DSTO/job/tox/218/console

vlorentz requested changes to this revision.Mar 27 2019, 10:28 AM

Tests fail because there are other parts of the code that need to be changed. See the description of task T1349. You should also take a look at https://docs.softwareheritage.org/devel/developer-setup.html to see how to run tests on your computer instead of waiting for the public CI

This revision now requires changes to proceed.Mar 27 2019, 10:28 AM
ashryaagr added a comment.Mar 28 2019, 4:40 AM

Ok I will make the changes in other parts of code.

vlorentz commandeered this revision.May 22 2019, 1:20 PM
vlorentz abandoned this revision.
vlorentz edited reviewers, added: ashryaagr; removed: vlorentz.

Content Hidden

The content of this revision is hidden until the author has signed all of the required legal agreements.
About · Developer information · Legal