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

RHELAI-429: Adding upgrade informer service #689

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

javipolo
Copy link
Collaborator

@javipolo javipolo commented Jul 23, 2024

This is a duplication of #663, since @tsorya is on PTO and we really need to push those changes

/cc @rhatdan
/cc @jeremyeder

Upgrade informer will run every couple of our and will be triggered by systemd timer.

In order to start it on boot and run once i enabled it and timer. Disabling auto upgrade service in order to remove unexpected reboots. Service will run "bootc upgrade --check" and in case new version exists it will create motd file with upgrade info.

@javipolo javipolo force-pushed the upgrade-informer-service branch 3 times, most recently from a7d6fe2 to 720b9ba Compare July 23, 2024 13:20
@rhatdan
Copy link
Member

rhatdan commented Jul 23, 2024

This should run only once per day. If you run it more often then that you will see complaints about flooding the logs.

We are not going to be updating that often that it requires more then once/day to run.

Upgrade informer will run every couple of our and will be triggered by
systemd timer.

In order to start it on boot and run once i enabled it and timer.
Disabling auto upgrade service in order to remove unexpected reboots.

Service will run "bootc upgrade --check" and in case new version exists
it will create motd file with upgrade info.

Signed-off-by: Igal Tsoiref <itsoiref@redhat.com>
Signed-off-by: Javi Polo <jpolo@redhat.com>
@javipolo javipolo force-pushed the upgrade-informer-service branch from 720b9ba to 5362188 Compare July 23, 2024 18:18
@javipolo
Copy link
Collaborator Author

This should run only once per day

Agree, changed the timer to 1day

@romfreiman
Copy link
Contributor

/lgtm

@rhatdan rhatdan merged commit 55610a8 into containers:main Jul 23, 2024
1 check passed
@javipolo javipolo deleted the upgrade-informer-service branch July 23, 2024 19:12
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