https://www.puppet.com/community logo
Join Slack
Powered by
# voxpupuli
  • b
    I think the new thing is called PCT
  • v
    I don't like that is written in Go and uses Go templating language
  • b
  • b
  • t
    Bets on how long it'll last?
  • b
    in two years Perforce will accept that msync is the superior tool for keeping modules in sync
  • b
    in addition to msync update, just add an msync create subcommand to create classes/new modules, done \ol
  • t
    And that beaker is a superior testing system
  • b
    nods
  • t
    At least the new stuff is being written in go so it's harder to contribute to
  • b
    ever contributed to facter 3? :D
  • t
    Once
  • t
    But I didn't write tests for it
  • t
    Never to puppetserver though
  • b
    a bit of cmake here, a bit of boost there!
  • t
    Got no time for that Clojure nonsense
  • b
    the ruby part in it still fun though
  • t
    But why tho
  • v
  • b
    @Alex Fisher and everybody interesting/using catalog-diff. Can I get some eyes on https://github.com/camptocamp/puppet-catalog-diff/pull/61/files ?
  • v
    People use catalog diff to help them upgrade from old versions of Puppet. As such, I think for this module we do actually want to try to make sure it still works with non supported puppet versions.
  • v
    eg. in the last 12 months, I used it to diff 2000 catalogs from puppetserver 4 to puppetserver 6.
  • v
    we could tag this as a new major version
  • v
    I need support for different TLS certs and that's kinda tricky / I'm not sure if possible at all, with the older client
  • v
    Is it not possible to make it use the new API if available?
  • v
    If you are going to break it for Puppet < 6 (which I don't think is a good idea), then that leaves behind loads of puppetdb compatibility code that would then be redundant.
  • v
  • v
    The original code already looks a bit wonky. Why is there a call to 
    get_puppetdb_version
    and a few lines later a heuristic for determining the version based on the return code of querying puppetdb?
  • v
    I need to say the new http client only needs to be available on the system that runs the differ. That box needs to be puppet 6.11 or newer. you can still connect to every old puppetdb/puppetserver
  • v
    ah, that might be true. Have you got an old puppetserver available to verify this against?
1...383940...648Latest