Fixes lp:1638706 azure tests fail on non amd64 #6560

Merged
merged 1 commit into from Nov 10, 2016

Conversation

Projects
None yet
3 participants
Contributor

reedobrien commented Nov 10, 2016

Forward port (cherry picked from commit cd17ec2)

QA: Run the target test on arm64-slave
1 .Verify it fails on develop

$ git checkout develop
Switched to branch 'develop'
Your branch is up-to-date with 'upstream/develop'.

$ go test -check.f TestBootstrapInstanceConstraints

----------------------------------------------------------------------
FAIL: environ_test.go:1005: environSuite.TestBootstrapInstanceConstraints

[LOG] 0:00.001 DEBUG juju.provider.azure opening model "testenv"
[LOG] 0:00.001 DEBUG juju.provider.azure.internal.azuretesting Senders.Do(https://api.azurestack.local/subscriptions/22222222-2222-2222-2222-222222222222?api-version=2015-11-01)
[LOG] 0:00.001 DEBUG juju.provider.azure.internal.azureauth discovered auth URI: https://testing.invalid/11111111-1111-1111-1111-111111111111
[LOG] 0:00.001 DEBUG juju.provider.azure.internal.azuretesting Senders.Do(https://testing.invalid/11111111-1111-1111-1111-111111111111/oauth2/token?api-version=1.0)
[LOG] 0:00.002 DEBUG juju.environs.bootstrap model "testenv" supports service/machine networks: false
[LOG] 0:00.002 DEBUG juju.environs.bootstrap network management by juju enabled: true
[LOG] 0:00.002 INFO cmd Loading image metadata
[LOG] 0:00.002 DEBUG juju.provider.azure.internal.azuretesting Senders.Do(https://api.azurestack.local/subscriptions/22222222-2222-2222-2222-222222222222/providers/Microsoft.Compute/locations/westus/vmSizes?api-version=2016-03-30)
[LOG] 0:00.003 INFO juju.environs.bootstrap looking for bootstrap agent binaries: version=1.2.3
[LOG] 0:00.003 INFO juju.environs.tools finding agent binaries in stream "released"
[LOG] 0:00.003 INFO juju.environs.tools reading agent binaries with major.minor version 2.1
[LOG] 0:00.003 INFO juju.environs.tools filtering agent binaries by version: 1.2.3
[LOG] 0:00.003 INFO juju.environs.tools filtering agent binaries by series: quantal
[LOG] 0:00.003 INFO juju.environs.tools filtering agent binaries by architecture: arm64
[LOG] 0:00.003 INFO juju.environs.bootstrap found 0 packaged agent binaries
environ_test.go:1030:
    c.Assert(err, jc.ErrorIsNil)
... value *errors.Err = &errors.Err{message:"model \"testenv\" of type azure does not support instances running on \"arm64\"", cause:error(nil), previous:error(nil), file:"github.com/juju/juju/environs/bootstrap/tools.go", line:51} ("model \"testenv\" of type azure does not support instances running on \"arm64\"")
... error stack:
	github.com/juju/juju/environs/bootstrap/tools.go:51: model "testenv" of type azure does not support instances running on "arm64"

OOPS: 0 passed, 1 FAILED
--- FAIL: TestPackage (0.02s)
FAIL
exit status 1
FAIL	github.com/juju/juju/provider/azure	0.202s
  1. Verify it passes with forward port.
$ go test -check.f TestBootstrapInstanceConstraints
OK: 1 passed
PASS
ok  	github.com/juju/juju/provider/azure	0.183s

bz2 approved these changes Nov 10, 2016

Looks good, thanks!

Contributor

reedobrien commented Nov 10, 2016

$$merge$$

Contributor

reedobrien commented Nov 10, 2016

$$merge$$

Contributor

jujubot commented Nov 10, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

reedobrien commented Nov 10, 2016

!!??!!

@jujubot jujubot merged commit d07e31b into juju:develop Nov 10, 2016

1 check was pending

github-check-merge-juju Jenkins queued the run (PR was triggered by comment)
Details

@reedobrien reedobrien deleted the reedobrien:lp/1638706_forwardport branch Nov 10, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment