bitmap: I just found out that login code is not working at the moment just because musicbrainz_db is not yet created
The code should have use musicbrainz_selenium instead right ?
That is probably the reason I was facing the user exist even thought I reset the databases except musicbrainz_db. Let me know how do I make plackup use musicbrainz_selenium instead of musicbrainz_db
bitmap
yes... do you have sub USE_SET_DATABASE_HEADER { 1 } in DBDefs.pm?
Cyna
Yes
bitmap
how did you determine it's using musicbrainz_db?
Cyna
When I ran the test with plackup working... It ran the first test which when clicked on signup and gave the following error
oh! chrome broke something recently where they don't send localhost requests through any configured proxy anymore
it works for me since I just started using a host alias for localhost with plackup etc.
I'm trying to see if I can get this proxy-bypass-list option working on the spawned chrome instance
Cyna
ohh
rahul24 joined the channel
rahul24 has quit
shivam-kapila joined the channel
c1e0 joined the channel
bitmap
anyway in the meantime you can get it working by adding something like `127.0.0.1 mycoolpc` to /etc/hosts and starting plackup with --host mycoolpc or whatever hostname you use
replace localhost:5000 with that hostname in DBDefs.pm too
proxy-bypass-list doesn't want to work here so I'm trying to come up with a better solution
Cyna
ohh I'll try making the change
yvanzo
hi Cyna: which “performance issues” did you experience?
Cyna
Latency on each requests
I used containers... Not the current mb-docker support
iliekcomputers
ruaok: we should seriously consider merging labs and listenbrainz. the fact that we have to open two different pull requests in two different repos to build any actual feature is a PITA.
Gazooo has quit
Gazooo joined the channel
BrainzGit
[listenbrainz-server] paramsingh opened pull request #691 (master…param/merge-labs-into-listenbrainz): [wip] Merge listenbrainz-labs into listenbrainz https://github.com/metabrainz/listenbrainz-serv...
MBS-10572: Pages that display release events trigger an error when a non-English UI language is selected: "Domain `countries` was not found." https://tickets.metabrainz.org/browse/MBS-10572
Cyna
bitmap: using a different name worked
running selenium test suite is seriously a joke :(
i am in total favour of merging this and archiving labs, it is annoying to have to change code in two different repositories and get two pull requests reviewed for what is almost always one piece of work.
surprised that labs is only around 4.4k LOC
Nyanko-sensei has quit
shivam-kapila has quit
sumedh has quit
c1e0 has quit
cleo joined the channel
cleo is now known as Guest34712
Nyanko-sensei joined the channel
shivam-kapila joined the channel
ruaok
iliekcomputers: ok, let's do it.
iliekcomputers
🥳🥳
CODEin_JDB joined the channel
all of the code in the PR is from labs and is code reviewed. this is the commit that fixes tests and seperates the manage.py files for spark and server (https://github.com/metabrainz/listenbrainz-serv...)
CODEin_JDB
Hey CatQuest can i make a whole new family tree about sri lankan intruments.
> <iliekcomputers> We'd have to add labs as a remote and merge the branch in progress to master:
duh. forgetting how git works, lol.
iliekcomputers
sure, let's wait for pristine__.
ruaok approves the PR
ruaok
since its all previously reviewed code, I'm good with it.
iliekcomputers
👍🏽
pristine__
Hey. So the I think that all-changes-mapping and master diverge. We may have merge conflicts and test fails after merging it into master. Otherwise it is to merge
fine*
iliekcomputers
pristine__: i'll fix the merge conflicts after merging this.
pristine__
Cool
iliekcomputers
pristine__: ok to merge?
thanks for the reviews on a weekend, pristine__ and ruaok :)