Skip to content

Comments

Refs #38173 - correct variable name to ntp_server#10873

Merged
ofedoren merged 1 commit intotheforeman:3.18-stablefrom
evgeni:ntp318
Feb 19, 2026
Merged

Refs #38173 - correct variable name to ntp_server#10873
ofedoren merged 1 commit intotheforeman:3.18-stablefrom
evgeni:ntp318

Conversation

@evgeni
Copy link
Member

@evgeni evgeni commented Feb 19, 2026

Fixes: c9d6db8
(cherry picked from commit 7a2fc6c)

@pr-processor pr-processor bot added Not yet reviewed Stable branch PRs that are opened against a stable branch. Usually a cherry pick labels Feb 19, 2026
Copy link
Member

@ofedoren ofedoren left a comment

Choose a reason for hiding this comment

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

A bit weird to see Refs here, but 🤷

@ofedoren ofedoren merged commit c47ff00 into theforeman:3.18-stable Feb 19, 2026
20 of 21 checks passed
@evgeni
Copy link
Member Author

evgeni commented Feb 19, 2026

A bit weird to see Refs here, but 🤷

Why? That's the Redmine that introduced the bug and it was not in a stable release yet.

@jcpunk
Copy link
Contributor

jcpunk commented Feb 19, 2026

Separately, it is a bit weird to see this wasn't caught by the unit tests. Should those be updated to ensure the problem doesn't return?

@evgeni
Copy link
Member Author

evgeni commented Feb 19, 2026

Yeah, we don't test that specific branch (rhel6 or use-ntp), so it has fallen through.

@shubhamsg199 caught it in Satellite tests tho :)

@evgeni evgeni deleted the ntp318 branch February 19, 2026 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stable branch PRs that are opened against a stable branch. Usually a cherry pick Templates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants