2017-11-11T00:22:25 *** kl_eisbaer has quit IRC 2017-11-11T00:29:39 *** kl_eisbaer has joined #opensuse-admin 2017-11-11T00:29:52 *** kl_eisbaer has left #opensuse-admin 2017-11-11T01:56:34 *** Fraser_Bell has joined #opensuse-admin 2017-11-11T02:01:45 *** Fraser_Bell has quit IRC 2017-11-11T03:06:35 *** plinnell has quit IRC 2017-11-11T03:57:31 *** okurz has quit IRC 2017-11-11T03:59:20 *** okurz has joined #opensuse-admin 2017-11-11T04:18:06 PROBLEM: Hosts syslog on monitor.infra.opensuse.org - CRITICAL: Found files older than 600 minutes /var/log/opensuse/hosts//linux.log was last modified on Fri Nov 10 18:09:15 2017 ; See https://monitor.opensuse.org/icinga/cgi-bin/extinfo.cgi?type=2&host=monitor.infra.opensuse.org&service=Hosts%20syslog 2017-11-11T04:47:33 *** plinnell has joined #opensuse-admin 2017-11-11T04:47:33 *** plinnell has joined #opensuse-admin 2017-11-11T07:43:11 *** Son_Goku has joined #opensuse-admin 2017-11-11T09:46:18 *** fvogt has joined #opensuse-admin 2017-11-11T10:02:27 *** cboltz has joined #opensuse-admin 2017-11-11T10:08:26 *** Son_Goku has quit IRC 2017-11-11T10:08:28 *** Sir_Gallantmon has joined #opensuse-admin 2017-11-11T11:10:26 cboltz: https://gitlab.infra.opensuse.org/infra/salt/merge_requests/82 up for review again 2017-11-11T11:16:53 *** ldevulder has quit IRC 2017-11-11T11:32:03 *** ldevulder has joined #opensuse-admin 2017-11-11T12:52:30 tampakrap: looks good, just a question: 2017-11-11T12:52:51 you use sed -i -e 's/production/base/' /srv/{salt,pillar}/top.sls to avoid adding some salt configuration 2017-11-11T12:53:24 wouldn't it be easier to drop a small salt config sniplet into /etc/salt/ instead? 2017-11-11T12:53:45 oh, and I'd prefer if you use s/^production:/base:/ ;-) 2017-11-11T12:54:29 I don't know 2017-11-11T12:54:42 I am installing only salt, no master/minion package 2017-11-11T12:54:49 so I am not sure if the configs are installed at all 2017-11-11T12:55:03 I tried also to pass saltenv=production, didn't work 2017-11-11T12:55:20 yeah, that would have been too easy ;-) 2017-11-11T12:57:47 so - what about using ^production: to make sure you change only the line you want to change? 2017-11-11T12:58:07 sure, doing it now 2017-11-11T12:59:02 my question would be 2017-11-11T12:59:11 I am doing the same sed twice, can I create a function? 2017-11-11T12:59:29 I don't know if it is possible to create a function in bash and pass args to it 2017-11-11T13:00:44 try yourself ;-) 2017-11-11T13:00:47 function foo { echo "$1"; } ; foo bar 2017-11-11T13:02:00 you can also use multiple sed commands without -e: sed -i 2017-11-11T13:02:19 you can also use multiple sed commands without -e: sed -i 's/foo/bar/ ; s/baz/asdf/' 2017-11-11T13:02:33 mind giving me a patch please? 2017-11-11T13:03:33 https://gitlab.infra.opensuse.org/infra/salt/-/jobs/298 2017-11-11T13:15:25 looks good, feel free to merge 2017-11-11T13:25:43 I will rename the script to test_show_highstate 2017-11-11T13:25:50 it does much more than syntax checking 2017-11-11T13:27:35 no objections, but that should be the last change in this branch 2017-11-11T13:27:45 merge it before the diff becomes even bigger ;-) 2017-11-11T13:28:32 lol okay 2017-11-11T13:28:47 really amazing that we have finally proper tests 2017-11-11T13:31:27 I will merge also the other two open ones, they seem fine 2017-11-11T13:31:43 wanna have a look before I do? 2017-11-11T13:32:19 yes, give me a minute ;-) 2017-11-11T13:32:58 k 2017-11-11T13:33:43 !84 looks good (assuming "gitlab-runner" is the correct username - but if not, it won't be more broken than before *g*) 2017-11-11T13:35:23 !83 looks good on the technical level (I don't know enough about the network setup to judge if the IPs and device names are correct) 2017-11-11T13:36:15 !83 is based on the comments of !73 2017-11-11T13:38:00 the changes match the comments, so I'll assume it's correct ;-) 2017-11-11T13:38:44 good thanks, all merged 2017-11-11T13:38:48 next one coming soon 2017-11-11T13:40:29 * cboltz hides 2017-11-11T13:52:48 *** ldevulder has quit IRC 2017-11-11T15:10:43 *** Sir_Gallantmon is now known as Son_Goku 2017-11-11T15:17:31 cboltz: https://gitlab.infra.opensuse.org/infra/salt/merge_requests/85 2017-11-11T15:31:34 *** sven15 has joined #opensuse-admin 2017-11-11T15:33:00 *** plinnell has quit IRC 2017-11-11T16:11:52 *** Hawk3ye has joined #opensuse-admin 2017-11-11T16:24:17 Hi folks, I've just created a new VG and LVM volume but the volume wont come active on boot (the VG does). Any ideas? 2017-11-11T16:54:57 *** plinnell has joined #opensuse-admin 2017-11-11T17:18:04 Hawk3ye: this channel is used to discuss the administration of the openSUSE infrastructure. For technical support, please ask in #suse 2017-11-11T17:18:46 tampakrap: !85 looks good :-) 2017-11-11T17:19:47 if you plan to add more tests, it might be a good idea to have everything in one script (or one script that calls all scripts) to avoid the test setup overhead 2017-11-11T17:20:43 in this case, using "unittest" might make sense because it will run all tests even if the first test fails 2017-11-11T17:58:44 *** asmorodskyi has joined #opensuse-admin 2017-11-11T20:17:05 *** asmorodskyi has quit IRC 2017-11-11T21:12:20 *** asmorodskyi has joined #opensuse-admin 2017-11-11T21:18:15 *** Son_Goku has quit IRC 2017-11-11T21:42:45 *** Son_Goku has joined #opensuse-admin 2017-11-11T22:45:42 *** cboltz has quit IRC 2017-11-11T23:13:09 *** Son_Goku has quit IRC 2017-11-11T23:28:34 *** Hawk3ye has quit IRC 2017-11-11T23:36:30 *** Son_Goku has joined #opensuse-admin 2017-11-11T23:45:27 *** fvogt has quit IRC