Utopic test failures due to addition of vivid series

Bug #1384175 reported by Martin Packman
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-core
Fix Released
Critical
Tim Penhey
1.20
Fix Released
Critical
Anastasia

Bug Description

The name for the next release of ubuntu has been announced, and added to utopic:

<http://www.markshuttleworth.com/archives/1425>

This is causing two test failures in the juju-core suite:

PANIC: local_test.go:335: localServerSuite.TestStartInstanceHardwareCharacteristics
...
... Panic: osVersion reported an error: invalid series "vivid" (PC=0x414676)

/usr/lib/go/src/pkg/runtime/panic.c:248
  in panic
/home/ubuntu/juju-core_1.21-alpha2/src/github.com/juju/juju/version/osversion.go:36
  in MustOSFromSeries
/home/ubuntu/juju-core_1.21-alpha2/src/github.com/juju/juju/environs/tools/simplestreams.go:178
  in ToolsMetadata.binary
/home/ubuntu/juju-core_1.21-alpha2/src/github.com/juju/juju/environs/tools/simplestreams.go:327
  in MergeMetadata
/home/ubuntu/juju-core_1.21-alpha2/src/github.com/juju/juju/environs/tools/simplestreams.go:430
  in MergeAndWriteMetadata
/home/ubuntu/juju-core_1.21-alpha2/src/github.com/juju/juju/environs/testing/tools.go:184
  in UploadFakeToolsVersions
/home/ubuntu/juju-core_1.21-alpha2/src/github.com/juju/juju/environs/testing/tools.go:192
  in AssertUploadFakeToolsVersions
local_test.go:341
  in localServerSuite.TestStartInstanceHardwareCharacteristics

...

FAIL: supportedseries_linux_test.go:23: supportedSeriesSuite.TestSupportedSeries

supportedseries_linux_test.go:27:
    c.Assert(series, gc.DeepEquals, expectedSeries)
... obtained []string = []string{"precise", "quantal", "raring", "saucy", "trusty", "utopic", "vivid"}
... expected []string = []string{"precise", "quantal", "raring", "saucy", "trusty", "utopic"}

OOPS: 21 passed, 1 FAILED

<http://juju-ci.vapour.ws:8080/job/run-unit-tests-utopic-amd64/829/console>

Ian Booth (wallyworld)
Changed in juju-core:
importance: High → Critical
assignee: nobody → Anastasia (anastasia-macmood)
Revision history for this message
Curtis Hovey (sinzui) wrote :

This kind of test failed last year too. My fix was to limit the array to the number of known items. New series can be added, beyond 6, but we will only check the first 6.

tags: added: vivid
Aaron Bentley (abentley)
tags: added: regression
Revision history for this message
Aaron Bentley (abentley) wrote :
Revision history for this message
Nate Finch (natefinch) wrote :

I have a fix for this here, btw: http://reviews.vapour.ws/r/237/

Once the review is finished I can backport to 1.20.11.

Tim Penhey (thumper)
Changed in juju-core:
assignee: Anastasia (anastasia-macmood) → Tim Penhey (thumper)
status: Triaged → In Progress
Ian Booth (wallyworld)
Changed in juju-core:
status: In Progress → Fix Committed
Martin Packman (gz)
Changed in juju-core:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.