Skip to content

Conversation

@insider89
Copy link
Contributor

@insider89 insider89 commented Feb 10, 2026

Summary

  • Renames the validator node address output label to nodeAddress across all output types (screen, file, kubernetes)
  • Validator ConfigMap artifact name changes from *-address to *-node-address and key from "address" to "nodeAddress"
  • Faucet account output remains unchanged — still uses address label and *-address ConfigMap name

Test plan

  • bun test — all 19 tests in bootstrap.output.test.ts pass
  • bun run typecheck — no type errors
  • bun run check — lint and format clean

Summary by cubic

Renamed the validator node "address" label to "nodeAddress" across CLI, file, and Kubernetes outputs to avoid confusion with faucet account addresses.

  • Migration
    • Update Kubernetes consumers:
      • ConfigMap name: -address -> -node-address (e.g., besu-node-validator-0-node-address)
      • ConfigMap key: address -> nodeAddress
    • If parsing CLI or file outputs, read nodeAddress instead of address for validator nodes.

Written for commit c4dd169. Summary will update on new commits.

Rename the validator node "address" label to "nodeAddress" across screen,
file, and kubernetes outputs to distinguish it from faucet account addresses.
Faucet output remains unchanged.
@gemini-code-assist
Copy link
Contributor

Summary of Changes

Hello @insider89, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request refines the naming conventions for validator node addresses within the CLI's bootstrap output. The primary goal is to introduce greater clarity by distinguishing validator node addresses with a "nodeAddress" label and corresponding ConfigMap naming, while maintaining the existing "address" label for faucet accounts. This change improves consistency and reduces potential ambiguity in identifying different types of addresses.

Highlights

  • Validator Address Output Renaming: The output label for validator node addresses has been renamed from "address" to "nodeAddress" across all output types (screen, file, Kubernetes).
  • ConfigMap Naming Convention Update: Validator ConfigMap artifact names have been updated from "-address" to "-node-address", and the corresponding key within these ConfigMaps has changed from "address" to "nodeAddress".
  • Faucet Account Unchanged: The faucet account output continues to use the "address" label and "*-address" ConfigMap naming convention, ensuring no breaking changes for faucet-related outputs.
Changelog
  • src/cli/commands/bootstrap/bootstrap.output.test.ts
    • Updated test assertions to reflect the new "node-address" naming for validator ConfigMaps.
    • Modified expected error messages to include the updated ConfigMap name.
  • src/cli/commands/bootstrap/bootstrap.output.ts
    • Changed the display label for validator addresses from "address" to "nodeAddress" in console output.
    • Updated the ConfigMap name and key for validator addresses to use "nodeAddress".
Activity
  • No human activity has been recorded on this pull request yet.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@github-actions github-actions bot added the qa:running QA workflow is currently running label Feb 10, 2026
@github-actions
Copy link

github-actions bot commented Feb 10, 2026

To view in Slack, search for: 1770735481.036809

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request effectively renames the validator node address output to nodeAddress across screen, file, and Kubernetes outputs. The changes are implemented correctly and consistently, and the test files have been updated accordingly to reflect these changes. The distinction made between validator node addresses and faucet addresses improves clarity. The code looks good, and I have no further suggestions.

@github-actions github-actions bot added qa:success QA workflow passed successfully feat New feature status:ready-for-review Pull request is ready for review and removed qa:running QA workflow is currently running labels Feb 10, 2026
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@insider89 insider89 merged commit b4948c5 into main Feb 10, 2026
13 checks passed
@insider89 insider89 deleted the feat/rename-validator-address-to-node-address branch February 10, 2026 15:01
@github-actions github-actions bot added status:merged Pull request has been merged and removed status:ready-for-review Pull request is ready for review labels Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat New feature qa:success QA workflow passed successfully status:merged Pull request has been merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant