Skip to content

Conversation

@tuminoid
Copy link
Contributor

Last year, support for FQDN hostnames instead of only IP addresses were added in #866. In that PR, no documentation changes were made. Variable names are still for compatibility reasons ending with _ip, which might be confusing. This commit adds relevant documentation what the fields accept in the config file inline help.

@tuminoid
Copy link
Contributor Author

/cc @ansasaki

@sarroutbi sarroutbi mentioned this pull request Nov 26, 2025
36 tasks
@sarroutbi
Copy link
Contributor

/packit retest-failed

@tuminoid
Copy link
Contributor Author

tuminoid commented Dec 8, 2025

Can I get a review for this docs change?

Filed #1161 for the bot crash, in case that somehow makes this not be visible on someone radar...

@sergio-correia
Copy link
Contributor

Would you mind rebasing the commit on top of the current tree?

Last year, support for FQDN hostnames instead of only IP addresses
were added. In that PR, no documentation changes were made. Variable
names are still for compatibility reasons ending with _ip, which might
be confusing. This commit adds relevant documentation what the fields
accept in the config file inline help.

Signed-off-by: Tuomo Tanskanen <[email protected]>
@tuminoid tuminoid force-pushed the tuomo/add-fqdn-docs branch from 2264d7e to 8c7a198 Compare December 9, 2025 10:57
@tuminoid
Copy link
Contributor Author

tuminoid commented Dec 9, 2025

Would you mind rebasing the commit on top of the current tree?

Done.

@codecov
Copy link

codecov bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.24%. Comparing base (d89668d) to head (8c7a198).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
Flag Coverage Δ
e2e-testsuite 58.24% <ø> (-0.03%) ⬇️
upstream-unit-tests 58.24% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sergio-correia sergio-correia merged commit 4ef7acb into keylime:master Dec 9, 2025
15 checks passed
@tuminoid tuminoid deleted the tuomo/add-fqdn-docs branch December 9, 2025 12:37
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