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-32494) ci: deploy systemd man to GitHub Pages #427

Merged
merged 3 commits into from
Apr 11, 2024

Conversation

jamacku
Copy link
Member

@jamacku jamacku commented Feb 9, 2024

rhel-only

Related: RHEL-1087

@jamacku jamacku added this to the RHEL-8.10 milestone Feb 9, 2024
@github-actions github-actions bot changed the title ci: deploy systemd man to GitHub Pages (RHEL-1087) ci: deploy systemd man to GitHub Pages Feb 9, 2024
@github-actions github-actions bot added pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review labels Feb 9, 2024
Copy link

github-actions bot commented Feb 9, 2024

Commit validation

Tracker - Missing issue tracker ✋

The following commits meet all requirements

commit upstream
b2d267e - ci: deploy systemd man to GitHub Pages rhel-only

The following commits need an inspection

commit note
11ad297 - apply suggestions Missing issue tracker
Missing upstream reference ‼️
aeb5713 - change branch Missing issue tracker
Missing upstream reference ‼️

Tracker validation

🔴 Missing tracker or Unknown tracker type; type: 'unknown'


Pull Request validation

Failed

🔴 Failed or pending checks - deploy[failure],gather-pull-requests[queued],source-git-automation[in_progress],build (stream8, GCC_ASAN)[in_progress],build (stream8, GCC)[in_progress],Analyze (cpp)[in_progress] Failed or pending statuses - CentOS CI (CentOS Stream 8)[pending]

Success

🟢 Review - Reviewed by a member
🟢 Approval - Changes were approved

@github-actions github-actions bot removed the pr/needs-ci Formerly needs-ci label Feb 9, 2024
@dtardon
Copy link
Member

dtardon commented Feb 12, 2024

@jamacku I guess I'm missing the context here. I understand what the action does, but not why do we want it 🤔

@jamacku
Copy link
Member Author

jamacku commented Feb 12, 2024

@dtardon, it was requested by @lnykryn. So we would have an easy way to access the systemd.net-naming-scheme man page.

I have already done the same for rhel9, so I thought, why not add it also to rhel8.
https://redhat-plumbers.github.io/systemd-rhel9/

.github/workflows/deploy-man-pages.yml Outdated Show resolved Hide resolved
.github/workflows/deploy-man-pages.yml Outdated Show resolved Hide resolved
.github/workflows/deploy-man-pages.yml Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the pr/needs-review Formerly needs-review label Feb 12, 2024
@jamacku
Copy link
Member Author

jamacku commented Feb 12, 2024

@mrc0mmand Thanks for review. I needed to make it work fast for Lukaš to share it. I'll try to minimize the list of dependencies.

@dtardon
Copy link
Member

dtardon commented Feb 13, 2024

@dtardon, it was requested by @lnykryn. So we would have an easy way to access the systemd.net-naming-scheme man page.

Such way does already exist: ./build/man/man and ./build/man/html. These commands build and display a specified man page, e.g., ./build/man/man systemd.net-naming-scheme. They have the additional advantage that one can see the effect of local changes immediately.

@jamacku
Copy link
Member Author

jamacku commented Feb 13, 2024

true, but it was for sharing with others, as a reference for technical writers.

@jamacku jamacku changed the base branch from main to rhel-8.10.0 March 11, 2024 15:09
@github-actions github-actions bot added rhel-8.10.0 pr/needs-ci Formerly needs-ci pr/needs-manual-merge and removed pr/needs-ci Formerly needs-ci labels Mar 11, 2024
@jamacku jamacku force-pushed the pages-github branch 2 times, most recently from e3fce46 to 9dc11f3 Compare April 11, 2024 08:31
@github-actions github-actions bot added the pr/needs-ci Formerly needs-ci label Apr 11, 2024
@github-actions github-actions bot changed the title (RHEL-1087) ci: deploy systemd man to GitHub Pages (RHEL-32494) ci: deploy systemd man to GitHub Pages Apr 11, 2024
@github-actions github-actions bot added the tracker/missing Formerly needs-bz label Apr 11, 2024
jamacku and others added 3 commits April 11, 2024 10:44
rhel-only

Related: RHEL-32494
Co-authored-by: Frantisek Sumsal <frantisek@sumsal.cz>
@jamacku jamacku merged commit aeb5713 into redhat-plumbers:rhel-8.10.0 Apr 11, 2024
1 check was pending
@jamacku
Copy link
Member Author

jamacku commented Apr 11, 2024

Oops, for some reason, the origin was pointing to upstream 😞

@jamacku jamacku deleted the pages-github branch April 11, 2024 08:53
@jamacku jamacku added ci-waived released and removed pr/needs-ci Formerly needs-ci tracker/missing Formerly needs-bz dont-merge labels Apr 11, 2024
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.

4 participants