Skip to content

docs: add troubleshooting steps for Ubuntu minion Salt version mismatch - #4733

Open
YatesMold wants to merge 1 commit into
uyuni-project:masterfrom
YatesMold:fix-ubuntu-salt-version-mismatch
Open

docs: add troubleshooting steps for Ubuntu minion Salt version mismatch#4733
YatesMold wants to merge 1 commit into
uyuni-project:masterfrom
YatesMold:fix-ubuntu-salt-version-mismatch

Conversation

@YatesMold

Copy link
Copy Markdown

Hello! 👋

While setting up my development environment for the GSoC 2026 project "AI-Powered Intelligent Monitoring and Root Cause Analysis for Uyuni" (#251), I encountered a critical edge case when bootstrapping an Ubuntu 22.04 minion.

The bootstrap script defaults to downloading Salt 3007+, which breaks the cryptographic handshake with a 3006 Uyuni Server (The Salt Master server's public key did not authenticate). Attempting to force a downgrade directly via the bootstrap script fails due to apt downgrade protections and missing dependencies (dctrl-tools, net-tools) on minimal Ubuntu instances.

As suggested by @srbarrios in the GSoC issue regarding initial contributions during the environment setup phase, I have documented the complete "Nuke & Pave" resolution path. By using {productname} tags, I ensured the docs remain compatible across builds. This ensures future contributors or users facing this version parity issue can resolve it quickly without getting stuck on ZMQ logs or package manager blockers.

Let me know if any format changes or structural adjustments are needed for the AsciiDoc!

@agraul agraul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for your pull request and interest in improving our documentation! I think these changes go a bit in the wrong direction. They describe how to use https://github.com/saltstack/salt-bootstrap, but that's not what we mean with "bootstrap script". In Uyuni, a bootstrap script is generated on the Uyuni server. Clients can download it from the Uyuni Server, or the server can push it to the clients over SSH.

Your experience raises a good point, the term "bootstrap script" is somewhat overloaded and can easily be confused with Salt's upstream bootstrap script. Maybe it's worth point that out in our documentation.

Comment on lines +363 to +364
curl -fsSL https://github.com/saltstack/salt-bootstrap/releases/latest/download/bootstrap-salt.sh -o bootstrap-salt.sh
sh bootstrap-salt.sh -A <uyuni_server_fqdn> onedir 3006

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

salt-bootstrap is unsupported. All minions must run our Salt Bundle package from the Client Tools repository for the given distribution (venv-salt-minion-xyz.deb). This script installs an incompatible Salt version, Uyuni expects patches from openSUSE/salt to be present on all minions.

@YatesMold

Copy link
Copy Markdown
Author

Hi @agraul, thank you so much for the detailed explanation and the review.

You are right. I fell exactly into the trap of confusing the upstream SaltStack bootstrap script with the Uyuni-generated one. This completely explains the version mismatch and the missing patches I was experiencing during my local setup.
I will pivot this PR right now to add a clear [WARNING] block in the documentation. I will explicitly clarify the terminology overload and state that the upstream salt-bootstrap is unsupported and the venv-salt-minion bundle from the Client Tools must be used instead.

I'll push the updated commit shortly.

@YatesMold
YatesMold force-pushed the fix-ubuntu-salt-version-mismatch branch from 36c7763 to 6ead94a Compare February 25, 2026 11:01
@YatesMold
YatesMold requested a review from agraul February 25, 2026 17:11
@0rnela

0rnela commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

Status: Outstanding feedback needs addressing.

@YatesMold
YatesMold force-pushed the fix-ubuntu-salt-version-mismatch branch from 6ead94a to 46c26b5 Compare April 29, 2026 17:26
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