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

docker: Upgrade os-autoinst_dev image to Tumbleweed #1586

Merged
merged 1 commit into from Jan 19, 2021

Conversation

okurz
Copy link
Member

@okurz okurz commented Dec 10, 2020

No description provided.

@okurz

This comment has been minimized.

@mergify

This comment has been minimized.

@okurz
Copy link
Member Author

okurz commented Jan 18, 2021

@Martchus is this what you have seen recently elsewhere as well?:

6:     # No tests run!
6: 
6: #   Failed test 'No tests run for subtest "load test success when casedir and productdir are relative path"'
6: #   at ./14-isotovideo.t line 200.
6: "/usr/bin/perl" failed to start: "No child processes" at ./14-isotovideo.t line 34.
6: # Tests were run but no plan was declared and done_testing() was not seen.
6: # Looks like your test exited with 255 just after 10.
6: [17:11:51] ./14-isotovideo.t ........................ 
6: Dubious, test returned 255 (wstat 65280, 0xff00)
6: Failed 1/10 subtests 

@kalikiana
Copy link
Member

@Martchus is this what you have seen recently elsewhere as well?:

6:     # No tests run!
6: 
6: #   Failed test 'No tests run for subtest "load test success when casedir and productdir are relative path"'
6: #   at ./14-isotovideo.t line 200.
6: "/usr/bin/perl" failed to start: "No child processes" at ./14-isotovideo.t line 34.
6: # Tests were run but no plan was declared and done_testing() was not seen.
6: # Looks like your test exited with 255 just after 10.
6: [17:11:51] ./14-isotovideo.t ........................ 
6: Dubious, test returned 255 (wstat 65280, 0xff00)
6: Failed 1/10 subtests 

I think I was the one bringing this one up. I saw it once in #1611 but couldn't reproduce it. Even running the subtest in a loop doesn't reproduce it.

@Martchus
Copy link
Contributor

Yes, that's exactly the problem I've seen recently as well. #1613 contains a fix. I've restarted the CI (which didn't work at all a few hours ago). Let's see whether it passes now.

Tested locally with

```
podman build -t os-autoinst_dev -f container/os-autoinst_dev/Dockerfile container/os-autoinst_dev/
podman run --rm -it --entrypoint='' os-autoinst_dev /bin/bash -c 'cd /opt && git clone https://github.com/os-autoinst/os-autoinst && cd os-autoinst && mkdir build && cd build && cmake -GNinja .. && ninja'
```

Related progress issue: https://progress.opensuse.org/issues/80962
@okurz okurz changed the title docker: Upgrade os-autoinst_dev image to Tumbleweed + use travis CI independant name docker: Upgrade os-autoinst_dev image to Tumbleweed Jan 19, 2021
@codecov
Copy link

codecov bot commented Jan 19, 2021

Codecov Report

Merging #1586 (308b3ad) into master (06a8eb3) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1586      +/-   ##
==========================================
- Coverage   62.31%   62.30%   -0.02%     
==========================================
  Files          58       58              
  Lines        6605     6605              
==========================================
- Hits         4116     4115       -1     
- Misses       2489     2490       +1     
Impacted Files Coverage Δ
backend/ipmi.pm 87.12% <0.00%> (-1.00%) ⬇️

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 06a8eb3...308b3ad. Read the comment docs.

@mergify mergify bot merged commit d4854e5 into os-autoinst:master Jan 19, 2021
@okurz okurz deleted the enhance/update_dev_image branch January 20, 2021 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants