Code Monkey home page Code Monkey logo

Comments (7)

oxc avatar oxc commented on August 10, 2024

I believe the readme is quite clear on how external configs work: https://github.com/oxc/puppet-dovecot/blob/master/README.md#external-config-files

I'm not totally sure which functionality you are missing, please elaborate.

from puppet-dovecot.

Yamakasi avatar Yamakasi commented on August 10, 2024

It's explained but I wondered if we should not go the dovecot way, even when it's puppetized. That's why, also refering to this quote:

In a puppet-based setup, this should not be necessary, and is thus currently not supported by this module.

I think it adds the flexibility which is taken out now ?

from puppet-dovecot.

oxc avatar oxc commented on August 10, 2024

This quote has nothing to do with dovecot-ldap.conf.ext, so perhaps you can explain your use-case.
Flexibility in this case comes with additional complexity, as I have explained in the original issue. If you have a use-case, that additional complexity might be warranted, but until then it's not.

from puppet-dovecot.

Yamakasi avatar Yamakasi commented on August 10, 2024

Sure it has everything todo with it otherwise you won't quote the situation. There is no specific use-case, there are standards we could adopt and indeed can add some complexity but is very doable to keep the config design/layout as native as possible.

from puppet-dovecot.

oxc avatar oxc commented on August 10, 2024

No, it does not. This text you quoted is about includes in conf.d, as the readme clearly states. Please read the text before you pick it apart, and please try the examples from the readme before you complain about missing features (as you have done before for other modules). Otherwise you're just trolling, if deliberately or not.

I'm closing this, because the extconfigs parameter is delegating to extconfigfile type, so this is already exactly what you want.

You're always welcome to create pull requests, of course.

from puppet-dovecot.

Yamakasi avatar Yamakasi commented on August 10, 2024

I must say it's a pity you always try to "point a finger" with a bad attitude when someone, like me in 2 cases now, mentions something to make your work more clear or try to go forward on standards, improvements, design.

Your way is a way to have the same type of config at the end but it's not like the standards of dovecot which you might want to keep in mind as it makes it indeed unclear and your comment, the quote in this "issue", creates an open end and you clearly try to "point a finger" again there which is rude and doesn't make people enthusiastic to create a PR at all.

from puppet-dovecot.

oxc avatar oxc commented on August 10, 2024

If you finally care to explain what you are trying to achieve, and why you think this is not working with this module, we could have a constructive discussion, but you refuse to provide that information.

Managing external configs files like dovecot-ldap.conf.ext is totally possible from both puppet and hiera, and the readme has an example that explains how it might look like.

The readme also explains why it's necessary to have that separation in the first place, and provides a link to the dovecot manual for further reference.

It explains why this differs from the includes in conf.d and why those (and only those) are currently not supported, and what is the required prerequisite to get them supported.

I'm really not sure what more you need, and so far you have not made any attempt to clarify that. Neither have you responded to any of my remarks in my replies.
Instead you are complaining about my tone, which - granted - is somewhat annoyed (and for good reason, see previous paragraphs), but really not the issue here.

from puppet-dovecot.

Related Issues (14)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.