alastairp, based on your whether cron is running PR, how about we also add an action to validate and display the crontabs in human readable form if a crontab is modified?
2021-05-14 13441, 2021
_lucifer
we do not need to do this now but seems a useful future enhancement
2021-05-14 13449, 2021
ruaok
reosarevok: seems good.
2021-05-14 13419, 2021
Freso
I would remove the "if you ask nicely", but otherwise seems good.
2021-05-14 13420, 2021
alastairp
mmm, maybe itercron will raise an exception if a pattern is invalid? I don't know if it can turn a pattern into a descriptive text
2021-05-14 13448, 2021
_lucifer
there's a is_valid function to check if the pattern is valid
2021-05-14 13457, 2021
alastairp
I guess ruaok and I have been around long enough that (most of the time) we know how to write a cron without screwing it up
2021-05-14 13416, 2021
alastairp
would be happy to add additional cron management commands if you'd find it useful
2021-05-14 13456, 2021
ruaok snaps his suspenders in agreement
2021-05-14 13407, 2021
reosarevok
Freso: well, the point is we are not obliged to remove the name AFAICT and I certainly won't do it if they are dicks about it
2021-05-14 13411, 2021
_lucifer
👍, human readable text would have been nice but i do not see that in the library docs so let's leave it.
hmm. interesting. I should read up on the USB-3 spec. I wonder how the data is multiplexed
2021-05-14 13427, 2021
alastairp
there's so much going on there
2021-05-14 13437, 2021
alastairp
usbc/thunderbolt/alt modes, etc
2021-05-14 13419, 2021
alastairp
a friend of mine has a 4k monitor with built in ethernet and usb hub, and if he wants 60hz he can't run the hub at usb-3/gigabit ethernet because there's not enough bandwidth
alastairp, ruaok: we do not have buildx on bono. can we install that?
2021-05-14 13456, 2021
_lucifer
*docker buildx
2021-05-14 13422, 2021
ruaok
sure, have at it. bono is our wild west, right?
2021-05-14 13437, 2021
alastairp
you'll have to install a new version of docker
2021-05-14 13443, 2021
_lucifer
lol
2021-05-14 13455, 2021
_lucifer
i checked we have 19.03 so it should work
2021-05-14 13420, 2021
_lucifer
only need to install buildx
2021-05-14 13440, 2021
alastairp
maybe there's a setting to add in the daemon.json file
2021-05-14 13457, 2021
ruaok
alastairp: for `COPY ./docker/services/spark_reader/consul-template-spark-reader.conf /etc/consul-template-spark-reader.conf` how does the base image know to look at this file?
2021-05-14 13412, 2021
alastairp
ruaok: sorry - missed one more item
2021-05-14 13419, 2021
alastairp
we add `down` to all services
2021-05-14 13427, 2021
alastairp
and then in rc.local rm the down based on env variables
2021-05-14 13446, 2021
ruaok
understood, but this is a one service container that I always want up. so I should not need down, right?
2021-05-14 13418, 2021
alastairp
right, so add a new runit service with a `run` file
any tips on how to debug this? the flags for run-consul-template are not terribly helpful.
2021-05-14 13425, 2021
alastairp
are you just running it from a `docker run`?
2021-05-14 13433, 2021
_lucifer
i woudl suggest copying stuff from the statup improvements PR
2021-05-14 13403, 2021
alastairp
consul-agent requires syslog to be started up (which runit will do)
2021-05-14 13411, 2021
ruaok
the container just restarts, so I am doing a `docker run --rm -it ... bash`
2021-05-14 13403, 2021
ruaok
yep, i have a valid looking /etc/service/syslog-ng/run
2021-05-14 13434, 2021
alastairp
I normally see this happening - consul-template starts up, doesn't see syslog, prints the error, syslog starts up, consul-template restarts, connects to syslog, and goes silent
2021-05-14 13400, 2021
_lucifer
reosarevok, i need to create a couple of fake users on MB for testing. can't use test.mb. what's the best way to go about it. i go ahead and create. after i am done, delete the account?
2021-05-14 13418, 2021
alastairp
but I'm unsure about the restart, sorry. I'm in a call, but can take a look at it afterwards if you get the code up
2021-05-14 13431, 2021
reosarevok
_lucifer: or don't delete them
2021-05-14 13444, 2021
reosarevok
I see no problem with having ListenBrainz Test User or something
ruaok, just saw your consul template, i think the startup improvements won't benefit you :(. those improvements are useful when loading stuff like keys from consul.
2021-05-14 13453, 2021
ruaok
ah, I feel better now. I looked at your links and wasn't certain what I should be copying.
2021-05-14 13403, 2021
ruaok
thanks though.
2021-05-14 13413, 2021
_lucifer
one thing though, you might need a `cd /code/metrics` somewhere.