| *** rlandy is now known as rlandy|afk | 00:15 | |
| *** harlowja has quit IRC | 03:18 | |
| *** rlandy|afk is now known as rlandy | 04:21 | |
| tristanC | Hello folks, could we tag nodepool version 3 or are we waiting for some changes? | 04:50 |
|---|---|---|
| *** harlowja has joined #zuul | 05:12 | |
| *** smyers has quit IRC | 05:15 | |
| *** smyers has joined #zuul | 05:16 | |
| *** myoung|ruck|bbl is now known as myoung|ruck | 05:21 | |
| *** myoung|ruck is now known as myoung|ruck|off | 05:22 | |
| tristanC | regarding zuul release, could we approve: https://review.openstack.org/547790 ? | 05:22 |
| tristanC | i'm waiting for the release to publish fedora and centos packages... | 05:23 |
| tristanC | would appreciate an heads-up on an estimated date for the tags | 05:26 |
| SpamapS | holy giant commit message batman | 05:29 |
| SpamapS | tristanC: any reason that wasn't split up a bit more? It looks kind of like a massive stack squashed into one. | 05:32 |
| * SpamapS gets tired after 200 or so lines of review | 05:33 | |
| SpamapS | currently on line 1700 or so of the diff :-P | 05:39 |
| tristanC | SpamapS: the review used to be splitted, but it was meant to be useful as a whole so a single review makes more sense imo | 05:40 |
| SpamapS | sure... it's a pendulum.. it's swung away from my happy place into your happy place. We'll get it merged. | 05:41 |
| tristanC | javascript makefile bits apart, the change is relatively small | 05:41 |
| SpamapS | well I'd complain about the amount of python in tests, but I will never, ever, ever, complain about too many tests | 05:43 |
| SpamapS | also having written some nasty ansible to do a lot of this, it's making me happy that I can reduce it to a few npm's | 05:45 |
| tristanC | actually that npm things makes rpm packaging tricky, we used to yum install zuul-webui which pulled jquery and angular packages | 05:50 |
| SpamapS | some day, we'll all look back and laugh at how much time we spent clinging to rpm/deb before we realized that the cloud doesn't have a DVD drive. | 05:51 |
| SpamapS | tristanC: I +3'd, but the parent is massive too | 05:51 |
| SpamapS | so like. | 05:51 |
| SpamapS | no wonder it hasn't landed | 05:51 |
| SpamapS | :-P | 05:51 |
| SpamapS | tristanC: you're in the gate. :) | 05:55 |
| SpamapS | And TIL that zuul-web is getting really fancy | 05:55 |
| tristanC | SpamapS: have you seen the job graph? here is a demo of https://review.openstack.org/537869 : https://softwarefactory-project.io/zuul/openstack.org/jobs.html | 05:57 |
| *** haint has quit IRC | 05:57 | |
| tristanC | thanks for the review, so we are now only missing the zuul_stream refactor | 05:57 |
| tobiash | SpamapS: yes, monty squashed most of the javascript stack | 06:05 |
| tobiash | tristanC: this javascript stack is one of the last two blockers for the v3 release | 06:05 |
| tobiash | so when they are sorted out v3 will be tagged probably next week | 06:06 |
| tobiash | and nodepool and zuul will be tagged at the same time | 06:06 |
| tristanC | tobiash: thanks for the information :) | 06:07 |
| openstackgerrit | Merged openstack-infra/zuul master: Make Info.endpoint a config override https://review.openstack.org/547576 | 06:07 |
| *** haint has joined #zuul | 06:07 | |
| tobiash | so the zuul_stream refactor is the last missing piece then | 06:08 |
| tobiash | tristanC: so you're already running with mordred's javascript stack? | 06:09 |
| tobiash | is there anything to think about when upgrading? | 06:10 |
| openstackgerrit | Merged openstack-infra/zuul master: Use yarn and webpack to manage zuul-web javascript https://review.openstack.org/547790 | 06:10 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Support brew for installing the js tools https://review.openstack.org/548235 | 06:13 |
| SpamapS | tristanC: pretttty | 06:14 |
| tristanC | tobiash: not really, fwiw here is the deployment change we used: https://softwarefactory-project.io/r/10971 | 06:14 |
| *** threestrands has quit IRC | 06:14 | |
| tristanC | not sure how the js would get installed by pip though | 06:15 |
| tobiash | ok, will have a look what pip will do | 06:16 |
| tobiash | thanks | 06:16 |
| tristanC | tobiash: i mean yes i've been cherry-picking all the v3 change in the zuul package to be ready for the release, and i couldn't make the zuul_stream refactor work | 06:18 |
| tobiash | tristanC: I think that's still WIP by mordred | 06:18 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Add queue size to tenant overview https://review.openstack.org/548248 | 06:21 |
| *** harlowja has quit IRC | 06:29 | |
| *** xiexianbin has joined #zuul | 06:57 | |
| xiexianbin | is http://zuul.openstack.org/ down? i get 502 from shanghai, China. | 06:58 |
| tobiash | xiexianbin: it's working for me | 06:58 |
| tobiash | maybe the great firewall is blocking you? | 06:59 |
| xiexianbin | i also get 502 in us vultr | 07:00 |
| xiexianbin | [root@vultr ~]# curl zuul.openstack.org <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN"> <html><head> <title>502 Proxy Error</title> </head> | 07:01 |
| tobiash | xiexianbin: you might want to ask in #openstack-infra, maybe there is something odd | 07:01 |
| xiexianbin | tks | 07:02 |
| ianw | ok it look like deployment is really broken with those javascript changes | 07:50 |
| AJaeger | tobiash: is there any change up for fixing deployment? | 07:54 |
| tobiash | AJaeger: oh, due to the zuul-web change that landed? | 07:54 |
| tobiash | probably not | 07:54 |
| tobiash | I think that merged a bit too early ;) | 07:55 |
| AJaeger | tobiash: yes, that's ianw's suggestion - see his comment on #openstack-infra as well. | 07:55 |
| AJaeger | tobiash: is that a single change? Should we revert it for now? | 07:56 |
| ianw | https://review.openstack.org/549608 is the change for the puppet | 07:56 |
| ianw | i've tried to fix it up ... | 07:56 |
| AJaeger | ianw: thanks! | 07:56 |
| ianw | also, i have now manually installed npm/yarn on the host as i ran the tools/install_javascript_tools and manually pip installed it | 07:56 |
| ianw | but i don't think that matters | 07:56 |
| tristanC | ianw: could you try dropping http://logs.openstack.org/90/547790/6/gate/build-javascript-content/73011cf/npm/html/ inside the /usr/local/lib/python3.5/dist-packages/zuul/web/static directory? | 07:59 |
| ianw | tristanC: well i sort of did that by installing yarn and pip reinstalling so the setup hook ran | 07:59 |
| ianw | the problem is that hte apache config is all wrong to pass through to the new files | 08:00 |
| ianw | to serve the new files i mean | 08:00 |
| ianw | which the puppet fix should correct | 08:00 |
| *** jpena|off is now known as jpena | 08:00 | |
| AJaeger | http://zuul.openstack.org/ runs now but looks rather empty | 08:01 |
| *** jpena is now known as jpena|off | 08:01 | |
| *** jpena|off is now known as jpena | 08:01 | |
| ianw | AJaeger: right, that's the issue above, if you see it's trying to source files that aren't served | 08:01 |
| *** sshnaidm|ptg has joined #zuul | 08:10 | |
| *** hashar has joined #zuul | 08:24 | |
| *** snapiri- has joined #zuul | 08:39 | |
| *** snapiri has quit IRC | 08:42 | |
| *** electrofelix has joined #zuul | 08:45 | |
| *** sshnaidm|ptg has quit IRC | 08:52 | |
| tobiash | SpamapS: do you also have occationally merge failed errors with github? | 09:40 |
| tobiash | not often, but sometimes it seems that the merger cannot fetch the branch | 09:41 |
| *** snapiri- has quit IRC | 11:18 | |
| *** xiexianbin has quit IRC | 11:29 | |
| *** sshnaidm|ptg has joined #zuul | 11:49 | |
| openstackgerrit | Andrea Frittoli proposed openstack-infra/zuul-jobs master: Revert "Revert "Deploy the build ssh key to auth_keys"" https://review.openstack.org/550027 | 11:53 |
| *** jpena is now known as jpena|lunch | 12:10 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Tenant config can be read from an external script https://review.openstack.org/535878 | 12:14 |
| *** sshnaidm|ptg has quit IRC | 12:38 | |
| *** jpena|lunch is now known as jpena | 12:56 | |
| *** snapiri has joined #zuul | 13:00 | |
| openstackgerrit | Andrea Frittoli proposed openstack-infra/zuul-jobs master: Fix the copy-build-sshkey role https://review.openstack.org/548049 | 13:13 |
| openstackgerrit | Andrea Frittoli proposed openstack-infra/zuul-jobs master: Revert "Revert "Deploy the build ssh key to auth_keys"" https://review.openstack.org/550027 | 13:13 |
| *** rlandy has joined #zuul | 13:35 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Tenant config can be read from an external script https://review.openstack.org/535878 | 13:36 |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Tenant config can be read from an external script https://review.openstack.org/535878 | 13:47 |
| *** myoung|ruck|off is now known as myoung|ruck | 14:13 | |
| *** electrofelix has quit IRC | 14:14 | |
| *** dkranz has joined #zuul | 14:22 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Tenant config can be read from an external script https://review.openstack.org/535878 | 14:25 |
| *** sshnaidm|ptg has joined #zuul | 14:35 | |
| mordred | tristanC: pip should transparently do the right thing - if you're installing from source, you need the javascript tools. if you install from a published python tarbal you should not need the javascript tooling installed | 14:41 |
| *** sshnaidm|ptg has quit IRC | 14:44 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul-jobs master: Revert "Revert "Deploy the build ssh key to auth_keys"" https://review.openstack.org/550027 | 14:48 |
| openstackgerrit | Andrea Frittoli proposed openstack-infra/zuul-jobs master: Fix the copy-build-sshkey role https://review.openstack.org/548049 | 14:50 |
| openstackgerrit | Andrea Frittoli proposed openstack-infra/zuul-jobs master: Revert "Revert "Deploy the build ssh key to auth_keys"" https://review.openstack.org/550027 | 14:50 |
| openstackgerrit | Andrea Frittoli proposed openstack-infra/zuul-jobs master: Revert "Revert "Deploy the build ssh key to auth_keys"" https://review.openstack.org/550027 | 14:51 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Set node version for javascript publish job https://review.openstack.org/550106 | 15:06 |
| Shrews | tristanC: I think your sequence handling code in the zk retry review looks good. I'm now concerned about the effects on locks (mentioned in my recent email response to the ML). | 15:49 |
| openstackgerrit | Merged openstack-infra/zuul master: Set node version for javascript publish job https://review.openstack.org/550106 | 15:54 |
| SpamapS | tobiash: yes, I believe the code needs to surface the error message from Github rather than just say "I couldn't merge this" | 16:40 |
| tobiash | SpamapS: that would be helpful | 16:40 |
| SpamapS | tobiash: I just had one of those recently, where the branch was configured to wait for a jenkins check, which was taking longer than normal, so the check/gate completed before jenkins and the user was confused.. "It merges just fine.. ?" | 16:40 |
| SpamapS | The error logs for the scheduler had the response from github. | 16:41 |
| SpamapS | Would also really be nice to be able to set requirements for a pipeline per-project sometimes. | 16:41 |
| tobiash | SpamapS: the response in my case was ref not found | 16:41 |
| tobiash | at the same time github was under more load than usual | 16:41 |
| SpamapS | I have that jenkins check listed as a trigger, so when it finishes, the change goes back into the gate, but it would be better if gate waited for the check. | 16:42 |
| SpamapS | tobiash: yeah I've had that too | 16:42 |
| SpamapS | though, they recently upgraded the GHE installation and I haven't seen it in a couple months | 16:42 |
| *** myoung|ruck is now known as myoung|ruck|lunc | 16:48 | |
| *** myoung|ruck|lunc is now known as myoung|ruck|food | 16:48 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Normalize hostname for statsd https://review.openstack.org/547309 | 17:03 |
| *** haint_ has joined #zuul | 17:11 | |
| *** jpena is now known as jpena|off | 17:12 | |
| *** haint has quit IRC | 17:13 | |
| *** hashar is now known as hasharDinner | 17:19 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Use zuul.tag in fetch-javascript-tarball https://review.openstack.org/550175 | 17:26 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Switch to publish-openstack-javascript-tarball https://review.openstack.org/550179 | 17:36 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Remove version-from-git role https://review.openstack.org/550180 | 17:39 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Normalize hostname for statsd https://review.openstack.org/547309 | 17:46 |
| *** jimi|ansible has quit IRC | 18:17 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Handle repos with no tags in version-from-git https://review.openstack.org/550175 | 18:23 |
| openstackgerrit | Andrea Frittoli proposed openstack-infra/zuul-jobs master: Fix the copy-build-sshkey role https://review.openstack.org/548049 | 18:25 |
| openstackgerrit | Andrea Frittoli proposed openstack-infra/zuul-jobs master: Fix the copy-build-sshkey role https://review.openstack.org/548049 | 18:26 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Handle repos with no tags in version-from-git https://review.openstack.org/550175 | 18:27 |
| openstackgerrit | Merged openstack-infra/zuul master: Switch to publish-openstack-javascript-tarball https://review.openstack.org/550179 | 18:29 |
| *** harlowja has joined #zuul | 18:39 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Handle repos with no tags in version-from-git https://review.openstack.org/550175 | 18:39 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Handle repos with no tags in version-from-git https://review.openstack.org/550175 | 18:44 |
| openstackgerrit | Andrea Frittoli proposed openstack-infra/zuul-jobs master: Revert "Revert "Deploy the build ssh key to auth_keys"" https://review.openstack.org/550027 | 18:51 |
| *** pwhalen has quit IRC | 19:09 | |
| openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Tenant config can be read from an external script https://review.openstack.org/535878 | 19:10 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul-jobs master: Handle repos with no tags in version-from-git https://review.openstack.org/550175 | 19:14 |
| *** myoung|ruck|food is now known as myoung|ruck | 19:15 | |
| *** pwhalen has joined #zuul | 19:16 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Fix the copy-build-sshkey role https://review.openstack.org/548049 | 19:18 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Revert "Revert "Deploy the build ssh key to auth_keys"" https://review.openstack.org/550027 | 19:18 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul master: Revert "Switch to publish-openstack-javascript-tarball" https://review.openstack.org/550217 | 19:21 |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Add role for fetching javascript built content https://review.openstack.org/550223 | 19:33 |
| dmsimard | SpamapS: https://landscape.cncf.io/selected=zuul :/ | 20:18 |
| openstackgerrit | Merged openstack-infra/zuul master: Revert "Switch to publish-openstack-javascript-tarball" https://review.openstack.org/550217 | 20:24 |
| *** hasharDinner is now known as hashar | 20:44 | |
| *** hashar is now known as hasharL4D2 | 21:11 | |
| mordred | dmsimard: sigh | 21:17 |
| mordred | dmsimard: also - enjoy how it lists Market Cap on an entry for an "Open Source" project | 21:18 |
| -openstackstatus- NOTICE: The infrastructure team is aware of replication issues between review.openstack.org and github.com repositories. We're planning a maintenance to try and address the issue. We recommend using our official supported mirrors instead located at https://git.openstack.org. | 21:19 | |
| clarkb | mordred: theoretically you can sell the copyright of an open source project right? (that said market cap makes no sense to me) | 21:22 |
| *** dkranz has quit IRC | 21:25 | |
| mordred | clarkb: well, that's ok - they list 'swift' as a cloud-native storage solution, but list it with swiftstack info rather than openstack | 21:28 |
| mordred | clarkb: I'd guess that shows the worldview | 21:28 |
| mordred | dmsimard: https://github.com/cncf/landscape/pull/315 | 21:28 |
| clarkb | mordred: thats even worse than getting the wrong zuul | 21:28 |
| dmsimard | mordred: I mentioned it here but it came up in #openstack-tc | 21:29 |
| dmsimard | I added a reference to https://github.com/Netflix/zuul/issues/315 to your PR | 21:32 |
| dmsimard | clarkb, mordred: I think the whole point of the thing might just be that sentence they put at the top: "You are viewing 468 cards with a total of 914,966 stars, market cap of $4.79T and funding of $19.1B." | 21:40 |
| dmsimard | (re: putting market cap on open source projects) | 21:41 |
| dmsimard | It's an odd metric | 21:41 |
| mordred | well - it's the Embrace-Extend-Destroy that's been running for several years now | 21:42 |
| mordred | equate "Open Source" with "Venture-Backed Open Source" | 21:42 |
| mordred | then get enough "Open Source" supporters to believe that Open Core is a good thing - by redefining what it means to be an "Open Source" supporter | 21:43 |
| mordred | then, voila! magic! the peskey parts of Open Source that the money people didn't like in the first place have been undercut - but we call it open source so 'everyone' is happy | 21:44 |
| dmsimard | Yup open core is being mixed up. It's an increasingly popular model too. | 21:44 |
| mordred | of course, I'm just that guy who believes that open source projects should only use open source tools for their development. silly me | 21:45 |
| dmsimard | Sensu, Gitlab, Influx, etc. | 21:45 |
| mordred | dmsimard: of course it is - people made it socially unacceptable to talk about the 'why' we do open source | 21:45 |
| mordred | complaining about people getting 'religious' every time they do | 21:45 |
| mordred | and responding that they "don't care about all that, just want to Get Stuff Done" | 21:46 |
| dmsimard | Then again, you could argue that Ansible and Ceph were open core as well (Tower and Calamari respectively) | 21:46 |
| mordred | dmsimard: well, I've always argued that tower doesn't matter to me because it's an additional product on top of ansible and I can happily ignore it and still use ansible ... and the callback interface is still full-featured enough for you to have written ara | 21:47 |
| pabelanger | I'm very happy at the beginning of openstack we had people cared about that, over get stuff done | 21:47 |
| mordred | pabelanger: ++ | 21:47 |
| mordred | dmsimard: BUT - I've also always been pretty vocal against tower being a thing I cannot use due to licensing | 21:48 |
| dmsimard | mordred: right, it gets bad even you cripple your "open core" on purpose to extort money | 21:48 |
| mordred | dmsimard: eucalyptus being open core is the reason openstack exists | 21:48 |
| mordred | (back when it was open core) | 21:48 |
| mordred | s/the reason/one of the reasons/ | 21:49 |
| mordred | ldap integration was an enterprise for-pay feature | 21:49 |
| dmsimard | It was also java (or is that CloudStack?) :P | 21:49 |
| mordred | meaning there was basically no ability to do auth integrations with the open source version | 21:49 |
| dmsimard | Funny you mention ldap, that's what gitlab tried to monetize off of me a few years ago | 21:50 |
| mordred | dmsimard: ldap integration is a popular upsell | 21:50 |
| mordred | the assumptoin being that only larger corporations have ldap that they care about | 21:50 |
| * mordred apologizes to the humans in the zuul channel who did not want to talk abot this topic | 21:51 | |
| *** threestrands has joined #zuul | 21:54 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Add role for fetching javascript built content https://review.openstack.org/550223 | 21:58 |
| mordred | getting closer ... | 22:02 |
| *** qwc has quit IRC | 22:02 | |
| *** qwc has joined #zuul | 22:02 | |
| *** hasharL4D2 is now known as hashar | 22:03 | |
| openstackgerrit | Merged openstack-infra/zuul master: Support brew for installing the js tools https://review.openstack.org/548235 | 22:31 |
| dmsimard | "Quadrupling Ansible performance with Mitogen": http://pythonsweetness.tumblr.com/post/171589071872/quadrupling-ansible-performance-with-mitogen | 22:49 |
| *** myoung|ruck is now known as myoung|ruck|off | 22:52 | |
| *** hashar has quit IRC | 22:57 | |
| *** hashar has joined #zuul | 23:02 | |
| pabelanger | read that this morning | 23:06 |
| *** jimi|ansible has joined #zuul | 23:06 | |
| pabelanger | good news, zuul / nodepool / diskimage-builder / etc all seem to just work on bionic :) http://logs.openstack.org/16/550216/3/check/windmill-ubuntu-bionic/3b97275/ | 23:18 |
| pabelanger | going to give bwrap a spin tomorrow and see how the version in bionic works | 23:19 |
| *** hashar has quit IRC | 23:21 | |
| persia | pabelanger: Woohoo! | 23:31 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!