-
Notifications
You must be signed in to change notification settings - Fork 242
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
Configure Renovate #1011
Configure Renovate #1011
Conversation
6074fce
to
c19bdde
Compare
c19bdde
to
572527e
Compare
572527e
to
1df6ba0
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.
Looks good overall. One question around one of the regex patterns (the other one looks good to me!)
renovate.json5
Outdated
// similar to the examples shown here: | ||
// https://docs.renovatebot.com/modules/manager/regex/#advanced-capture | ||
"matchStrings": [ | ||
"# renovate: datasource=(?<datasource>.*?) depName=(?<depName>.*?)\\s+.+version: (?<currentValue>.*)" |
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.
@chancez Have you tested this by running Renovate locally? The currentValue
capture here seems to include the '
quotes and I'm not sure if Renovate will properly parse them:
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.
I haven't, I haven't had much success getting it running locally. Getting all the deps locally has been a struggle, and the docker image is x86 only and too slow if emulated. I copied 99.9% of the setup from cilium though, so if this is wrong, it's broken in cilium too. I'll try taking another stab at running it locally.
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.
Ah we don't use quotes in cilium's CI for go-version
. That's the difference.
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.
Okay, new commit will update the regex. Here's what I've got instead, which should handle quotes and no-quotes:
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.
Added a couple of cases at https://regex101.com/r/oErr7U/3
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.
Oh man the escaping for this is painful, I miss YAML (words I usually never say).
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.
Thanks @chancez! Some questions / comments
Alright I've figured out running renovate. Currently practicing on my fork https://github.com/chancez/hubble. You may notice unexpected PRs as I'm going to revert some recent ones in my fork so I can test how it handles updates for things that are already up to date. |
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.
Thanks @chancez, LGTM now!
@@ -145,6 +145,16 @@ After a new stable `v$MAJOR.$MINOR` release branch has been created, update the | |||
`target-branch` and `commit-message.prefix` fields in `.github/dependabot.yml` | |||
to point to the newly created branch, instead of the old stable branch. | |||
|
|||
## Update the `renovate` configuration | |||
|
|||
After a new stable `v$MAJOR.$MINOR` release branch has been created, update `renovate.json5`: |
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.
Any reason we're keeping renovate.json5 at the root of the project instead of .github/renovate.json5
like we have at cilium/cilium
?
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 original PR put it at renovate.json
at the root, but your right, I can move it into .github
I think.
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.
Thanks!
This comment was marked as outdated.
This comment was marked as outdated.
Okay I've pushed another commit with the latest changes I'm thinking about after iterating in my fork.
Take a look at my fork to see what it would look like with some outdated dependencies. I reverted and changed the versions in my master branch to make it easier to test, not all these PRs will need to be opened once this PR is merged. https://github.com/chancez/hubble/pulls |
.github/renovate.json5
Outdated
"groupName": "golang-stable", | ||
// Restrict the alpine version that can be used on stable branches | ||
"matchPackageNames": [ | ||
"docker.io/library/alpine" | ||
], |
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.
group named golang-stable
for the alpine image? 🤔
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.
Yes, it's part of the Golang image updates. I suppose this made more sense when the group name was Golang-images
but I changed it to Golang and now it's less apt.
This is the same as cilium, it's just less obvious with the inheritance, hence explicitly adding the groupName. Now I question if this should be part of that group or if I should rename the group.
The group name mostly changes the branch/PR name and is how the PRs get grouped together. Do we want a separate PR for alpine updates? That would seem reasonable to me.
Thanks for the update @chancez, LGTM beside my comment about golang/alpine.
nitpicking: instead of removing, we could comment them (json5 ftw) so that we're less likely to forget to get them back at some point? |
66c54f0
to
67aca4e
Compare
Signed-off-by: Chance Zibolski <chance.zibolski@gmail.com>
67aca4e
to
d5e1acb
Compare
@kaworu if you want to make one final pass I can get this merged today/tomorrow and we can work on merging renovate PRs/disabling dependabot if all goes well. |
Welcome to Renovate! This is an onboarding PR to help you understand and configure settings before regular Pull Requests begin.
🚦 To activate Renovate, merge this Pull Request. To disable Renovate, simply close this Pull Request unmerged.
Detected Package Files
Dockerfile
(dockerfile).github/workflows/build-images-release.yaml
(github-actions).github/workflows/codeql-analysis.yml
(github-actions).github/workflows/integration-tests.yaml
(github-actions).github/workflows/release.yml
(github-actions).github/workflows/tests.yml
(github-actions)go.mod
(gomod)Configuration
🔡 Renovate has detected a custom config for this PR. Feel free to ask for help if you have any doubts and would like it reviewed.
Important: Now that this branch is edited, Renovate can't rebase it from the base branch any more. If you make changes to the base branch that could impact this onboarding PR, please merge them manually.
You have configured Renovate to use the following baseBranches:
master
,v0.11
.What to Expect
With your current configuration, Renovate will create 4 Pull Requests:
fix(deps): pin dependencies (master)
renovate/master-all-go-deps-master
master
12126ed593697635c525b302836b292b657ea573
a97318bf6562f2ed2632c5f985db51b1bc5bdcd0
f8bf7650dccb756cea26edaf9217aab85500fe07
2b0eb0f724e320b655240e331aef36d1175986c2
4dd4b25de38418174a6e859e8a32eaccca32dccc
2e9d26c8c37aae03e3f9d4e90b7116f5accb7cab
518241257478c557633ab36e474dfcaeb9a3c623
f36bfe3c337aa95c86f04c721acdbafb5ffb1611
ca59edaa5a761e1d0ea91d6c07b063f85ef24f78
deeee5cf87c57e7f205795f0d88290f862855c47
f221882bfb484564f1714ae05f197dea2c76898d
f6f7691b1fdeb513f56608cd2c32c51f8194bf51
chore(deps): update all github action dependencies (master)
renovate/master-all-github-action
master
f095bcc56b7c2baf48f3ac70d6d6782f4f553222
4c11c9fe1dcd9636620a16455165783b20fc7ea0
e9ef242655d12993efdcda9058dee2db83a2cb9b
fac708d6674e30b6ba41289acaab6d4b75aa0753
65d862660abb392b8c4a3d1195a2108db131dd05
ac5ee1fca8fba06d82bfc39af726f19ffe037a49
eafaea8d0f5853934deece2ffa67af59d936562b
40891eba8c2bcd1309b07ba8b11232f313e86779
a946f06b5ddbde456d0115dc585e8dc3c7a0c3e0
95cfca769b334017179a1fbe879b6d9b5cddfe54
0d51521d4226fe362fc8aa8ce484ce3cd5299478
2d96ac1d715d2ea5abee55927b0cce7ec07ca148
44887b225ceca96efd8a912d39c09ad70312af31
fix(deps): update module gopkg.in/yaml.v2 to v3 (master)
renovate/master-major-3-all-go-deps-master
master
f6f7691b1fdeb513f56608cd2c32c51f8194bf51
chore(deps): update all github action dependencies (v0.11)
renovate/v0.11-all-github-action
v0.11
f095bcc56b7c2baf48f3ac70d6d6782f4f553222
4c11c9fe1dcd9636620a16455165783b20fc7ea0
e9ef242655d12993efdcda9058dee2db83a2cb9b
fac708d6674e30b6ba41289acaab6d4b75aa0753
65d862660abb392b8c4a3d1195a2108db131dd05
eafaea8d0f5853934deece2ffa67af59d936562b
40891eba8c2bcd1309b07ba8b11232f313e86779
a946f06b5ddbde456d0115dc585e8dc3c7a0c3e0
95cfca769b334017179a1fbe879b6d9b5cddfe54
0d51521d4226fe362fc8aa8ce484ce3cd5299478
44887b225ceca96efd8a912d39c09ad70312af31
🚸 Branch creation will be limited to maximum 2 per hour, so it doesn't swamp any CI resources or overwhelm the project. See docs for
prhourlylimit
for details.❓ Got questions? Check out Renovate's Docs, particularly the Getting Started section.
If you need any further assistance then you can also request help here.
This PR has been generated by Mend Renovate. View repository job log here.