Skip to content
This repository has been archived by the owner on Aug 1, 2022. It is now read-only.

PMM-7925: Add SystemD service with podman to run PMM #355

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

denisok
Copy link
Contributor

@denisok denisok commented Apr 11, 2022

https://jira.percona.com/browse/PMM-7925
percona/pmm-doc#759

ToDo:

  • add patter to modify tags during a build in spec files and service file
  • need to figure out something with user service tag, where to put it and how to update

@denisok denisok changed the title Add SystemD service with podman to run PMM PMM-7925: Add SystemD service with podman to run PMM Apr 11, 2022
@denisok denisok marked this pull request as ready for review May 31, 2022 10:16
DISABLE_UPDATES=1

# Enable DBaaS feature
#ENABLE_DBAAS=1
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we split environment variables between this file and service?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

service has environment for systemd/podman and env file is passed to pmm.

It is not split, it just another file to pass to env to pmm


* Tue May 31 2022 Denys Kondratenko <denys.kondratenko@percona.com> - 2.29.0-2
- PMM-7925 add systemd service with podman support

* Mon May 16 2022 Nikita Beletskii <nikita.beletskii@percona.com> - 2.29.0-1
- PMM-10027 remove useless packages

Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we use this spec? We use this spec for internal pmm-server package (it contains some js files) and I'm not sure that we need systemd service in container :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it was easier for PoC to use existing spec. And actually it fits it well in term that is a part of pmm-server. We don't need it in container for sure, but probably after build we need to sync it into some repo that available for everyone.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants