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

CI: move CodeQL tests to default.yaml #1971

Closed
wants to merge 1 commit into from

Conversation

kinkie
Copy link
Contributor

@kinkie kinkie commented Dec 28, 2024

Github CodeQL is complaining that it can't find
the configuration for the tests
There is no documented reason for this, but there
is also no documented way to change this setting

Move CodeQL workflows back to default.yaml to
make Github happy again

Github CodeQL is complaining that it can't find
the configuration for the codeql tests.
There is no documented reason for this, but there
is also no documented way to change this setting.

Move CodeQL workflows back to default.yaml to
make Github happy again
Copy link
Contributor

@rousskov rousskov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for attacking this old problem!

Error: Code Scanning could not process the submitted SARIF file:
CodeQL analyses from advanced configurations cannot be processed when the default setup is enabled

Even if the odd naming requirement does exist, the fact that this PR does not fix CodeQL tests suggests that there may be more to this story than that odd naming requirement.

If you have not tried that already, and there is no known reason to fear some kind of serious information/configuration loss, then I suggest completely disabling CodeQL and then re-enabling it from scratch in hope that GitHub will find current/official workflows and does the right thing... This is a very weak recommendation -- CodeQL is not my area of expertise.

Please ping me when the tests are fixed.

Note to self: We appear to be using "default setup" for CodeQL. Some related configuration details are at https://github.com/squid-cache/squid/settings/code-scanning/default-setup

Comment on lines +1 to +2
# CodeQL tests apparently have to be deefined in a file
# named default.yaml - this is not well documented
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you point me to any documentation that states this odd requirement? If not, I suggest:

Suggested change
# CodeQL tests apparently have to be deefined in a file
# named default.yaml - this is not well documented
# Apparently, CodeQL tests must be defined in a file named default.yaml.
# We cannot find where that requirement is documented, but these tests
# fail with "configurations not found" errors if defined in quick.yaml.

The above suggestion also fixes a misspelling.

@rousskov rousskov added the S-waiting-for-author author action is expected (and usually required) label Dec 29, 2024
@kinkie
Copy link
Contributor Author

kinkie commented Dec 29, 2024

I think I found why: it's a very well hidden settings page at https://github.com/squid-cache/squid/security/code-scanning/tools/CodeQL/status

@rousskov
Copy link
Contributor

I think I found why: it's a very well hidden settings page at https://github.com/squid-cache/squid/security/code-scanning/tools/CodeQL/status

Glad you are making progress! FWIW, I do not see any settings at the above status page.

@kinkie
Copy link
Contributor Author

kinkie commented Dec 29, 2024 via email

@kinkie
Copy link
Contributor Author

kinkie commented Dec 31, 2024

The stuck PRs seem to be dequeueing, including CodeQL checks which we perform but not enforce.
This change seems to be no longer needed. Closing it

@kinkie kinkie closed this Dec 31, 2024
@squid-anubis squid-anubis added the M-failed-other https://github.com/measurement-factory/anubis#pull-request-labels label Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
M-failed-other https://github.com/measurement-factory/anubis#pull-request-labels S-waiting-for-author author action is expected (and usually required)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants