Skip to content

Conversation

msugakov
Copy link
Contributor

Description

Part of

Validation

Only ran the config validator. It complains about the unknown rpm-lockfile. Otherwise, it's ok.
In general and as always, I don't know how it will work. Let's apply and see.

As part of https://issues.redhat.com/browse/ROX-30592

Note that I switched syntax from "Later" to "Cron" because that's
what Renovate docs suggest.
See https://docs.renovatebot.com/configuration-options/#schedule

> Renovate supports the standard Cron syntax, as well as deprecated
> support for a subset of Later syntax. We recommend you always use
> Cron syntax, due to its superior testing and robustness.

I checked the new expressions with https://crontab.guru
@msugakov msugakov requested a review from a team as a code owner September 16, 2025 08:55
Our Cron expression for Konflux task updates plus the default limit
of 2 may not allow creating all necessary PRs over the weekend.
Therefore, I bump the value.
Value `4` was unscientifically chosen.

Found out about this setting from docs
https://konflux-ci.dev/docs/mintmaker/user/#how-to-limit-the-number-of-prsmrs
https://docs.renovatebot.com/configuration-options/#prhourlylimit
Copy link

openshift-ci bot commented Sep 16, 2025

@msugakov: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-tests 4f7b284 link false /test e2e-tests

Full PR test history. Your PR dashboard.

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-sigs/prow repository. I understand the commands that are listed here.

@msugakov
Copy link
Contributor Author

Split just the rename into #2201

@tommartensen
Copy link
Contributor

@msugakov okay to close this after #2201 ?

@msugakov
Copy link
Contributor Author

No, I'll take it to draft to do https://issues.redhat.com/browse/ROX-30592

@msugakov msugakov marked this pull request as draft September 17, 2025 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants