git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 2/2] tests: introduce --stress-jobs=<N>

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Mar 3, 2019, 14:47 UTC
Message-ID
<nycvar.QRO.7.76.6.1903031544310.45@tvgsbejvaqbjf.bet>
In-Reply-To
<20190303141946.GA28939@szeder.dev>
Hi,
On Sun, 3 Mar 2019, SZEDER Gábor wrote:
Show 10 quoted lines
> On Sat, Mar 02, 2019 at 01:19:44PM -0800, Johannes Schindelin via GitGitGadget wrote:
> > The --stress option currently accepts an argument, but it is confusing
> > to at least this user that the argument does not define the maximal
> > number of stress iterations, but instead the number of jobs to run in
> > parallel per stress iteration.
> 
> Well, these options' description in 't/README' is quite clear on what
> they do.  If the lack of updates to 't/README' in these patches is any
> indication, then you haven't read that :) but doing so might very well
> have avoided your confusion in the first place.

Yep, hadn't read it, assumed that it was obvious that `--stress=<N>` meant: try for at most <N> times to replicate the issue.

Which to me is a good indicator that the UI could be improved...
Show 9 quoted lines
> According to my Bash history, I used '--stress=<even-more-cpu-cores>'
> about 20x more often than '--stress-limit=<N>'.  That's not surprising
> at all, since the main point is to try to trigger rare, hard to
> reproduce failures, no matter how many repetitions it takes.  And even
> if there is an upper bound, it is usually not the number of repetitions
> I know in advance, but rather the time I'm willing to sacrifice on it,
> e.g. how long I'm on lunch break or doing groceries, or when I need my
> CPUs for more important things, or simply when I give up, and hit
> ctrl-C.

I don't doubt that *you* need `--stress-jobs=<N>` more often than `--stress-limit=<N>`, but I really think that common users will need the latter a lot more often (if they need the former at all).

> And with --stress-jobs=<N> I will have to type more :)
Sorry ;-)

Ciao, Dscho

Previous: SZEDER GáborNext: Johannes Schindelin via GitGitGadget
Message 6 of 15 in “tests: some touchups related to the --stress feature”
  1. 0/2 tests: some touchups related to the --stress featureJohannes Schindelin via GitGitGadget, Mar 2, 2019
  2. 2/2 tests: introduce --stress-jobs=<N>Johannes Schindelin via GitGitGadget, Mar 2, 2019
  3. Eric SunshineMar 3, 2019
  4. Junio C HamanoMar 3, 2019
  5. SZEDER GáborMar 3, 2019
  6. Johannes SchindelinMar 3, 2019
  7. 1/2 tests: let --stress-limit=<N> imply --stressJohannes Schindelin via GitGitGadget, Mar 2, 2019
  8. Junio C HamanoMar 3, 2019
  9. 0/2 tests: some touchups related to the --stress featureJohannes Schindelin via GitGitGadget, Mar 3, 2019
  10. 1/2 tests: let --stress-limit=<N> imply --stressJohannes Schindelin via GitGitGadget, Mar 3, 2019
  11. 2/2 tests: introduce --stress-jobs=<N>Johannes Schindelin via GitGitGadget, Mar 3, 2019
  12. Eric SunshineMar 3, 2019
  13. Junio C HamanoMar 4, 2019
  14. Eric SunshineMar 4, 2019
  15. Jeff KingMar 3, 2019

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.