[00:00:27] (CR) jerkins-bot: [V: -1] Test unravelling the ci db pre-create [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651648 (owner: Eileen) [00:05:38] (PS2) Eileen: Test unravelling the ci db pre-create [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651648 [00:07:59] (CR) jerkins-bot: [V: -1] Test unravelling the ci db pre-create [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651648 (owner: Eileen) [00:12:24] (PS3) Eileen: Test unravelling the ci db pre-create [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651648 [00:17:34] (CR) jerkins-bot: [V: -1] Test unravelling the ci db pre-create [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651648 (owner: Eileen) [00:24:11] (PS4) Eileen: Test unravelling the ci db pre-create [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651648 [00:24:28] (PS5) Eileen: Test unravelling the ci db pre-create [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651648 [00:24:40] (CR) jerkins-bot: [V: -1] Test unravelling the ci db pre-create [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651648 (owner: Eileen) [00:24:45] (PS6) Eileen: Test unravelling the ci db pre-create [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651648 [00:32:29] (CR) jerkins-bot: [V: -1] Test unravelling the ci db pre-create [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651648 (owner: Eileen) [02:31:37] Wikimedia-Fundraising-Banners: [BETA] QA of Birthday Banners 2021 - https://phabricator.wikimedia.org/T268153 (Shizhao) In Firefox 84, open https://en.wikipedia.org/w/index.php?title=NASA&banner=B2021_181120_WP20_control&uselang=en&force=1 Browser prompt: > Content Security Policy violation detected! Trie... [03:07:36] (PS1) Eileen: Fix install of wmf reports such that they are managed by .mgd files [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651675 (https://phabricator.wikimedia.org/T270684) [03:10:48] (Abandoned) Eileen: Remove legacy method of adding to paths [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651641 (owner: Eileen) [03:11:29] (CR) Ejegg: [C: +2] "Deletion looks good, though commit message ends on a cliffhanger" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651310 (owner: Eileen) [03:11:44] (CR) Ejegg: [C: +2] "Aww, somebody had high hopes for this. Seems like it would be a bit impractical at today DB size even if it worked." [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651311 (https://phabricator.wikimedia.org/T270684) (owner: Eileen) [03:12:22] (CR) Eileen: "Will roadrunner survive. Will the acme tnt explode. Tune in next week" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651310 (owner: Eileen) [03:13:37] (PS2) Eileen: Remove export UI extension [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651642 [03:13:39] (PS2) Eileen: Fix install of wmf reports such that they are managed by .mgd files [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651675 (https://phabricator.wikimedia.org/T270684) [03:21:02] (PS1) Eileen: Remove pseudo-bao from gateway reconcilliation [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651676 (https://phabricator.wikimedia.org/T270684) [03:29:08] (Merged) jenkins-bot: Remove legacy 'contribution tracking' remnant from wmf_reports [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651310 (owner: Eileen) [03:29:10] (CR) jerkins-bot: [V: -1] Remove attempt to add Contacts & Contributions export [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651311 (https://phabricator.wikimedia.org/T270684) (owner: Eileen) [03:38:33] (CR) Eileen: "recheck" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651311 (https://phabricator.wikimedia.org/T270684) (owner: Eileen) [04:26:37] (PS2) Eileen: Remove pseudo-bao from gateway reconcilliation [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651676 (https://phabricator.wikimedia.org/T270684) [04:26:39] (PS1) Eileen: Move gateway reconcilliation report into wmf-civicrm [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651681 (https://phabricator.wikimedia.org/T270684) [04:28:02] Fundraising Sprint Zeitgeistbusters, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Fr-drupal-upgrade-2021, Patch-For-Review: Determine if wmf_reports is used by anyone & possibly remove - https://phabricator.wikimedia.org/T270684 (Eileenmcnaughton) I have removed most of the code in thi... [04:46:16] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Fr-drupal-upgrade-2021: Try to remove wmf_common_date_unix_to_civicrm - https://phabricator.wikimedia.org/T270753 (Eileenmcnaughton) [05:00:42] PROBLEM - check_puppetrun on civi1001 is CRITICAL: CRITICAL: Puppet has 2 failures. Last run 5 minutes ago with 2 failures. Failed resources (up to 3 shown): Package[openssl],Package[python-libxml2] [05:02:34] that puppet alarm is me. it's being held up by the updates running. [05:04:02] just so happens to be running during the daily virus scan time. [07:05:36] PROBLEM - check_puppetrun on civi1001 is CRITICAL: CRITICAL: Puppet has 2 failures. Last run 3 minutes ago with 2 failures. Failed resources (up to 3 shown): Package[openssl],Package[python-libxml2] [07:30:42] RECOVERY - check_puppetrun on civi1001 is OK: OK: Puppet is currently enabled, last run 1 minute ago with 0 failures [08:04:29] (CR) DannyS712: "looked at everything, a few issues" (5 comments) [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/650975 (owner: Umherirrender) [08:33:00] (PS56) AndyRussG: Campaign filtering [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/604279 (https://phabricator.wikimedia.org/T268646) (owner: AlexPinchuk) [08:45:23] (CR) AndyRussG: "Changes in patch sets 55 and 56" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/604279 (https://phabricator.wikimedia.org/T268646) (owner: AlexPinchuk) [09:08:37] (CR) AndyRussG: "This patch set just consolidates some repeated code in ext.centralNotice.display/index.js. Thanks!!" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/607461 (https://phabricator.wikimedia.org/T268646) (owner: ItSpiderman) [09:10:32] Fundraising Sprint Xtreme Lolcats, Fundraising Sprint Yellow hornets of kindness and healing, Fundraising Sprint Zeitgeistbusters, Fundraising-Backlog, and 2 others: Code review and release management for CN campaign filtering - https://phabricator.wikimedia.org/T268646 (AndyRussG) >>! In T268646... [14:41:51] (CR) Ejegg: [C: +2] Remove export UI extension [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651642 (owner: Eileen) [14:48:50] Fundraising Sprint Yellow hornets of kindness and healing, Fundraising Sprint Zeitgeistbusters, Fundraising-Backlog: on endowment payments page, change ways to give link - https://phabricator.wikimedia.org/T269716 (Ejegg) Open→Resolved [14:48:57] Fundraising Sprint Yellow hornets of kindness and healing, Fundraising Sprint Zeitgeistbusters, Fundraising-Backlog: Apply Mediawiki 1.35.1 update to payments-wiki - https://phabricator.wikimedia.org/T270507 (Ejegg) Open→Resolved [14:49:47] Fundraising Sprint Yellow hornets of kindness and healing, Fundraising Sprint Zeitgeistbusters, Fundraising-Backlog, MW-1.36-notes (1.36.0-wmf.22; 2020-12-15), Patch-For-Review: on endowment payments page, change FAQ link - https://phabricator.wikimedia.org/T269717 (Ejegg) Open→Resolved [14:59:26] (Merged) jenkins-bot: Remove export UI extension [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651642 (owner: Eileen) [15:22:39] (CR) Ejegg: "So for deploy, we make sure to run drush updb before running any Civi extension updates?" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651675 (https://phabricator.wikimedia.org/T270684) (owner: Eileen) [15:24:11] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Importing Activity Updates - https://phabricator.wikimedia.org/T219191 (CaitVirtue) @DStrine Can you give us a sense of when Fr-Tech will be able to asses this? Even just knowing that (ie not even when this work could possibly be done) will help us wit... [16:14:56] (PS4) Umherirrender: Add missing @var and improve documentation [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/650975 [16:15:08] (CR) Umherirrender: Add missing @var and improve documentation (4 comments) [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/650975 (owner: Umherirrender) [16:45:54] Fundraising Sprint Zeitgeistbusters, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Fr-drupal-upgrade-2021, Patch-For-Review: Determine if wmf_reports is used by anyone & possibly remove - https://phabricator.wikimedia.org/T270684 (DStrine) Gateway reconciliation is probably still used b... [16:51:40] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Importing Activity Updates - https://phabricator.wikimedia.org/T219191 (DStrine) Sorry I didn't see the previous comments on this. I'll talk with the team today. [17:16:20] Fundraising Sprint Zeitgeistbusters, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Fr-drupal-upgrade-2021, Patch-For-Review: Determine if wmf_reports is used by anyone & possibly remove - https://phabricator.wikimedia.org/T270684 (EMartin) Confirming that I do indeed use the Gateway rec... [17:17:30] (PS1) Ejegg: Civi ext to batch update activities [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651808 (https://phabricator.wikimedia.org/T219191) [17:21:49] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Patch-For-Review: Importing Activity Updates - https://phabricator.wikimedia.org/T219191 (DStrine) We could get this done today. @CaitVirtue @NNichols @jkim_wikimedia I'm trying to ping Nora and Jerry on fr-all in slack. I'm not aware of a MG channe... [17:41:22] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Patch-For-Review: Importing Activity Updates - https://phabricator.wikimedia.org/T219191 (Ejegg) I enabled the extension suggested by @Eileenmcnaughton on staging, however I'm not sure it's going to do what's necessary. According to the doc at http... [17:45:06] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Patch-For-Review: Importing Activity Updates - https://phabricator.wikimedia.org/T219191 (Ejegg) @jkim_wikimedia and @RLewis, want to see if you can find data on staging to test with? You would do the search under Search->activities, from the resul... [17:54:08] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Patch-For-Review: Importing Activity Updates - https://phabricator.wikimedia.org/T219191 (DStrine) I am hearing from MG that this doesn't have to be done right now or over the break. @jkim_wikimedia will get the list onto the server and we'll take a... [17:55:12] Wikimedia-Fundraising-Banners: QA for I already donated variant - https://phabricator.wikimedia.org/T270658 (jbolorinos-ctr) Screenshot Test Results - Desktop: https://app.crossbrowsertesting.com/public/i15b74a2dc92badb/screenshots/zbf86eab97bebd227336 [18:06:42] (Abandoned) Zfilipin: WIP The first sample Selenium test in Node.js [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/443093 (https://phabricator.wikimedia.org/T262684) (owner: Zfilipin) [18:48:27] Wikimedia-Fundraising-Banners: Thermometer variant - https://phabricator.wikimedia.org/T270719 (jbolorinos-ctr) Screenshot Test Results - Desktop: https://app.crossbrowsertesting.com/public/i15b74a2dc92badb/screenshots/zed664debeb4dcade7c6 [18:48:55] Wikimedia-Fundraising-Banners: QA for I already donated variant - https://phabricator.wikimedia.org/T270658 (jbolorinos-ctr) Open→Resolved No blockers found. Signing off on QA for this variant @spatton This banner is now READY TO TEST! [20:06:06] Fundraising Sprint Zeitgeistbusters, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Fr-drupal-upgrade-2021, Patch-For-Review: Determine if wmf_reports is used by anyone & possibly remove - https://phabricator.wikimedia.org/T270684 (Eileenmcnaughton) Re the Lybunt report - there are 2 lyb... [20:08:00] (CR) Eileen: "Yeah - my expectation is that the extension update will give an error until this has run or create duplicate option values (depending on w" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/651675 (https://phabricator.wikimedia.org/T270684) (owner: Eileen) [20:24:56] ejegg: so that extension you tried for MG - it does updates & they want to add activities? [20:25:04] or what was the limitation? [20:26:30] eileen: that's what I thought from how dstrine explained their needs in standup [20:27:02] yeah - I was just pondering it - is it a mix of add / update - or just that they want to add? [20:27:17] Sounds like a mix of add and update [20:29:22] ah & is it a case where they won't know if the contact has an activity? ie they would want to select a bunch & it would be lookup & then add or update [20:34:37] eileen: really not sure - we need to clarify with MG but apparently it's not so urgent as to do it today [20:34:55] yeah I just wanted to be thinking about it [20:37:48] eileen: those mgd files won't have any issues autoloading what they need to run CRM_Core_Component::getComponentID('CiviContribute') , right? [20:38:09] no - they are loaded in a civi context [20:38:12] I mean, they're always included from something that's fully bootstrapped [20:38:15] cool cool [20:38:27] from a hok [20:38:29] hook [20:38:34] Wikimedia-Fundraising-Banners: Thermometer variant - https://phabricator.wikimedia.org/T270719 (jbolorinos-ctr) No blockers found! Signing off on QA for this variant @spatton [20:41:57] fr-tech, hope you all a have great holiday period and a happy new year. See you on the other side! all the best and bye for now! :) [20:42:15] jgleeson|away :D same!!! [20:43:35] happy holidays jgleeson|away! [20:55:18] same jgleeson|away !! [21:17:26] Wikimedia-Fundraising-Banners: Thermometer variant - https://phabricator.wikimedia.org/T270719 (jbolorinos-ctr) Open→Resolved [21:30:49] (PS1) QChris: Allow “Gerrit Managers” to import history [wikimedia/fundraising/crm/civiproxy] (refs/meta/config) - https://gerrit.wikimedia.org/r/651841 [21:30:51] (CR) QChris: [V: +2 C: +2] Allow “Gerrit Managers” to import history [wikimedia/fundraising/crm/civiproxy] (refs/meta/config) - https://gerrit.wikimedia.org/r/651841 (owner: QChris) [21:31:05] (PS1) QChris: Import done. Revoke import grants [wikimedia/fundraising/crm/civiproxy] (refs/meta/config) - https://gerrit.wikimedia.org/r/651842 [21:31:07] (CR) QChris: [V: +2 C: +2] Import done. Revoke import grants [wikimedia/fundraising/crm/civiproxy] (refs/meta/config) - https://gerrit.wikimedia.org/r/651842 (owner: QChris) [21:47:22] Wikimedia-Fundraising-Banners: QA for donation amount animations - https://phabricator.wikimedia.org/T270659 (jbolorinos-ctr) Screenshot Test Results - Desktop: https://app.crossbrowsertesting.com/public/i15b74a2dc92badb/screenshots/z4aeb897e5d11a0d060b [21:48:41] Wikimedia-Fundraising-Banners: M Sm - Swipeable variant - https://phabricator.wikimedia.org/T270718 (jbolorinos-ctr) Screenshot Test Results - Mobile: https://app.crossbrowsertesting.com/public/i15b74a2dc92badb/screenshots/z02df4553c12936cc15a [21:54:25] Wikimedia-Fundraising-Banners: M Sm - Swipeable variant - https://phabricator.wikimedia.org/T270718 (jbolorinos-ctr) It's a little awkward that only the top part swipes when this banner is expanded imo. @scervantes would there be anyway to make the expanded version of the banner swipeable instead? Or would... [21:55:14] (PS57) AndyRussG: Campaign filtering [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/604279 (https://phabricator.wikimedia.org/T268646) (owner: AlexPinchuk) [21:57:39] Wikimedia-Fundraising-Banners: [BETA] QA of Birthday Banners 2021 - https://phabricator.wikimedia.org/T268153 (jbolorinos-ctr) @Shizhao that's due to this banner currently being in the staging environment, that popup message will not appear on production. Thanks for adding the note though! And please feel... [22:11:05] (PS26) AndyRussG: Control displaying campaigns based on user preferences [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/607461 (https://phabricator.wikimedia.org/T268646) (owner: ItSpiderman) [22:12:23] (CR) AndyRussG: "Removed unneeded FIXME in CentralNoticeTestFixtures.php, and also smoke tested campaign fallback with opt-out (seems to work fine). Thanks" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/607461 (https://phabricator.wikimedia.org/T268646) (owner: ItSpiderman) [22:16:06] (CR) AndyRussG: Campaign filtering (1 comment) [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/604279 (https://phabricator.wikimedia.org/T268646) (owner: AlexPinchuk) [22:55:03] AndyRussG: oh shoot, patch-notice-type-log.sql is giving me a Error 1118: Row size too large. The maximum row size for the used table type, not counting BLOBs, is 8126. [22:55:15] does it let you run the update.php without error locally? [22:56:12] looks like most of the rest of that table is int or blob [22:56:27] sorry, I'll put this in gerrit to be properly async [22:57:37] ejegg: hey [22:57:59] no worries, lemme check [22:58:12] I'mma try with 'blob' and make sure that works [22:58:43] ejegg: I guess I didn't smoke test that exact file, since I installed fresh with the new sql so it should have run CentralNotice.sql [22:58:49] ejegg: what database backend are u using? [22:59:03] mysql [23:00:11] ejegg: found it, there's a comma at the end of the line 2 [23:00:14] instead of a semicolon [23:00:34] ooh [23:00:59] hmm? [23:01:09] ejegg: ahh no that's not wrong [23:01:12] wait, you can't add two columns in the same alter? [23:01:19] I'm pretty sure it IS the row size [23:01:37] that log table uses blob or text for all the other text-y columns [23:01:42] hmm, except comment [23:02:06] not sure why it didn't complain on a fresh install tho [23:02:16] ejegg: I mean the meain CentralNotice.sql is identical [23:02:39] I'll try wiping this clean, installing the CN stuff from master, then updating to this path and update.php-ing [23:02:45] one sec [23:02:48] * AndyRussG hugs docker [23:07:18] ejegg: K I un-hug docker 'cause it's giving me errors now, I guess 'cause I updated the computer while the containers were running? gonna try restarting ahhh! [23:12:51] oh phoo [23:13:34] well let me see if my local is the same as on prod [23:15:00] ejegg: working fine for me! [23:15:02] Adding not_type field to table cn_notices ...done. [23:15:04] Adding notlog_begin_type field to table cn_notice_log ...done. [23:15:36] (for docker restarting the computer fixed it) [23:16:03] hmm hmm, locally I seem to have varbinary(255) for not_name, begin/end_projects rather than varchar [23:16:29] when you say 'describe cn_notice_log' does it say varchar there for you? [23:16:38] hmmm lemme see [23:16:58] feels like those should take the same space tho [23:17:42] oh.. "varbinary uses less space than varchar if multi-byte collation is used for the varchar column" so it shouldn't be bigger for me [23:18:09] ejegg: huh also I get varbinary for everything [23:19:05] ejegg: https://paste.toolforge.org/view/e6489c12 [23:22:04] ejegg: maybe paste your describe results there for comparison? [23:22:15] which version of mariadb r u on? [23:22:21] also r u on Docker or bare metal? [23:22:53] AndyRussG: I just compared and mine is the same for all [23:22:55] just a diff order [23:23:04] and without the type columns [23:23:10] ints are (11) etc [23:23:34] bare metal, one sec for the maria version [23:23:51] Server version: 10.3.24-MariaDB-2 [23:24:01] hmm, that's a bit old, no? [23:24:28] what's prod on, 10.5? [23:24:47] heh, i assumed I had the latest, being on deb experimental [23:25:07] but I guess debian doesn't bump db server versions unless explicitly told to? [23:26:59] ejegg: huh no idea... the Docker-based mariadb I'm running is 10.4.17 [23:27:18] ejegg: wanna try on Docker? [23:27:45] k, will try that [23:27:48] yeah. prod is 10.4. haven't made the jump to 10.5 yet. [23:27:55] ok, thanks dwisehaupt [23:28:18] that'll be a big one as the binaries change names so we'll have to update some ancillary stuff to handle it. [23:28:47] oh fun [23:29:03] erf [23:29:09] ejegg: k lmk if there are tips I can share? haven't yet documented CN on docker [23:29:46] thanks! Guess I'll just use the mediawiki docker & link my cn checkout in to the extensions dir + add the LocalSettings bits? [23:30:56] ejegg hmmm lemme just paste stuff into an etherpad [23:32:46] ejegg: https://etherpad.wikimedia.org/p/centralnotice-docker [23:32:51] (still filling it in) [23:41:38] hmm, InnoDB strict mode disabled looks like it will let you alter the column [23:41:48] but then might throw an error later on insert [23:41:55] https://mariadb.com/kb/en/troubleshooting-row-size-too-large-errors-with-innodb/ [23:43:41] so that's ON in my db [23:44:00] ejegg: huh... I have no trouble with inserts in that table either [23:44:22] says the inserts can throw errors if the data is too large [23:44:47] so maybe a row where all the binaries are full etc? [23:45:02] maxlength comment [23:45:14] dunnnooooo [23:45:25] ejegg: finished writing out the Docker instructions on the etherpad [23:45:39] k, i'mma check if that var is on for the docker db [23:45:48] There are also instructions for resetting everything [23:45:49] and thanks! [23:45:54] :) thank u! [23:46:12] You can also try the update.php with CN first on master, then switching to the new changes and running update.php again [23:46:23] And also just running update.php directly on the new changes [23:46:28] to test the different versions of the SQL [23:49:38] ejegg: btw another thing to note, when it actually comes time to modify the schema on prod, it will be done by a person deeply familiar with prod db stuff now [23:49:45] it's like a special thing we book [23:50:04] and they'll actually modify some of the details of the schema because, prod [23:50:42] also this'll go to the beta cluster which will smoke test the schema changes for us, I assume on the same version of stuff as prod? [23:51:14] yep, that's always nice [23:51:46] still, if we're relying on strict mode being off for something to work I'm a bit suspicious of that [23:52:03] k, containers up [23:52:05] installing [23:53:00] hmmm ok fair point [23:53:01] hmm, can't access the database [23:53:18] on that docker-composer exec -w line [23:53:38] did you do the mkdir dbdata¡ [23:53:40] ? [23:53:44] aha, nope [23:53:52] K gotta remove that as root now [23:54:05] yarp [23:54:19] imagine I want to bring down the containers first [23:54:47] ah yeee [23:55:34] nice, that was it [23:55:57] hmm, should that install script return my cli when it's done? [23:55:57] so, even though the containers are set to run as your local user, if the host target of the bind mount doesn't exist, it'll still get created as owned by root [23:56:08] yep! [23:57:07] k, that works. I wonder why it doesn't like my Vector symlink [23:57:15] ahh, probs relative path [23:57:23] ejegg: ahh symlinks yeah don't work as expected on Docker [23:57:37] k, will just check it out again [23:57:57] or hardlink [23:58:35] or can you do that with dirs? [23:58:50] nope [23:58:58] fresh checkout it is [23:59:45] hmmmm