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

Disable codeql to avoid timeouts / long runtimes #31726

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

Conversation

scbedd
Copy link
Member

@scbedd scbedd commented Nov 12, 2024

This is how it used to run.

In the past week, we started seeing very long builds due to codeQL kicking on:

@benbp
Copy link
Member

benbp commented Nov 13, 2024

Should we start a thread with 1es?

@benbp
Copy link
Member

benbp commented Nov 13, 2024

Seeing a ton of errors like this, wonder if it's related to the sparse checkout changes?

[2024-11-11 18:57:21] [build-stdout] sdk/schemaregistry/schema-registry/samples/v1/typescript/package.json: Main file not found

@scbedd
Copy link
Member Author

scbedd commented Nov 14, 2024

Seeing a ton of errors like this, wonder if it's related to the sparse checkout changes?

[2024-11-11 18:57:21] [build-stdout] sdk/schemaregistry/schema-registry/samples/v1/typescript/package.json: Main file not found

When you say "sparse checkout changes" what do you mean? If you mean my sparse-checkout changes to generate-job-matrix, then I would answer "no" because I didn't change the checkout of release deployments at all. Which is where those "missing main file" errors are visible.

However, if you were to tell me

the reason you're seeing these main file not found errors is because of sparse checkout

I'd agree with you 👍

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

Successfully merging this pull request may close these issues.

2 participants