Page MenuHomeSoftware Heritage
Feed All Stories

Sep 8 2021

ardumont updated the diff for D6211: client: Open last_visit method to retrieve last visit of an origin.

Complete the test (i had forgotten to)

Sep 8 2021, 1:11 PM
ardumont requested review of D6211: client: Open last_visit method to retrieve last visit of an origin.
Sep 8 2021, 1:07 PM
ardumont added a comment to D6210: wip: collection api: Open filtering deposit per user and origin_url.

I'm more and more inclined towards using the web client instead.
See D6211

Sep 8 2021, 1:06 PM
ardumont added a revision to T3192: Add possibility to fetch last SWHID for a deposit using an origin on deposit cli: D6211: client: Open last_visit method to retrieve last visit of an origin.
Sep 8 2021, 1:05 PM · Easy hack, SWORD deposit
swh-public-ci added a comment to D6139: cassandra: Add option to select (hopefully) more efficient batch insertion algos.

Build is green

Sep 8 2021, 12:05 PM
vlorentz updated the diff for D6139: cassandra: Add option to select (hopefully) more efficient batch insertion algos.
  • rebase
  • directory_entry_add_batch: Reduce churn of prepared statements
Sep 8 2021, 11:58 AM
anlambert closed D6207: browse/snapshot_context: Ensure pull request branches can be browsed.
Sep 8 2021, 11:54 AM
anlambert committed rDWAPPSb5ed69554056: browse/snapshot_context: Ensure pull request branches can be browsed (authored by anlambert).
browse/snapshot_context: Ensure pull request branches can be browsed
Sep 8 2021, 11:54 AM
olasd placed T2100: Bootstrap Debian packaging for swh.graph up for grabs.
Sep 8 2021, 11:36 AM · Compressed graph service
olasd placed T2103: (Debian) package py4j up for grabs.
Sep 8 2021, 11:36 AM · Compressed graph service
olasd placed T3168: Proper deployment of swh-graph with debian package up for grabs.
Sep 8 2021, 11:36 AM · Compressed graph service, Puppet recipes
vlorentz added a comment to D6139: cassandra: Add option to select (hopefully) more efficient batch insertion algos.

After analyze, the batch mode uses huge prepared statements (PS). It seems with the concurrency of the replayers, they are evicted from the PS cache before they can be executed. The client silently reprepare the statement and retry fall in a prepared/evicted loop.

Sep 8 2021, 11:17 AM
vlorentz accepted D6207: browse/snapshot_context: Ensure pull request branches can be browsed.

I wonder if we should add a "Pull Request" tab on snapshot views. Seems like a rabbit hole to implement it correctly for all forges though...

Sep 8 2021, 11:14 AM
aeviso committed rDPROV26061a0a15a0: Remove old debug logging and improve other's messages (authored by aeviso).
Remove old debug logging and improve other's messages
Sep 8 2021, 10:41 AM
aeviso closed D6208: Avoid flushing empty buffers in the `Provenance` backend.
Sep 8 2021, 10:41 AM
aeviso committed rDPROV00f6d9acfb2e: Avoid flushing empty buffers in the `Provenance` backend (authored by aeviso).
Avoid flushing empty buffers in the `Provenance` backend
Sep 8 2021, 10:41 AM
aeviso committed rDPROVbd6de9eaba19: Fix two minor bugs on the MongoDB backend (authored by aeviso).
Fix two minor bugs on the MongoDB backend
Sep 8 2021, 10:41 AM
aeviso closed D6205: Fix two minor bugs on the MongoDB backend.
Sep 8 2021, 10:41 AM
vsellier added a comment to D6139: cassandra: Add option to select (hopefully) more efficient batch insertion algos.

According to the documentation of the cassandra concurrent api[1], it seems the concurrency can by specified as an argument of the execute_concurrent_with_args method. The default is 100, but it could be interesting to check with higher or lower values

Sep 8 2021, 10:27 AM
moranegg requested changes to D6201: Add an overview of the metadata workflow.
Sep 8 2021, 10:13 AM
vsellier added a comment to D6139: cassandra: Add option to select (hopefully) more efficient batch insertion algos.

These are more results with different number of replayers.
Each line represents a server with 20 directory replayers, the renages are for one-by-one, concurrent, batch

  • 1 node

  • 2 nodes
Sep 8 2021, 10:11 AM

Sep 7 2021

ardumont removed a reviewer for D6210: wip: collection api: Open filtering deposit per user and origin_url: System administrators.
Sep 7 2021, 8:11 PM
olasd accepted D6208: Avoid flushing empty buffers in the `Provenance` backend.

ACK. These were the ones I was indeed thinking of. All good then :-)

Sep 7 2021, 5:42 PM
ardumont added inline comments to D6210: wip: collection api: Open filtering deposit per user and origin_url.
Sep 7 2021, 5:31 PM
ardumont requested review of D6210: wip: collection api: Open filtering deposit per user and origin_url.
Sep 7 2021, 5:25 PM
ardumont added a revision to T3192: Add possibility to fetch last SWHID for a deposit using an origin on deposit cli: D6210: wip: collection api: Open filtering deposit per user and origin_url.
Sep 7 2021, 5:22 PM · Easy hack, SWORD deposit
jayeshv accepted D6205: Fix two minor bugs on the MongoDB backend.
Sep 7 2021, 5:18 PM
vlorentz added inline comments to D6201: Add an overview of the metadata workflow.
Sep 7 2021, 5:10 PM
aeviso added a comment to D6208: Avoid flushing empty buffers in the `Provenance` backend.
In D6208#160587, @olasd wrote:

I'm not too sure about the flat-out removal of a lot of these logging statements (rather than just turning them into logging statements with deferred formatting), but the changes to the flush() logic at least look sound.

Sep 7 2021, 5:09 PM
anlambert requested review of D6207: browse/snapshot_context: Ensure pull request branches can be browsed.
Sep 7 2021, 5:04 PM
moranegg added a comment to D6201: Add an overview of the metadata workflow.

Good idea to add this documentation!
I have added a few comments.

Sep 7 2021, 5:03 PM
olasd added a comment to D6208: Avoid flushing empty buffers in the `Provenance` backend.

I'm not too sure about the flat-out removal of a lot of these logging statements (rather than just turning them into logging statements with deferred formatting), but the changes to the flush() logic at least look sound.

Sep 7 2021, 5:03 PM
swh-public-ci added a comment to D6165: Add new RabbitMQ-based client/server API.

Build is green

Sep 7 2021, 4:56 PM
aeviso requested review of D6209: Remove locks from PostgreSQL backend insertions.
Sep 7 2021, 4:56 PM
aeviso requested review of D6208: Avoid flushing empty buffers in the `Provenance` backend.
Sep 7 2021, 4:55 PM
aeviso updated the diff for D6165: Add new RabbitMQ-based client/server API.
  • Add new RabbitMQ-based client/server API
  • Split set method's requests into several queues on server side
  • Remove get queues and have client read from ProvenanceStorage directly
  • Add support for relation_add to the RabbitMQ server
  • Refactor server to use multiple sub-processes instead of threads
  • Rework ProvenanceStorageRabbitMQWorker to handle connection loss
  • Remove old client/server storage based on swh.core.api.RPCClient
  • Improve connection error handling on both client and server side
Sep 7 2021, 4:53 PM
vsellier requested review of D6206: webapp: support new metadata search backend configuation.
Sep 7 2021, 4:08 PM
vsellier added a revision to T3433: Deploy swh.search v0.10/v0.11: D6206: webapp: support new metadata search backend configuation.
Sep 7 2021, 4:08 PM · System administration, Archive search
vlorentz added inline comments to D6201: Add an overview of the metadata workflow.
Sep 7 2021, 4:07 PM
swh-public-ci added a comment to D6165: Add new RabbitMQ-based client/server API.

Build is green

Sep 7 2021, 3:55 PM
aeviso updated the diff for D6165: Add new RabbitMQ-based client/server API.

rebase

Sep 7 2021, 3:53 PM
aeviso requested review of D6205: Fix two minor bugs on the MongoDB backend.
Sep 7 2021, 3:51 PM
jayeshv added inline comments to D6201: Add an overview of the metadata workflow.
Sep 7 2021, 3:45 PM
swh-public-ci added a comment to D6186: misc: Add iframe view for contents and directories.

Build is green

Sep 7 2021, 3:44 PM
swh-public-ci added a comment to D6203: Retry on concurrent conflicting updates.

Build is green

Sep 7 2021, 3:36 PM
vlorentz closed T3562: [swh-search] Document version conflict during parallel indexation as Resolved by committing rDSEA7479282c70db: Retry on concurrent conflicting updates.
Sep 7 2021, 3:31 PM · Archive search
vlorentz closed D6203: Retry on concurrent conflicting updates.
Sep 7 2021, 3:31 PM
vlorentz committed rDSEA7479282c70db: Retry on concurrent conflicting updates (authored by vlorentz).
Retry on concurrent conflicting updates
Sep 7 2021, 3:31 PM
vlorentz updated the diff for D6203: Retry on concurrent conflicting updates.

rebase

Sep 7 2021, 3:31 PM
anlambert updated the diff for D6186: misc: Add iframe view for contents and directories.

Update:

  • rebase
  • address @vlorentz comments
  • fix a couple of spotted issues
  • add sphinx documentation
  • add modals displayable from the SWHIDs tab explaining how to integrate iframe views in external web page
Sep 7 2021, 3:28 PM
swh-public-ci added a comment to D6198: Allow all hosts in tests.

Build is green

Sep 7 2021, 3:28 PM
olasd closed D6204: Use a temporary table to add relations.
Sep 7 2021, 3:26 PM
olasd committed rDPROV759d39a44c45: Use a temporary table to add relations (authored by olasd).
Use a temporary table to add relations
Sep 7 2021, 3:26 PM
olasd added inline comments to D6204: Use a temporary table to add relations.
Sep 7 2021, 3:24 PM
aeviso accepted D6204: Use a temporary table to add relations.
Sep 7 2021, 3:24 PM
vsellier accepted D6203: Retry on concurrent conflicting updates.

LGTM thanks

Sep 7 2021, 3:24 PM
olasd requested review of D6204: Use a temporary table to add relations.
Sep 7 2021, 3:23 PM
marla.dasilva moved T3536: Blog post Easter Eggs NLNet from Restricted Project Column to Restricted Project Column on the Unknown Object (Project) board.
Sep 7 2021, 3:23 PM · Unknown Object (Project)
marla.dasilva added a comment to T3536: Blog post Easter Eggs NLNet.

The link of a collaborative document has been created for publication of a blog post:

Sep 7 2021, 3:22 PM · Unknown Object (Project)
aeviso committed rDPROV9076e8a1dd34: Add SQL enum for relation get filter options (authored by aeviso).
Add SQL enum for relation get filter options
Sep 7 2021, 3:15 PM
aeviso committed rDPROVec1827e6eb00: Improve denormalized PostgreSQL schema to properly check for duplicates (authored by aeviso).
Improve denormalized PostgreSQL schema to properly check for duplicates
Sep 7 2021, 3:15 PM
aeviso committed rDPROVe529372cf218: Move PosgreSQL's backend method `relation_add` to SQL function (authored by aeviso).
Move PosgreSQL's backend method `relation_add` to SQL function
Sep 7 2021, 3:14 PM
aeviso committed rDPROVb523dfd8c1f8: Refactor `relation_add` prepearing to move SQL queries into functions (authored by aeviso).
Refactor `relation_add` prepearing to move SQL queries into functions
Sep 7 2021, 3:14 PM
aeviso closed D6090: Refactor `ProvenanceStoragePostgreSql`.
Sep 7 2021, 3:14 PM
aeviso committed rDPROVacd76e442fbe: Move PosgreSQL's backend method `relation_get` to SQL function (authored by aeviso).
Move PosgreSQL's backend method `relation_get` to SQL function
Sep 7 2021, 3:14 PM
olasd accepted D6090: Refactor `ProvenanceStoragePostgreSql`.
Sep 7 2021, 3:11 PM
vlorentz added inline comments to D6201: Add an overview of the metadata workflow.
Sep 7 2021, 3:11 PM
vlorentz closed D6198: Allow all hosts in tests.
Sep 7 2021, 3:10 PM
vlorentz committed rDWAPPSb552b01dc7bd: Allow all hosts in tests (authored by vlorentz).
Allow all hosts in tests
Sep 7 2021, 3:10 PM
vlorentz updated the diff for D6198: Allow all hosts in tests.

rebase

Sep 7 2021, 3:09 PM
swh-public-ci added a comment to D6198: Allow all hosts in tests.

Build is green

Sep 7 2021, 3:09 PM
vsellier closed D6202: explicitly name the metadata search configuration property.
Sep 7 2021, 3:06 PM
vsellier committed rDWAPPSc302b9a5e40d: explicitly name the metadata search configuration property (authored by vsellier).
explicitly name the metadata search configuration property
Sep 7 2021, 3:06 PM
anlambert accepted D6201: Add an overview of the metadata workflow.
Sep 7 2021, 3:04 PM
anlambert accepted D6202: explicitly name the metadata search configuration property.

Looks good to me.

Sep 7 2021, 3:03 PM
vsellier requested review of D6202: explicitly name the metadata search configuration property.
Sep 7 2021, 2:57 PM
vlorentz requested review of D6203: Retry on concurrent conflicting updates.
Sep 7 2021, 2:57 PM
vlorentz updated the diff for D6198: Allow all hosts in tests.

Remove unneeded lines

Sep 7 2021, 2:55 PM
anlambert accepted D6198: Allow all hosts in tests.

Looks good to me.

Sep 7 2021, 2:53 PM
vlorentz added a revision to T3562: [swh-search] Document version conflict during parallel indexation: D6203: Retry on concurrent conflicting updates.
Sep 7 2021, 2:53 PM · Archive search
vlorentz requested review of D6198: Allow all hosts in tests.
Sep 7 2021, 2:47 PM
ardumont committed rDDEPf87539ca3705: test: Fix repetition and make pytest happier (authored by ardumont).
test: Fix repetition and make pytest happier
Sep 7 2021, 2:26 PM
ardumont committed rDDEP3f55520d7e58: Fix swh deposit list description (authored by ardumont).
Fix swh deposit list description
Sep 7 2021, 2:09 PM
ardumont updated the task description for T3564: Puppetize graph service and add icinga alert.
Sep 7 2021, 1:49 PM · System administration, Compressed graph service, Puppet recipes
ardumont updated the summary of D6200: Add icinga checks around the graph service.
Sep 7 2021, 1:48 PM
ardumont added a revision to T3564: Puppetize graph service and add icinga alert: D6200: Add icinga checks around the graph service.
Sep 7 2021, 1:48 PM · System administration, Compressed graph service, Puppet recipes
ardumont added a revision to T3564: Puppetize graph service and add icinga alert: D6199: Install graph services as-is.
Sep 7 2021, 1:48 PM · System administration, Compressed graph service, Puppet recipes
ardumont updated the summary of D6199: Install graph services as-is.
Sep 7 2021, 1:48 PM
vlorentz requested review of D6201: Add an overview of the metadata workflow.
Sep 7 2021, 1:46 PM
anlambert added inline comments to D6190: Add link to extrinsic metadata API from the browse view.
Sep 7 2021, 1:17 PM
anlambert added inline comments to D6190: Add link to extrinsic metadata API from the browse view.
Sep 7 2021, 1:16 PM
ardumont updated the task description for T3564: Puppetize graph service and add icinga alert.
Sep 7 2021, 12:34 PM · System administration, Compressed graph service, Puppet recipes
ardumont updated the task description for T3564: Puppetize graph service and add icinga alert.
Sep 7 2021, 12:33 PM · System administration, Compressed graph service, Puppet recipes
ardumont updated the task description for T3168: Proper deployment of swh-graph with debian package.
Sep 7 2021, 12:33 PM · Compressed graph service, Puppet recipes
ardumont triaged T3564: Puppetize graph service and add icinga alert as High priority.
Sep 7 2021, 12:32 PM · System administration, Compressed graph service, Puppet recipes
swh-public-ci added a comment to D6190: Add link to extrinsic metadata API from the browse view.

Build is green

Sep 7 2021, 12:27 PM
ardumont updated the summary of D6200: Add icinga checks around the graph service.
Sep 7 2021, 12:24 PM
ardumont added inline comments to D6200: Add icinga checks around the graph service.
Sep 7 2021, 12:23 PM
ardumont retitled D6200: Add icinga checks around the graph service from Add icinga checks around the graph services to Add icinga checks around the graph service.
Sep 7 2021, 12:22 PM
ardumont retitled D6199: Install graph services as-is from Install graph services as-is and add icinga checks to Install graph services as-is.
Sep 7 2021, 12:22 PM