Skip to content
This repository was archived by the owner on Sep 8, 2024. It is now read-only.

Conversation

@krisgesling
Copy link
Contributor

This is a test to see if sleeping after a Skills report being loaded improves the stability of the first Voight Kampff tests in a Jenkins environment.

DO NOT MERGE

@krisgesling krisgesling added the Status: Work in progress PR being actively worked on, not yet ready for review. label Jun 30, 2020
@devops-mycroft devops-mycroft added the CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) label Jun 30, 2020
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

2 similar comments
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

@krisgesling krisgesling force-pushed the bugfix/random-vk-fail branch from 42821d3 to ac76220 Compare July 3, 2020 02:52
@devops-mycroft
Copy link

Voight Kampff Integration Test Failed (Results)

@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

1 similar comment
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

@krisgesling krisgesling force-pushed the bugfix/random-vk-fail branch from fa0e316 to d4abc37 Compare July 17, 2020 06:25
@krisgesling krisgesling removed the Status: Work in progress PR being actively worked on, not yet ready for review. label Jul 17, 2020
@krisgesling
Copy link
Contributor Author

As discussed in dev sync, I haven't had a date-time failure while the extra sleep here has been activated. However as the failure is so sporadic, this might be a coincidence.

To validate my assumption we're going to merge this and see if the issue arises anytime over the next few weeks. If it doesn't then there is indeed something that quite ready when the tests are initiated. If the test fails again, then the hypothesis is false and we can look elsewhere.

Either way we can then remove the sleep and be in a better position to find the underlying cause.

@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

@forslund
Copy link
Collaborator

Sounds good

Copy link
Collaborator

@forslund forslund left a comment

Choose a reason for hiding this comment

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

Feel free to merge away :)

@krisgesling krisgesling merged commit 6f33cc0 into dev Jul 17, 2020
@krisgesling krisgesling deleted the bugfix/random-vk-fail branch September 2, 2020 21:02
krisgesling added a commit that referenced this pull request Oct 28, 2020
krisgesling added a commit that referenced this pull request Jun 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants