chaoss / chaoss/CollectOSS

Unsure what to do about the facade path for moved repos

Open
#396 1 comment 0 reactions 0 assignees View on GitHub
data issues task:facade
Dominant language
Python
Stars
13
Forks
17
Avg merge
6h 59m
Merged PRs (30d)
1

Description

When we detect a repo has moved (301), we create a new dict of repo table data with the `repo_path` set to `None`

https://github.com/chaoss/CollectOSS/blob/115b4eec7c2d08d1bda4cef60c2c4c3dd8ea9546/collectoss/tasks/github/detect_move/core.py#L112

This line was last touched in 34d5c7db2c10af03faf8c650a6197198bd4031fa

We do this without resetting the facade collection state such that it will be cloned fresh next run, which essentially breaks collection/puts facade in a weird state.

Contributor guide

Open the contributing guide

Research direction

Start at collectoss/tasks/github/detect_move/core.py around line 112 and trace how the facade collection state is handled after a 301 redirect. Reproduce the next collection run for a moved repository, then verify that the repository data and facade state no longer leave collection broken or in an inconsistent state.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
data-engineering
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.