Skip to content

update codeql-action to v2#7568

Merged
fuweid merged 1 commit intocontainerd:mainfrom
akhilerm:code-ql-version-bump
Oct 21, 2022
Merged

update codeql-action to v2#7568
fuweid merged 1 commit intocontainerd:mainfrom
akhilerm:code-ql-version-bump

Conversation

@akhilerm
Copy link
Copy Markdown
Member

The current codeql version v1 that is being used in the github actions are stated for deprecation on 7-Dec-2022 Ref.
Currently it shows as a warning in the github action runs.

Updating the versions to v2 for the codeql actions. The version for codeql-action/upload-sarif is pinned to commit corresponding to tag v2.1.28 as per the recommendation in this PR

Signed-off-by: Akhil Mohan [email protected]

updating codeql github actions to v2, as v1 version will be deprecated
in Dec'22.

Signed-off-by: Akhil Mohan <[email protected]>
@k8s-ci-robot
Copy link
Copy Markdown

Hi @akhilerm. Thanks for your PR.

I'm waiting for a containerd member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

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

LGTM

@fuweid fuweid merged commit b9206de into containerd:main Oct 21, 2022
@akhilerm akhilerm deleted the code-ql-version-bump branch October 22, 2022 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants