Page MenuHomeSoftware Heritage

bitbucket: Fix full lister with fallback [start, end] if not provided
ClosedPublic

Authored by ardumont on Jun 21 2019, 3:49 PM.

Details

Summary

It nows starts without failing.
It encounters the same behavior as the incremental one though (cf. T1826#34034).
The fix for that behavior is on its way and out of the scope of this diff.

Related T1826
Depends on D1628

Test Plan
  • tox
  • docker-dev: schedule list-bitbucket-full (check swh-lister's logs)

Diff Detail

Repository
rDLS Listers
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

swh/lister/bitbucket/lister.py
55

@olasd in the end, i kept the db_last_index because tests were annoying me...

The proper syntax for phabricator to pick up the dependency is "Depends on Dxyz".

This revision is now accepted and ready to land.Jun 21 2019, 5:05 PM

The proper syntax for phabricator to pick up the dependency is "Depends on Dxyz".

Indeed, you told me time and again and i knew this time. I did not notice i missed it.
Thanks

That probably explains the build failure.

That probably explains the build failure.

Build is green

Indeed, jenkins agrees \m/

This revision was automatically updated to reflect the committed changes.