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

CodeQL Workflow Fails after "upload" stage #2034

Closed
astrogeco opened this issue Jan 21, 2022 · 0 comments · Fixed by #2035 or #2043
Closed

CodeQL Workflow Fails after "upload" stage #2034

astrogeco opened this issue Jan 21, 2022 · 0 comments · Fixed by #2035 or #2043

Comments

@astrogeco
Copy link
Contributor

astrogeco commented Jan 21, 2022

Describe the bug
CodeQL workflow fails in the code ql analyze step, specifically right after the "upload" stage.

See failed run in

https://github.com/nasa/cFE/actions/runs/1726828015

Also silent "failure" in

https://github.com/nasa/cFE/runs/4892024047?check_suite_focus=true

Cleaning up databases
  /opt/hostedtoolcache/CodeQL/0.0.0-20211208/x64/codeql/codeql database cleanup /home/runner/work/_temp/codeql_databases/cpp --mode=brutal
  Cleaning up existing TRAP files after import...
  TRAP files cleaned up (3ms).
  Cleaning up scratch directory...
  Scratch directory cleaned up (0ms).

fatal: ambiguous argument 'refs/remotes/pull/2019/merge': unknown revision or path not in the working tree.

To Reproduce
Run latest CodeQL workflow

Expected behavior
Passing workflow run

Code snips
If applicable, add references to the software.

System observed on:

  • Github Actions, Ubuntu 18.04

Additional context
Can be fixed by updating workflow to use nasa/cFS#413

@astrogeco astrogeco changed the title CodeQL Workflow Fails after "upload" stage #412 CodeQL Workflow Fails after "upload" stage Jan 21, 2022
astrogeco added a commit to astrogeco/cFE that referenced this issue Jan 21, 2022
astrogeco added a commit to astrogeco/cFE that referenced this issue Jan 21, 2022
astrogeco added a commit to astrogeco/cFE that referenced this issue Jan 21, 2022
astrogeco added a commit to astrogeco/cFE that referenced this issue Jan 24, 2022
Adds new parameters to match updated cFS-CodeQL workflow interface
introduced in nasa/cFS#2035
astrogeco added a commit to astrogeco/cFE that referenced this issue Jan 24, 2022
Adds new parameters to match updated cFS-CodeQL workflow interface
introduced in nasa/cFS#2035
astrogeco added a commit to astrogeco/cFE that referenced this issue Jan 24, 2022
Adds new parameters to match updated cFS-CodeQL workflow interface
introduced in nasa/cFS#413
astrogeco added a commit to astrogeco/cFE that referenced this issue Jan 26, 2022
Adds new parameters to match updated cFS-CodeQL workflow interface
introduced in nasa/cFS#413
astrogeco added a commit to astrogeco/cFE that referenced this issue Jan 26, 2022
- Add new parameters to match updated cFS-CodeQL workflow interface
introduced in nasa/cFS#413
- Add file-exclusion checks to Action trigger so workflow doesn't run if
only changes in commit or pull request are to documentation
astrogeco added a commit to astrogeco/cFE that referenced this issue Feb 3, 2022
- Add new parameters to match updated cFS-CodeQL workflow interface
introduced in nasa/cFS#413
- Add file-exclusion checks to Action trigger so workflow doesn't run if
only changes in commit or pull request are to documentation
astrogeco added a commit that referenced this issue Feb 3, 2022
astrogeco added a commit to nasa/cFS that referenced this issue Feb 3, 2022
*cFE v7.0.0-rc4+dev70*

nasa/cFE#2041, Improve CFE_SB_IsValidMsgId handler
nasa/cFE#2034, Update CodeQL workflow
nasa/cFE#2042, Replace CFE_SB_ValueToMsgId(0) with CFE_SB_INVALID_MSG_ID

*osal v6.0.0-rc4+dev32*

nasa/osal#1209, typesafe definition of osal_id_t

*sample_app v1.3.0-rc4+dev9*

nasa/sample_app#165, Use preferred UT patterns

Co-authored-by: Jacob Hageman   <skliper@users.noreply.github.com>
Co-authored-by: Paul            <paulober@users.noreply.github.com>
Co-authored-by: Ariel Adams     <ArielSAdamsNASA@users.noreply.github.com>
@skliper skliper added this to the Draco milestone Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants