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

    bastelfreak

    06/10/2022, 9:32 AM
    and again, if the defaults are fine, you don't need to set it at all. your to options are already in there
  • v

    VoxBot

    06/10/2022, 9:33 AM
    I think I need to change them, I'm just to get the config to generate clean/known config before I start changing
  • a

    Alex Fisher

    06/10/2022, 10:16 AM
    @ewoud https://gist.github.com/alexjfisher/b0287dc84a5ce451093d43f2de16f4ed
  • v

    VoxBot

    06/10/2022, 10:23 AM
    bastelfreak: thanks a lot, the problem was I was double setting proxy_set_header, rather than an array of values for the module to loop through
  • v

    VoxBot

    06/10/2022, 10:23 AM
    I would have wasted loads more time not finding that, thank you
  • v

    VoxBot

    06/10/2022, 10:28 AM
    :)
  • b

    bastelfreak

    06/10/2022, 10:28 AM
    @Alex Fisher having that in the stdlib docs would help a lot
  • a

    Alex Fisher

    06/10/2022, 10:29 AM
    ?
  • b

    bastelfreak

    06/10/2022, 10:29 AM
    an example dependency graph for ensure_packages
  • a

    Alex Fisher

    06/10/2022, 10:30 AM
    It's just a thing I need to fix. I'm really not trying to introduce any changes. Especially not really subtle, (but breaking ones) like this.
  • b

    bastelfreak

    06/10/2022, 10:30 AM
    yes sure
  • b

    bastelfreak

    06/10/2022, 10:31 AM
    but one of the graphs would be nice I think
  • v

    VoxBot

    06/10/2022, 10:31 AM
    bastelfreak: I do need to change them, I just wanted the config generation working first before I then tried to change them, start from a working place and build on it
  • a

    Alex Fisher

    06/10/2022, 10:32 AM
    Whatever the behaviour was before, I want to maintain it. The PR was only supposed to be a first step towards fixing the breaking change that got into 8.0.0
  • b

    bastelfreak

    06/10/2022, 10:32 AM
    yes, sure
  • v

    VoxBot

    06/10/2022, 10:54 AM
    afisher: "great", more subtle things
  • v

    VoxBot

    06/10/2022, 10:56 AM
    afisher: do you end up with a similar to old.png if you add Package['tzdata'] -> Class['foo']
  • v

    VoxBot

    06/10/2022, 10:57 AM
    (might also be helpful to add a quick comment on how to generate the diagram in case people want to reproduce it)
  • v

    VoxBot

    06/10/2022, 12:01 PM
    can I get a review for this tiny rubocop update https://github.com/voxpupuli/puppet-catalog-diff/pull/73
  • b

    bastelfreak

    06/10/2022, 12:01 PM
    @Alex Fisher do you have some time to debug puppet-catalog-diff / any ideas on how to handle trusted facts?
  • v

    VoxBot

    06/10/2022, 1:05 PM
    can I get a review for https://github.com/voxpupuli/puppet-catalog-diff/pull/81
  • y

    Yury Bushmelev

    06/10/2022, 1:10 PM
    good $time_of_the_day here! I found voxpopuli puppet lint plugins enabled in my PDK and it affects my bolt plans 🙂
  • y

    Yury Bushmelev

    06/10/2022, 1:11 PM
    I found few weird things.. shall I report an issue on github or can we discuss here before?
  • b

    bastelfreak

    06/10/2022, 1:12 PM
    I guess a github issue a good idea. then we can also reference it in pull requests/others can find it
  • y

    Yury Bushmelev

    06/10/2022, 1:12 PM
    ok, I'll raise few issues there then on the weekend!
  • a

    Alex Fisher

    06/10/2022, 1:17 PM
    @bastelfreak Where are you using trusted facts? Just your hiera?
  • b

    bastelfreak

    06/10/2022, 1:18 PM
    a good question
  • b

    bastelfreak

    06/10/2022, 1:18 PM
    in hiera, but maybe also in the puppet code
  • a

    Alex Fisher

    06/10/2022, 1:21 PM
    What version puppetserver(s)?
  • b

    bastelfreak

    06/10/2022, 1:21 PM
    6.something is old and 7.something is the new one. currently I get old catalogs from puppetdb and don't let them compile
1...666768...648Latest