mirror of https://github.com/kubernetes/kops.git
Merge pull request #11879 from rifelpet/node-os-arch
Kubetest2 - set node-os-arch flag instead of skipping kubectl test on arm64
This commit is contained in:
commit
0432737d43
|
|
@ -70,17 +70,6 @@ func (t *Tester) setSkipRegexFlag() error {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
igs, err := t.getKopsInstanceGroups()
|
|
||||||
if err != nil {
|
|
||||||
return err
|
|
||||||
}
|
|
||||||
for _, ig := range igs {
|
|
||||||
if strings.Contains(ig.Spec.Image, "arm64") {
|
|
||||||
skipRegex += "|Simple.pod.should.handle.in-cluster.config"
|
|
||||||
break
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// Ensure it is valid regex
|
// Ensure it is valid regex
|
||||||
if _, err := regexp.Compile(skipRegex); err != nil {
|
if _, err := regexp.Compile(skipRegex); err != nil {
|
||||||
return err
|
return err
|
||||||
|
|
|
||||||
|
|
@ -302,6 +302,21 @@ func (t *Tester) getZones() ([]string, error) {
|
||||||
return zoneNames, nil
|
return zoneNames, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func (t *Tester) addNodeOSArchFlag() error {
|
||||||
|
igs, err := t.getKopsInstanceGroups()
|
||||||
|
if err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
for _, ig := range igs {
|
||||||
|
if strings.Contains(ig.Spec.Image, "arm64") {
|
||||||
|
klog.Info("Setting --node-os-arch=arm64")
|
||||||
|
t.TestArgs += " --node-os-arch=arm64"
|
||||||
|
break
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
|
||||||
func (t *Tester) execute() error {
|
func (t *Tester) execute() error {
|
||||||
fs, err := gpflag.Parse(t)
|
fs, err := gpflag.Parse(t)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
|
@ -355,6 +370,10 @@ func (t *Tester) execute() error {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if err := t.addNodeOSArchFlag(); err != nil {
|
||||||
|
return err
|
||||||
|
}
|
||||||
|
|
||||||
return t.Test()
|
return t.Test()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue