[infra] Dont upload a build status for all projects

[jonathanmetzman]

Projects that are deleted/disabled/incomplete should not get them.

Fixes #5608

[asraa]

Thanks! Code looks great, could I ask a quick question I was wondering when looking at the code?

Eventually history will be None for projects when we process them and we'll just skip them, so this fixes when there's some historical builds but now the project is deleted/incomplete/etc? I like your solution more than removing history if not project_should_build.

[jonathanmetzman]

Thanks! Code looks great, could I ask a quick question I was wondering when looking at the code?

Eventually history will be None for projects when we process them and we'll just skip them, so this fixes when there's some historical builds but now the project is deleted/incomplete/etc? I like your solution more than removing history if not project_should_build.

Exactly.

[oliverchang]

This won't work unfortunately, because this is not run as part of an OSS-Fuzz checkout.

Instead, what we can do to fix this here is to do a datastore query for the Project entity:

oss-fuzz/infra/build/functions/project_sync.py

Line 132 in 6ca344a

Project(name=project_name,

If it does not exist, then don't upload.

These entities are already being deleted when a project is deleted, but not when it's disabled:

oss-fuzz/infra/build/functions/project_sync.py

Line 117 in 6ca344a

delete_project(cloud_scheduler_client, project)

Probably need to change something around here to check for the disabled attribute:

oss-fuzz/infra/build/functions/project_sync.py

Line 202 in 6ca344a

def get_projects(repo):

[inferno-chromium]

request changes as per last review.