[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

There should be no timeouts (because wait_visibility no longer is used in 
tests). The fix can introduce new failures (like in issue42507), but it is a 
separate issue and may be related to bugs in the implementation of Tk on macOS.

--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.8, Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[miss-islington]

miss-islington  added the comment:


New changeset 03ae7e4518dae6547576c616173106d2eb283ae2 by Miss Islington (bot) 
in branch '3.9':
bpo-42142: Try to fix timeouts in ttk tests (GH-23474)
https://github.com/python/cpython/commit/03ae7e4518dae6547576c616173106d2eb283ae2


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[miss-islington]

miss-islington  added the comment:


New changeset a5b0c17e0d04931e639c4aa7a908a69cd0b529b3 by Miss Islington (bot) 
in branch '3.8':
bpo-42142: Try to fix timeouts in ttk tests (GH-23474)
https://github.com/python/cpython/commit/a5b0c17e0d04931e639c4aa7a908a69cd0b529b3


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[miss-islington]

Change by miss-islington :


--
pull_requests: +22447
pull_request: https://github.com/python/cpython/pull/23566

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:


New changeset 6cc2c419f6cf5ed336609ba01055e77d7c553e6d by Serhiy Storchaka in 
branch 'master':
bpo-42142: Try to fix timeouts in ttk tests (GH-23474)
https://github.com/python/cpython/commit/6cc2c419f6cf5ed336609ba01055e77d7c553e6d


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[miss-islington]

Change by miss-islington :


--
nosy: +miss-islington
nosy_count: 5.0 -> 6.0
pull_requests: +22446
pull_request: https://github.com/python/cpython/pull/23565

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[Terry J. Reedy]

Terry J. Reedy  added the comment:

'Intermittent' appears to mean under 1%, but to much to ignore.  Serhiy's patch 
is based on some definite hypothesis as to the reason.  The test of his patch 
would be to merge it and see if failure re-occurs or not. Doing so seems better 
than nothing.  Additional comment on the PR.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[E. Paine]

E. Paine  added the comment:

PR 23156 [Skip select ttk tests when on Ubuntu] proposes Ubuntu-specific test 
skips and is given as a justification for PR 28468 [Add 
platform.freedesktop_osrelease]. However, PR 23474 [Fix timeouts in ttk tests] 
attempts to correctly address the issue, though proving it does this will be 
hard as the issue is an intermittent failure and I don't believe any of us can 
reproduce locally (hence why it was initially blamed on Azure).

Serhiy also raised concerns about skipping on Ubuntu (though for slightly 
different reasons) and I have attempted to explain the reasoning in 
https://github.com/python/cpython/pull/23156#issuecomment-732056985.

My problem with using PR 23156 as a justification for PR 28468 is that the 
changes I proposed were only intended to be temporary (hence why I considered 
it acceptable to use the os.uname hack rather than something more reliable) and 
I don't think it, in itself, should cause a permanent API addition.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[Matthias Klose]

Matthias Klose  added the comment:

how is this test run on the CI?  at least it succeeds for me when run locally.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[Matthias Klose]

Matthias Klose  added the comment:

I became aware of this by the patch proposal "Please submit a PR to skip on 
Ubuntu", which results in the re-introduction of a distro specific API.

I would rather help fixing this issue, instead on relying on some distro 
behavor.

--
nosy: +doko

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[Serhiy Storchaka]

Change by Serhiy Storchaka :


--
pull_requests: +22367
pull_request: https://github.com/python/cpython/pull/23474

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[STINNER Victor]

Change by STINNER Victor :


--
nosy:  -vstinner

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[Terry J. Reedy]

Terry J. Reedy  added the comment:

> Would this not also stop a number of IDLE tests? (Terry?)

Yes, at least half of them.  It took years to get us to run gui tests with the 
X simulator mock.  When we first added gui tests for IDLE in 2013, they (and 
the t/tk gui tests) were only run on Windows.

Please do not undo that.  Better to kill just wait-visibility on Linux.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[E. Paine]

E. Paine  added the comment:

> Could that bot be configured to run tests with -u-gui?

This has been found to not just affect one machine: it is both Azure and Github 
Ubuntu. Therefore, the only Ubuntu bot still running gui tests would be Travis.

It could be done (with a very small patch) but:
1. I not sure about disabling all tkinter gui tests when most appear to be fine 
still
2. Would this not also stop a number of IDLE tests? (Terry?)
3. Would you intend for this to be temporary or more permanent?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

Could that bot be configured to run tests with -u-gui? Tests on Linux I can run 
manually.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[STINNER Victor]

STINNER Victor  added the comment:

bpo-42400 has been marked as a duplicate of this issue.

--
nosy: +vstinner

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[E. Paine]

E. Paine  added the comment:

This is starting to get *very* annoying. Today's failure was a new one (but 
still `wait_visibility`):

test_variable_change (tkinter.test.test_ttk.test_extensions.LabeledScaleTest) 
... Timeout (0:20:00)!
Thread 0x7f31dade1080 (most recent call first):
  File "/home/runner/work/cpython/cpython/Lib/tkinter/__init__.py", line 696 in 
wait_visibility
  File 
"/home/runner/work/cpython/cpython/Lib/tkinter/test/test_ttk/test_extensions.py",
 line 147 in test_variable_change

Again, this was on the Github Ubuntu test 
(https://github.com/python/cpython/pull/23200/checks?check_run_id=1370499785#step:10:912).
 I will update my PR to skip *all* tests that use `wait_visbility` on Ubuntu 
and ask that this would be committed sooner rather than later.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue42142] test_ttk timeout: FAIL tkinter ttk LabeledScale test_resize, and more

[STINNER Victor]

Change by STINNER Victor :


--
title: FAIL tkinter ttk LabeledScale test_resize, and more -> test_ttk timeout: 
FAIL tkinter ttk LabeledScale test_resize, and more

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com