-
Notifications
You must be signed in to change notification settings - Fork 84
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
Codecov blocking CI jobs #250
Comments
I'm working on cancelling the jobs right now. If anyone wants to submit a PR to upgrade to codecov-action v4, under a copier question, that would be helpful. |
Now cancelling all queued jobs: import json
import subprocess
def get_repos(org: str):
res = subprocess.check_output(
["gh", "repo", "list", "--limit", "1000", "--json", "name", org]
)
for repo in json.loads(res):
yield f"{org}/{repo['name']}"
def get_queued_jobs(repo: str):
res = subprocess.check_output(
[
"gh",
"run",
"--repo",
repo,
"list",
"--status",
"queued",
"--json",
"databaseId",
]
)
for job in json.loads(res):
yield job["databaseId"]
for repo in get_repos("OCA"):
print(repo)
for job in get_queued_jobs(repo):
subprocess.run(["gh", "run", "--repo", repo, "cancel", str(job)]) |
Can this be done adding a organization-wide secret? In such case, I guess it's better to update to the newest version, now that we do it. |
Yes absolutely. I have already added an org wide secret named See example here: OCA/mis-builder#606 This does not block anymore but fails to find the coverage report, though. |
I'm on it |
Didn't know there was this issue too, so I'll stop bothering in discord (https://discord.com/channels/737652535149592587/761225067798462465) and start here 😄
..and it went well, see https://github.com/OCA/l10n-italy/actions/runs/8892070599/job/24415363278. Also note that the token configuration is not apparently needed because https://github.com/OCA/l10n-italy/actions/runs/8892015940/job/24415185936 is successful too, but I suppose explicit is better than implicit 😉 |
It says So we need to check it works without token on ocabot merge and in main branches too. |
Right, so we have to use the token, thanks for pointing that out :) - uses: codecov/codecov-action@v4
with:
fail_ci_if_error: true # optional (default = false)
files: ./coverage1.xml,./coverage2.xml # optional
flags: unittests # optional
name: codecov-umbrella # optional
token: ${{ secrets.CODECOV_TOKEN }} # required
verbose: true # optional (default = false) |
So, we have a working PoC, correct? Anyone going for the PR? |
Yes, the PR already exists: #251. |
PR merged, so now a release of the template should be done and do a massive repo update. |
I'll do that today. |
@sbidoul I think you should run it automatically also for 15.0, not only 16.0 and 17.0 |
yesyes, and all other branches from 11... Problem is that with all the repos/branches where pre-commit does not pass, I'm creating PRs for PSC to review, but until that is merged, the broken codecov will remain present and blocking. I'll look at patching the codecov step only, but that may create copier conflict later. What a mess... |
OK, I see. I'm starting to fix incorrect branches, as you can see in OCA/account-payment#726. The lack of "line too long" check on 17.0 branches when switching to ruff is doing a lot of harm... |
Sorry about that. I apparently had forgotten to push that last change back then. |
Well, better later than never. Let's go with it! |
Thanks @sbidoul for your amazing job 💪 |
@sbidoul can you please cancel old Codecov jobs that are still blocking actions for the dotfiles PRs? |
There are none at the moment. |
OK, then maybe it's just the regular actions corresponding to the pushes. I'm waiting for example this PR to start its actions: OCA/account-invoicing#1718 |
16 and 17 are done. 14 and 15 in progress. |
This should help in the future to protect against situations like OCA#250, when Codecov suddenly changes something, starts to fail and becomes blocking for everybody here.
It doesn't look like so. AFAICS the timesheet 16.0 repo still hasn't the update. I don't mind updating the template myself, but I'll wait for your answer. I just wanted to warn you in case something went under the radar in your process. Thank you very much @sbidoul. I hope #253 smooths future problems. |
@yajo Look at : OCA/timesheet#672 Should be treated manually |
I have fixed and merged all the remaining pull request that require manual intervention except the |
Thank you Pedro! |
In the meantime, codecov has fixed the issue, so I think I'll leave the older branches alone for the moment. |
OK, better to not open the Pandora's box... |
Since yesterday, the codecov action step is blocking all our CI jobs.
This blocks basically everything as there is a timeout of several hours on GitHub actions.
Issue submitted to codecov here: codecov/feedback#354
Things to try
The text was updated successfully, but these errors were encountered: