Skip to content

Conversation

@AndrewCharlesHay
Copy link
Contributor

Update test label to Go 1.25 in metrics and ensure workflow uses correct Go version. References #5869.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. metrics Issues or PRs related to metrics labels Oct 3, 2025
@k8s-ci-robot k8s-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 3, 2025
@mloiseleur
Copy link
Collaborator

Thanks @AndrewCharlesHay . 🤔 Cannot we have a compile time ? (and so avoid stupid error like this)

@AndrewCharlesHay AndrewCharlesHay force-pushed the fix/go-1-25-label-update branch from 21b39f4 to 533262c Compare October 3, 2025 14:44
@coveralls
Copy link

coveralls commented Oct 3, 2025

Pull Request Test Coverage Report for Build 18225753053

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 78.553%

Totals Coverage Status
Change from base Build 18168258555: 0.0%
Covered Lines: 15801
Relevant Lines: 20115

💛 - Coveralls

@AndrewCharlesHay
Copy link
Contributor Author

I'm curious how import statements get factored into test code coverage

@AndrewCharlesHay
Copy link
Contributor Author

Does coveralls not run on every commit?

@mloiseleur
Copy link
Collaborator

Does coveralls not run on every commit?
It updates the comment with the info on the last build made with Go / Test

@mloiseleur
Copy link
Collaborator

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 3, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mloiseleur

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mloiseleur mloiseleur changed the title Update goversion label to 1.25 in metrics test test: update goversion label to 1.25 in metrics test Oct 3, 2025
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 3, 2025
@k8s-ci-robot k8s-ci-robot merged commit caa4fc7 into kubernetes-sigs:master Oct 3, 2025
17 checks passed
@ivankatliarchuk
Copy link
Member

This line with version was not part of testing, this is why there is no change.

ivankatliarchuk added a commit to gofogo/k8s-sigs-external-dns-fork that referenced this pull request Oct 8, 2025
* master: (175 commits)
  chore(deps): bump renovatebot/github-action (kubernetes-sigs#5890)
  chore(cloudflare): migrate DeleteCustomHostname() to new lib (kubernetes-sigs#5880)
  docs(advanced): configuration precedence (kubernetes-sigs#5871)
  test: update goversion label to 1.25 in metrics test (kubernetes-sigs#5886)
  ci(linter): add go-critic (kubernetes-sigs#5875)
  docs(providers): add info about Myra protection option and docker image (kubernetes-sigs#5879)
  refactor(pihole): reduce cyclomatic complexity of TestProviderV6 (kubernetes-sigs#5876)
  test(source/service): add serviceTypeFilter edge case (kubernetes-sigs#5872)
  chore(ci): speed-up & coveralls (kubernetes-sigs#5870)
  feat(provider/cloudflare): add support for tags (kubernetes-sigs#5862)
  chore(deps): bump renovatebot/github-action (kubernetes-sigs#5874)
  feat: add new flags to allow migration of OwnerID (kubernetes-sigs#4823)
  docs(volcengine): add volcengine provider to readme (kubernetes-sigs#5866)
  chore(deps): bump renovatebot/github-action (kubernetes-sigs#5856)
  docs improve txt registry documentation formatting and examples for apex record (kubernetes-sigs#5863)
  chore: upgrade ExternalDNS to go v1.25 and golangci-lint v2.5 (kubernetes-sigs#5869)
  refactor(pihole): reduce cyclomatic complexity of TestProvider (kubernetes-sigs#5865)
  refactor(service): reduce cyclomatic complexity of extractHeadlessEndpoints (kubernetes-sigs#5822)
  test(cloudflare): clear environment variables before setting test values (kubernetes-sigs#5851)
  fix(endpoint/source) Allow '.' in TXT Records (kubernetes-sigs#5844)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. metrics Issues or PRs related to metrics size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants