be954f2fde39cefa80baeda01381a29582772684
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Apr 8 2020
lgtm
Also, add yourself to the CONTRIBUTORS file (at the top of the repository ;)
(I think it's alphabetic order on the firstname there.)
Update: Address @ardumont comment.
Remains mostly nitpicks to fix (type, indentation, docstring updates...)
Build has FAILED
Build has FAILED
Build has FAILED
Build has FAILED
Build is green
rebase
rebase
rebase
rebase
In D2960#71967, @ardumont wrote:Looks good to me.
Don't you want to implement your callable here (the one which actually exclude stuff)?
Because now, the diff title is no longer in sync with the code ;)
Or do you plan to add it in swh-scanner?
Build is green
Looks good to me.
Build is green
- storage*: Add clear_buffers operation for proxy storages (including interface, rpc/server)
- test_filter: Extract the filter storage into a fixture
requested changes
You really need to add the clear_buffers method to interface.py as well. The method needs to be available on the rpc client/server, because we're recursively calling it on all backends.
The get_storage_with_filter_config refactoring could have been a new storage fixture (local to this module) instead
The get_storage_with_filter_config refactoring could have been a new storage fixture (local to this module) instead (and it should be in a separate commit).
An interesting experiment, disabling the proxy buffer storage in the loader nixguix configuration.
And the number of hashcollision dropped to 0 (no new event for that loader since yesterday around 6pm our time).
Build is green
Getting there, see my previous comments ;)
I did not reply as val did (and i agreed with him ;)
Build has FAILED
requested changes
Apr 7 2020
Build is green
Closing this as Django has been successfully upgraded to version 2.2 in production.
cd swh-lister git checkout master git pull arc patch D2974 tox -r -- -x -s
@ardumont It seems the build error did not go away
@ardumont It seems the build error did not go away
Build has FAILED
Update: use pytest fixture instead of private module variable
In D2967#71861, @douardda wrote:why calling it _wfi_timeout (with the leading _)?
Also, in the glorious pytest way, this could be implemented as a fixture!
But yeah.
why calling it _wfi_timeout (with the leading _)?
Could you improve the first line of commit message before merging? (eg. Remove useless copy)
Build has FAILED
Build has FAILED