#theforeman

/

      • gwmngilfen
        sigh, dont hit enter
      • as per mhulan's comments in that ticket, any plugin using "prepend" will conflict with those using alias_method_chain
      • aruzicka
        wubooo: only for transaction handling, not for "find me a thing with id 3". dynflow internaly uses sequel for manipulating execution plans
      • gwmngilfen
        unless you're actually seeing openscap in the stacktraces, it's probably something else
      • spectr has quit
      • ikonia
        gwmngilfen: seeing openscap in the stack trace
      • and I'm seeing openscap in the menu items, hence my shock at it being there
      • gwmngilfen
        then I guess you have the plugin loaded, indeed, start with removing it and then figure out what installed it ;)
      • or sure it's not a default option, and i don't believe its a dependency of anything either
      • s/or/for
      • FriedBob has quit
      • hgiessel has quit
      • wubooo
        aruzicka: right okay that makes sense, so basically need to develop a makara equivalent for sequel?
      • aruzicka
        wubooo: or implement activerecord persistence adapter for dynflow
      • wubooo
        it's a bit annoying that foreman literally can't function with both makara and remote execution
      • wubooo has quit
      • ikonia
        gwmngilfen: it doesn't "look" like a default option, but that said, I've not specificed it (and I've ever tried using the --no-enable option for it, and it still pops up after deployment, thats why I was wondering if it had been made standard/default and the installer options not cleaned up
      • gwmngilfen
        ikonia: its not a default. can you reproduce on a clean host?
      • aruzicka_ has quit
      • Z3NF1N1TY joined the channel
      • ikonia
        gwmngilfen: trying to do it at the moment
      • wuboooooo joined the channel
      • wuboooooo
        oops, quit by accident, aruzicka: , any advice on best approach?
      • dLobatog has quit
      • suresh12 has quit
      • ikonia
        so should /usr/share/foreman-installer/config/foreman-answers.yaml
      • be the correct answer file that the installer stores the answers for each time it's run
      • just want to clarify %100 I'm verifying the right answer fire
      • file
      • gwmngilfen
        it's i/etc/ iirc, although that may be a link
      • ikonia
        thats the installer options isn't it ?
      • /etc/foreman-installer/scenarios.d/foreman-answers.yaml
      • dLobatog joined the channel
      • isn't that the options that the installer presents, not the actual answers for storing
      • gwmngilfen
        wasy enough to test
      • *easy
      • ikonia
        a fair point
      • gwmngilfen
        pretty sure thats the file though
      • ikonia
        good job I asked, as I was certain it was the other way around
      • gwmngilfen tests just to be sure
      • I'm doing the same now
      • gwmngilfen
        yep, thats the one
      • ikonia
        interesting, you win
      • gwmngilfen
        i enabled a plugin, and it changed from false to {}
      • i usually do
      • ;)
      • ikonia
        cheat
      • gwmngilfen
        nah, practice ;)
      • aruzicka
        wottop: I'm not sure what would amount to less work
      • kpease_ joined the channel
      • jyejare_ has quit
      • asharvit joined the channel
      • wottop
        aruzicka: huh?
      • devmodem joined the channel
      • gwmngilfen
        @aruzicka:matrix.org: wuboooooo i think it's fair to say we're weren't really considering tools like makara. Perhaps the first step would be a discussion on the mailing list about wether we should support it, and what that work might need?
      • kTitan has quit
      • AtuM has quit
      • sharvit has quit
      • sharvita joined the channel
      • asharvit has quit
      • nagoor has quit
      • nagoor joined the channel
      • nagoor has quit
      • FriedBob joined the channel
      • nagoor joined the channel
      • hardbot joined the channel
      • nagoor has quit
      • nagoor joined the channel
      • lzap has quit
      • wuboooooo
        gwmngilfen: thanks for the info, i'll raise a topic on the mailing list
      • jpavel joined the channel
      • jpavel
        Hey folks - got a question about smart class parameters.
      • I have a puppet module with: Variant[Boolean, Enum['sometimes']]. When I have it set to sometimes as a smart class parameter, I get parameter 'manage_puppetca' expects a value of type Boolean or Enum['sometimes'], got String
      • joshbenner has quit
      • The override is set to String - is there another way to configure the smart class parameter override for Enum?
      • I tried setting an inputer validator since that is Enum-like, but it made no difference. :-D
      • bryan_kearney has quit
      • gwmngilfen
        jpavel: so you're hitting one of life's great mysteries - why is it so damn hard to send booleans in YAML
      • basically we have a Boolean type in Foreman just for this purpose
      • but ofc, you cant use that because you need strings for the Enum
      • however sending "true" for the other type is going to come out as a tring not a boolean
      • (like, thanks for that YAML)
      • jpavel
        Right. :-( I use "Boolean" where applicable - I do like the dropdown to simplify that for me so I don't fat-finger my typing
      • gwmngilfen
        is this a home-written modue?
      • jpavel
        it is
      • gwmngilfen
        so the easiest solution is probably to add true and false to the enum
      • and then use a validator in foreman to select one of those 3 values
      • on the puppet side you can massage the true-string into a true-boolean
      • devmodem has quit
      • seethaler has quit
      • suresh12 joined the channel
      • not perfect, but should work i think?
      • jpavel
        I can try it and see!
      • nstrug has quit
      • gwmngilfen
        the validator should mean you still get a drop-down, iirc
      • if you use a list style, not a regex style ofc
      • Z3NF1N1TY has left the channel
      • jpavel
        Actually, I got it to work - I used yaml instead of String, with the validator
      • ernelson has quit
      • thanks!!!
      • oprazak has quit
      • gwmngilfen
        jpavel: ah nice
      • ernelson joined the channel
      • Philambdo has quit
      • tbrisker has quit
      • TKersten has left the channel
      • hmw has quit
      • spectr joined the channel
      • spectr-RH has quit
      • spectr has quit
      • bryan_kearney joined the channel
      • StuartMI joined the channel
      • jclaret joined the channel
      • jclaret has quit
      • jclaret joined the channel
      • smeyer has quit
      • fbrychta is now known as fbrychta-bbl
      • swapab has quit
      • jcpunk joined the channel
      • pondrejk has quit
      • swapab joined the channel
      • swapab has quit
      • pgagne_ has quit
      • afeferku has quit
      • shimshtein has quit
      • lpramuk has quit
      • hardbot joined the channel
      • jomitsch is now known as jomitsch-afk
      • sharvita has quit
      • orabin has quit
      • mshira_ has quit
      • wuboooooo has quit
      • hmw joined the channel
      • timogoebel has quit
      • tamarin joined the channel
      • preethi joined the channel
      • rplevka_ has quit
      • fabianvf
        is there a good way for me to apply a small patch (in a distributable way) to my foreman installation without messing too much stuff up? I want to remove the skip_orchestration! from https://github.com/theforeman/foreman/blob/1f6a... so that my dns updates when I get puppet reports
      • gwmngilfen
        fabianvf: i'd suggest starting a discussion (either a feature request or the mailing list) about it - I'd guess we could accept that as a Setting, and then you can contribute it upstream :)
      • much easier than carrying it yourself
      • fabianvf
        gwmngilfen: certainly happy to contribute it upstream :) was just curious if there is a way I could do it for right now since presumably there will be a bit of a delay between proposal/submission and release
      • gwmngilfen
        indeed. you could do it as a plugin, but for such a one-line patch that may be overkill
      • if you do plan to go upstream with it, then editing the file is probably ok - it's only a burden when you have many manual changes to port at upgrade time
      • fabianvf
        gwmngilfen: cool, that makes sense. Should I make an issue and then submit a patch or just submit the patch and include reasoning?
      • gwmngilfen
        all core changes need a ticket anyway, so I'd start with a "Feature" ticket, give your reasoning and that you're happy to write it.
      • a setting makes the most sense to me, but perhaps others will disagree
      • suresh12 has quit
      • pid1 joined the channel
      • nkathole has quit
      • jpavel has quit