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

tooling: kicking off setec tooling #33967

Closed
wants to merge 2 commits into from

Conversation

alyssawilk
Copy link
Contributor

Right now (assuming we have permissions right which I likely don't) this will

  1. call us out on unassigned setec bugs
  2. call out idle bugs (1w no action)

in the long run I'm hoping it can do much of the "where are we for release tracking" figuring out which cve-next issues have fixes in progress, approved, etc but starting off simple.

Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #33967 was opened by alyssawilk.

see: more, trace.

- cron: '0 5 * * 1,2,3,4,5'

permissions:
contents: read # to fetch code (actions/checkout)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

not sure if there's any way for this to "run" off envoyproxy/envoy but look at setec (I assume no, which means updates will be annoying but I'm starting with that anyway just to see

Copy link
Member

@phlax phlax May 6, 2024

Choose a reason for hiding this comment

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

ci cant run in envoy using setec without giving it permissions to do so, which apart from weakening the security there would add a lot of complexity

the easier option is what i suggested elsewhere - strip setec's main to just ci and just add any necessary ci there

Signed-off-by: Alyssa Wilk <[email protected]>
@alyssawilk alyssawilk closed this May 6, 2024
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.

2 participants