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

Update qm /etc/agent/agent.conf that when the qm.service is started #57

Merged
merged 1 commit into from
May 17, 2023

Conversation

rhatdan
Copy link
Member

@rhatdan rhatdan commented May 17, 2023

This pull request will cause the /usr/lib/qm/rootfs/etc/hirte/agent.conf to be created on every qm.service start. This way if the admin changes the /etc/hirte/agent.conf on the host, the equivalent inside of the qm will match with the exception that the nodename is prefixed with qm.

If the nodename in the hosts /etc/hirte/agent.conf is changed to foobar, then the qm's /etc/hirte/agent.conf will have the same settings except the node name will be qm.foobar.

@rhatdan rhatdan requested a review from lsm5 as a code owner May 17, 2023 13:21
@rhatdan
Copy link
Member Author

rhatdan commented May 17, 2023

@alexlarsson @pypingou @lsm5 @imcleod

With this change the qm agent.conf will always match the hosts agent.conf sans the nodename.

@dougsland
Copy link
Collaborator

LGTM

Copy link
Member

@lsm5 lsm5 left a comment

Choose a reason for hiding this comment

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

LGTM

@pypingou
Copy link
Member

Worth a try, we should document that behavior somewhere though as it means we won't be able to tweak the agent.conf in the QM side any longer (we could tweak the conf by only via agent.conf.d)

@rhatdan
Copy link
Member Author

rhatdan commented May 17, 2023

Yes or modify the qm.container.

This pull request will cause the /usr/lib/qm/rootfs/etc/hirte/agent.conf
to be created on every qm.service start.  This way if the admin changes
the /etc/hirte/agent.conf on the host, the equivalent inside of the qm
will match with the exception that the nodename is prefixed with qm.

If the nodename in the hosts /etc/hirte/agent.conf is changed to foobar,
then the qm's /etc/hirte/agent.conf will have the same settings except
the node name will be qm.foobar.

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
@rhatdan rhatdan merged commit 683e988 into containers:main May 17, 2023
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.

4 participants