WIP: email_alias new type #378

Closed
smwltr wants to merge 1 commit from smwltr/cdist:email_alias into master
First-time contributor

needs a man page

anything else?

needs a man page anything else?
smwltr added 1 commit 2024-05-07 20:30:04 +00:00
Collaborator
There already is a `__mail_alias` type in cdist-contrib: https://code.ungleich.ch/ungleich-public/cdist-contrib/src/branch/master/type/__mail_alias
fnux closed this pull request 2024-05-13 05:56:13 +00:00
Author
First-time contributor

There already is a __mail_alias type in cdist-contrib: https://code.ungleich.ch/ungleich-public/cdist-contrib/src/branch/master/type/__mail_alias

Ah yes. I remember now wondering why there is so much code in that and thinking that I'd try making something simple using the __key_value type, which seems like it's better for code maintance.

IMO this functionality is needed in a server orchestration tool. For example, confiuring the email alias for root on many servers/distros is needed.

If I add support for alias removal, which hopefully make it on par with the one in contrib, and I'll make anther PR with a comment explicitly about this.

> There already is a `__mail_alias` type in cdist-contrib: https://code.ungleich.ch/ungleich-public/cdist-contrib/src/branch/master/type/__mail_alias Ah yes. I remember now wondering why there is so much code in that and thinking that I'd try making something simple using the `__key_value` type, which seems like it's better for code maintance. IMO this functionality is needed in a server orchestration tool. For example, confiuring the email alias for `root` on many servers/distros is needed. **If** I add support for alias removal, which hopefully make it on par with the one in contrib, and I'll make anther PR with a comment explicitly about this.

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: ungleich-public/cdist#378
No description provided.