Page MenuHomeSoftware Heritage
Feed Advanced Search

May 5 2022

Ericson2314 closed T4224: `/api/1/known/` returns wrong answers the SWHIDs only differ by object type as Resolved.

Fixed with D7748

May 5 2022, 7:56 PM · Easy hack, Web app
Ericson2314 closed D7748: Fix `api_swhid_known` when swhids share hash part.
May 5 2022, 7:39 PM
Ericson2314 committed rDWAPPS55122f0a2e29: Fix `api_swhid_known` when swhids share hash part (authored by Ericson2314).
Fix `api_swhid_known` when swhids share hash part
May 5 2022, 7:39 PM
Ericson2314 updated the diff for D7748: Fix `api_swhid_known` when swhids share hash part.

Fix commit messsage as @vlorentz requested

May 5 2022, 7:18 PM
Ericson2314 requested review of D7748: Fix `api_swhid_known` when swhids share hash part.
May 5 2022, 6:46 PM
Ericson2314 claimed T4224: `/api/1/known/` returns wrong answers the SWHIDs only differ by object type.
May 5 2022, 5:30 PM · Easy hack, Web app
Ericson2314 added a comment to T4224: `/api/1/known/` returns wrong answers the SWHIDs only differ by object type.

As I suspected, api_swhid_known in swh/web/api/views/identifiers.py does some keying by hashes, and that logic is not quite right :).

May 5 2022, 5:23 PM · Easy hack, Web app
Ericson2314 created T4224: `/api/1/known/` returns wrong answers the SWHIDs only differ by object type.
May 5 2022, 5:20 PM · Easy hack, Web app
Ericson2314 added inline comments to D7746: docs: Add /raw/(swhid)/ endpoint in Web API documentation.
May 5 2022, 5:15 PM
Ericson2314 accepted D7746: docs: Add /raw/(swhid)/ endpoint in Web API documentation.
May 5 2022, 5:14 PM

May 4 2022

Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

Ah. and I guess there needs to be another round of tagging and whatnot to trigger a website redeploy?

May 4 2022, 12:20 AM

May 3 2022

Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

(Post merge, the diff that shows up is just the first commit, but they all are on master.)

May 3 2022, 10:11 PM
Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

Eh, I'll just assume, given the timezone latency, that going down to one review was intentional. (With the directory algorithm factored out into swh-storage, this patch is now simpler and more "just do the obvious thing" too.)

May 3 2022, 10:09 PM
Ericson2314 committed rDWAPPS25a999821b3f: Adds an API route for fetching objects as Git objects (authored by plt-amy).
Adds an API route for fetching objects as Git objects
May 3 2022, 10:08 PM
Ericson2314 closed D7192: Route for fetching Git-encoded objects.
May 3 2022, 10:08 PM
Ericson2314 committed rDWAPPS43b197d8357f: Deduplicate `SWHID_RE` (authored by Ericson2314).
Deduplicate `SWHID_RE`
May 3 2022, 10:08 PM
Ericson2314 committed rDWAPPS9cd95df4775c: Extend test utits with new functions (authored by Ericson2314).
Extend test utits with new functions
May 3 2022, 10:08 PM
Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

@vlorentz is anyone else (@anlambert?) going to review, or should I push this once it passes again?

May 3 2022, 7:57 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Rebase as master changed since

May 3 2022, 7:56 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Use new swh.storage.algos.directory.directory_get

May 3 2022, 4:25 PM

May 2 2022

Ericson2314 updated the summary of D7722: Add function storage.algos.directory.directory_get.
May 2 2022, 6:57 PM
Ericson2314 accepted D7722: Add function storage.algos.directory.directory_get.
May 2 2022, 6:56 PM
Ericson2314 updated the diff for D7722: Add function storage.algos.directory.directory_get.

Update commit message in the way @anlambert requested

May 2 2022, 6:54 PM
Ericson2314 abandoned D7720: Add `algos.directory.directory_get_all_entries`.
May 2 2022, 4:29 PM
Ericson2314 accepted D7722: Add function storage.algos.directory.directory_get.

Oh awesome! Thank so much!!

May 2 2022, 4:29 PM
Ericson2314 added a comment to D7720: Add `algos.directory.directory_get_all_entries`.

Do you think it would be possible to land this, and a version of the main one that depends on this, and then circle back to add the additional tests for malformed objects?

May 2 2022, 7:50 AM

May 1 2022

Ericson2314 requested review of D7720: Add `algos.directory.directory_get_all_entries`.
May 1 2022, 9:09 PM
Ericson2314 added inline comments to D7192: Route for fetching Git-encoded objects.
May 1 2022, 8:08 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Include raw manifest in directory object

May 1 2022, 7:20 PM
Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

(Mark notes to fix copyright as done)

May 1 2022, 7:16 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix copyright dates

May 1 2022, 7:15 PM
Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

Do you all know what is causing these CI failures?

just Cypress tests timing out as usual

May 1 2022, 6:44 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Kick off CI again after I saw swh-model's wor

May 1 2022, 6:31 PM
Ericson2314 closed D7719: Dedup `format_git_object_from_headers`.
May 1 2022, 6:28 PM
Ericson2314 committed rDMODfe4ebb2f51aa: Dedup `format_git_object_from_headers` (authored by Ericson2314).
Dedup `format_git_object_from_headers`
May 1 2022, 6:28 PM
Ericson2314 requested review of D7719: Dedup `format_git_object_from_headers`.
May 1 2022, 5:40 PM

Apr 29 2022

Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

Do you all know what is causing these CI failures?

Apr 29 2022, 5:50 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Push same stuff again to restart job

Apr 29 2022, 6:15 AM

Apr 28 2022

Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

I'll wait for now pending @anlambert review, but let me know if I should push again to to trigger a new build after the last one seems to have died spuriously (if I am not misunderstanding).

Apr 28 2022, 8:24 PM
Ericson2314 added inline comments to D7192: Route for fetching Git-encoded objects.
Apr 28 2022, 7:32 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Remove dead code, remove 400 from docs

Apr 28 2022, 7:31 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

It would seem I did a no-op revise before

Apr 28 2022, 6:26 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Remove unused imports

Apr 28 2022, 5:25 PM
Ericson2314 added inline comments to D7192: Route for fetching Git-encoded objects.
Apr 28 2022, 5:24 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Address some review comments

Apr 28 2022, 5:11 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

It works! Clean up commit history and stop filtering the tests

Apr 28 2022, 12:57 AM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Try to fix without with_data

Apr 28 2022, 12:37 AM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Try to fix content test with with_data

Apr 28 2022, 12:15 AM

Apr 27 2022

Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Try again to fix tests

Apr 27 2022, 11:45 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix import of snapshot_get_all_branches

Apr 27 2022, 11:17 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix some issues, use @vlorentz's suggestion to vastly simplify!

Apr 27 2022, 11:10 PM
Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

@vlorentz Thanks! That is much nice!

Apr 27 2022, 10:51 PM
Ericson2314 abandoned D7705: Format `test_archive.py`.

My version of black was too old!

Apr 27 2022, 10:44 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Try to fix tests

Apr 27 2022, 10:15 PM
Ericson2314 requested review of D7705: Format `test_archive.py`.
Apr 27 2022, 10:06 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix typo, I wasn't running flake8 on enough files!

Apr 27 2022, 9:52 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.
  • Follow @vlorentz's advice to hopfully get mocking to work
Apr 27 2022, 9:45 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Reuse storage from archive

Apr 27 2022, 8:57 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Use storage directly rather than archive

Apr 27 2022, 8:50 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix flake8 errors, fix messed up imports

Apr 27 2022, 7:11 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Try new swh_model, unparse object_id

Apr 27 2022, 7:01 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix url_args keys

Apr 27 2022, 6:39 PM
Ericson2314 closed D7653: Add missing `content_git_object`.
Apr 27 2022, 6:28 PM
Ericson2314 committed rDMOD08c69e63794a: Add missing `content_git_object` (authored by Ericson2314).
Add missing `content_git_object`
Apr 27 2022, 6:28 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

-k in pytest.ini

Apr 27 2022, 6:20 PM
Ericson2314 updated the diff for D7653: Add missing `content_git_object`.

Use git-style not phab-style commit message

Apr 27 2022, 6:17 PM
Ericson2314 added a comment to D7653: Add missing `content_git_object`.

@vlorentz do you mean repeat the title in the summary field?

Apr 27 2022, 5:54 PM
Ericson2314 updated the summary of D7653: Add missing `content_git_object`.
Apr 27 2022, 5:51 PM
Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

On a diferrent note, once D7653 lands is there any synchronization work, or can the next version of this just start using content_git_object right away?

Apr 27 2022, 5:16 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Make URL param name and python param name align

Apr 27 2022, 5:13 PM
Ericson2314 updated subscribers of D7192: Route for fetching Git-encoded objects.

Yeah I am afraid I haven not yet gotten the container setup instructions from @plt-amy (we are both on NixOS) and so I am flying blind (ab)using CI. I was already planning on doing that today, but in the meantime I might see if I can temp change CI just to run the one test :).

Apr 27 2022, 5:11 PM
Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

Ah thanks so much, @vlorentz! I was even looking for something along those lines, but just totally failed to find it.

Apr 27 2022, 4:46 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Add swh/web/api/urls.py import, fix positional param name

Apr 27 2022, 4:45 PM
Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

django.urls.exceptions.NoReverseMatch: Reverse for 'api-1-raw-object' not found. 'api-1-raw-object' is not a valid view function or pattern name.

Apr 27 2022, 12:34 AM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix flake8 asking for more blank lines

Apr 27 2022, 12:04 AM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix api_route decoration, factor out SWHID_RE

Apr 27 2022, 12:02 AM

Apr 26 2022

Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix typo

Apr 26 2022, 10:42 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix typo

Apr 26 2022, 10:39 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Tweak some things

Apr 26 2022, 10:36 PM
Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

BTW, is there a way to convert one of these back to a "draft" state?

Apr 26 2022, 10:17 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix flake8 issues

Apr 26 2022, 10:16 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix CI failures

Apr 26 2022, 10:07 PM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Try adding tests, simplify implementation

Apr 26 2022, 9:59 PM
Ericson2314 planned changes to D7192: Route for fetching Git-encoded objects.
Apr 26 2022, 9:58 PM
Ericson2314 commandeered D7192: Route for fetching Git-encoded objects.
Apr 26 2022, 9:58 PM
Ericson2314 added a comment to D7653: Add missing `content_git_object`.

Thanks @vlorentz for all your advice and help. I think this is good now!

Apr 26 2022, 8:55 PM
Ericson2314 updated the summary of D7653: Add missing `content_git_object`.
Apr 26 2022, 8:53 PM
Ericson2314 requested review of D7653: Add missing `content_git_object`.
Apr 26 2022, 8:52 PM
Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

I suppose a next step is testing this?

Apr 26 2022, 7:56 AM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Fix missing imports

Apr 26 2022, 6:00 AM
Ericson2314 retitled D7192: Route for fetching Git-encoded objects from Route for fetching Git-encoded snapshots to Route for fetching Git-encoded objects.
Apr 26 2022, 4:56 AM
Ericson2314 updated the diff for D7192: Route for fetching Git-encoded objects.

Convert to follow mailing list plan

Apr 26 2022, 4:54 AM

Apr 6 2022

Ericson2314 added a comment to D7192: Route for fetching Git-encoded objects.

So in https://github.com/obsidiansystems/go-ipfs-swh-plugin/blob/66272b4a7d903171e2a1dd3d75309b3c1cad5ece/bridge/bridge.go#L119-L155 you can see how we are using the content route today. Hopefully seeing some working code makes things clearer!

Apr 6 2022, 10:58 PM