Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix minio path mapping #97

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

derklaro
Copy link

this fixes an issue introduced in 220d850. In that commit the routing map file name was changed from routing.map to codecov.map, but that was never changed for minio. Therefore accesses to <host>/archive are not correctly proxied to minio, but rather to the default backend. This causes uploads to the storage to fail, after this change the upload works successfully.

Old:

debug - 2024-06-27 11:06:48,946 -- Upload request to Codecov complete. --- {"response": {"external_id": "...", "created_at": "2024-06-27T11:06:48.824828Z", "raw_upload_location": "https://<host>/archive/v4/raw/2024-06-27/7A629CCD.....", "state": "", "provider": null, "upload_type": "uploaded", "url": "...", "ci_url": "...", "flags": [], "env": {}, "name": "...", "job_code": "Test & Coverage"}}
debug - 2024-06-27 11:06:48,947 -- Sending upload to storage
warning - 2024-06-27 11:09:03,464 -- Request failed. Retrying --- {"retry": 0}
warning - 2024-06-27 11:11:18,632 -- Request failed. Retrying --- {"retry": 1}
warning - 2024-06-27 11:13:33,800 -- Request failed. Retrying --- {"retry": 2}

New:

debug - 2024-06-27 12:53:31,659 -- Upload request to Codecov complete. --- {"response": {"external_id": "...", "created_at": "2024-06-27T12:53:31.496127Z", "raw_upload_location": "https://<host>/archive/v4/raw/2024-06-27/62CD7E...", "state": "", "provider": null, "upload_type": "uploaded", "url": "...", "ci_url": "...", "flags": [], "env": {}, "name": "...", "job_code
debug - 2024-06-27 12:53:31,659 -- Sending upload to storage
info - 2024-06-27 12:53:32,088 -- Process Upload complete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants