Skip to content

Log workflow key when listing JD nodes#515

Merged
vyzaldysanchez merged 6 commits intomainfrom
task/jd-node-list/log-workflow-key
Oct 15, 2025
Merged

Log workflow key when listing JD nodes#515
vyzaldysanchez merged 6 commits intomainfrom
task/jd-node-list/log-workflow-key

Conversation

@vyzaldysanchez
Copy link
Contributor

Quality of life improvement: we need to know the workflow key of the nodes after they are connected to JD to then configure the capabilities registry.

This PR addresses that request by logging the WorkflowKey when list JD nodes.

@vyzaldysanchez vyzaldysanchez requested a review from a team as a code owner October 14, 2025 19:45
@changeset-bot
Copy link

changeset-bot bot commented Oct 14, 2025

🦋 Changeset detected

Latest commit: 571189b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
chainlink-deployments-framework Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link

👋 vyzaldysanchez, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

krehermann
krehermann previously approved these changes Oct 15, 2025
@graham-chainlink
Copy link
Collaborator

Hey @vyzaldysanchez , please create a changeset file following the instructions here as that helps with versioning and the changelog

@vyzaldysanchez vyzaldysanchez added this pull request to the merge queue Oct 15, 2025
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

Merged via the queue into main with commit e458ad5 Oct 15, 2025
14 checks passed
@vyzaldysanchez vyzaldysanchez deleted the task/jd-node-list/log-workflow-key branch October 15, 2025 04:43
github-merge-queue bot pushed a commit that referenced this pull request Oct 15, 2025
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## chainlink-deployments-framework@0.57.0

### Minor Changes

-
[#482](#482)
[`62ed5d0`](62ed5d0)
Thanks [@rodrigombsoares](https://github.com/rodrigombsoares)! -
Implement SUI proposal analyzer

-
[#510](#510)
[`8fd65fe`](8fd65fe)
Thanks [@graham-chainlink](https://github.com/graham-chainlink)! - feat:
enable strict yaml unmarshalling

When unmarshalling from yaml input for pipelines, if there is a field
not defined in the struct, an error will be returned. This helps catch
typos and misconfigurations early.

-
[#512](#512)
[`c035859`](c035859)
Thanks [@jkongie](https://github.com/jkongie)! - Adds a new option to
test engine environment loading for setting NodeIDs

    `WithNodeIDs` - option to set NodeIDs into the test environment

### Patch Changes

-
[#507](#507)
[`3ba8202`](3ba8202)
Thanks [@graham-chainlink](https://github.com/graham-chainlink)! -
fix(catalog/memory): convert to true inmemory implementation

    Remove dependency on pgtest

-
[#515](#515)
[`e458ad5`](e458ad5)
Thanks [@vyzaldysanchez](https://github.com/vyzaldysanchez)! - Add extra
info(workflow key and p2p key bundles) when logging JD nodes on a table
format.

---------

Co-authored-by: app-token-issuer-engops[bot] <144731339+app-token-issuer-engops[bot]@users.noreply.github.com>
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.

3 participants