Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(RHEL-40924) man: remove mentions of ssh-generator #24

Closed
wants to merge 1 commit into from

Conversation

jamacku
Copy link
Member

@jamacku jamacku commented Jul 23, 2024

rhel-only: doc

Related: RHEL-40924

rhel-only: doc

Related: RHEL-40924
@jamacku jamacku added this to the RHEL-10.0.BETA milestone Jul 23, 2024
@jamacku jamacku requested a review from msekletar July 23, 2024 07:17
@github-actions github-actions bot changed the title man: remove mentions of ssh-generator (RHEL-40924) man: remove mentions of ssh-generator Jul 23, 2024
Copy link

github-actions bot commented Jul 23, 2024

Commit validation

Tracker - RHEL-40924

The following commits meet all requirements

commit upstream
5121eae - man: remove mentions of ssh-generator rhel-only: doc

Tracker validation

Success

🟢 Tracker RHEL-40924 has set desired product: CentOS Stream 10
🟢 Tracker RHEL-40924 has set desired component: systemd
🟢 Tracker RHEL-40924 has been approved


Pull Request validation

Failed

🔴 Review - Missing review from a member

Success

🟢 CI - All checks have passed

@dtardon
Copy link
Member

dtardon commented Jul 23, 2024

I'm not against this, but I wonder whether it's necessary. There are other references for stuff we don't ship in the man pages, so why single out systemd-ssh-generator? (E.g., timedatectl(1) describes several verbs that are only usable if systemd-timesyncd and systemd-networkd are used.)

Or is the reason "we don't want people to know about this, as otherwise someone might ask that we include the generator"?

@jamacku jamacku closed this Jul 23, 2024
@jamacku jamacku deleted the remove-ssh branch July 23, 2024 12:36
@donnieelmore
Copy link

Why was this closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants