Skip to content

Comments

Fix etcdctl output redirect typo.#791

Merged
DmitriiRabenok merged 1 commit intomainfrom
fix/etcdctl-redirect-typo
Feb 18, 2026
Merged

Fix etcdctl output redirect typo.#791
DmitriiRabenok merged 1 commit intomainfrom
fix/etcdctl-redirect-typo

Conversation

@iagusev
Copy link
Collaborator

@iagusev iagusev commented Feb 3, 2026

Description

Fix incorrect stderr redirection in etcdctl wrapper script when pulling etcd image via containerd.
The previous redirection syntax suppressed logs incorrectly due to a typo.

Solution

Fixed shell redirection typo in etcdctl wrapper script:

  • Replaced invalid 2&>1 redirection with correct 2>&1
  • Ensured stderr is properly redirected together with stdout to /dev/null
  • No functional behavior change besides correct log suppression

Test Cases

Steps:

  1. Run etcdctl member list on control-plane node
  2. Ensure etcd image is pulled via wrapper script and command completed
  3. Verify no stderr output is leaked to terminal or to file 1 in current directory

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • There is no breaking changes, or migration patch is provided
  • Integration CI passed
  • No changes in unit tests required
  • There is no merge conflicts

@DmitriiRabenok DmitriiRabenok merged commit f3fb998 into main Feb 18, 2026
31 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants