Skip to content

docs: add a note about satellite#385

Merged
bcl merged 1 commit intoosbuild:mainfrom
lzap:sat-note1
Dec 2, 2025
Merged

docs: add a note about satellite#385
bcl merged 1 commit intoosbuild:mainfrom
lzap:sat-note1

Conversation

@lzap
Copy link
Contributor

@lzap lzap commented Nov 26, 2025

Building from Satellite repositories is possible via force-repo.

@lzap lzap requested a review from a team as a code owner November 26, 2025 06:17
@lzap lzap requested review from croissanne, mvo5 and supakeen and removed request for a team November 26, 2025 06:17
supakeen
supakeen previously approved these changes Nov 26, 2025
To completely replace the default base repositories during a build the
option `--force-repo=file:///path/to/repos` can be used.

When replacing repositories with Katello/Satellite kickstart trees, make sure
Copy link
Contributor

Choose a reason for hiding this comment

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

How does this usually work with Katello/Stellite repos, how are the repos made available to dnf? I'm asking because when we construct the buildroot we just use the dnf from the host, are there any special vars we could pass to dnf to make these entitlements work?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Kickstart repositories are not subject of HTTP client authentication, Satellite users can just copy and paste them from the UI or CLI. There is no need to get anything from the host.

Copy link
Contributor

Choose a reason for hiding this comment

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

I would love if @thozza could also have a quick look at this, I personally lack experience in this (satelite/katello) area. But happy to +1 it as its certainly a very useful hint, I just wonder if it needs more fleshing out of if people familiar with the topic will get enough info from it (I guess they will?)

Copy link
Member

Choose a reason for hiding this comment

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

I thought about this a bit, and I think that we should instead by default suggest using the --data-dir instead. The main reason is that --force-repo replaces the base repositories for all distributions, while with --data-dir, you could pass different base repos per distro (minor) version.

So while I find documenting a workflow to use Satellite helpful, putting it under --force-repo section is IMHO not the best choice. It would probably warrant for a separate section just about Satellite.

Otherwise, the text looks OK.

mvo5
mvo5 previously approved these changes Nov 26, 2025
Copy link
Contributor

@mvo5 mvo5 left a comment

Choose a reason for hiding this comment

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

Approving but my approval is more a +0.5

@supakeen
Copy link
Member

The spellcheck will need to be fixed anyhow before this should merge :)

@lzap lzap dismissed stale reviews from mvo5 and supakeen via 29d0252 November 27, 2025 10:13
@lzap
Copy link
Contributor Author

lzap commented Nov 27, 2025

Rebased, added a note about GPG.

Copy link
Contributor

@bcl bcl left a comment

Choose a reason for hiding this comment

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

Looks good

@bcl bcl added this pull request to the merge queue Dec 2, 2025
Merged via the queue into osbuild:main with commit 04417da Dec 2, 2025
38 checks passed
@lzap lzap deleted the sat-note1 branch December 2, 2025 16:03
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.

5 participants