https://www.puppet.com/community logo
Join Slack
Powered by
# voxpupuli
  • v
    mostly looks some like some small visual changes
  • v
    ewoud: ah I just saw it on a PR
  • v
    just looks like a UI change?
  • y
    hmm.. do we have auditd module? 🤔
  • v
    I don't think so
  • y
    I'm not ready to deploy SIMP stack.. so nexcess-auditd seems another most popular one..
  • d
  • y
    meh.. no Puppet 8 support it seems.. also no Debian
  • y
    well.. maybe I'll just make a profile with 2 templates instead..
  • v
    I understand the SIMP stack concern, but I think you don't need too much for their auditd module
  • y
    https://github.com/kemra102/puppet-auditd <- this one looks good also, but archived
  • y
    well.. simp depends on simplib -> simpkv, which is ok-ish.. as long as I don't want syslog.. otherwise it'll explode 🙂
  • y
    but maybe I'll try to update nexcess one instead
  • y
    ah, btw.. wrt dropping EOL OSes/adding new OSes - is it better to do in different PRs or just different commits in one PR is ok?
  • y
    IIRC, it was something regarding changelog
  • v
    different PRs are better. so one shows up as an enhancement in the changelog, the other one as a breaking change
  • v
    and that makes it easier to mention the OS versions in the PR title
  • v
    the goal should be that a user reads the changelog and knows if their OS is supported or not. Without reading the actual PR diffs, just the changelog
  • v
    bastelfreak: yes, though I think I saw a few improvements
  • v
    for example, they only show you the allowed ways to merge and no longer the disabled ones
  • v
    and explicitly list failed jobs
  • v
    the checks are indeed also a bit different and it's probably an improvement
  • y
    order of drop/add PR doesn't matter, right?
  • v
    so if the PR that removes versions would remove the whole OS, I provide that second
  • v
    just to keep the diff in the metadata.json shorter, but it doesn't make a difference
  • v
  • v
  • t
    @Yury Bushmelev i use genv/auditd on Debian
    🤝🏻 1
  • d
  • v
    bastelfreak: colleague points out misleading text in gitlab_ci_runner's README, "To use the Gitlab CI runners it is required to have the puppetlabs/docker module." - it is only required if you want to use docker executor, surely?
1...614615616...648Latest