search

ManageIQ / manageiq-providers-embedded_terraform

ManageIQ plugin for the Embedded Terraform provider.
Apache License 2.0
0 stars 10 forks source link

Fix some issues with opentofu-runner systemd service #52

Closed agrare closed 3 months ago

agrare commented 3 months ago

Fix issues with --cgroup=systemd missing systemd user session

The cgroupv2 manager is set to systemd but there is no systemd user session available
For using systemd, you may need to log in using a user session
Alternatively, you can enable lingering with: `loginctl enable-linger 985` (possibly as root)
Falling back to --cgroup-manager=cgroupfs

Also move our service file closer to what would be generated by the Quadlet systemd generator in https://github.com/ManageIQ/manageiq-providers-embedded_terraform/pull/50

We aren't able to use that entirely yet but this is inspired by blatantly copies the resulting transient service file.

I split out the fix for each issue into a separate commit for easier review

miq-bot commented 3 months ago

Checked commits https://github.com/agrare/manageiq-providers-embedded_terraform/compare/642c9c2efaa96dca37fe1678b9ab9fa31ddb0226~...daa6ad82980724e384065846a6e475063db5fd6b with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 1 file checked, 0 offenses detected Everything looks fine. :star:

Fryguy commented 3 months ago

Backported to radjabov in commit 3ffb2bdc0ae4ced0b4654c866ec618b2fa9e2421.

commit 3ffb2bdc0ae4ced0b4654c866ec618b2fa9e2421
Author: Jason Frey <fryguy9@gmail.com>
Date:   Mon Jun 3 14:01:59 2024 -0400

    Merge pull request #52 from agrare/improve_opentofu_worker_systemd_service

    Fix some issues with opentofu-runner systemd service

    (cherry picked from commit d07d56520b76ebf6daf6f1f6b39d1982e1092f3c)