Skip to content

[PRODCRE-1077] Flag for local capabilities + Launcher metrics#19919

Merged
bolekk merged 1 commit intodevelopfrom
feature/PRODCRE-1077-local-only-flag
Oct 17, 2025
Merged

[PRODCRE-1077] Flag for local capabilities + Launcher metrics#19919
bolekk merged 1 commit intodevelopfrom
feature/PRODCRE-1077-local-only-flag

Conversation

@bolekk
Copy link
Copy Markdown
Contributor

@bolekk bolekk commented Oct 16, 2025

Explicitly mark capabilities as local and skip them in Launcher (depends on smartcontractkit/chainlink-common#1616)

Additionally export a few new metrics from Launcher.

Also had to bump go to 1.25 with the new common.

@github-actions
Copy link
Copy Markdown
Contributor

I see you updated files related to core. Please run pnpm changeset in the root directory to add a changeset as well as in the text include at least one of the following tags:

  • #added For any new functionality added.
  • #breaking_change For any functionality that requires manual action for the node to boot.
  • #bugfix For bug fixes.
  • #changed For any change to the existing functionality.
  • #db_update For any feature that introduces updates to database schema.
  • #deprecation_notice For any upcoming deprecation functionality.
  • #internal For changesets that need to be excluded from the final changelog.
  • #nops For any feature that is NOP facing and needs to be in the official Release Notes for the release.
  • #removed For any functionality/config that is removed.
  • #updated For any functionality that is updated.
  • #wip For any change that is not ready yet and external communication about it should be held off till it is feature complete.

Comment thread go.mod
module github.com/smartcontractkit/chainlink/v2

go 1.24.5
go 1.25.3
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is anything forcing a go bump? or you just think its about time?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, new common is set to 1.25

expectedError: "no such host",
},
{
/*{ // TODO: failing with go 1.25
Copy link
Copy Markdown
Contributor Author

@bolekk bolekk Oct 16, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cedric-cordenier / @jmank88 - any idea why these started to break with Go 1.25? In playground they don't work with either... https://go.dev/play/p/XYO2ekdS5bx?v=goprev

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

even weirder: tip does not error!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Likely this? golang/go#75713
Only in newer patch release of 1.24 and 1.25

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed yesterday: golang/go#75815

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wow good find thanks!

@cl-sonarqube-production
Copy link
Copy Markdown

@jmank88 jmank88 mentioned this pull request Oct 17, 2025
@bolekk bolekk closed this Oct 17, 2025
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

Successfully merging this pull request may close these issues.

4 participants