journal: Migrate to latest storage.origin_visit_upsert/add api change

Related to swh-storage!913 (closed) Related to D2820

Test Plan

tox

---

Migrated from D2826 (view on Phabricator)

Build has FAILED

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

Looks good to me.

Merge request was accepted

approved this merge request

Build has FAILED

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

Some references in the commit message have been migrated:

• D2813 is now swh-storage!913 (closed)
• D2820 is now swh-storage!916 (closed)

Fix build

Build is green See https://jenkins.softwareheritage.org/job/DJNL/job/tox/397/ for more details.

I don't understand the issue. What do you mean by "without consistency"?

And why is the origin visit date missing at all?

I don't understand the issue. What do you mean by "without consistency"?

It means origin_visit's origin reference generated is not necessarily connected with the origin generated otherwise.

And why is the origin visit date missing at all?

no idea.

! In !248 (closed), @ardumont wrote: I don't understand the issue. What do you mean by "without consistency"?

It means origin_visit's origin reference generated is not necessarily connected with the origin generated otherwise.

You could evolve it

I don't understand the issue. What do you mean by "without consistency"?

It means origin_visit's origin reference generated is not necessarily connected with the origin generated otherwise.

You could evolve it

sure.

That's not an issue for landing this diff. I made the comment so that we see that future improvment in our hypothesis strategies in the model module.

Merge request was merged

closed