https://www.puppet.com/community logo
Join Slack
Powered by
# voxpupuli
  • y

    Yury Bushmelev

    07/14/2023, 7:27 PM
    at least according to the ERB template.. maybe we can drop some variants though..
  • y

    Yury Bushmelev

    07/14/2023, 7:27 PM
    I see no nested hashes in the unit tests but I see this in the original ERB
  • v

    VoxBot

    07/14/2023, 7:29 PM
    https://github.com/voxpupuli/beaker-hostgenerator/pull/313 I took a stab at further reducing the size of the file data.rb
  • y

    Yury Bushmelev

    07/14/2023, 7:58 PM
    just curious if anyone saw a config which uses this in
    *_cfg_{append,prepend}
    in nginx: https://github.com/voxpupuli/puppet-nginx/blob/master/templates/mailhost/mailhost.erb#L8
  • y

    Yury Bushmelev

    07/14/2023, 7:58 PM
    I mean exactly the case when it’s Hash[String, Hash[String, Array[String]]]
  • y

    Yury Bushmelev

    07/14/2023, 7:59 PM
    I cannot imagine in which case in nginx.conf this can be useful..
  • y

    Yury Bushmelev

    07/14/2023, 8:02 PM
    well.. found it can be used within upstream maybe:
    Copy code
    upstream big_server_com {
        server 127.0.0.3:8000 weight=5;
        server 127.0.0.3:8001 weight=5;
        server 192.168.0.1:8000;
        server 192.168.0.1:8001;
      }
    so
    { 'server' => { '127.0.0.3:8000' => 'weight=3' }}
    .. 😱
  • y

    Yury Bushmelev

    07/14/2023, 8:03 PM
    I just want it to be Array[String] and that’s it please..
  • y

    Yury Bushmelev

    07/14/2023, 10:20 PM
    well.. I guess this batch if ready finally: https://github.com/voxpupuli/puppet-nginx/pull/1568
    🎉 1
  • y

    Yury Bushmelev

    07/15/2023, 9:16 AM
    had a look over the puppet-nginx issues and found some complaints regarding the
    *_cfg_{append,prepend}
    parameters behaviour 😞
  • d

    Dr Bunsen Honeydew

    07/15/2023, 9:16 AM
    See the
    puppet-nginx
    module at https://forge.puppet.com/puppet/nginx?src=slack&channel=voxpupuli
  • y

    Yury Bushmelev

    07/15/2023, 10:53 AM
    ewoud: I saw you switched ‘{ ’ to ' {' when finalising an
    if
    line.. not sure what was the reason but now it produces double space before the
    {
    and no space after:
    <% if $default  {%>
  • v

    VoxBot

    07/16/2023, 8:52 PM
    VoxBot: hmm, interesting because when I tried to use it in a test it showed up in a weird way
  • y

    Yury Bushmelev

    07/17/2023, 8:18 AM
    ah.. I didn’t tried unit tests there yet.. maybe that’s why it’s unclear for me
  • e

    edwin

    07/17/2023, 10:38 AM
    Hallochen Vox! hattip ☕ Most of you know this already from our Vox sync last week, but here we go! This is just a reminder of the upcoming maintenance on the Forge this Wednesday 👇 https://puppetcommunity.slack.com/archives/C0W298S9G/p1689068073536019 Thanks!
  • v

    VoxBot

    07/17/2023, 4:34 PM
    Got a tiny PR for facter 3 compat on falco https://github.com/voxpupuli/puppet-falco/pull/31
  • v

    VoxBot

    07/17/2023, 5:07 PM
    https://github.com/voxpupuli/puppet-falco/pull/32
  • v

    VoxBot

    07/17/2023, 5:15 PM
    Hunner: as a ruby non-expert, why do those braces make a difference?
  • v

    VoxBot

    07/17/2023, 5:17 PM
    The original result from the command util resolution is passed directly to the
    confine
    function, and the curly braces turn that into a block (similar to
    do...end
    ) that can be executed differently
  • v

    VoxBot

    07/17/2023, 5:17 PM
    Puppet 7 and facter 4 seem to handle non-block confines, but puppet 6 and facter 3 stil needs blocks
  • v

    VoxBot

    07/17/2023, 5:19 PM
    ah I see, hmm, thanks
  • v

    VoxBot

    07/17/2023, 6:07 PM
    there's probably also a difference that a non-block is executed greedy (so when the file is loaded) while a block is loaded lazily (when block.call is invoked)
  • v

    VoxBot

    07/17/2023, 6:08 PM
    so I suspect that if you run facter os it will never call the block when you use confine { … } but confine … will run
  • v

    VoxBot

    07/17/2023, 6:13 PM
    ewoud: ohhh quite a good point
  • v

    VoxBot

    07/17/2023, 6:42 PM
    The non-block version would show issues when the command becomes available but a persistent puppet agent isn't restarted. Anyway, I wonder why facter 4 didn't also throw an error when no block is supplied? Seems like something that should be an issue and not an intended behavior
  • e

    emerson_prado

    07/17/2023, 8:27 PM
    Hi all! Do this result from yum module GitHub merge hook indicate the problem is indeed the outdated
    REFERENCE.md
    file? If so, how can I get details on the file problem - in other words, to know what exactly is outdated?
  • k

    kenyon

    07/17/2023, 8:31 PM
    @emerson_prado
    REFERENCE.md
    is generated by
    bundle exec rake reference
    👍 1
  • e

    emerson_prado

    07/17/2023, 8:34 PM
    Copy code
    bundle exec rake reference
      The "reference" task is deprecated.
    
      Prefer "release:prepare" which manage all pre-release steps, or directly run
      the "strings:generate:reference" task.
    Should I run
    bundle exec rake strings:generate:reference
    ?
  • k

    kenyon

    07/17/2023, 8:34 PM
    ah yeah, thought that was the case, I was in the process of checking
  • e

    emerson_prado

    07/17/2023, 8:43 PM
    @kenyon Fixed. Thanks!
    👍 1
1...556557558...648Latest