zalando-incubator / zalando-incubator/stackset-controller
Some low hanging fruits to improve the coverage:
Open
Nobody has claimed this yet.
test
- Dominant language
- Go
- Stars
- 183
- Forks
- 29
- Avg merge
- 3d 18h
- Merged PRs (30d)
- 5
Description
- the error cases in StackSetController.collectResources()
- the normalizWeight edge cases in traffic.go
via: https://github.com/zalando-incubator/stackset-controller/pull/251#issuecomment-694266780
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by inspecting StackSetController.collectResources() and the normalizWeight function in traffic.go, along with any existing Go tests for those areas. Add coverage for the error cases and normalization edge cases identified in the issue, then run the relevant Go test suite and confirm those paths are exercised.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go, kubernetes
- Domain
- devops, testing
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100