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

    Alex Fisher

    07/15/2022, 10:44 AM
    @bastelfreak Hi! Did you ever have a crack at fixing https://github.com/kuleuven/puppet-lint-manifest_whitespace-check/issues/10 ?
  • b

    bastelfreak

    07/15/2022, 11:58 AM
    no :(
  • a

    Alex Fisher

    07/15/2022, 1:18 PM
    I'm struggling to write a failing spec test.
  • a

    Alex Fisher

    07/15/2022, 1:25 PM
    @bastelfreak In https://github.com/kuleuven/puppet-lint-manifest_whitespace-check/issues/10#issue-973616662, he you mean
    closing_bracket
    instead of
    closing_brace
    ? 😉
  • b

    bastelfreak

    07/15/2022, 1:26 PM
    possible, but usually I copy and paste those messages 🤔
  • a

    Alex Fisher

    07/15/2022, 1:29 PM
    Works a lot better if I put your example in ./spec/puppet-lint/plugins/manifest_whitespace_closing_bracket_spec.rb instead of ./spec/puppet-lint/plugins/manifest_whitespace_closing_brace_spec.rb
  • a

    Alex Fisher

    07/15/2022, 1:30 PM
    Anyway... that was the easy bit. Now to see if I can actually fix it!
  • a

    Alex Fisher

    07/15/2022, 2:01 PM
    @bastelfreak https://github.com/kuleuven/puppet-lint-manifest_whitespace-check/pull/15
    🦜 2
    👍 2
  • a

    Alex Fisher

    07/15/2022, 2:01 PM
    @bastelfreak https://github.com/kuleuven/puppet-lint-manifest_whitespace-check/pull/15
  • s

    Slackbot

    07/15/2022, 2:35 PM
    This message was deleted.
    b
    l
    a
    • 4
    • 6
  • s

    Slackbot

    07/15/2022, 2:40 PM
    This message was deleted.
    👍 1
    b
    o
    • 3
    • 3
  • o

    op-ct

    07/15/2022, 2:40 PM
    We have some community contributions coming to overhaul https://github.com/puppetlabs/beaker-google/. All the other beaker stuff has migrated to voxpupuli with―should beaker-google come, too?
  • v

    VoxBot

    07/16/2022, 11:59 AM
    bastelfreak, it's a belgian university, maybe .be people from the cfgmgmt bubble know someone there?
  • v

    VoxBot

    07/16/2022, 9:07 PM
    I'm pretty sure some people from that university regularly attend cfgmgmtcamp and are Foreman users, but I could be mistaken (and that they were from the Ghent university)
  • v

    VoxBot

    07/18/2022, 9:06 AM
    https://github.com/voxpupuli/voxpupuli-test/pull/80 should be a simple review :)
  • v

    VoxBot

    07/18/2022, 9:08 AM
    I'm completely lost in how this ever worked: https://github.com/voxpupuli/rspec-puppet-facts/commit/21b679f45b9e20902af53161b5e1bd0b4fb723f7
  • v

    VoxBot

    07/18/2022, 9:08 AM
    custom facts are strings, but all other facts are symbols?
  • v

    VoxBot

    07/18/2022, 9:10 AM
    and why is this biting me only now?
  • v

    VoxBot

    07/18/2022, 9:11 AM
    https://github.com/voxpupuli/rspec-puppet-facts/pull/91 mhm
  • v

    VoxBot

    07/18/2022, 9:11 AM
    This should also be fixed upstream in FacterDB
  • v

    VoxBot

    07/18/2022, 9:11 AM
    I now see that there's later a PR that adds a setting to it
  • v

    VoxBot

    07/18/2022, 9:11 AM
    but there was a reason for the facts being symbols in facterdb
  • v

    VoxBot

    07/18/2022, 9:11 AM
    but custom facts do not respect that
  • v

    VoxBot

    07/18/2022, 9:14 AM
    this is not what I wanted to spend my time on today, but here we are
  • v

    VoxBot

    07/18/2022, 9:15 AM
    how did you notice this? I think not all custom facts are strings?
  • v

    VoxBot

    07/18/2022, 9:15 AM
    or are they always stringified?
  • v

    VoxBot

    07/18/2022, 9:15 AM
    I'm quite sure I used them as symbols previously
  • v

    VoxBot

    07/18/2022, 9:16 AM
    I did this: add_custom_fact :systemd, ->(os, facts) { require 'pry' ; binding.pry ; facts[:service_provider] == 'systemd' }
  • v

    VoxBot

    07/18/2022, 9:16 AM
    and now I wonder how this ever worked, it probably didn't
  • v

    VoxBot

    07/18/2022, 9:17 AM
    not sure I ever tried it that way
1...103104105...648Latest