oxidecomputer / oxidecomputer/cluster-api-provider-oxide

E2E test improvements

Open
#29 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Task
Dominant language
Go
Stars
2
Forks
1
Avg merge
3h 57m
Merged PRs (30d)
6

Description

We have a single e2e test today, which spins up a kind management cluster and a minimal workload cluster. We may want to test more scenarios: do we successfully reattach the control plane's floating IP if the attached control plane instance vanishes? Do we adopt the floating IP if it already exists? And so on. We should also discuss which tests belong as e2e tests on a real oxide cluster, which as unit tests, and which as integration tests with a stubbed k8s and oxide client.

We should also e2e test a simple workload on the workload cluster, because I have a suspicion that the current e2e test cluster actually won't produce a working network overlay; I'll file a separate issue about this.

Finally, we should consider running the e2e tests on ci, since they're only run on developer machines today.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the existing single end-to-end test that creates the kind management and workload clusters, then review how the repository separates end-to-end, unit, and integration tests. Define coverage for floating-IP recovery, existing-IP adoption, and a simple workload, and determine how these tests should run in CI; done means the selected scenarios and test placement are implemented.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, kubernetes
Domain
ci-cd, infrastructure, testing-qa
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.