feat: add karmada compatibility scraper and resolved conflicts#3579
feat: add karmada compatibility scraper and resolved conflicts#3579smyja wants to merge 2 commits into
Conversation
Greptile SummaryAdds Karmada as a new compatibility entry: a Python scraper that parses the Kubernetes compatibility table from the Karmada README, a seed YAML with three initial versions (1.15–1.17), and the manifest registration that wires it into the aggregation pipeline.
Confidence Score: 3/5The static seed data and manifest changes are safe, but the wrong Helm repository URL means the scraper will fail to auto-populate chart versions for any new Karmada releases going forward. The helm_repository_url in karmada.yaml points at the raw GitHub source tree rather than the published Helm chart repository, so every future scraper run will silently fail to resolve chart versions for new Karmada releases. The three existing seed entries are unaffected because they already carry hardcoded chart_version values, but the ongoing maintenance path is broken until the URL is corrected. static/compatibilities/karmada.yaml (wrong helm_repository_url) and utils/compatibility/scrapers/karmada.py (unused constant, version normalization truncation)
|
| Filename | Overview |
|---|---|
| utils/compatibility/scrapers/karmada.py | New scraper parses the Karmada README markdown compatibility table; has an unused HELM_REPO_URL constant and a normalize_version function that silently truncates patch versions to .0 |
| static/compatibilities/karmada.yaml | New compatibility data for Karmada 1.15-1.17; helm_repository_url points to the raw GitHub source directory instead of the published Helm chart repository, which will cause chart version lookups to fail |
| static/compatibilities/manifest.yaml | Adds karmada to the scraper manifest list; change is correct and consistent with other entries |
| static/compatibilities.yaml | Aggregated file updated with the karmada addon entry; inherits the wrong helm_repository_url from karmada.yaml but otherwise structurally correct |
Reviews (1): Last reviewed commit: "feat: add karmada compatibility scraper" | Re-trigger Greptile
https://www.cncf.io/projects/karmada/
https://github.com/karmada-io/karmada?tab=readme-ov-file#kubernetes-compatibility
Closed previous stale pr and reopened new one with fixes
Test Plan
Test environment: https://console.your-env.onplural.sh/
Checklist
Plural Flow: console