seems the build failed while trying to install python
2021-05-07 12720, 2021
_lucifer
i'll rerun the jobs and see if that goes away
2021-05-07 12757, 2021
Mr_Monkey
ruaok: Yes, as mentionned I need to update Node version, but I started that last night.
2021-05-07 12757, 2021
Mr_Monkey
Alternatively, you can merge as-is, I sexuple-checked that the mechanism works correctly, and using a more recent Node version all tests pass.
2021-05-07 12750, 2021
ruaok
Mr_Monkey: great, thanks for verifying.
2021-05-07 12701, 2021
Strezless joined the channel
2021-05-07 12717, 2021
sumedh joined the channel
2021-05-07 12743, 2021
_lucifer
ruaok, i want to test some further improvements for the spark request consumer. will need to take down/up the container few times and issue some requests. is that fine?
ruaok, what's the right way to run a script in the background? i found nohup, setsid or & but not sure which to use?
2021-05-07 12736, 2021
ruaok
script.py &
2021-05-07 12750, 2021
ruaok
will run in the background, but if you close that terminal, the process gets killed.
2021-05-07 12700, 2021
ruaok
nuhup script.py &
2021-05-07 12709, 2021
ruaok
will continue running in the background.
2021-05-07 12747, 2021
ruaok
nohup script.py 2>&1 > <file> &
2021-05-07 12702, 2021
ruaok
should capture output into a file and keep running if the terminal disconnects.
2021-05-07 12712, 2021
_lucifer
yup that the last command is what i am exactly looking for. thanks!
2021-05-07 12712, 2021
ruaok
likely to only work in bash.
2021-05-07 12726, 2021
ruaok
alias m='make 2>&1 | less'
2021-05-07 12742, 2021
ruaok
is an alias I have for make'ing C programs.
2021-05-07 12743, 2021
_lucifer
ruaok, driver running on newleader without docker now :D
2021-05-07 12717, 2021
_lucifer
and python deps also managed inside a venv so no issues there.
2021-05-07 12721, 2021
_lucifer
alastairp, after our discussion yesterday, i revisted the docs. i had misunderstood those, the deps do get distributed to the driver at least in the yarn client mode which is the one we use. thanks!
2021-05-07 12737, 2021
ruaok
nice. remind me, what was the motivation for getting rid of docker?
2021-05-07 12734, 2021
_lucifer
doesn't add anything and requires additional setup. right we have to build the driver image and update the git repo on newleader to ensure things work correctly.
2021-05-07 12754, 2021
_lucifer
now its just update the repo on newleader and execute the startup script.
2021-05-07 12714, 2021
alastairp
ruaok: I did a preliminary review of the PR, but skipped the listenstore file for now in order to get some comments to you. Looking at listenstore now
2021-05-07 12725, 2021
ruaok
alastairp: great, thanks.
2021-05-07 12752, 2021
ruaok
_lucifer: ahh, I see. I think we may want to make the request_consumer into an actual daemon, rather than starting with with nohup.
2021-05-07 12718, 2021
alastairp
unit tests are passing in github, but failing for me (`self.listenstore.insert(listens)` -> AttributeError: 'NoneType' object has no attribute 'insert'), looking into why
2021-05-07 12753, 2021
ruaok
I'll get moving on the other comments.
2021-05-07 12734, 2021
_lucifer
ruaok, sounds interesting. but i do not know how to do that. let's discuss it in detail later when you are available.
2021-05-07 12742, 2021
ruaok
+1
2021-05-07 12719, 2021
alastairp
_lucifer: in fact, the app startup process inside our docker (service files, etc) is exactly the same process as this, so we could reuse the mechanism
Oh, just seeing your comment reosarevok are you on this?
2021-05-07 12727, 2021
reosarevok
Mr_Monkey (and Freso): I blocked their jira editing privs (if I'm right about how that works) because just deleting the user makes it so that they can re-add it immediately
2021-05-07 12732, 2021
alastairp
reosarevok: it looks like it's not fixed yet. leave open (or review submittedn) thanks
2021-05-07 12733, 2021
reosarevok
But this is the second time they do this
2021-05-07 12744, 2021
reosarevok
So, we could use a better way
2021-05-07 12751, 2021
reosarevok
They also tried to spam our github at some point
2021-05-07 12717, 2021
Mr_Monkey
Thanks for dealing with it. Can't imagine what they're trying to get out of that.
2021-05-07 12732, 2021
ruaok
I dont understand most spammers.
2021-05-07 12708, 2021
reosarevok
Chances are this specific one just has some sort of mental issue? The whole thing is super rambling. But who knows
2021-05-07 12725, 2021
reosarevok
Anyway, if you see it happen again, please do ping me, Freso or yvanzo ASAP
2021-05-07 12734, 2021
reosarevok
Any of us should be able to deal with it
2021-05-07 12740, 2021
alastairp
Mr_Monkey: I'm seeing some weird frontend interaction issues with the new time ranges PR
2021-05-07 12715, 2021
alastairp
after loading some listens from spotify. I'm looking into it a bit more, but I'm not sure if it's due to the time range storage/redis, or something in the frontend code
2021-05-07 12745, 2021
alastairp
(local development version)
2021-05-07 12724, 2021
ruaok
try deleting the redis volume and restarting.
2021-05-07 12744, 2021
alastairp
trying that
2021-05-07 12737, 2021
ruaok
first bits of feedback fixes pushed.
2021-05-07 12738, 2021
alastairp
that doesn't seem to be it - I think it's mostly to do with the older/oldest newer/newest buttons. it's making the queries to get the updated data, but doesn't seem to be displaying it
2021-05-07 12747, 2021
ruaok
Mr_Monkey: alastairp: https://vacunacovid.catsalut.gencat.cat/ is now starting to have useful info. Even though I fit the age bracket that is being vaccinated now, I tried for a cita and got a: Your group is hasn't been called yet. I'll keep trying.
ruaok: this is what I'm trying to work out with the frontend - now that I see some problems with the frontend I'll try and use the API to test the listenstore independently
2021-05-07 12714, 2021
ruaok
what problems are you seeing? anything I can help with?
