#general
- e
enough-analyst-54434
02/05/2016, 9:54 PMnot sure on the java8 scenario - tried ssh'ing in and no-longer can - b
bored-art-40741
02/05/2016, 9:54 PMThose machines are ephemeral - b
bored-art-40741
02/05/2016, 9:55 PMThey do have java8 - e
enough-analyst-54434
02/05/2016, 9:55 PMthats what I figured - new instance - b
bored-art-40741
02/05/2016, 9:55 PMThe AMI has java8 baked in--and if it didn't, it would just skip the test anyway - b
bored-art-40741
02/05/2016, 9:55 PMMy hypothesis is that Travis doesn't have java8, so the test has been skipped - b
bored-art-40741
02/05/2016, 9:55 PMAnd if it hadn't been, it would have failed - e
enough-analyst-54434
02/05/2016, 9:55 PMwait, what doe it even mean for travis to have a java? - e
enough-analyst-54434
02/05/2016, 9:56 PMThat makes no sense - b
bored-art-40741
02/05/2016, 9:56 PMErr, how does it not make sense? - e
enough-analyst-54434
02/05/2016, 9:56 PMTravis is a program - b
bored-art-40741
02/05/2016, 9:56 PMClearly we are running all of these tests against jvms - b
bored-art-40741
02/05/2016, 9:57 PMI'm talking about the VM environment tests are run in - e
enough-analyst-54434
02/05/2016, 9:57 PMit just shells out to bash right away - e
enough-analyst-54434
02/05/2016, 9:57 PMthen our pytho code runs against the os - b
bored-art-40741
02/05/2016, 9:57 PMUh huh - e
enough-analyst-54434
02/05/2016, 9:57 PMI'm not getting it - b
bored-art-40741
02/05/2016, 9:57 PMAnd then runs all kinds of tests against JVMs that have to be present on the VM - e
enough-analyst-54434
02/05/2016, 9:58 PMand we have jvm location code - in python - e
enough-analyst-54434
02/05/2016, 9:58 PMthe jvm jenkins uses should matter not a whit - b
bored-art-40741
02/05/2016, 9:58 PMOkay, I think I explained my hypothesis really clearly above. Take a look attests/python/pants_test/backend/jvm/tasks/test_junit_run_integration.py:35 - b
bored-art-40741
02/05/2016, 9:59 PMThere are tests that don't run depending on which versions of the jdk are available in the environment - b
bored-art-40741
02/05/2016, 9:59 PMI'm claiming that Travis doesn't have jdk8 available, and I know that Jenkins does. In the first scenario, the test is just skipped - e
enough-analyst-54434
02/05/2016, 9:59 PMDistributionLocator.locate(minimum_version=version, maximum_version='{}.9999'.format(version)) - b
bored-art-40741
02/05/2016, 9:59 PMIn the second scenario, the test runs but fails. And I think it fails correctly--our code is wrong in this particular scenario - e
enough-analyst-54434
02/05/2016, 9:59 PMthat looks at the AMI - e
enough-analyst-54434
02/05/2016, 9:59 PMnot jenkins - b
bored-art-40741
02/05/2016, 9:59 PMThe "jenkins environment", yeah - e
enough-analyst-54434
02/05/2016, 10:00 PMit should be scanning PATH, etc - e
enough-analyst-54434
02/05/2016, 10:00 PMused to look in /usr/lib/jvm iirc, etc