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

Merged
merged 1 commit into from Nov 10, 2016

Conversation

Projects
None yet
3 participants
Contributor

reedobrien commented Nov 10, 2016

For azure provider environ test that is failing, verify error message on non amd64 arches and succeed, otherwise continue test on amd64.

QA: Run unit tests on non amd64 arch and see if they pass.

  1. fails on 2.0
jenkins@swirlix-30:~/redir/src/github.com/juju/juju/provider/azure$ ./azure-2.0.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.002 DEBUG juju.provider.azure.internal.azureauth discovered auth URI: https://testing.invalid/11111111-1111-1111-1111-111111111111
[LOG] 0:00.002 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.0
[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:
    ...
... 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
  1. Passes when patched:
jenkins@swirlix-30:~/redir/src/github.com/juju/juju/provider/azure$ ./azure-patched.test -check.f TestBootstrapInstanceConstraints
OK: 1 passed
PASS

bz2 approved these changes Nov 10, 2016

Looks good, thanks!

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

@jujubot jujubot merged commit a83896d into juju:2.0 Nov 10, 2016

1 check passed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details

@reedobrien reedobrien deleted the reedobrien:lp/1638706_environSuite-fails-rare-archs branch Nov 10, 2016

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