| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-sphinx master: Improve warning details https://review.openstack.org/599469 | 01:10 |
|---|---|---|
| *** SpamapS has joined #zuul | 01:17 | |
| * SpamapS finally got past /usr/bin/python being missing | 01:19 | |
| SpamapS | 2018-09-04 01:21:47,233 DEBUG zuul.AnsibleJob: [build: 4635c79b877d42e8b5e57bf242a3987a] Ansible output: b"ERROR! the role 'remove-build-sshkey' was not found in | 01:22 |
| SpamapS | /tmp/tmp8iptk_kh/4635c79b877d42e8b5e57bf242a3987a/trusted/project_0/git.zuul-ci.org/zuul-base-jobs/playbooks/base/roles:/tmp/tmp8iptk_kh/4635c79b877d42e8b5e57bf242a3987a/work/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles:/tmp/tmp8iptk_kh/4635c79b877d42e8b5e57bf242a3987a/trusted/project_0/git.zuul-ci.org/zuul-base-jobs/playbooks/base" | 01:22 |
| SpamapS | hrmph | 01:23 |
| SpamapS | I do have zuul-jobs as an untrusted project.. hrm | 01:24 |
| SpamapS | for whatever reason my executor hasn't pulled in zuul-jobs | 01:29 |
| SpamapS | do I have to template it in as a required project for every job? | 01:29 |
| SpamapS | ohhh.. I think I namespaced it wrong | 01:30 |
| SpamapS | interesting. So.. if you set up your zuul git source as git.zuul-ci.org, zuul-base-jobs don't work, becauase the repos aren't under openstack-infra | 01:31 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: web: add change status page https://review.openstack.org/599472 | 01:37 |
| *** jiapei has joined #zuul | 02:55 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Increase timeout on streaming tests https://review.openstack.org/599495 | 05:24 |
| *** tobiash has joined #zuul | 05:25 | |
| *** eumel8 has quit IRC | 05:48 | |
| *** gundalow has quit IRC | 05:49 | |
| *** mugsie has quit IRC | 05:49 | |
| *** gundalow has joined #zuul | 05:49 | |
| *** zigo has quit IRC | 05:49 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Increase timeout on streaming tests https://review.openstack.org/599495 | 05:52 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Increase hard test timeout https://review.openstack.org/599495 | 05:56 |
| *** hashar has joined #zuul | 06:09 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Cap cherrypy to below 18.0.0 https://review.openstack.org/599502 | 06:34 |
| *** snapiri has joined #zuul | 06:40 | |
| openstackgerrit | Markus Hosch proposed openstack-infra/zuul master: Add a sanity check for all refs returned by Gerrit https://review.openstack.org/599011 | 06:45 |
| *** quiquell has joined #zuul | 06:46 | |
| tobiash | corvus, clarkb: we have to cap cherrypy to below 18.0.0 to unbreak tests | 06:58 |
| tobiash | corvus, clarkb: I also identified the commit in cherrypy that breaks us: https://github.com/cherrypy/cherrypy/commit/d3fa6b80b227069a795a15fc56129f44b9108292 | 06:58 |
| SpamapS | FYI, I have my zuul via AWS/EKS/Kubernetes working-ish now. Still need to add log storage and builds DB | 07:03 |
| SpamapS | also I don't quite have the ingress/nginx config right yet | 07:03 |
| tobiash | cool | 07:12 |
| *** quiquell has quit IRC | 07:24 | |
| tobiash | corvus, clarkb: I pushed up a PR that fixes the issue: https://github.com/cherrypy/cherrypy/pull/1736 | 07:33 |
| SpamapS | Wondering if I can package it up with a helm chart | 07:34 |
| SpamapS | this has been kind of massive as an effort. Would love to share it with anyone else who wants to run zuul on k8s | 07:34 |
| *** jpena|off is now known as jpena | 07:45 | |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Cap cherrypy to below 18.0.0 https://review.openstack.org/599502 | 07:51 |
| *** electrofelix has joined #zuul | 09:14 | |
| *** panda|rover has quit IRC | 09:54 | |
| *** panda has joined #zuul | 09:59 | |
| *** panda has quit IRC | 10:04 | |
| *** bhavikdbavishi has joined #zuul | 10:48 | |
| *** mugsie has joined #zuul | 10:48 | |
| *** jpena is now known as jpena|lunch | 11:02 | |
| *** panda has joined #zuul | 11:34 | |
| dmsimard | SpamapS: k8s operators are all the rage right now. | 11:42 |
| dmsimard | With CRDs. | 11:43 |
| openstackgerrit | Tobias Henkel proposed openstack-infra/zuul master: Use gearman client keepalive https://review.openstack.org/599573 | 11:57 |
| fungi | SpamapS: we should be able to patch the zuul-base-jobs repo to assume it's not using an openstack-infra namespaced zuul-jobs repo | 12:02 |
| fungi | the openstack community doesn't use what's in zuul-base-jobs, it's there as an example anyway | 12:02 |
| *** samccann has joined #zuul | 12:02 | |
| *** jpena|lunch is now known as jpena | 12:03 | |
| *** darkwisebear has joined #zuul | 12:21 | |
| *** rlandy has joined #zuul | 12:30 | |
| *** darkwisebear has quit IRC | 12:30 | |
| *** darkwisebear has joined #zuul | 12:31 | |
| *** jbryce has joined #zuul | 12:38 | |
| tristanC | corvus: mordred: jhesketh: fwiw, i wrote a blog post about React from a python dev point of view: http://www.softwarefactory-project.io/react-for-python-developers.html | 13:11 |
| jhesketh | tristanC: awesome, will read that tomorrow :-) | 13:17 |
| *** bhavikdbavishi has quit IRC | 13:37 | |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-base-jobs master: Remove zuul-jobs namespace https://review.openstack.org/599607 | 13:58 |
| fungi | SpamapS: ^ would that have helped? | 13:58 |
| pabelanger | fungi: SpamapS: we should make that fqdn, - zuul: git.zuul-ci.org/zuul-jobs | 14:01 |
| pabelanger | also, does zuul.o.o have git.zuul-ci.org connection? | 14:01 |
| fungi | pabelanger: it does not, but zuul.o.o doesn't use the zuul-base-jobs repo | 14:02 |
| fungi | pabelanger: also, won't relative naming work fine there if we're assuming that zuul-jobs is being consumed via the same connection as zuul-base-jobs? | 14:03 |
| openstackgerrit | Merged openstack-infra/nodepool master: Fix node state checks after aquiring lock https://review.openstack.org/599343 | 14:04 |
| fungi | pabelanger: also in theory it makes zuul-base-jobs more reconsumable if the user has a forked/local zuul-jobs repo? | 14:04 |
| pabelanger | fungi: maybe, but as you add more connections into zuul, it gets much harder to know where a repo lives. I've found, at least in rdoproject, namespacing projects to be much easier on end users | 14:04 |
| fungi | since the url to zuul-jobs isn't hardcoded in the base job | 14:04 |
| fungi | the base and base-test jobs in the zuul-base-jobs project are almost never likely to be used by complex deployments anyway, so i think the idea is to solve for the simple case anyway | 14:06 |
| *** maxamillion has joined #zuul | 14:10 | |
| *** jiapei has quit IRC | 14:34 | |
| dmsimard | corvus: Would you be okay with an iterative approach to the AWS EC2 nodepool driver ? I feel like attempting to land feature parity in a single patch might make it harder to review and land. | 15:16 |
| dmsimard | By features I mean like managing leaks, quotas (I see max-cores/max-ram as different than max-servers), diskimage (especially because this touches stuff outside of nodepool-launcher), cloud-image, etc | 15:17 |
| *** openstackgerrit has quit IRC | 15:20 | |
| *** shachar has joined #zuul | 15:23 | |
| Shrews | I wouldn't expect other drivers to have exact feature parity | 15:24 |
| Shrews | But we should probably establish a set of "basic things" a driver should do before we land it | 15:25 |
| Shrews | i think managing leaks falls under that | 15:25 |
| corvus | dmsimard: how about a big patch series? | 15:26 |
| dmsimard | corvus: yeah a patch series is what I had in mind | 15:27 |
| corvus | we can get the whole series looking right before we land it. you can order it by importance too, so once we hit a critical mass, we could land a chunk. | 15:27 |
| *** snapiri has quit IRC | 15:27 | |
| *** openstackgerrit has joined #zuul | 15:31 | |
| openstackgerrit | Markus Hosch proposed openstack-infra/nodepool master: Add metric for image build result https://review.openstack.org/590412 | 15:31 |
| *** sshnaidm is now known as sshnaidm|afk | 15:37 | |
| fungi | the osf marketing team is working on the one-pager glossy handout for zuul (e.g., to have on hand at ansiblefest) and is hoping we can give them a shorter url for https://zuul-ci.org/docs/zuul/admin/zuul-from-scratch.html | 15:40 |
| fungi | would we want to add a redirect via a .htaccess file for that? | 15:41 |
| clarkb | I wonder if sphinx supports that functionality in a sphinxy way too | 15:42 |
| clarkb | but .htaccess should work as long as vhost allows for it | 15:42 |
| fungi | well, the sphinx roots are fairly far down too | 15:43 |
| fungi | https://zuul-ci.org/docs/zuul/zfs wouldn't really be too much better | 15:43 |
| fungi | i think they're wanting something easy to type with your thumb on a cell phone while you're wandering away from the booth with your flyer | 15:44 |
| clarkb | good point | 15:44 |
| * fungi thinks "this is what qrcodes are for" | 15:45 | |
| pabelanger | how has foundation done it in the past with log URLs? | 15:47 |
| fungi | what log urls? | 15:48 |
| pabelanger | sorry, long* URLs | 15:49 |
| fungi | they've mostly only promoted things on https://www.openstack.org/ in the past and set up one-word redirects at the docroot level for things they want to put in print media like brochures | 15:50 |
| fungi | i think we'd rather they use a zuul-ci.org domain in the url rather than openstack.org | 15:52 |
| clarkb | right so to make it more concret https://zuul-ci.org/get-started or similar would be what they are looking for | 15:53 |
| clarkb | in the past that has been https://openstack.org/get-started for openstack and they manage that webserver | 15:53 |
| pabelanger | yah, that seems user friendly | 15:57 |
| *** jlvilla-viva is now known as jlvillal | 15:58 | |
| corvus | i think a .htaccess at root sounds like a good solution | 16:00 |
| *** hashar is now known as hasharAway | 16:21 | |
| *** darkwisebear has quit IRC | 16:28 | |
| *** bhavikdbavishi has joined #zuul | 16:47 | |
| *** bramwelt has joined #zuul | 16:50 | |
| *** dkehn has quit IRC | 16:56 | |
| bhavikdbavishi | @here need help on 'nodepool', we are process of migrating existing zuul v2 to zuul v3. | 16:57 |
| bhavikdbavishi | existing slaves are configured and get connect with 'jenkins' user. | 16:57 |
| bhavikdbavishi | now while setting up with nodepool, how do I make use of same 'jenkins' user and 'id_rsa'? | 16:57 |
| bhavikdbavishi | I did update '.ssh/config' with below content, but when I try to run 'ssh-keyscan -t ed25519 <slave>' it results empty. | 16:57 |
| bhavikdbavishi | Host * | 16:57 |
| bhavikdbavishi | IdentityFile /var/lib/zuul/.ssh/jenkins_id_rsa | 16:57 |
| bhavikdbavishi | User jenkins | 16:57 |
| bhavikdbavishi | I can specify user name to nodepool.yaml, but I couldn't find way to specify 'sshkey' info. OR do I need to specify elsewhere? | 16:57 |
| *** hwoarang_ has joined #zuul | 16:57 | |
| *** hwoarang__ has joined #zuul | 16:58 | |
| *** dkehn has joined #zuul | 16:59 | |
| pabelanger | bhavikdbavishi: nodepool no longer connected to a remote node via SSH, it just does a keyscan to validate the node is online | 17:00 |
| pabelanger | also, nodepoolv3 doesn't have support for jenkins. I have seen some project that try to add support for it | 17:00 |
| pabelanger | IIRC: https://wiki.jenkins.io/display/JENKINS/NodePool+Agents+Plugin | 17:01 |
| *** hwoarang has quit IRC | 17:01 | |
| *** hwoarang_ has quit IRC | 17:02 | |
| *** hwoarang has joined #zuul | 17:03 | |
| *** hwoarang__ has quit IRC | 17:08 | |
| *** jpena is now known as jpena|off | 17:16 | |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-website master: Add a get-started redirect https://review.openstack.org/599699 | 17:19 |
| fungi | corvus: clarkb: pabelanger: thanks. straw-man ^ | 17:20 |
| fungi | also it might be nice to get a second review on https://review.openstack.org/591870 (promotional banner mentioning ansiblefest presence) | 17:20 |
| corvus | fungi: tobiash +2d, i +3d | 17:21 |
| *** openstackgerrit has quit IRC | 17:22 | |
| fungi | thanks! | 17:22 |
| *** openstackgerrit has joined #zuul | 17:25 | |
| openstackgerrit | Merged openstack-infra/zuul-website master: Add a promotional message banner and events list https://review.openstack.org/591870 | 17:25 |
| bhavikdbavishi | pabelanger: ok, thanks for the info. | 17:27 |
| fungi | corvus: clarkb: pabelanger: if we can at least get consensus on what the url should be, that will avoid getting in the way of print deadlines to have the materials in time for ansiblefest | 17:36 |
| pabelanger | +2 | 17:39 |
| fungi | i chose /get-started in honor of the "get started" button it'll be backing | 17:40 |
| corvus | fungi: 301 and not 302? | 17:43 |
| corvus | fungi: mostly thinking -- if we're considering changing the target in the future, should it be 302? | 17:43 |
| fungi | yes, thanks! i did in fact mean temporary redirect. will fix | 17:43 |
| corvus | fungi: cool. other than that, lgtm. | 17:44 |
| fungi | as for the global url in the index page there, was there a reason we didn't make it a local link instead? | 17:44 |
| fungi | not that it particularly helps in this case because we can't currently test the redirect via logs.o.o anyway | 17:44 |
| fungi | oh, and the absoluteness would present a problem regardless | 17:45 |
| corvus | fungi: i don't recall a reason for that; probably just copy/pasted a url into the text | 17:46 |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-website master: Add a get-started redirect https://review.openstack.org/599699 | 17:46 |
| fungi | there are a few other global urls for https://zuul-ci.org/something in there anyway (for example, docs) | 17:46 |
| *** electrofelix has quit IRC | 17:58 | |
| clarkb | fungi: minor thing on https://review.openstack.org/#/c/599699/2 wasn't sure if that is itnentional or not | 18:28 |
| fungi | clarkb: thanks, i'll have to test it | 18:33 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/nodepool master: uncap sphinx https://review.openstack.org/599742 | 18:45 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul master: Uncap sphinx https://review.openstack.org/599743 | 18:48 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul master: Uncap sphinx https://review.openstack.org/599743 | 18:52 |
| *** bhavikdbavishi has quit IRC | 19:07 | |
| openstackgerrit | Jeremy Stanley proposed openstack-infra/zuul-website master: Add a get-started redirect https://review.openstack.org/599699 | 19:17 |
| openstackgerrit | Doug Hellmann proposed openstack-infra/nodepool master: uncap sphinx https://review.openstack.org/599742 | 19:19 |
| openstackgerrit | Andreas Jaeger proposed openstack-infra/zuul master: Uncap sphinx https://review.openstack.org/599743 | 19:21 |
| *** goern has quit IRC | 19:34 | |
| *** elyezer_ has joined #zuul | 20:27 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: add-build-sshkey: Remove only the master key https://review.openstack.org/599073 | 20:27 |
| *** elyezer__ has quit IRC | 20:30 | |
| corvus | i've seen two changes fail 3 TestStreaming tests on both py35 and py36 | 20:57 |
| corvus | that seems unusually reliable; i wonder if a dependency changed | 20:57 |
| corvus | there is a new cherrypy | 20:58 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Add private key storage migration https://review.openstack.org/592213 | 20:59 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Change project.private_key to private_secrets_key https://review.openstack.org/597302 | 20:59 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Create per-project ssh keys https://review.openstack.org/597711 | 20:59 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Provide per-project ssh key to executor https://review.openstack.org/597727 | 20:59 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Serve project SSH keys and document https://review.openstack.org/599063 | 20:59 |
| *** samccann has quit IRC | 21:01 | |
| corvus | there definitely seems to be a problem with the new ws4py | 21:30 |
| corvus | er, sorry ws4py is the same, but that's where the error is coming from | 21:33 |
| clarkb | cherrypy update incompatible with existing ws4py then? | 21:34 |
| corvus | i suspect this: https://github.com/cherrypy/cherrypy/commit/a95e619f | 21:36 |
| clarkb | corvus: cherrpy changes file says the big change in 18.0.0 is dropping python2.7 support. We don't python27 so probably not related, however 17.x is an LTS supported version as a result if we want to pin to 17.0 | 21:36 |
| corvus | clarkb: yeah, ping to 17 is probably the quick fix | 21:36 |
| corvus | though i think we'd have to pin to 17.3.0, not 17.4 since that ^ made it into 17.4 | 21:37 |
| corvus | ws4py.exc.HandshakeError: Invalid challenge response: b"b's7t5jqfxalziz8svzckiqazs08a='" wanted b'S7t5jQFxaLzIZ8svZckiQAZS08A=' | 21:37 |
| corvus | that's from my local debugging ^ | 21:37 |
| corvus | i'll try 17.4 locally and check | 21:38 |
| corvus | there is no 17.4.0 | 21:39 |
| corvus | https://pypi.org/project/CherryPy/#history | 21:39 |
| corvus | so i guess any 17 will work. until they release 17.4.0 :) | 21:40 |
| corvus | i'm puzzled. https://github.com/cherrypy/cherrypy/commit/a95e619f looks exactly equivalent when running under py3, which we are and have been | 21:41 |
| clarkb | is it actually running under python2 somehow? | 21:42 |
| corvus | no, we don't work under py2 at all | 21:42 |
| corvus | must be a different change | 21:42 |
| corvus | it's this one: https://github.com/cherrypy/cherrypy/commit/d3fa6b80b227069a795a15fc56129f44b9108292 | 21:50 |
| corvus | ws4py supplies the header as bytes | 21:50 |
| clarkb | odd that they would use six if python3 only | 21:51 |
| clarkb | corvus: so it converts the byte string to "b'foo'" then converts that to bytes again as b"b'foo'" | 21:51 |
| pabelanger | corvus: clarkb: https://github.com/cherrypy/cherrypy/pull/1736 is in backscoll from tobiash | 21:54 |
| *** hasharAway has quit IRC | 21:56 | |
| corvus | oh missed that :) | 21:58 |
| corvus | and... there's the change from tobiash to pin :) | 21:59 |
| corvus | that's what i get for taking a holiday | 22:00 |
| corvus | clarkb, pabelanger: want to +3 https://review.openstack.org/599502 ? | 22:00 |
| pabelanger | ha, I missed that too | 22:00 |
| clarkb | done | 22:00 |
| *** ianychoi has quit IRC | 22:22 | |
| corvus | i think we're going to have to squash the cherrypy and sphinx fixes | 22:38 |
| corvus | i'll do that now | 22:38 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Cap cherrypy and uncap sphinx https://review.openstack.org/599502 | 22:41 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Add private key storage migration https://review.openstack.org/592213 | 22:42 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Change project.private_key to private_secrets_key https://review.openstack.org/597302 | 22:42 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Create per-project ssh keys https://review.openstack.org/597711 | 22:42 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Provide per-project ssh key to executor https://review.openstack.org/597727 | 22:42 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul master: Serve project SSH keys and document https://review.openstack.org/599063 | 22:42 |
| corvus | i went ahead and rebased those on the dependency fix, so hopefully we can go ahead and approve those | 22:43 |
| *** rlandy is now known as rlandy|bbl | 22:45 | |
| openstackgerrit | Merged openstack-infra/zuul-sphinx master: Improve warning details https://review.openstack.org/599469 | 22:46 |
| corvus | ianw: i'll tag 6106a9051f393b6140fb60e452cf384cb96616fa as 0.2.4 | 22:57 |
| *** ianychoi has joined #zuul | 23:16 | |
| ianw | thanks! | 23:20 |
| openstackgerrit | Merged openstack-infra/zuul master: Cap cherrypy and uncap sphinx https://review.openstack.org/599502 | 23:47 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!