-
Notifications
You must be signed in to change notification settings - Fork 42
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
Update test matrix to run faster & add 18.04 #107
Conversation
Signed-off-by: Amy Aronsohn <WagThatTail@Me.com>
Signed-off-by: Amy Aronsohn <WagThatTail@Me.com>
On both suites it's not the authoritive that is failing now but Postgresql test suite, they are timing out. I did not have this issue when I ran all the 18.04 tests locally but I will run the full suite and see if I can replicate |
Signed-off-by: Amy Aronsohn <WagThatTail@Me.com>
PoswerDNS did not make a 409 repo for Bionic Signed-off-by: Amy Aronsohn <WagThatTail@Me.com>
The problem is we cannot test installing 40 on 18.04, there is no repo for it. The tests were hiding the problem because they have allow failure on, so things broke later.
|
We install a specific minor now Signed-off-by: Amy Aronsohn <WagThatTail@Me.com>
Signed-off-by: Amy Aronsohn <WagThatTail@Me.com>
Signed-off-by: Amy Aronsohn <WagThatTail@Me.com>
I'm passing all locally now so just waiting for tests to finish in Travis |
Signed-off-by: Amelia Aronsohn <WagThatTail@Me.com>
right now tracing down what test failed in a huge suite is problimatic. Lets break this up to make it easier to troubleshoot. I stole all of this from chef-cookbooks/chef-client Signed-off-by: Amelia Aronsohn <WagThatTail@Me.com>
Signed-off-by: Amelia Aronsohn <WagThatTail@Me.com>
Signed-off-by: Amelia Aronsohn <WagThatTail@Me.com>
Trying to make the test suite faster I enabled concurrent builds for this repo, there is not any resources it needs that should require them. I also broke out all instances so they are easier to see where the failures are. This gives us much more concurrency than the two we originally had |
Signed-off-by: Amelia Aronsohn <WagThatTail@Me.com>
While we still have the one test failing and timing out at least it runs way faster Before: Ran for 1 hr 29 min 42 sec |
If we're ok with the 18.04 PG test not completing we can either remove it or comment it out with a note. |
Signed-off-by: Amelia Aronsohn <WagThatTail@Me.com>
I've only removed it in the travis matrix, as the test runs locally without issue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic work! Sooooooo much faster too, I love it!
Travis numbers:
Before: Ran for 1 hr 29 min 42 sec
After: Ran for 12 min 12 sec
Using travis' concurrency with separated kitchen instances makes finding errors, parsing the logs, and all general troubleshooting better too.