Skip to content
This repository has been archived by the owner on May 7, 2021. It is now read-only.

Commit

Permalink
kola/tests: disable tests on Azure
Browse files Browse the repository at this point in the history
The Azure SDK requires a hostname to be set via the API during machine
creation and then overwrites the hostname post-boot via wa-agent.

Update the metadata/contents test to validate only COREOS_AZURE_IPV4_DYNAMIC
as the machines being created are not behind a load balancer.
  • Loading branch information
arithx committed Nov 29, 2017
1 parent 4ce382b commit c5a5a18
Show file tree
Hide file tree
Showing 5 changed files with 27 additions and 9 deletions.
22 changes: 14 additions & 8 deletions kola/tests/ignition/sethostname.go
Original file line number Diff line number Diff line change
Expand Up @@ -59,17 +59,23 @@ func init() {
]
}
}`)

// These tests are disabled on Azure because the hostname
// is required by the API and is overwritten via waagent.service
// after the machine has booted.
register.Register(&register.Test{
Name: "coreos.ignition.v1.sethostname",
Run: setHostname,
ClusterSize: 1,
UserData: configV1,
Name: "coreos.ignition.v1.sethostname",
Run: setHostname,
ClusterSize: 1,
UserData: configV1,
ExcludePlatforms: []string{"azure"},
})
register.Register(&register.Test{
Name: "coreos.ignition.v2.sethostname",
Run: setHostname,
ClusterSize: 1,
UserData: configV2,
Name: "coreos.ignition.v2.sethostname",
Run: setHostname,
ClusterSize: 1,
UserData: configV2,
ExcludePlatforms: []string{"azure"},
})
}

Expand Down
4 changes: 3 additions & 1 deletion kola/tests/metadata/contents.go
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,9 @@ func verifyAWS(c cluster.TestCluster) {
}

func verifyAzure(c cluster.TestCluster) {
verify(c, "COREOS_AZURE_IPV4_DYNAMIC", "COREOS_AZURE_IPV4_VIRTUAL")
verify(c, "COREOS_AZURE_IPV4_DYNAMIC")
// kola tests do not spawn machines behind a loasd balancer on Azure
// which is required for COREOS_AZURE_IPV4_VIRTUAL to be present
}

func verifyPacket(c cluster.TestCluster) {
Expand Down
1 change: 1 addition & 0 deletions kola/tests/misc/install.go
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ func init() {
}]
}
}`),
ExcludePlatforms: []string{"azure"},
})
}

Expand Down
6 changes: 6 additions & 0 deletions kola/tests/misc/nfs.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,11 +49,17 @@ func init() {
Run: NFSv3,
ClusterSize: 0,
Name: "linux.nfs.v3",
// Disabled on Azure because setting hostname
// is required at the instance creation level
ExcludePlatforms: []string{"azure"},
})
register.Register(&register.Test{
Run: NFSv4,
ClusterSize: 0,
Name: "linux.nfs.v4",
// Disabled on Azure because setting hostname
// is required at the instance creation level
ExcludePlatforms: []string{"azure"},
})
}

Expand Down
3 changes: 3 additions & 0 deletions kola/tests/systemd/journald.go
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,9 @@ func init() {
Run: journalRemote,
ClusterSize: 0,
Name: "systemd.journal.remote",
// Disabled on Azure because setting hostname
// is required at the instance creation level
ExcludePlatforms: []string{"azure"},
})
}

Expand Down

0 comments on commit c5a5a18

Please sign in to comment.