Skip to content

arm64: fix kubeadm.* tests#217

Merged
tormath1 merged 5 commits intoflatcar-masterfrom
tormath1/fix-kubeadm-test
Sep 3, 2021
Merged

arm64: fix kubeadm.* tests#217
tormath1 merged 5 commits intoflatcar-masterfrom
tormath1/fix-kubeadm-test

Conversation

@tormath1
Copy link
Copy Markdown
Contributor

@tormath1 tormath1 commented Aug 26, 2021

In this PR:

  • we try to fix the arm64 tests for kubeadm.* by updating the generated config in order to use checksum based on the arch
  • we increase the API server timeout value

How to use

sudo ./bin/kola run --board arm64-usr ... kubeadm.v1.22.0.cilium.base

Testing done

The API server seems to boot but I don't know why, the container crashes right after for no reason and randomly. I suspect hitting some memory / CPU issue. Let's try with CI.

@tormath1 tormath1 self-assigned this Aug 26, 2021
@tormath1 tormath1 force-pushed the tormath1/fix-kubeadm-test branch from 5638471 to 2c5438e Compare August 30, 2021 10:13
@tormath1 tormath1 changed the title [wip] arm64: fix kubeadm.* tests arm64: fix kubeadm.* tests Aug 30, 2021
@tormath1 tormath1 requested a review from a team August 30, 2021 10:26
@tormath1 tormath1 marked this pull request as ready for review August 30, 2021 10:26
Mathieu Tortuyaux added 5 commits August 31, 2021 13:22
we access the checksum conditionnaly based on the arch

Signed-off-by: Mathieu Tortuyaux <mathieu@kinvolk.io>
Signed-off-by: Mathieu Tortuyaux <mathieu@kinvolk.io>
the main change is that we need to generate checksum in function of
the architecture.

Signed-off-by: Mathieu Tortuyaux <mathieu@kinvolk.io>
on ARM64, API server takes some time to boot up.

Signed-off-by: Mathieu Tortuyaux <mathieu@kinvolk.io>
Signed-off-by: Mathieu Tortuyaux <mathieu@kinvolk.io>
@tormath1 tormath1 force-pushed the tormath1/fix-kubeadm-test branch from 2c5438e to 609b149 Compare August 31, 2021 11:24
@tormath1 tormath1 merged commit 8f95956 into flatcar-master Sep 3, 2021
@tormath1 tormath1 deleted the tormath1/fix-kubeadm-test branch September 3, 2021 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants