[01:34:35] 6Labs, 10Tool-Labs: toolsbeta-webproxy is unreachable - https://phabricator.wikimedia.org/T109853#1679837 (10scfc) 5Open>3Invalid a:3scfc I needed the quota so I deleted the instance. [01:47:01] 6Labs, 10Tool-Labs: Customization of /etc/hosts deletes alias ubuntu.openstack.eqiad.wmflabs - https://phabricator.wikimedia.org/T113922#1679840 (10scfc) 3NEW [01:49:29] Change on 12wikitech.wikimedia.org a page Nova Resource:Tools/Access Request/Esperluette was created, changed by Esperluette link https://wikitech.wikimedia.org/wiki/Nova_Resource:Tools/Access_Request/Esperluette edit summary: Created page with "{{Tools Access Request |Justification=I'm hoping to create something like hatnote's Weekly Wikipedia, but for Wiktionary |Completed=false |User Name=Esperluette }}" [02:28:15] PROBLEM - Puppet failure on tools-webgrid-lighttpd-1402 is CRITICAL: CRITICAL: 62.50% of data above the critical threshold [0.0] [04:03:13] RECOVERY - Puppet failure on tools-webgrid-lighttpd-1402 is OK: OK: Less than 1.00% above the threshold [0.0] [04:08:30] RECOVERY - Puppet failure on tools-webgrid-lighttpd-1402 is OK: OK: Less than 1.00% above the threshold [0.0] [06:11:13] could anybody here kindly tell me whom to contact if i want to create a new project [06:13:33] hey khushbu do you have any knowledge regarding my prev question [07:04:48] 6Labs, 10Tool-Labs, 3Labs-Sprint-114, 3Labs-Sprint-115: Add support to dynamicproxy for kubernetes based web services - https://phabricator.wikimedia.org/T111916#1679962 (10yuvipanda) a:3Joe [07:04:59] 6Labs, 10Tool-Labs, 3Labs-Sprint-115: Write admission controller disabling mounting of unauthorized volumes - https://phabricator.wikimedia.org/T112718#1679964 (10yuvipanda) a:3yuvipanda [07:51:43] (03PS2) 10Jean-Frédéric: Split out replace_default_cat_with_new_categories_in_image in four methods [labs/tools/heritage] - 10https://gerrit.wikimedia.org/r/241568 [07:52:12] (03CR) 10Jean-Frédéric: "recheck" [labs/tools/heritage] - 10https://gerrit.wikimedia.org/r/241568 (owner: 10Jean-Frédéric) [07:52:27] (03CR) 10Jean-Frédéric: [C: 032] Split out replace_default_cat_with_new_categories_in_image in four methods [labs/tools/heritage] - 10https://gerrit.wikimedia.org/r/241568 (owner: 10Jean-Frédéric) [07:56:20] (03Merged) 10jenkins-bot: Split out replace_default_cat_with_new_categories_in_image in four methods [labs/tools/heritage] - 10https://gerrit.wikimedia.org/r/241568 (owner: 10Jean-Frédéric) [09:51:23] (03CR) 10Hashar: [C: 032] Fix indentation in config.yaml.sample [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241527 (owner: 10Polybuildr) [09:56:01] (03Merged) 10jenkins-bot: Fix indentation in config.yaml.sample [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241527 (owner: 10Polybuildr) [09:58:04] (03CR) 10Hashar: [C: 04-1] "Thank you! I will do the matching CI change to adjust the jobs being triggered." [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241523 (https://phabricator.wikimedia.org/T95891) (owner: 10Polybuildr) [10:00:41] (03CR) 10Hashar: "CI https://gerrit.wikimedia.org/r/241615" [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241523 (https://phabricator.wikimedia.org/T95891) (owner: 10Polybuildr) [11:04:01] RECOVERY - Puppet failure on tools-webgrid-lighttpd-1407 is OK: OK: Less than 1.00% above the threshold [0.0] [11:06:05] RECOVERY - Puppet staleness on tools-webgrid-lighttpd-1407 is OK: OK: Less than 1.00% above the threshold [3600.0] [11:15:44] Geohack (https://tools.wmflabs.org/geohack/) is down, please restart. [11:34:54] you should contact geohack's maintainer [11:35:08] tools.wmflabs.org/?tool=geohack [11:35:32] I'd offer to help, but I still need to get all my powers back [11:35:35] (03CR) 10Hashar: "recheck" [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241523 (https://phabricator.wikimedia.org/T95891) (owner: 10Polybuildr) [11:36:08] maintainer of geohack is "magnus manske" and "Kolossos". [11:36:35] (03CR) 10Hashar: [C: 032] Use `npm test` to lint YAML [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241523 (https://phabricator.wikimedia.org/T95891) (owner: 10Polybuildr) [11:36:51] (03Merged) 10jenkins-bot: Use `npm test` to lint YAML [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241523 (https://phabricator.wikimedia.org/T95891) (owner: 10Polybuildr) [11:40:24] revi: a tool that is used by millions of wikipedia pages should be restarted immidiately by whoever... aren't there server admins or anything like this? [11:41:08] the Magnus Manske and kolossos is the only who can restart them [11:41:18] (well labs sysadmin can but that's excluded) [12:15:19] 10Tool-Labs-tools-Other: Resetup templatecheck tool - https://phabricator.wikimedia.org/T113943#1680576 (10scfc) 3NEW a:3scfc [13:44:07] Change on 12wikitech.wikimedia.org a page Nova Resource:Tools/Access Request/Esperluette was modified, changed by Tim Landscheidt link https://wikitech.wikimedia.org/w/index.php?diff=185724 edit summary: [13:58:30] (03PS1) 10Addshore: Also exclude TCB-Team- from #wikimedia-fundraising [labs/tools/wikibugs2] - 10https://gerrit.wikimedia.org/r/241652 [15:01:57] 6Labs, 10Tool-Labs, 3Labs-Sprint-115: Attribute cache issue with NFS on Trusty - https://phabricator.wikimedia.org/T106170#1681046 (10coren) After some experimenting, there appear to be two plausible solutions: * mount /data/project (at least) with the `noac` NFS option; this forces attribute caching to be d... [15:02:23] 6Labs, 10Tool-Labs, 5Patch-For-Review: continuous jobs killed during restart despite rescheduling - https://phabricator.wikimedia.org/T109362#1547071 (10coren) [15:02:24] 6Labs, 10Tool-Labs, 3Labs-Sprint-115: Attribute cache issue with NFS on Trusty - https://phabricator.wikimedia.org/T106170#1681049 (10coren) [15:04:48] * Coren kicked geohack. [15:06:11] * gifti gives coren a cookie [15:24:26] !log tools rebooting tools-shadow after mount option changes. [15:24:29] Logged the message at https://wikitech.wikimedia.org/wiki/Nova_Resource:Tools/SAL, Master [15:56:16] Coren or yuvipanda, is the disk space alert on labstore1002 ok for me to ignore for some reason? [15:56:54] andrewbogott: It's a false alert caused by check_disk being dumb. There should have been a workaround in hiera but it doesn't work. I'll be begging for help at the meeting. :-) [15:58:19] Coren: ok! [16:01:57] yuvipanda: Meeting? [16:29:44] 6Labs, 10Tool-Labs, 3Labs-Sprint-103, 3Labs-Sprint-115: Labs: Move tools-shadow off the same host as tool-master - https://phabricator.wikimedia.org/T103390#1681441 (10coren) 5Open>3Resolved Now on labvirt1008 [16:29:45] 6Labs, 10Tool-Labs: Move tools-master and tools-shadow to trusty - https://phabricator.wikimedia.org/T94791#1681443 (10coren) [16:54:00] yuvipanda: have a few moments to approve this? https://meta.wikimedia.org/wiki/Special:OAuthListConsumers/view/c683af90c0ed69e3a4dc855eaa52d9e9 [16:58:37] 6Labs, 10Labs-Infrastructure, 3labs-sprint-116: Make sure nova is re-using old private IPs - https://phabricator.wikimedia.org/T113648#1681636 (10Andrew) [17:00:04] 6Labs, 3Labs-Sprint-109, 7Monitoring, 5Patch-For-Review, 3labs-sprint-116: Monitor nova services - https://phabricator.wikimedia.org/T90784#1681637 (10Andrew) [17:13:15] 6Labs, 10Tool-Labs, 3Labs-Sprint-115, 3labs-sprint-116: Write admission controller disabling mounting of unauthorized volumes - https://phabricator.wikimedia.org/T112718#1681691 (10yuvipanda) [17:13:21] 6Labs, 10Tool-Labs, 3Labs-Sprint-114, 3Labs-Sprint-115, 3labs-sprint-116: Add support to dynamicproxy for kubernetes based web services - https://phabricator.wikimedia.org/T111916#1681692 (10yuvipanda) [17:13:27] 6Labs, 3Labs-Q4-Sprint-1, 3Labs-Q4-Sprint-2, 3Labs-Q4-Sprint-4, and 3 others: Labs NFSv4/idmapd mess - https://phabricator.wikimedia.org/T87870#1681693 (10coren) [17:13:35] 6Labs, 3Labs-Sprint-114, 3Labs-Sprint-115, 3labs-sprint-116: Make a flowchart for locating and halting misbehaving NFS clients - https://phabricator.wikimedia.org/T101744#1681694 (10coren) [17:13:45] 6Labs, 10Tool-Labs, 3Labs-Sprint-115, 5Patch-For-Review, 3labs-sprint-116: Attribute cache issue with NFS on Trusty - https://phabricator.wikimedia.org/T106170#1681695 (10coren) [17:14:51] 6Labs, 10Tool-Labs, 3labs-sprint-116: Allow direct ssh access to tools - https://phabricator.wikimedia.org/T113979#1681701 (10yuvipanda) 3NEW [17:16:54] 6Labs, 6Discovery, 7Elasticsearch, 3labs-sprint-116: Replicate production elasticsearch indices to labs - https://phabricator.wikimedia.org/T109715#1681726 (10yuvipanda) [17:17:40] 6Labs, 7Database, 5Patch-For-Review, 3labs-sprint-116: watchlist table not available on labs - https://phabricator.wikimedia.org/T59617#1681728 (10coren) [17:17:48] 6Labs, 3labs-sprint-116: centralauth_p is missing tables - https://phabricator.wikimedia.org/T68533#1681729 (10coren) [17:18:01] 6Labs, 10Labs-Infrastructure: Audit private IP allocation for Labs instances - https://phabricator.wikimedia.org/T113982#1681732 (10Andrew) 3NEW a:3Andrew [17:18:03] 6Labs, 10Tool-Labs, 3labs-sprint-116: Tool Labs: Provide anonymized view of the user_properties table - https://phabricator.wikimedia.org/T60196#1681730 (10coren) [17:21:04] 6Labs, 10Labs-Infrastructure, 3labs-sprint-116: Audit private IP allocation for Labs instances - https://phabricator.wikimedia.org/T113982#1681778 (10Andrew) [17:22:40] 6Labs, 3labs-sprint-116: Fix check_disk bogus alerts on labstore1002 - https://phabricator.wikimedia.org/T113435#1681802 (10coren) There is already a fix applied that does not work as expected. This is the same issue as T104975 but the fix there didn't quite work. [17:24:48] 6Labs, 10Labs-Infrastructure, 10hardware-requests, 6operations: New server: labdns1001 - https://phabricator.wikimedia.org/T106147#1681821 (10Andrew) a:5Andrew>3mark Reassigned to Mark for approval of the misc-server allocation [17:25:18] PROBLEM - Puppet failure on tools-webgrid-lighttpd-1402 is CRITICAL: CRITICAL: 22.22% of data above the critical threshold [0.0] [17:31:17] 6Labs: Disable NFS for the toolserver-legacy project - https://phabricator.wikimedia.org/T104256#1681852 (10coren) Doesn't need nfs. [17:32:40] 6Labs: Eliminate SPOFs in Labs infrastructure (Tracking) - https://phabricator.wikimedia.org/T105723#1681864 (10Andrew) [17:32:41] 6Labs, 10Labs-Infrastructure, 3Labs-Sprint-107: labnet1001 is a spof - https://phabricator.wikimedia.org/T106141#1681861 (10Andrew) 5Open>3Resolved a:3Andrew now there are two [17:32:52] 6Labs, 7Database, 5Patch-For-Review, 3labs-sprint-116: watchlist table not available on labs - https://phabricator.wikimedia.org/T59617#1681865 (10Krd) Just if anyone cares, after nearly 2 years I have somewhat lost interest in this issue, so I'm fine if somebody closes this at not done. [17:35:46] 6Labs, 10Labs-Infrastructure, 10hardware-requests, 6operations: New server: labdns1001 - https://phabricator.wikimedia.org/T106147#1681882 (10mark) The assignment of a misc box for this is approved for now - but I think we shouldn't just have DNS on it in the future. As discussed, LDAP would also be fine... [17:36:03] 6Labs, 10Labs-Infrastructure, 10hardware-requests, 6operations: New server: labdns1001 - https://phabricator.wikimedia.org/T106147#1681883 (10mark) a:5mark>3None [17:37:59] 6Labs: Disable NFS for the toolserver-legacy project - https://phabricator.wikimedia.org/T104256#1681894 (10coren) a:5yuvipanda>3coren [17:50:53] 6Labs, 10Tool-Labs, 3labs-sprint-116: Allow direct ssh access to tools - https://phabricator.wikimedia.org/T113979#1681982 (10valhallasw) There might be an issue with logging, though. We'd like to know who of the maintainers logged in, and I'm not sure if that's something that's easily logged (although we lo... [17:54:37] 6Labs, 10Incident-20150617-LabsNFSOutage, 3Labs-Sprint-102, 3Labs-Sprint-103, and 3 others: Audit projects' use of NFS, and remove it where not necessary - https://phabricator.wikimedia.org/T102240#1681989 (10coren) [17:54:39] 6Labs, 5Patch-For-Review: Disable NFS for the toolserver-legacy project - https://phabricator.wikimedia.org/T104256#1681987 (10coren) 5Open>3Resolved Project unnfsized. [18:09:40] Hi, anyone know what's the procedure for getting an OAuth consumer approved? [18:10:14] jimmyxu: you request it, and it's reviewed [18:10:57] Hm it's been lying in OAuthConsumerRegistration for a week or so. Is there a venue where I should lodge a request? [18:15:02] ragesoss, jimmyxu: incidentally, see https://meta.wikimedia.org/wiki/Requests_for_comment/OAuth_handover [18:15:16] RECOVERY - Puppet failure on tools-webgrid-lighttpd-1402 is OK: OK: Less than 1.00% above the threshold [0.0] [18:15:51] tgr: nice to see that moving forward. [18:15:55] jimmyxu: I think it might be 'not reviewed' on purpose, because you say it's 'internal'. And if you just use it yourself, it doesn't need to be apporved [18:16:35] valhallasw`cloud: it's been my experience that consumers don't get reviewed unless you ask someone. [18:17:21] hm, maybe I was lucky :-) [18:18:03] I'm not sure who can review consumers these days. It's not a specific right on mw.org? *confused* [18:18:37] valhallasw`cloud: it's a right on meta.wikimedia.org now. [18:18:43] ohhh. [18:18:45] valhallasw`cloud: but then I get an error of E005 when trying to authorize my bot, apparently I can only use that consumer with my non-bot account [18:18:45] that makes sense [18:18:52] jimmyxu: correct [18:19:00] just the account with which you requested the consumer [18:20:08] jimmyxu: https://meta.wikimedia.org/w/index.php?title=Special:ListUsers&group=oauthadmin [18:21:45] valhallasw`cloud: If pinging someone directly is the way, I think I'll do that then. thx :) [18:25:54] 6Labs, 10Tool-Labs: Install flex on bastions - https://phabricator.wikimedia.org/T114003#1682112 (10scfc) 3NEW a:3scfc [19:06:43] 6Labs, 5Patch-For-Review: Disable NFS for the toolserver-legacy project - https://phabricator.wikimedia.org/T104256#1682232 (10yuvipanda) Thanks! [19:21:12] PROBLEM - Puppet failure on tools-checker-01 is CRITICAL: CRITICAL: 100.00% of data above the critical threshold [0.0] [19:21:19] !log tools.jouncebot Restarted to pick up I529e774 (Filter example ircnick from patch owners list) [19:21:22] Logged the message at https://wikitech.wikimedia.org/wiki/Nova_Resource:Tools.jouncebot/SAL, Master [19:21:39] Krenair: ^ [19:21:46] ty [19:38:57] 6Labs, 10Labs-Infrastructure, 10hardware-requests, 6operations, 3labs-sprint-116: New server: labdns1001 - https://phabricator.wikimedia.org/T106147#1682384 (10Andrew) [19:43:33] (03PS1) 10Polybuildr: Add node_modules to .gitignore [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241747 [19:44:55] (03CR) 10Polybuildr: [C: 04-1] "Oops. Wrong file added." [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241747 (owner: 10Polybuildr) [19:47:08] (03PS2) 10Polybuildr: Add node_modules to .gitignore [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241747 [19:48:07] tgr: I'm confused about 'authentication only with real name/email. Is that info that is also available automatically to a consumer with 'basic rights'? [19:48:47] (03CR) 10Yuvipanda: [C: 032] Add node_modules to .gitignore [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241747 (owner: 10Polybuildr) [19:48:52] This table doesn't mention real name/email anywhere, except in the name (but not the 'rights' column) for that authetnication only with email right: https://meta.wikimedia.org/wiki/Special:OAuth/grants#useoauth [19:49:27] ragesoss: whoops sorry [19:49:33] I see halfak had taken care of it for you [19:49:55] yuvipanda: no prob. Do you know the answer to that question, though? ^ [19:50:23] * halfak wants to know too. [19:50:48] ragesoss: nope, legoktm or anomie probably [19:52:32] yuvipanda: why does grrrit have an empty .jshintrc? :P [19:52:33] (03Merged) 10jenkins-bot: Add node_modules to .gitignore [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241747 (owner: 10Polybuildr) [19:52:52] polybuildr: I think that's supposed to prevent jenkins from freaking out over somethings? [19:53:01] yuvipanda: ha, okay. [19:53:11] polybuildr: you're fully welcome to add one and make it pass if you want to :D I'll happily merge [19:53:20] ragesoss: For Special:OAuth/identify? The real name and email are available to clients with the special 'authonlyprivate' grant, or granted the 'viewmyprivateinfo' right. [19:53:37] yuvipanda: wait, what? I thought it passes default JSHint already. :o [19:53:50] polybuildr: I don't know if it does [19:53:50] Yeah, I think it does. [19:54:03] legoktm: do you know why there's an empty jshintrc? [19:54:19] hashar put in the empty .jshintrc [19:54:21] * anomie notes there aren't any currently-configured grants that grant 'viewmyprivateinfo', though [19:54:22] anomie, can consumers with authonly still use /identify? [19:54:54] halfak: Yes. That's how they're intended to auth. [19:55:22] "auth" is overloaded. [19:55:32] To mean authorization and authentication [19:55:39] Here, I want to understand "identify" [19:56:04] Another question: what can cause "Insufficient OAuth user rights, E007"? Will that happen if you try to auth through a blocked IP? [19:56:24] I have a user who says he gets this error when trying to log in from his classroom, but not when he logs in from elsewhere. [19:56:26] polybuildr: jshint will traverse upwards until it finds a .jshintrc and it'll use that. So committing a blank one prevents that upwards traversal [19:56:41] duploktm: oh, right! thanks [19:56:51] Why are you duploktm? :P [19:57:01] halfak: "authonly" and "authonlyprivate" are intended for something like Phab that doesn't need to do anything with the user's MediaWiki account except verify "Yes, that's the account owner". The way they do that is by hitting Special:OAuth/identify. Normal clients can also use Special:OAuth/identify, in which case it's basically a signed version of api.php?action=query&meta=userinfo. [19:57:21] yuvipanda: polybuildr: we needed an empty .jshintrc because jshint running on gallium would end up looking for a .jshintrc in the parent directories. Eventually finding the one in /srv/ which comes from integration/docroot.git [19:57:22] anomie, yeah. IO' [19:57:38] yuvipanda: polybuildr: nowadays, I guess the empty jshint can be removed or better, adjusted to whatever standard [19:57:43] hashar: right, thanks :D [19:57:49] I can't remember the details [19:57:50] 6Labs, 10Labs-Infrastructure, 6operations: install/setup labdns1001 - https://phabricator.wikimedia.org/T106584#1682446 (10RobH) [19:57:54] anomie, I'm with you. I've built a lot of tools like that and I'm an oauth approver. I just need to know what these things *mean* [19:58:07] but Timo told me that an empty .jshintrc {} does not have the same behavior as running jshint without any jshintrc [19:58:10] anome: question above about "Insufficient OAuth user rights, E007" ^ [19:58:12] apparently they are slightly different [19:59:09] ragesoss: https://www.mediawiki.org/wiki/Help:OAuth/Errors#E007. It means that User::isBlocked() returned true. [19:59:13] polybuildr: some days I'm a toddler. https://en.wikipedia.org/wiki/Lego_Duplo [19:59:40] duploktm: okay... :P [19:59:45] halfak: So what is it that you need to know what it means? [19:59:45] duploktm, lol [19:59:50] anomie: cool. that would explain it; must be an IP block on his classroom. [20:00:15] I think I understand. What I'd really like is a good reference for these new authorization classes. [20:00:22] Right now, the table says nothing about what they enable. [20:00:28] How how they related to basic rights [20:02:00] 6Labs, 10Labs-Infrastructure, 10hardware-requests, 6operations, 3labs-sprint-116: New server: labdns1001 - https://phabricator.wikimedia.org/T106147#1682465 (10RobH) I'm not sure why this is bare-metal to run labs dns in a public vlan (like holmium) Can't this be a ganeti vm? [20:03:27] "authonly" and "authonlyprivate" don't give any rights on the wiki, not even "basic rights". They're also supposed to display a less-scary message on the authorization popup, although I don't recall offhand exactly what the messages are. [20:04:47] (03PS1) 10Polybuildr: Add Grunt to run JSHint [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241754 [20:05:33] 6Labs, 10Labs-Infrastructure, 10hardware-requests, 6operations, 3labs-sprint-116: New server: labdns1001 - https://phabricator.wikimedia.org/T106147#1682496 (10yuvipanda) Let's definitely not put any labs support things on ganeti - that'll definitely complicate something or the other in the future (mixi... [20:05:49] 6Labs, 10Labs-Infrastructure, 6operations: install/setup labservices1001 - https://phabricator.wikimedia.org/T106584#1682499 (10RobH) [20:06:01] hashar, yuvipanda ^^ [20:06:39] (03CR) 10Yuvipanda: [C: 032] "woo" [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241754 (https://phabricator.wikimedia.org/T95891) (owner: 10Polybuildr) [20:07:01] (03Merged) 10jenkins-bot: Add Grunt to run JSHint [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241754 (https://phabricator.wikimedia.org/T95891) (owner: 10Polybuildr) [20:07:41] hashar: would you like to remove the jsonlint check even from jenkins jobs, or only remove jshint? [20:09:28] polybuildr: oh the main goal was to get rid of yamllint and add jshint [20:09:40] hashar: what about jsonlint? [20:09:42] polybuildr: jsonlint, as you said, it is probably not of any use for that repo [20:09:48] alright :) [20:09:52] polybuildr: your call :-} [20:10:12] I am just clicking [ ] +2 all day long! [20:10:40] I'm ok giving polybuildr +2 on the krrrit-wm repo if wanted :) [20:11:04] hashar: just tried to run an `npm test` with an invalid package.json. Lots of npm errors, and status code 1, so I think it's safe to get rid of jsonlint. [20:11:18] yuvipanda: and I guess what ever admin right to deploy changes [20:11:33] polybuildr: yup lets do that [20:11:46] hashar: right now it is running in a docker container on the new kubernetes cluster so only I can deploy changes atm unforunately [20:11:49] have to do some paper flipping, be back in a few [20:12:02] yuvipanda: what??? [20:12:23] yuvipanda: so we have yet another vm system in addition to Ganeti and OpenStack? [20:12:36] hashar: kubernetes is not a VM system :) [20:12:46] hashar: you didn't read the happy emails on wikitech-l about the new cluster? :P [20:13:16] it's an alternative to gridengine, not OpenStack or Ganeti [20:19:29] polybuildr: na I haven't carefully read the announcement :D [20:19:40] yuvipanda: but it get to run docker containers doesn't it ? [20:19:58] hashar: yes? and containers aren't VMs no [20:22:54] yuvipanda: hashar : Don't use empty jshintrc [20:23:14] If you find a problem because it's using the wrong jshintrc because you don't have one in the repo, disable the jshint job and go back to square 1 [20:23:35] jshint without a non-empty jshintrc is just silly and confusing [20:23:48] yuvipanda: dunno, most people treat VMs as 'a sandbox for files and processes', which is definitely also true for docker containers :-p [20:23:53] Put a proper one in there and the problem is solved. [20:24:12] Krinkle: But an empty jshintrc does the default jshint checks, doesn't it? [20:24:23] Or rather, there is no problem to begin with. It's only going wrong because people keep plastering jshint jobs on random repos without thinking logically first. [20:24:27] polybuildr: No, it does not. [20:24:35] And it shoudn't. You should specify the rules you want, or not use jshint. [20:24:40] but V+1 and auto-merge! [20:24:41] valhallasw`cloud: doesn't have its own kernel, hence not VM :D also people will be using it far more similar to gridengine than openstack which I guess is a more important distinction [20:24:53] Krinkle: But I managed to get it to error by removing a semicolon. [20:24:54] https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript [20:25:08] polybuildr: It will error for very very basic syntax violations and bad patterns. [20:25:11] But that's is nearly useless [20:25:21] Certainly not in any quality standards compliance in our circles. [20:25:31] Krinkle: That's true. [20:34:14] polybuildr: flipped too many papers sorry. I am giving up for today [20:34:19] 6Labs, 3Labs-Sprint-114, 3Labs-Sprint-115, 3labs-sprint-116: Make a flowchart for locating and halting misbehaving NFS clients - https://phabricator.wikimedia.org/T101744#1682630 (10coren) 5Open>3Resolved Added information to https://wikitech.wikimedia.org/wiki/Labs_NFS [20:34:27] hashar: alright! goodnight o/ [20:34:51] polybuildr: will look at the patches tomorrow morning. But maybe someone else will merge them overnight :-} [20:34:58] polybuildr: thank you for the CI integration for grrrit [20:34:58] hashar: sure :) [20:35:05] my pleasure :D [20:35:56] polybuildr: actually it is straightforward [20:37:04] 6Labs, 3labs-sprint-116: centralauth_p is missing tables - https://phabricator.wikimedia.org/T68533#1682656 (10coren) [20:37:42] hashar: the CI integration? yeah, pretty straightforward :) [20:40:29] (03PS1) 10Catrope: Don't duplicate #wikimedia-collaboration notifications in #wikimedia-dev [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241788 [20:40:45] polybuildr: it is in the gate queue, waiting for mediawiki-phpunit-zend as usual :-} [20:46:34] (03PS2) 10Yuvipanda: Don't duplicate #wikimedia-collaboration notifications in #wikimedia-dev [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241788 (owner: 10Catrope) [20:47:20] polybuildr: should be good now [20:47:24] (03CR) 10Legoktm: [C: 04-2] "Dupe of https://gerrit.wikimedia.org/r/#/c/230945/" [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241788 (owner: 10Catrope) [20:47:40] hashar: yeah, merged :) [20:49:36] (03PS1) 10Polybuildr: Update js-yaml and replace require()s with read file and yaml.safeLoad [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241871 (https://phabricator.wikimedia.org/T114024) [20:49:57] yuvipanda: I don't have the repo set up for use, could you please test ^? [20:50:22] It's blocking the addition of a proper .jshintrc (details in linked task) [21:03:25] (03CR) 10Yuvipanda: "I'll note that the migration of -mobile into -dev was more of a 'eh, get all these bots out of here!' than a 'oooh we should integrate int" [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/230945 (owner: 10Jforrester) [21:43:43] yuvipanda, hey [21:43:58] yuvipanda, have you seen my patches about getting rid of ircecho? [21:44:26] Krenair: no I thought ori was helping you CR those? [21:44:35] I'm all for making those two things one [21:44:45] (03PS1) 10Polybuildr: Add a proper .jshintrc and pass JSHint [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241904 [21:45:03] yuvipanda, well he is [21:45:10] but I wanted your thoughts about something icinga-specific [21:45:18] Krenair, yuvipanda, done ^^ :) [21:45:22] at least, sort of [21:45:27] I guess it's kind of generic [21:46:25] polybuildr: I'll test it in an hour or so? [21:46:28] Krenair: go on :D [21:46:42] yuvipanda, so it needs a nickserv account [21:46:50] but that password presumably needs to be kept somewhere private [21:47:09] I'm not sure how to do this properly in labs in a way that doesn't involve referring to unpuppetised files [21:47:51] Krenair: 1. don't register it :P or 2. unpuppetized file [21:48:04] I don't think not registering it is an option. [21:48:12] yuvipanda: sure :) that change is dependent on another though, and I'm not sure if I broke anything :P o/ [21:49:18] Krenair: why not [21:49:23] Krenair: is it just for testing? [21:50:03] er, well [21:50:07] I meant shinken earlier [21:50:19] I think shinken in labs is not really for testing, is it? [21:50:44] no [21:50:59] but it hasn't had a problem not being registered, no? [21:51:16] ircecho allows you to not be registered [21:51:36] I don't think tcpircbot does [21:51:47] the nickserv pass is required [21:51:56] I guess then we should maybe patch that? [21:52:02] or make it unpuppetized :) [21:52:10] just the password, that is [22:00:27] (03PS2) 10Polybuildr: Update js-yaml and replace require()s with read file and yaml.safeLoad [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241871 (https://phabricator.wikimedia.org/T114024) [22:02:06] (03CR) 10Polybuildr: [C: 04-1] "The patch that this is dependent on has changed." [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241904 (owner: 10Polybuildr) [22:11:57] (03PS2) 10Polybuildr: Add a proper .jshintrc and pass JSHint [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241904 [22:17:34] (03PS1) 10Polybuildr: Add .npm-debug.log* to .gitignore [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241919 [22:20:04] (03PS1) 10Polybuildr: Add connections.yaml to .gitignore [labs/tools/grrrit] - 10https://gerrit.wikimedia.org/r/241921 [22:24:54] yuvipanda: lots of patchsets for you to review :P [22:24:56] goodnight o/ [22:26:12] PROBLEM - Puppet failure on tools-webgrid-lighttpd-1402 is CRITICAL: CRITICAL: 33.33% of data above the critical threshold [0.0] [22:29:35] 10wikimedia-labs-wikistats: status of LXDE wikis - remove table? - https://phabricator.wikimedia.org/T111591#1683289 (10RobiH) It still is being linked to from the www. site. [22:31:06] 10wikimedia-labs-wikistats: status of LXDE wikis - remove table? - https://phabricator.wikimedia.org/T111591#1683297 (10RobiH) But what actually should be removed, are the ORAIN and PARDUS tables. Both domains expired for good. [23:08:34] 10Tool-Labs-tools-Matthewrbowker's-tools: MATTHEWRBOWKER-9 Move opening and closing HTML to functions - https://phabricator.wikimedia.org/T61066#1683463 (10Matthewrbowker) [23:29:35] 6Labs, 10Labs-Infrastructure, 6operations: install/setup labservices1001 - https://phabricator.wikimedia.org/T106584#1683533 (10RobH) [23:43:44] 6Labs: Kill ldapsupportlib.py - https://phabricator.wikimedia.org/T114063#1683588 (10yuvipanda) 3NEW [23:44:01] 10Tool-Labs-tools-Other: Resetup templatecheck tool - https://phabricator.wikimedia.org/T113943#1683598 (10scfc) 5Open>3Resolved [23:52:02] PROBLEM - Puppet failure on tools-exec-1209 is CRITICAL: CRITICAL: 44.44% of data above the critical threshold [0.0]