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.

Sign up for GitHub

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

Jump to bottom

Parallel skill test #2811

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Parallel skill test #2811

wants to merge 2 commits into from

Conversation

forslund
Copy link
Collaborator

@forslund forslund commented Jan 19, 2021 •
edited

Description

Experiment splitting skilltest into 2 set run in parallel. Hopefully this will reduce the total test time.

Still todo:

  • Make the docker building more effective
  • Reporting results

How to test

Assert that Voight Kampff still tests all skills in a sensible manner

Contributor license agreement signed?

CLA [ Yes ]

@forslund forslund changed the title Test/parallell skill test Parallel skill test Jan 19, 2021
@forslund forslund force-pushed the test/parallell-skill-test branch 7 times, most recently from 89501ca to 38b268e Compare January 19, 2021 19:52
@devops-mycroft devops-mycroft added the CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) label Jan 19, 2021
@forslund forslund force-pushed the test/parallell-skill-test branch 3 times, most recently from 8be9490 to f94954b Compare January 19, 2021 20:23
@forslund forslund added Status: Work in progress PR being actively worked on, not yet ready for review. Type: Tests Addition or improvement of automated tests labels Jan 19, 2021
@devops-mycroft
Copy link

Voight Kampff Integration Test Failed (Results).
Mycroft logs are also available: skills.log, audio.log, voice.log, bus.log, enclosure.log

1 similar comment
@devops-mycroft
Copy link

Voight Kampff Integration Test Failed (Results).
Mycroft logs are also available: skills.log, audio.log, voice.log, bus.log, enclosure.log

@devops-mycroft
Copy link

Voight Kampff Integration Test Failed (Results).
Mycroft logs are also available: skills.log, audio.log, voice.log, bus.log, enclosure.log

1 similar comment
@devops-mycroft
Copy link

Voight Kampff Integration Test Failed (Results).
Mycroft logs are also available: skills.log, audio.log, voice.log, bus.log, enclosure.log

@devops-mycroft
Copy link

Voight Kampff Integration Test Failed (Results).
Mycroft logs are also available: skills.log, audio.log, voice.log, bus.log, enclosure.log

2 similar comments
@devops-mycroft
Copy link

Voight Kampff Integration Test Failed (Results).
Mycroft logs are also available: skills.log, audio.log, voice.log, bus.log, enclosure.log

@devops-mycroft
Copy link

Voight Kampff Integration Test Failed (Results).
Mycroft logs are also available: skills.log, audio.log, voice.log, bus.log, enclosure.log

@devops-mycroft
Copy link

Voight Kampff Integration Test Failed (Results).
Mycroft logs are also available: skills.log, audio.log, voice.log, bus.log, enclosure.log

1 similar comment
@devops-mycroft
Copy link

Voight Kampff Integration Test Failed (Results).
Mycroft logs are also available: skills.log, audio.log, voice.log, bus.log, enclosure.log

@forslund forslund force-pushed the test/parallell-skill-test branch 2 times, most recently from eaef4a4 to 1902a11 Compare January 28, 2021 06:42
@MycroftAI MycroftAI deleted a comment from devops-mycroft Jan 28, 2021
@MycroftAI MycroftAI deleted a comment from devops-mycroft Jan 28, 2021
@MycroftAI MycroftAI deleted a comment from devops-mycroft Jan 28, 2021
@MycroftAI MycroftAI deleted a comment from devops-mycroft Jan 28, 2021
@MycroftAI MycroftAI deleted a comment from devops-mycroft Jan 28, 2021
@MycroftAI MycroftAI deleted a comment from devops-mycroft Jan 28, 2021
@MycroftAI MycroftAI deleted a comment from devops-mycroft Jan 28, 2021
@MycroftAI MycroftAI deleted a comment from devops-mycroft Jan 28, 2021
@MycroftAI MycroftAI deleted a comment from devops-mycroft Jan 28, 2021
@MycroftAI MycroftAI deleted a comment from devops-mycroft Jan 28, 2021
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

@forslund
Split integration test into 2 parallel stages
1a3b4e1 
This splits the default.yml into two test sets set1.yml and set2.yml,
roughly balanced for equal run time.

The Docker build now takes the test set as argument so a docker image for
either test set can be built in the Jenkinsfile
@forslund
Rebalance test sets
ab03897
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

@krisgesling krisgesling mentioned this pull request Jul 20, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chrisveilleux chrisveilleux Awaiting requested review from chrisveilleux

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) Status: Work in progress PR being actively worked on, not yet ready for review. Type: Tests Addition or improvement of automated tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@forslund @devops-mycroft