MetaCell / MetaCell/cloud-harness
Remove - to _ replacements in the deployment generation
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 19
- Forks
- 5
- Avg merge
- 3d 18h
- Merged PRs (30d)
- 3
Description
application names containing dashes are being replaced by underscores.
There initial reason for this change is that on helm is not straightforward to access keys with hashes (see https://github.com/helm/helm/issues/2192). At the same time, application names are not usually accessed directly, so using the index syntax as suggested in the issue above it's a viable solution, causing less troubles than the actual alternative:
There is a lot of code around dealing with the replacement and that's very prone to introduce bugs in the long term.
Contributor guide
No contributing guide indexed for this repository
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
No files or tests are named. Start by locating the deployment generation code and tracing the application-name dash-to-underscore replacement, then review the referenced Helm issue. Done means the replacement logic is removed, dashed application names remain usable through the proposed index syntax, and relevant deployment generation checks pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- helm, kubernetes
- Domain
- devops, infrastructure
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100