Skip to content

feat(clustertool): Own info pkg#120

Merged
alfi0812 merged 1 commit intomainfrom
infocmd
May 2, 2026
Merged

feat(clustertool): Own info pkg#120
alfi0812 merged 1 commit intomainfrom
infocmd

Conversation

@Boemeltrein
Copy link
Copy Markdown
Member

Description

Own pkg as there was forgetool mentioned.

⚒️ Fixes #

⚙️ Type of change

  • 🆕 New feature (e.g., bootstrap, cluster config, or automation)
  • 🐛 Bugfix (e.g., node setup, config generation)
  • 🔃 Chart Updates/Changes included with clustertool
  • 🔃 Refactor (internal changes, no functional impact)
  • ⚠️ Breaking change (changes Talos cluster behavior)

🧪 How Has This Been Tested?

  • Run clustertool commands locally
  • Verified cluster bootstrap on Talos nodes
  • Checked generated configs are valid
  • Validated cluster state after changes

📃 Notes:

✔️ Checklist:

  • ⚖️ Followed project style and contribution guidelines
  • 👀 Self-reviewed my changes
  • 🧪 Documented testing steps
  • 🏷️ PR title follows convention:
    • feat(clustertool):
    • fix(clustertool):
    • chore(clustertool):

Please don't blindly check all the boxes. Read them and only check those that apply.
Those checkboxes are there for the reviewer to see what is this all about and
the status of this PR with a quick glance.

@alfi0812 alfi0812 merged commit e2e9a44 into main May 2, 2026
8 checks passed
@alfi0812 alfi0812 deleted the infocmd branch May 2, 2026 20:50
@Crow-Control
Copy link
Copy Markdown
Member

@Boemeltrein this was a mistake on forgetool side, it shouldn't have any message on the info pkg there.
(or at least not combined with general system info)

alfi0812 pushed a commit that referenced this pull request May 3, 2026
* Revert "info is back (#120)"

This reverts commit e2e9a44.

* Refactor help text variable for clarity

Signed-off-by: Kjeld Schouten <info@kjeldschouten.nl>

* Import zerolog for logging in info.go

Added zerolog for enhanced logging capabilities.

Signed-off-by: Kjeld Schouten <info@kjeldschouten.nl>

---------

Signed-off-by: Kjeld Schouten <info@kjeldschouten.nl>
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