search

kubernetes / sig-security

Process documentation, non-code deliverables, and miscellaneous artifacts of Kubernetes SIG Security
Apache License 2.0
161 stars 53 forks source link

Fix a naming issue in CVE feed script #80

Closed mtardy closed 1 year ago

mtardy commented 1 year ago

The, just merged, new script is causing failure because of a stupid naming fail (it's my fault) https://testgrid.k8s.io/sig-security-cve-feed#auto-refreshing-official-cve-feed.

I wrote official_cve_feed.json instead of official-cve-feed.json, which messed up the job.

If someone can lgtm and approve this as soon as possible it would be charming so that I can pursue and continue merging the web side.

mtardy commented 1 year ago

cc @PushkarJ

mtardy commented 1 year ago

/priority critical-urgent

PushkarJ commented 1 year ago

/lgtm /approve

k8s-ci-robot commented 1 year ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mtardy, PushkarJ

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[sig-security-tooling/OWNERS](https://github.com/kubernetes/sig-security/blob/main/sig-security-tooling/OWNERS)~~ [PushkarJ] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment