Adapt what's possible from the last round of review
Thanks!
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Jan 25 2022
Wrap command call with chronic
Build is green
Build passed on Jenkins, great ! Abandoning this, will land it later.
- rebase
- update according the feedback
Make the script fail fast
Adapt according to review
In D7033#182832, @olasd wrote:Thanks! (and sorry for the hash algo ping-pong)
LGTM
thanks, just a small inlined remark regarding the httpie package installation
Build is green
add logging
remove the 'scribe' attribution
In D7029#182839, @olasd wrote:I wonder if that'd be worth a warning. May end up being a bit noisy though.
Actually send the email instead of just displaying it
I wonder if that'd be worth a warning. May end up being a bit noisy though.
Thanks! (and sorry for the hash algo ping-pong)
I'd reword the commit message as "Deduplicate hardcoded key size"
That sounds ok to me: scm entries are deduplicated. 2 pages corresponding to the same scm entry (i.e. with the same scm url) will produce only one output.
In this sample we should indeed have 2 jars and 1 scm.
I'm answering your questions now so you can get a change to come back to me before I start working on this again tomorrow.
I'm not 100% convinced we need to recheck the objects at every addition (within a transaction that can still fail to commit) instead of afterwards, but it doesn't /hurt/ either. We'll make a full pass on all objects again later anyway.
Build is green
doesn't matplotlib work with the built-in tkinter by default?
address review comments
swh-loader-svn v1.1.0 has been released, new tests have been scheduled on staging (T3875#77706).
Subversion origins that failed to load or with a partial visit status when using swh-loader-svn v1.0.0 (T3870#77580)
have been resubmitted on staging to check how many can now be fully loaded using swh-loader-svn v1.1.0.
Looks good. I trust you for the bzr part, so I didn't check it
Build is green
Rebase
staging:
- Package python3-swh.loader.svn upgraded
- Service swh-worker@loader_{svn,high_priority} restarted
Build is green