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

golang/go not updated #189

Closed
frezbo opened this issue Aug 3, 2022 · 17 comments
Closed

golang/go not updated #189

frezbo opened this issue Aug 3, 2022 · 17 comments

Comments

@frezbo
Copy link
Sponsor Contributor

frezbo commented Aug 3, 2022

Go 1.19 was released on 2nd Aug, 2022, but it seems I didn't get a renovate PR for the same.

The renovate config clearly sees the deps here:

image
seems something got broken recently. Creating it here to track why some deps doesn't gets updated. Had a similar issue with goimports as referenced in #185

@frezbo
Copy link
Sponsor Contributor Author

frezbo commented Aug 3, 2022

Seems renovate has failed to update go in aqua-registry too: https://github.com/aquaproj/aqua-registry/tree/main/pkgs/golang/go

@suzuki-shunsuke
Copy link
Member

suzuki-shunsuke commented Aug 3, 2022

@suzuki-shunsuke
Copy link
Member

Thank you for your report.

@frezbo
Copy link
Sponsor Contributor Author

frezbo commented Aug 3, 2022

I'll try to see what's broken when I get some time, seems renovate is a bit flaky this week

@suzuki-shunsuke
Copy link
Member

Thank you. Recently I'm a little busy so it is difficult to investigate the issue about Renovate for now.

@suzuki-shunsuke
Copy link
Member

Related tweet (Japanese).
https://twitter.com/ponkio_o/status/1555048136825262081

@suzuki-shunsuke
Copy link
Member

Maybe this warning is about go datasource?
#177

{
"fileMatch": ["\\.?aqua\\.ya?ml"],
"matchStrings": [
" +['\"]?(version|ref)['\"]? *: +['\"]?(?<currentValue>[^'\" \\n]+?)['\"]? +# renovate: depName=(?<depName>[^\\n]+)",
" +['\"]?name['\"]? *: +['\"]?(?<depName>[^\\n]+)*@(?<currentValue>[^'\" \\n]+)['\"]?"
],
"datasourceTemplate": "go"
}

@suzuki-shunsuke
Copy link
Member

I guess we can ignore this warning although we would like to remove the warning.
aqua-renovate-config's default preset uses two datasources, github_release datasource and go datasource.

Renovate tries to get each package's version from these datasources, but Renovate can get the version from only one datasource.

For example, Renovate would succeed to get the version of suzuki-shunsuke/tfcmt from github_release datasource
but would fail to get it from go datasource.
This failure would be shown as the warning, but I guess we can ignore this warning.

Note that the above description is just my assumption.

@suzuki-shunsuke
Copy link
Member

Hmm...
Maybe we can distinguish the datasource by the regular expression of package name.

go datasource package name includes a period ..

pkgs/golang.org/x/tools/cmd/goimports/registry.yaml
2:  - type: go_install
3-    name: golang.org/x/tools/cmd/goimports
4-    path: golang.org/x/tools/cmd/goimports
5-    repo_owner: golang
6-    repo_name: tools
7-    version_source: github_tag

pkgs/golang.org/x/tools/gopls/registry.yaml
2:  - type: go_install
3-    name: golang.org/x/tools/gopls
4-    aliases:
5-      - name: golang/tools/gopls
6-    path: golang.org/x/tools/gopls
7-    repo_owner: golang

pkgs/golang.org/x/perf/cmd/benchstat/registry.yaml
2:  - type: go_install
3-    path: golang.org/x/perf/cmd/benchstat
4-    description: Benchstat computes and compares statistics about benchmarks
5-    link: https://pkg.go.dev/golang.org/x/perf/cmd/benchstat
6-

@suzuki-shunsuke
Copy link
Member

#192

@suzuki-shunsuke
Copy link
Member

@suzuki-shunsuke
Copy link
Member

suzuki-shunsuke commented Aug 4, 2022

golang/go was updated from 1.18.4 to 1.18.5, but not updated to 1.19.
suzuki-shunsuke/test-renovate#158

I guess this is because 1.19 is invalid as semver.
When go1.19.x will be released, Renovate would update golang/go.

{
"fileMatch": ["\\.?aqua\\.ya?ml"],
"matchStrings": [
" +['\"]?version['\"]? *: +['\"]?(go)?(?<currentValue>[^'\" \\n]+)['\"]? +# renovate: depName=golang/go[ \\n]",
" +['\"]?name['\"]? *: +['\"]?golang/go@(go)?(?<currentValue>[^'\" \\n]+)['\"]?"
],
"extractVersionTemplate": "^go(?<version>.*)$",
"datasourceTemplate": "github-tags",
"depNameTemplate": "golang/go"
},

I checked the Renovate log. The version was semver.

{
            "depName": "golang/go",
            "currentValue": "go1.18.4",
            "datasource": "github-releases",
            "replaceString": " name: golang/go@go1.18.4",
            "depIndex": 0,
            "updates": [],
            "warnings": [],
            "versioning": "semver",
            "skipReason": "invalid-value"
          }

@suzuki-shunsuke
Copy link
Member

Renovate supports various versioning, but I don't know which is best for golang/go.
https://docs.renovatebot.com/modules/versioning/

@frezbo
Copy link
Sponsor Contributor Author

frezbo commented Aug 4, 2022

golang/go was updated from 1.18.4 to 1.18.5, but not updated to 1.19.

yeh, i guess that's right, i manually updated for now.

@frezbo
Copy link
Sponsor Contributor Author

frezbo commented Aug 4, 2022

I'm closing this issue for now, since this is a minor inconvenience.

@frezbo frezbo closed this as completed Aug 4, 2022
@frezbo
Copy link
Sponsor Contributor Author

frezbo commented Aug 4, 2022

Thanks for chasing the issue and putting up a fix 👍

@rust84
Copy link

rust84 commented Aug 5, 2022

I tried semver-coerced with no luck and ended up just setting it to loose.

  "packageRules": [
    {
      "matchDatasources": ["github-tags"],
      "matchPackageNames": ["golang/go"],
      "versioning": "loose"
    }
  ]

frezbo added a commit to frezbo/aqua-renovate-config that referenced this issue Feb 2, 2023
Golang major releases follow `major.minor` only and patch releases follow
`major.minor.patch` format. This change allows renovate to update Golang
major/patch release versions.

This should also fix an old issue aquaproj#189.

Signed-off-by: Noel Georgi <git@frezbo.dev>
frezbo added a commit to frezbo/aqua-renovate-config that referenced this issue Feb 2, 2023
Golang major releases follow `major.minor` only and patch releases follow
`major.minor.patch` format. This change allows renovate to update Golang
major/patch release versions.

This should also fix an old issue aquaproj#189.

Signed-off-by: Noel Georgi <git@frezbo.dev>
frezbo added a commit to frezbo/aqua-renovate-config that referenced this issue Feb 2, 2023
Golang major releases follow `major.minor` only and patch releases follow
`major.minor.patch` format. This change allows renovate to update Golang
major/patch release versions.

This should also fix an old issue aquaproj#189.

Signed-off-by: Noel Georgi <git@frezbo.dev>
frezbo added a commit to frezbo/aqua-renovate-config that referenced this issue Feb 2, 2023
Golang major releases follow `major.minor` only and patch releases follow
`major.minor.patch` format. This change allows renovate to update Golang
major/patch release versions.

This should also fix an old issue aquaproj#189.

Signed-off-by: Noel Georgi <git@frezbo.dev>
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

No branches or pull requests

3 participants