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

    VoxBot

    06/28/2024, 9:03 AM
    this if I remove the baseline_os filter as required in the ubuntu 24 PR
  • v

    VoxBot

    06/28/2024, 9:05 AM
    yeah, I think it makes sense to run the unit tests without the filter and then on a larger runner
  • v

    VoxBot

    06/28/2024, 9:13 AM
    mhm I can add a runner group to our org without a TOTP token, but to remove a runner group I need to authenticate again
  • r

    Robert Waffen

    06/28/2024, 9:26 AM
    review pls: https://github.com/voxpupuli/modulesync_config/pull/913
  • v

    VoxBot

    06/28/2024, 9:26 AM
    valantin: https://github.com/voxpupuli/puppet-zabbix/pull/967 that will make it fast
  • v

    VoxBot

    06/28/2024, 9:27 AM
    rwaffen: I review yours, you review mine?
  • v

    VoxBot

    06/28/2024, 9:33 AM
    question, is gentoo a OS supported by voxpupupli? I see it was removed from facterdb
  • v

    VoxBot

    06/28/2024, 9:33 AM
    we don't have something like officially supported operating systems
  • v

    VoxBot

    06/28/2024, 9:34 AM
    if something works and has tests, we can merge it
  • v

    VoxBot

    06/28/2024, 9:34 AM
    and on FacterDB we removed factsets from old Facter version, but there are still Gentoo factsets present
  • v

    VoxBot

    06/28/2024, 9:34 AM
    ok, so I need to understand why spec fails on gentoo-2-x86_64
  • v

    VoxBot

    06/28/2024, 9:35 AM
    https://gist.github.com/bastelfreak/051a7994325931883186d806c321dca4
  • v

    VoxBot

    06/28/2024, 9:38 AM
    ah ok I misunderstood this message "No facts were found in the FacterDB for Facter v4.2.8 on {:operatingsystem=>"Gentoo", :hardwaremodel=>"x86_64"}, using v3.14.5 instead"
  • v

    VoxBot

    06/28/2024, 9:39 AM
    that's fine. there's a complex logic to check which puppet version we use in unit tests, then we search for a matching facter version. then we check for a factset with the same facter minor version or an older factset
  • v

    VoxBot

    06/28/2024, 10:10 AM
    I need help, I don't understand why the tests fail on gentoo, it fail all the test
  • v

    VoxBot

    06/28/2024, 10:12 AM
    on which PR?
  • v

    VoxBot

    06/28/2024, 10:17 AM
    not pushed, I'm working to remove baselineos filter
  • v

    VoxBot

    06/28/2024, 10:23 AM
    fixed main problem i push it
  • v

    VoxBot

    06/28/2024, 10:24 AM
    \o/
  • v

    VoxBot

    06/28/2024, 10:31 AM
    can I get a review for https://github.com/voxpupuli/voxpupuli-rubocop/pull/64 ?
  • v

    VoxBot

    06/28/2024, 11:05 AM
    I've realise next relese of zabbix module was a major relese due to an already merged backwards-incompatible change
  • v

    VoxBot

    06/28/2024, 11:06 AM
    so probably next week I've massively require review to prepare next release
  • v

    VoxBot

    06/28/2024, 11:41 AM
    bastelfreak: this is the error in gentoo spec I'm tolking about https://github.com/voxpupuli/puppet-zabbix/actions/runs/9712083421/job/26806183342?pr=968
  • v

    VoxBot

    06/28/2024, 11:43 AM
    ah it assumes Gentoo has no systemd
  • v

    VoxBot

    06/28/2024, 11:44 AM
    ups I've probably found the problem, I've set manage_startup_script to true for Gentoo
  • v

    VoxBot

    06/28/2024, 11:47 AM
    is it, shame on me :D
  • v

    VoxBot

    06/28/2024, 11:47 AM
    not found before because not run the spec on gentoo...
  • v

    VoxBot

    06/28/2024, 11:49 AM
    that's exactly the reason why I don't like those filters
  • v

    VoxBot

    06/28/2024, 12:34 PM
    neat, Puppet now adopts voxpupuli-rubocop
  • v

    VoxBot

    06/28/2024, 12:36 PM
    bastelfreak: I'm forgot to rebase config update to https://github.com/voxpupuli/puppet-zabbix/pull/956 and mention it in the PR
1...600601602...648Latest