[13:11:44] [1/3] WikiCreate seems to be acting up again. The last two wikis created did not get the main page boiler plate and their requestors were not given bureau rights. [13:11:44] [2/3] scpsecretlabde [13:11:45] [3/3] thetrawlerproject [13:21:19] Huh [13:21:22] > [03c60b088c3360eb1dc6faa7] 2024-01-07 13:20:57: Fatal exception of type "MediaWiki\Revision\RevisionAccessException" [13:21:28] https://meta.miraheze.org/wiki/Meta:Wiki_creators?diff=prev&oldid=349789 [13:30:44] Do we no longer use https://cvt.miraheze.org/wiki/CVT_action_log? [13:31:23] @reception123 I'm picking up train ticket then python [14:31:41] thanks! [14:31:54] We _should_ but given that there's not really been much CVT activity since June... [14:32:12] For some reason that seems to ocasionally happen and no one has really figured out how to fix [14:32:49] Hm [14:32:57] Good to know if [14:33:26] If the four nominations for Steward are succesful this will hopefully change [14:33:37] Hopefully [14:34:10] I know there was discussion about running a script later today? Should we wait until after that before creating any more wikis? [14:34:22] (Technically there is a successful steward request, but it can’t close till NDA is done) [14:34:33] I'm not sure which script? If you mean the deletion script that's been running for a while already and is unrelated [14:35:39] [1/2] Im also hoping to try migrate one of fandoms anti vandalism rc tool things here(compared edits to a model feed on edits to score how likely it’s vandalism and notify CVT [14:35:40] [2/2] https://cdn.discordapp.com/attachments/1006789349498699827/1193563607284334704/image0.jpg?ex=65ad2bbb&is=659ab6bb&hm=f52cf10005d3363c61523ab663cc8c376aaa6320a6bb1717c2177461d2ffdc07& [14:35:55] Might be somewhat useful [14:35:57] No ide [14:36:00] oh that's great! We definitely need better CVN tools [14:36:10] and easier reporting for users is exactly one aspect [14:36:43] You wouldn’t happen to know how to brutal wrangle python libraries? I’ve run docker build more times than I care to admit [14:37:30] Lol [14:37:59] Eh I’ll probably get there eventually [14:38:15] May not be much but I wanna help out, and python is the only language I know so may as well use it [14:38:26] I'm no python expert though maybe RhinosF1 will be able to help [14:38:32] And discord bots happen to be one of my worked areas [14:38:44] I’ll ask him once I get the base code to work [14:38:54] Currently it’s a matter of dependencies [14:39:22] Than I can take a look at the code to see how to change it from fandom to Mira [14:39:23] I'm here [14:39:45] Morning [14:41:18] What python you need help with [14:43:50] Rn its less code and getting libraries installed [14:43:54] I’ll get the repo rq [14:44:14] This is what fandom uses, was hoping to adapt it for us https://github.com/soap-team/cynthia [14:44:16] [1/2] Ok. I thought CosmicAlpha was going to do something after he got some sleep. [14:44:17] [2/2] Should I reopen the phab task? [14:45:11] I have a wip dockerfile for this, and also had to loosen some stuff in requirements.txt. It’s… very messy [14:46:11] it was a problem w/ special page on a single wiki [14:46:12] Is this not the code I said I wouldn't let anywhere near production [14:46:33] When some fandom person offered to help with it at Miraheze [14:46:38] Was it? Was that to me or some else [14:48:27] Someone involved in the soap team [14:49:04] huh [14:49:19] Yes [14:50:02] When they offered it to us, I said I wouldn't let code of that poor standard anywhere near a production Miraheze server [14:50:11] @pixldev [14:51:58] Ah, well good to know ahead of time [14:54:19] Shame, could have been a useful asset [14:55:05] And I am in no way qualified to improve it [14:55:28] I’ve made it substantially worse trying to get it to run [15:19:43] @reception123 why has X-Debug broken? [15:20:47] @rhinosf1 CA made some changes to MHDebug yesterday [15:21:22] He may have broke it [15:21:30] as CI for puppt is complaining [15:23:41] Hmm yeah, we'll probably have to wait for CA to fix that [15:27:07] @reception123 I am currently working on rename-wiki [15:40:50] ok that was a stupid bug [15:43:20] Hm? [15:43:25] me [15:43:28] i made a bug [15:44:29] Smh take a look at my development process and then tell me what you wrote was a stupid bug [15:48:25] just need to fix modules/mediawiki/files/cookbooks/rename-wiki.py:36: error: Incompatible return value type (got "object", expected "str") [return-value] [15:59:34] @cosmicalpha not sure what happened with PageProperties but it's causing some wikis to be completely down - https://phabricator.miraheze.org/T11636 [15:59:40] and even insering SQL doesn't fix [16:18:06] @reception123 is passing CI [16:18:07] https://github.com/miraheze/puppet/pull/3589/files [16:18:18] be very careful testing [16:18:34] there is a guard up so you need to press c and then enter to allow a command [16:18:37] or s/a [16:34:57] Nice! Thanks! [16:35:04] I'll get to it later today [16:57:47] If I may ask, what parts in particular do you take issues with the quality? (Im not an exceptionally adept python developer by any means so curious) [16:58:08] the utter lack of CI [17:09:00] So your issue is with the development process, not the codebase? [17:09:33] poor process leads to poor code [17:10:59] Understandable [17:11:13] oh well [17:11:19] was worth a shot at least [17:12:37] CI is there for a reason [17:12:50] It can catch a lot [17:13:17] From security issues to unexpected error conditions to things that'll change in future versions [17:18:17] yeah the last one is a big one, I spend a good hour trying to get requirements.txt to work probably cause the code is a few years old [17:18:32] I might clone the repo and try to set up CI for the heck of it [17:19:14] Steal what's in puppet for Python [17:19:34] That will have you meet my standard of what's good [17:21:22] Sure thing [17:21:49] ill let ya know how it goes [17:40:25] MacFan4000: wm-bot is dead in #mediawiki [17:43:15] oh... that extension needs disabled pending another security review, I forgot about that. New version of it completely rewrote the extension and even changed the functionality of what the extension is to be something a lot more complex. Not saying it isn't safe but it needs a new review... [17:43:42] MacFan4000: problem is a user error [17:46:32] RhinosF1: looking at channel logs, I see a restricted phab ticket was linked [17:46:47] MacFan4000: yes [17:46:50] User error [19:52:09] @rhinosf1 salt syntax isn't fully right, as salt doesn't like "", only '' it seems [19:52:21] so I think we'll have to use `'` twice and escape it [19:52:32] Okay [19:52:44] What's the exact command at the execute line? [19:57:20] @reception123 [20:03:28] is there a role to ping for #New wiki - no permissions issues? [20:03:38] To grant rights [20:05:18] If it's just permissions then stewards [20:07:52] hmm, looks like it actually doesn't like it permissions wise either [20:07:54] ` ERROR 1045 (28000): Access denied for user 'root'@'localhost' (using password: NO)` [20:18:45] @reception123 that doesn't surprise me [20:19:02] But what does have access [20:25:46] access denied… to root [20:29:29] @paladox is that intentional? (salt not having access to mysql commands) [20:32:17] No [21:33:09] @rhinosf1 fixed thanks to CA's help but now it's being stupid [21:33:11] [1/4] `reception@puppet141:~$ sudo salt-ssh -E 'db131' cmd.run 'sudo -i mysql -e "SELECT wiki_dbcluster FROM mhglobal.cw_wikis WHERE wiki_dbname = 'surpisenerdswiki';"' [21:33:12] [2/4] db131.miraheze.org: [21:33:12] [3/4] ERROR 1054 (42S22) at line 1: Unknown column 'surpisenerdswiki' in 'where clause' [21:33:12] [4/4] ` [21:33:21] that makes no sense... and the command copy/pasted exactly works directly on db131 [21:34:38] ah, must be escaping [21:35:55] Final command: `sudo salt-ssh -E 'db131' cmd.run 'sudo -i mysql -e "SELECT wiki_dbcluster FROM mhglobal.cw_wikis WHERE wiki_dbname = '\''surpisenerdswiki'\'';"' ` [21:36:17] so once the escaping is done in the script we should be good to test again [21:40:00] @reception123 okay feel free to do that [21:42:17] @reception123 feel free to apply your suggestion [21:48:35] Ok, I'll try that tomorrow as I should be more awake for stuff like this [23:00:09] Hmm, if that misbehaves, you can always use c-style strings for cleaner escaping. eg: `salt-ssh -E 'db131' cmd.run $'sudo -i mysql -e "SELECT wiki_dbcluster FROM mhglobal.cw_wikis WHERE dbname = \'surpisenerdswiki\';"'`