marvel-nccr / ansible-role-aiida

An ansible role that installs and configures AiiDA on Ubuntu.
Other
2 stars 5 forks source link

fix systemd templates for profile name with dashes #49

Closed ltalirz closed 3 years ago

ltalirz commented 3 years ago

When a profile name included dashes (e.g. my-profile), the dashes were replaced by slashes in the systemd unit files for the AiiDA daemon / the REST API, resulting in an error upon start (profile "my/profile" not found).

This PR fixes the unit templates and also sets a profile name with a dash for the molecule CI tests.

ltalirz commented 3 years ago

@chrisjsewell tests pass; feel free to merge

chrisjsewell commented 3 years ago

thanks @ltalirz could you just explain the difference between %i and %I

ltalirz commented 3 years ago

sure, I've just updated the commit message see also https://www.freedesktop.org/software/systemd/man/systemd.unit.html#Specifiers

chrisjsewell commented 3 years ago

cheers