Fixed and deployed to production.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Feb 23 2022
Feb 23 2022
anlambert closed T3967: "Link" header is not properly displayed in apidoc when it contains [] as Resolved.
anlambert committed rDWAPPS29bf07ecfeab: requirements-test: Move ImportError workarounds in it (authored by anlambert).
requirements-test: Move ImportError workarounds in it
anlambert committed rDWAPPS7b917f18fd15: misc/coverage: Use django cache instead of LRU ones (authored by anlambert).
misc/coverage: Use django cache instead of LRU ones
anlambert committed rDWAPPScc5736263984: common/utils: Add utility functions to check swh-web run environment (authored by anlambert).
common/utils: Add utility functions to check swh-web run environment
Return adequate default value when value is not in cache.
anlambert added inline comments to D7226: deposit-list: Allow listing of deposit with their raw metadata if any.
Feb 22 2022
Feb 22 2022
Update comment in test
anlambert retitled D7224: misc/coverage: Use django cache instead of LRU ones from misc/coverage: misc/coverage: Use django cache instead of LRU ones to misc/coverage: Use django cache instead of LRU ones.
anlambert retitled D7224: misc/coverage: Use django cache instead of LRU ones from misc/coverage: Bypass LRU caches when not in production to misc/coverage: misc/coverage: Use django cache instead of LRU ones.
Use django cache instead of LRU caches.
In D7224#188075, @vlorentz wrote:This code feels rather weird. I think you should get rid of the lru_cache decorator on _get_listers_metrics, and implement caching directly in _get_listers_metrics, with a boolean argument to toggle it.
Fix typo in comment
anlambert requested review of D7223: common/utils: Add utility functions to check swh-web run environment.
anlambert committed rDWAPPSc14cfd1b2814: misc/coverage: Display counters for bzr/cvs visit types conditionally (authored by anlambert).
misc/coverage: Display counters for bzr/cvs visit types conditionally
anlambert added inline comments to D7221: misc/coverage: Display counters for bzr/cvs visit types conditionally.
anlambert committed rDDOC6aa5f08095e9: docs/Makefile: Fix warning about file not being included in any toctree (authored by anlambert).
docs/Makefile: Fix warning about file not being included in any toctree
anlambert requested review of D7219: docs/Makefile: Fix warning about file not being included in any toctree.
anlambert committed rDWAPPS937a1d59c592: common/swh_templatetags: Fix parsing issue in urlize_header_links (authored by anlambert).
common/swh_templatetags: Fix parsing issue in urlize_header_links
Feb 21 2022
Feb 21 2022
anlambert closed T3839: Optimize SVN loader performance and memory consumption on large repositories as Resolved.
Fixed and deployed.
Fixed and deployed.
anlambert requested review of D7213: common/swh_templatetags: Fix parsing issue in urlize_header_links.
anlambert committed rDWAPPS55aa143db84a: requirements: Workaround another ImportError when running pytest (authored by anlambert).
requirements: Workaround another ImportError when running pytest
anlambert committed rDDEPe1549dc0d3d9: requirements-test: Bump pytest-postgresql (authored by anlambert).
requirements-test: Bump pytest-postgresql
anlambert committed rDLDBZR77fd211f4ded: loader: Upgrade repository with old formats prior loading (authored by anlambert).
loader: Upgrade repository with old formats prior loading
anlambert requested review of D7207: requirements: Workaround another ImportError when running pytest.
In D7202#187560, @Alphare wrote:Should this be folded with the patch in T3963#79379?
anlambert added a project to T3923: Include submodules recursively when saving git repositories: Git loader.
Feb 18 2022
Feb 18 2022
Remove no longer used RepositoryNeedsUpgrade exception
anlambert added inline comments to D7202: loader: Upgrade repository with old formats prior loading.
anlambert committed rDWAPPSf7b35626cb3b: requirements: Workaround ImportError when running pytest (authored by anlambert).
requirements: Workaround ImportError when running pytest
anlambert committed rDWAPPS213828bfd4fd: package.json: Upgrade dependencies (authored by anlambert).
package.json: Upgrade dependencies
anlambert committed rDWAPPSd9944bdd56c4: browse/utils: Robustify content encoding detection (authored by anlambert).
browse/utils: Robustify content encoding detection
So after some tests, what I proposed in D7198#187379 simply does not work as iterators cannot be retried.
Feb 17 2022
Feb 17 2022
I misunderstood what is the repos object in the code so we did not use the throttling_retry decorator the right way.
anlambert committed rDLS6a7479553e6b: sourceforge: Fix origin URLs for CVS projects (authored by anlambert).
sourceforge: Fix origin URLs for CVS projects
Rebase
Looks good to me.
anlambert requested changes to D7195: lauchpad: Manage unhandled exception when reading page of result.
Looks good to me, I added some nitpick comments.
Feb 16 2022
Feb 16 2022
Great, thanks !
In D7183#186875, @vlorentz wrote:IMO ~ implies some sort of regexp or glob match, but here this is essentially a substring search. : is consistent with the syntax used by Google, Bing, DDG, ...
Bump log from debug to warning.
anlambert committed rDENV5c64a46ed699: docker/conf/lister: Add sourceforge listing tasks (authored by anlambert).
docker/conf/lister: Add sourceforge listing tasks
only : is still confusing to me, how about using := or ~ instead ?
Looks good to me, it is related to T3927 right ?
Feb 15 2022
Feb 15 2022
Thanks !
anlambert added a comment to D7177: Revert "package/loader: Remove no longer needed type ignore comment".
My bad, I had latest mypy (0.931) installed in my venv and it complained about the comment, but it is still needed for mypy 0.920.
anlambert committed rDENV3e432fc8c754: docker: Enable CVS loader use in docker environment (authored by anlambert).
docker: Enable CVS loader use in docker environment
docker: Cleanup dockerfile
Rebase
Reorder loader modules in pip install command.