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.

Sign up for GitHub

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

Jump to bottom

cleanup(ci): drop workflow dispatch trigger. #187

Merged
merged 1 commit into from
Oct 20, 2023
Merged

cleanup(ci): drop workflow dispatch trigger. #187

merged 1 commit into from
Oct 20, 2023

Conversation

FedeDP
Copy link
Collaborator

@FedeDP FedeDP commented Oct 20, 2023

What type of PR is this?

/kind cleanup

Any specific area of the project related to this PR?

/area ci

What this PR does / why we need it:

The workflow_dispatch trigger makes no sense now:

  • we already run crawler daily
  • even if we force a kernel-crawler run, test-infra update-dbg still runs daily, thus it has no effect at all

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

maxgio92
maxgio92 approved these changes Oct 20, 2023
View reviewed changes
Copy link
Member

@maxgio92 maxgio92 left a comment

Choose a reason for hiding this comment

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

/approve

@poiana poiana added the lgtm label Oct 20, 2023
@poiana poiana merged commit dd3f22e into main Oct 20, 2023
5 checks passed
@poiana poiana deleted the cleanup/drop_workflow_dispatch branch October 20, 2023 15:28
@FedeDP FedeDP restored the cleanup/drop_workflow_dispatch branch October 23, 2023 13:42
@FedeDP FedeDP deleted the cleanup/drop_workflow_dispatch branch October 23, 2023 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maxgio92 maxgio92 maxgio92 approved these changes

Assignees

@maxgio92 maxgio92

Labels
approved area/ci dco-signoff: yes kind/cleanup lgtm size/XS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@FedeDP @maxgio92 @poiana