Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: set to 10 minutes the kill timeout for tests failing on slow boards #10664

Merged
merged 1 commit into from
Aug 24, 2021

Conversation

sergiocazzolato
Copy link
Collaborator

Some of these tests are failing on pi2 and pi3 in the lab with kill
timeout.

Some boards are getting slow and the idea is to move from 5m to 10m to
avoid those issues.

The affected tests are the ones that run on ubuntu core.

The test services-refresh-mode initially was in 5 minutes, then moved to 12 but when tested with 10m it works fine.

Some of these tests are failing on pi2 and pi3 in the lab with kill
timeout.

Some boards are getting slow and the idea is to move from 5m to 10m to
avoid those issues.

The affected tests are the ones that run on ubuntu core.
@sergiocazzolato sergiocazzolato added the Simple 😃 A small PR which can be reviewed quickly label Aug 23, 2021
Copy link
Contributor

@stolowski stolowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov-commenter
Copy link

codecov-commenter commented Aug 23, 2021

Codecov Report

Merging #10664 (b241d6b) into master (8170654) will decrease coverage by 2.39%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #10664      +/-   ##
==========================================
- Coverage   80.71%   78.32%   -2.40%     
==========================================
  Files         727      882     +155     
  Lines       58158    99360   +41202     
==========================================
+ Hits        46940    77819   +30879     
- Misses       7544    16649    +9105     
- Partials     3674     4892    +1218     
Flag Coverage Δ
unittests 78.32% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
snap/snapenv/snapenv.go
store/details_v2.go
interfaces/builtin/jack1.go
advisor/pkgfinder.go
interfaces/builtin/content.go
sandbox/apparmor/apparmor.go
interfaces/builtin/location_control.go
x11/xauth.go
interfaces/builtin/device_buttons.go
overlord/devicestate/devicemgr.go
... and 1523 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ea6f44a...b241d6b. Read the comment docs.

Copy link
Contributor

@mvo5 mvo5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine, just one question inline.

@@ -3,7 +3,7 @@ summary: "Check that refresh-modes works"
# takes >1.5min to run
backends: [-autopkgtest]

kill-timeout: 12m
kill-timeout: 10m
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(nitpick) here the time decreases by 2min, is that intentional?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, see the bottom of PR desc: The test services-refresh-mode initially was in 5 minutes, then moved to 12 but when tested with 10m it works fine.

Copy link
Member

@anonymouse64 anonymouse64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@bboozzoo
Copy link
Collaborator

Test failures are unrealted, @mvo5 can you merge it?

@mvo5 mvo5 merged commit e551cf4 into snapcore:master Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Simple 😃 A small PR which can be reviewed quickly
Projects
None yet
6 participants