Differential D6329
Fix branch bookmark id format so ingestion can finish Authored by ardumont on Sep 23 2021, 10:14 AM. Tags None Subscribers None
Details
For bookmarks, the ids format listed are not aligned with the rest of the code. It's This commit adds the extra mapping to fix the issue Related to T3584#71046 (for the pdb insight) tox
Diff Detail
Event Timeline
Comment Actions Build is green Patch application report for D6329 (id=22993)Could not rebase; Attempt merge onto 8e3b880ebc... Updating 8e3b880..39619a2 Fast-forward swh/loader/mercurial/from_disk.py | 8 ++++++-- swh/loader/mercurial/tests/data/anomad-d.tgz | Bin 0 -> 2757941 bytes swh/loader/mercurial/tests/test_from_disk.py | 13 +++++++++++++ 3 files changed, 19 insertions(+), 2 deletions(-) create mode 100644 swh/loader/mercurial/tests/data/anomad-d.tgz Changes applied before testcommit 39619a24301ec0867cb8048fccae839626b41aec
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date: Thu Sep 23 10:12:31 2021 +0200
Identify & fix missing mapping scenario about mismatched bookmark id
Related to T3584
commit ef502bcdf3a2717a5d582db6c3658ae4788def7c
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date: Fri Sep 17 17:19:20 2021 +0200
Capture missing revision <-> hgnode-id scenario in a xfail test
anomad-d is the `user`-`repository` name.
It's a repository which presents an anomaly in some undefined way yet. That anomaly
makes the ingestion fail.
It's something that's happening currently once in a while with the bitbucket ingestion.
This commit is just a preparatory work to analyze the problem either if i have time to
or if someone else wants to.
To analyze comment the xfail mark and let the test fail, then debug. That should fairly
help.
Related to T3584See https://jenkins.softwareheritage.org/job/DLDHG/job/tests-on-diff/310/ for more details.
Comment Actions Build is green Patch application report for D6329 (id=22994)Could not rebase; Attempt merge onto 8e3b880ebc... Updating 8e3b880..a18db13 Fast-forward swh/loader/mercurial/hgutil.py | 13 ++++++++++++- swh/loader/mercurial/tests/data/anomad-d.tgz | Bin 0 -> 2757941 bytes swh/loader/mercurial/tests/test_from_disk.py | 13 +++++++++++++ 3 files changed, 25 insertions(+), 1 deletion(-) create mode 100644 swh/loader/mercurial/tests/data/anomad-d.tgz Changes applied before testcommit a18db136ab4fe7c55938eba2aa4531298ff97998
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date: Thu Sep 23 10:12:31 2021 +0200
Fix branch bookmark id format so ingestion can finish
For bookmarks, the ids format listed are not aligned with the rest of the code. It's
human readable id as bytes string instead of bytes string. As it's not what's expected
by the caller, this failed the build.
This commit adds the extra mapping to fix the issue
Related to T3584
commit ef502bcdf3a2717a5d582db6c3658ae4788def7c
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date: Fri Sep 17 17:19:20 2021 +0200
Capture missing revision <-> hgnode-id scenario in a xfail test
anomad-d is the `user`-`repository` name.
It's a repository which presents an anomaly in some undefined way yet. That anomaly
makes the ingestion fail.
It's something that's happening currently once in a while with the bitbucket ingestion.
This commit is just a preparatory work to analyze the problem either if i have time to
or if someone else wants to.
To analyze comment the xfail mark and let the test fail, then debug. That should fairly
help.
Related to T3584See https://jenkins.softwareheritage.org/job/DLDHG/job/tests-on-diff/311/ for more details. Comment Actions Build is green Patch application report for D6329 (id=22995)Could not rebase; Attempt merge onto 8e3b880ebc... Updating 8e3b880..395ea80 Fast-forward swh/loader/mercurial/hgutil.py | 9 ++++++++- swh/loader/mercurial/tests/data/anomad-d.tgz | Bin 0 -> 2757941 bytes swh/loader/mercurial/tests/test_from_disk.py | 13 +++++++++++++ 3 files changed, 21 insertions(+), 1 deletion(-) create mode 100644 swh/loader/mercurial/tests/data/anomad-d.tgz Changes applied before testcommit 395ea80518cd1f46f190e68790b5d52c4cea946f
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date: Thu Sep 23 10:12:31 2021 +0200
Fix branch bookmark id format so ingestion can finish
For bookmarks, the ids format listed are not aligned with the rest of the code. It's
human readable id as bytes string instead of bytes string. As it's not what's expected
by the caller, this failed the build.
This commit adds the extra mapping to fix the issue
Related to T3584
commit ef502bcdf3a2717a5d582db6c3658ae4788def7c
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date: Fri Sep 17 17:19:20 2021 +0200
Capture missing revision <-> hgnode-id scenario in a xfail test
anomad-d is the `user`-`repository` name.
It's a repository which presents an anomaly in some undefined way yet. That anomaly
makes the ingestion fail.
It's something that's happening currently once in a while with the bitbucket ingestion.
This commit is just a preparatory work to analyze the problem either if i have time to
or if someone else wants to.
To analyze comment the xfail mark and let the test fail, then debug. That should fairly
help.
Related to T3584See https://jenkins.softwareheritage.org/job/DLDHG/job/tests-on-diff/312/ for more details. Comment Actions Thanks, let's deploy this so we can ingest properly now. Cheers, | ||||||||||||||||||||||||||