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
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