I have six tickets assigned to me, which looks correct (I expected 5 but there's two for amazon cover art)
2022-03-14 07316, 2022
bitmap
right now I'm updating the descriptions or leaving comments to summarize the schema changes and how they affect mirrors - let's do that for any tickets we're assigned
reosarevok: yvanzo: are there any schema changes we need to discuss that were decision-required? whether in or outside of that fix version
2022-03-14 07326, 2022
reosarevok
I added the moods bit to that schema change fix version
2022-03-14 07336, 2022
yvanzo
bitmap: About AC MBIDs, it seems pretty clear except the merge. It can be discussed later on thought, mostly implementation details.
2022-03-14 07333, 2022
reosarevok
The main decision needed there is "should moods be its own entity just like genres are, or a hardcoded list of tags without MBIDs"
2022-03-14 07320, 2022
reosarevok
The only reason to go for the latter is because I understand mayhem and monkey are expecting a much smaller list that is controlled by MeB and without user requests, so avoiding the whole new-entity thing might be easier
2022-03-14 07337, 2022
reosarevok
That said, literally copying genre might still be easier just because of the less thinking required... :)
2022-03-14 07342, 2022
bitmap
yvanzo: ok, let's discuss merging at some point
2022-03-14 07323, 2022
CatQuest
bitmap: thnaks, it is mostly copy-pasta of your comments in the great "create tickets of stuff from the summit" doc tho :D
2022-03-14 07331, 2022
yvanzo
bitmap, reosarevok: aliases MBIDs should be dropped for now, until clarified.
2022-03-14 07345, 2022
reosarevok
I'm ok with that
2022-03-14 07309, 2022
bitmap
reosarevok: hard to say -- I feel like a hard-coded list would be easier until we're sure how the entities should work
2022-03-14 07315, 2022
yvanzo
(removed it)
2022-03-14 07337, 2022
bitmap
CatQuest: good copy-pasta then :D
2022-03-14 07339, 2022
reosarevok
bitmap: well, it'd just copy the genre code anyway, so it would work in the same way :)
2022-03-14 07353, 2022
reosarevok
I'm just not sure whether that's overkill or not
2022-03-14 07318, 2022
CatQuest
I think I also asked reo for a link to your presnetation since I remember you had made one
2022-03-14 07317, 2022
bitmap
I assume mbids and aliases would be useful to moods then?
2022-03-14 07332, 2022
monkey
Yes :)
2022-03-14 07348, 2022
monkey
Certainly aliases at least
2022-03-14 07302, 2022
reosarevok
Speaking of aliases, one thing we did talk about was adding edit support for genres
bitmap: any reason *not* to add an edit_genre table, even if we don't yet use it, which can later be used for edits adding genres and genre aliases?
2022-03-14 07355, 2022
CatQuest
oh there isn't an actual interface
2022-03-14 07307, 2022
CatQuest
yea that shoudl be in, makes it easier for reo to add these faster
2022-03-14 07318, 2022
reosarevok
CatQuest: adding a genre now works like adding a medium format type or whatever, there's an edit interface but it doesn't create an edit
2022-03-14 07324, 2022
reosarevok
(so no edit history / notes / whatnot)
2022-03-14 07325, 2022
CatQuest
huh
2022-03-14 07329, 2022
yvanzo
Is there a design document about moods? or a state of the art? or anything?
2022-03-14 07340, 2022
CatQuest didn't know that (sicne can't edit those :D)
2022-03-14 07346, 2022
CatQuest
I'm for creating edits of these tho
2022-03-14 07348, 2022
CatQuest
all these
2022-03-14 07350, 2022
bitmap
reosarevok: we should definitely add edit_genre before the edits are implemented
2022-03-14 07355, 2022
reosarevok
bitmap: sure, just making sure that implementing edits seems sensible and that there's no harm done in just adding edit_genre now so we can work on that without another schema change
2022-03-14 07313, 2022
reosarevok
Can you check the schema for a sec and make sure that I'm right that'd be the only needed addition?
2022-03-14 07327, 2022
reosarevok
If so, I'll make a ticket
2022-03-14 07319, 2022
bitmap
just the table + index and FKs should be enough
2022-03-14 07327, 2022
monkey
yvanzo: Not really, just a lot of notes I made. But I could assemble a document
2022-03-14 07353, 2022
reosarevok
monkey: it wouldn't hurt, for when I work on it, although I expect it won't change much of the MBS implementation
2022-03-14 07311, 2022
reosarevok
bitmap: Ok. I see a few of the edit_$entity have a materialized status column, but most do not. Do you know why?
2022-03-14 07344, 2022
bitmap
it would added to improve the performance of those but I'm not sure how relevant it is in our postgres anymore
right. for SEARCH, at least the series ordering_attribute, moods, and artist credit mbids
2022-03-14 07302, 2022
zas
atj: new server is up and ready, named rudi
2022-03-14 07305, 2022
CatQuest
:O
2022-03-14 07359, 2022
zas
bitmap: I noticed incorrect partition sizes on rakim (my fault), for now, not much an issue, but I'd like to resize them at some point, we'll need to move stuff from it elsewhere, we can use new server rudi for that (it's the one half of the future gateways), it is perhaps time to move to keydb and have some redundancy for those redis instances
2022-03-14 07339, 2022
zas
CatQuest: rudi is first half of new gateways, second half will be rex ;)
2022-03-14 07341, 2022
bitmap
heh, ok :) I can perhaps look into setting up a keydb cluster this week
2022-03-14 07345, 2022
CatQuest
i noticed that
2022-03-14 07348, 2022
CatQuest
cool
2022-03-14 07349, 2022
CatQuest
:D
2022-03-14 07350, 2022
Revenant has quit
2022-03-14 07335, 2022
zas
bitmap: great, I can give you an hand on this, as I configured redundant instances for gateways
2022-03-14 07332, 2022
zas
one thing I didn't solve is dockerized haproxy + keepalived, which are used to provide automatic failover. I fall back on deploying them on host directly, don't remember what was the issue exactly though. We can have a look at this too.
2022-03-14 07328, 2022
bitmap
sounds good, I'll ping you when I start looking at it
2022-03-14 07350, 2022
reosarevok
bitmap: so, we'll want an update that adds edit_genre, pk and index for all, and one that adds the fks for standalone, right?
2022-03-14 07322, 2022
bitmap
reosarevok: yup
2022-03-14 07304, 2022
reosarevok
bitmap: oh! I remembered the other thing that I wanted to add for genres. rel tables, so we can at least relate them to urls
2022-03-14 07327, 2022
reosarevok
I guess I would need to add the whole set just in case though, not just l_genre_url
2022-03-14 07320, 2022
bitmap
ah okay. if you can create a ticket and commit to working on that then it sounds fine
2022-03-14 07323, 2022
skelly37 joined the channel
2022-03-14 07355, 2022
reosarevok
Sure
2022-03-14 07300, 2022
bitmap
and we do have l_url_url so I think you are right, heh
2022-03-14 07307, 2022
reosarevok
bitmap: MBS-11599 also mentions annotations, while I'm not sure how useful those are rn, I guess we might as well do that too - that would bring genre tables mostly to parity with other entities, I think
No mailed in reviews, so people up: zas, monkey, atj, lucifer, yvanzo, akshaaatt, reosarevok, alastairp, bitmap, mayhem, CatQuest, Freso – anyone else who wish to give review, let me know ASAP!
2022-03-14 07303, 2022
Freso
zas: Go!
2022-03-14 07309, 2022
zas
Hey
2022-03-14 07340, 2022
Sophist-UK joined the channel
2022-03-14 07353, 2022
zas
Apart usual Picard PR reviews, edits, general maintenance and user support, I configured new server rakim
2022-03-14 07356, 2022
aerozol
Moin!
2022-03-14 07310, 2022
zas
and ordered another one to replace yehudi, we just received it
2022-03-14 07313, 2022
CatQuest
aw
2022-03-14 07318, 2022
CatQuest
morena aerozol
2022-03-14 07330, 2022
zas
yehudi is now retired, and new server was configured, welcome to rudi
2022-03-14 07339, 2022
zas
fin. monkey ?
2022-03-14 07300, 2022
monkey
hi!
2022-03-14 07334, 2022
monkey
Last week was quite busy, with more refactoring on LB and deploying/ announcing the new BrainzPlayer on beta
