-
Notifications
You must be signed in to change notification settings - Fork 24
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
Run konflux builds and tests on all PRs #1841
Conversation
This should help us prevent breaking this build on master.
0c62805
to
20f3874
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The comment/s is/are not blocking.
@@ -20,10 +20,85 @@ concurrency: | |||
cancel-in-progress: true | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[two things, not about this line]
One:
name: Run konflux workflows
may suggest that GitHub drives/triggers Konflux which is not the case. I'd propose to change the name along the lines of "Run post-Konflux workflows" or "Test Konflux builds" or something.
Two:
This has cron schedule, I wonder what's the use?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One:
name: Run konflux workflows
I agree, I'll rename the workflow.
Two:
This has cron schedule, I wonder what's the use?
I believe this was added by @Stringy in order to run the konflux tests on nightlies, but I don't think it has much point if we don't trigger the konflux build as well. I'll ask him about it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Description
This should help us prevent breaking this build on master.
Checklist
Automated testing
If any of these don't apply, please comment below.
Testing Performed