| *** frenzyfriday has joined #zuul | 00:23 | |
| *** frenzyfriday has quit IRC | 00:28 | |
| *** wuchunyang has joined #zuul | 00:53 | |
| *** frenzyfriday has joined #zuul | 01:03 | |
| *** frenzyfriday has quit IRC | 01:15 | |
| *** frenzyfriday has joined #zuul | 02:03 | |
| *** frenzyfriday has quit IRC | 02:08 | |
| *** frenzyfriday has joined #zuul | 02:57 | |
| *** frenzyfriday has quit IRC | 03:10 | |
| *** frenzyfriday has joined #zuul | 03:19 | |
| *** frenzyfriday has quit IRC | 03:32 | |
| *** bhavikdbavishi has joined #zuul | 03:34 | |
| *** bhavikdbavishi1 has joined #zuul | 03:37 | |
| *** bhavikdbavishi has quit IRC | 03:39 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:39 | |
| *** wuchunyang has quit IRC | 04:03 | |
| *** armstrongs has quit IRC | 04:13 | |
| *** frenzyfriday has joined #zuul | 04:19 | |
| *** bhavikdbavishi has quit IRC | 04:24 | |
| *** bhavikdbavishi has joined #zuul | 04:24 | |
| *** frenzyfriday has quit IRC | 04:33 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** mwhahaha has quit IRC | 04:39 | |
| *** mwhahaha has joined #zuul | 04:40 | |
| *** frenzyfriday has joined #zuul | 05:22 | |
| *** frenzyfriday has quit IRC | 05:35 | |
| openstackgerrit | Merged zuul/nodepool master: Ignore unparsable/empty image upload ZNode data https://review.opendev.org/738013 | 05:41 |
|---|---|---|
| *** reiterative has quit IRC | 05:45 | |
| *** reiterative has joined #zuul | 05:45 | |
| *** frenzyfriday has joined #zuul | 06:01 | |
| *** vorotech has joined #zuul | 06:08 | |
| *** bhavikdbavishi1 has joined #zuul | 06:40 | |
| *** bhavikdbavishi has quit IRC | 06:42 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 06:42 | |
| *** frenzyfriday has quit IRC | 06:48 | |
| *** hashar has joined #zuul | 06:50 | |
| *** sshnaidm|afk is now known as sshnaidm | 06:50 | |
| *** vorotech has quit IRC | 06:50 | |
| *** bhavikdbavishi has quit IRC | 07:07 | |
| *** bhavikdbavishi has joined #zuul | 07:07 | |
| *** jcapitao has joined #zuul | 07:09 | |
| tobiash | mhu: I wonder why https://review.opendev.org/749246 is necessary. I thought the poller is only used in http-only scenarios where the password is set? | 07:13 |
| *** bhavikdbavishi has quit IRC | 07:16 | |
| *** yoctozepto2 has joined #zuul | 07:20 | |
| *** yoctozepto has quit IRC | 07:22 | |
| *** yoctozepto2 is now known as yoctozepto | 07:22 | |
| mhu | tobiash, I've seen it happen on our instance, with a connection where the password is not set, so I thought I'd submit a patch | 07:30 |
| mhu | tristanC told me he looked into it too, he might have submitted a better patch for it earlier that I wasn't aware of | 07:30 |
| mhu | uh, it was merged 3 months ago actually https://review.opendev.org/#/c/730655/ | 07:33 |
| *** tosky has joined #zuul | 07:40 | |
| *** bhavikdbavishi has joined #zuul | 07:49 | |
| *** jpena|off is now known as jpena | 07:57 | |
| *** bhavikdbavishi1 has joined #zuul | 08:03 | |
| *** bhavikdbavishi has quit IRC | 08:04 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 08:04 | |
| fbo | Hi @zuul-maint the changes to support GitLab are almost all merged but 3 are stuck in Gerrit due to a missing W+1 on the parent change https://review.opendev.org/#/c/741931 | 08:12 |
| *** nils has joined #zuul | 08:18 | |
| *** mach1na has joined #zuul | 08:32 | |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: More E208 mode fixes https://review.opendev.org/748498 | 08:33 |
| *** vishalmanchanda has joined #zuul | 08:39 | |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: More E208 mode fixes https://review.opendev.org/748498 | 08:41 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: CLI: add autohold-info, autohold-delete via REST https://review.opendev.org/728410 | 08:41 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: improve promote endpoint testing https://review.opendev.org/728489 | 08:42 |
| *** CraigR has joined #zuul | 08:43 | |
| *** mach1na has quit IRC | 08:46 | |
| *** vorotech has joined #zuul | 08:50 | |
| *** hashar is now known as hasharAway | 09:00 | |
| *** CraigR has quit IRC | 09:08 | |
| *** bhavikdbavishi has quit IRC | 09:31 | |
| *** bhavikdbavishi has joined #zuul | 09:31 | |
| *** mach1na has joined #zuul | 09:45 | |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: More E208 mode fixes https://review.opendev.org/748498 | 09:58 |
| *** hasharAway is now known as hashar | 10:10 | |
| *** mach1na has quit IRC | 10:15 | |
| *** mach1na has joined #zuul | 10:15 | |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul master: Consolidate js/jsx indentation https://review.opendev.org/747377 | 10:19 |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul master: Consolidated javascript identation https://review.opendev.org/749702 | 10:26 |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul master: Consolidate js/jsx indentation https://review.opendev.org/747377 | 10:27 |
| *** mach1na has quit IRC | 10:37 | |
| *** bhavikdbavishi has quit IRC | 10:44 | |
| *** bhavikdbavishi has joined #zuul | 10:45 | |
| *** bhavikdbavishi has quit IRC | 10:49 | |
| *** bhavikdbavishi has joined #zuul | 10:50 | |
| *** bhavikdbavishi has quit IRC | 10:54 | |
| *** jcapitao is now known as jcapitao_lunch | 11:00 | |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: Add ensure-ansible role https://review.opendev.org/749706 | 11:03 |
| *** bhavikdbavishi has joined #zuul | 11:04 | |
| *** mach1na has joined #zuul | 11:13 | |
| *** hashar is now known as hasharAway | 11:25 | |
| *** bhavikdbavishi has quit IRC | 11:31 | |
| *** jpena is now known as jpena|lunch | 11:38 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Exercise github auth handling in tests https://review.opendev.org/749711 | 11:50 |
| *** arxcruz is now known as arxcruz|ruck | 11:52 | |
| *** hasharAway is now known as hashar | 11:54 | |
| *** vorotech has quit IRC | 11:56 | |
| *** mach1na has quit IRC | 11:59 | |
| *** mach1na has joined #zuul | 12:00 | |
| *** mattd01 has joined #zuul | 12:01 | |
| *** rlandy has joined #zuul | 12:08 | |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: More E208 mode fixes https://review.opendev.org/748498 | 12:11 |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: Add ensure-ansible role https://review.opendev.org/749706 | 12:13 |
| *** jcapitao_lunch is now known as jcapitao | 12:14 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: WIP: Handle review requirements in canMerge https://review.opendev.org/749040 | 12:18 |
| *** vorotech has joined #zuul | 12:20 | |
| *** Goneri has joined #zuul | 12:31 | |
| *** bhavikdbavishi has joined #zuul | 12:34 | |
| *** jpena|lunch is now known as jpena | 12:40 | |
| *** vorotech has quit IRC | 12:58 | |
| *** bhavikdbavishi has quit IRC | 13:13 | |
| *** mach1na has quit IRC | 13:24 | |
| openstackgerrit | Merged zuul/zuul master: gitlab: support the merge reporter https://review.opendev.org/741931 | 13:24 |
| openstackgerrit | Merged zuul/zuul master: gitlab: remove default mutables https://review.opendev.org/741938 | 13:24 |
| openstackgerrit | Merged zuul/zuul master: gitlab: support the MR merged event https://review.opendev.org/742107 | 13:24 |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: More E208 mode fixes https://review.opendev.org/748498 | 13:29 |
| *** vorotech has joined #zuul | 13:51 | |
| *** AshBullock has joined #zuul | 14:14 | |
| AshBullock | Hey all, I've been doing some research into developing a driver for bitbucket cloud, there's already some great progress on a driver for bitbucket server and at first I thought it might make sense to bundle them together, but the api models are quite a bit different - they would need largely different client and connector, I was wondering in this | 14:27 |
| AshBullock | case if it would be better to create a separate driver to manage bitbucket cloud? | 14:27 |
| corvus | AshBullock: yeah, if they're very different, it sounds like it may make sense to have two drivers. also, it seems really weird to me that they would be so different. why would they make everyone write two different integrations for their product? | 14:30 |
| AshBullock | yeah, yeah - from my initial research it seems like the cloud offerings api is quite a bit more developed, and has a few differences in the object model. I'm in the early stage of developing a driver so will report my findings soon so we can make a more informed decision | 14:34 |
| fungi | if it's just a matter of the api for bbcloud is more expansive and the bitbucket api is a subset of it, then they might be able to still share a lot of code | 14:35 |
| fungi | i do agree it would be strange for them to choose to maintain two entirely different apis | 14:35 |
| corvus | AshBullock: take a look at the gerrit driver; there is a somewhat similar situation there, in that we added support for an experimental "checks api" feature which is very different than the stream-events system | 14:35 |
| corvus | AshBullock: the overall framework was still similar enough it made sense to use the same driver, it was just the event ingestion was different | 14:36 |
| corvus | AshBullock: so you might look at that and see if you think something similar might apply to BB, or if it's just completely different, then 2 drivers would be fine | 14:36 |
| corvus | AshBullock: (i should say: the event ingestion and reporting were both different) | 14:37 |
| corvus | AshBullock: (git references, data models, event types, etc were all the same) | 14:37 |
| AshBullock | thanks, I'll take a look at that example and see what I think - it's been quite hard to find any concrete docs on the differences but I did find a few threads https://community.atlassian.com/t5/Answers-Developer-Questions/Is-there-documentation-on-REST-API-differences-Bitbucket-Server/qaq-p/497461 | 14:50 |
| zbr | fungi: https://review.opendev.org/#/c/749702/1 please (trivial) | 14:50 |
| corvus | AshBullock: that's reassuring; seems like we're looking at this the right way. :) | 14:52 |
| AshBullock | seems like a rebrand of Stash led to server whereas bitbucket cloud is developed separately :D thanks for the example though I'll continue my research ... | 14:53 |
| *** AshBullock has quit IRC | 14:58 | |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul-jobs master: More E208 mode fixes https://review.opendev.org/748498 | 15:00 |
| *** ashbullock has joined #zuul | 15:04 | |
| openstackgerrit | wes hayutin proposed zuul/zuul-jobs master: Make .sh browsable on swift logs https://review.opendev.org/731795 | 15:17 |
| *** iurygregory has quit IRC | 15:24 | |
| fungi | zbr: when you say "prevent future regressions" on 749702 do you mean regressions in functionality/behavior? or regressions in whitespace? | 15:27 |
| zbr | fungi: regressions on whitespace | 15:30 |
| zbr | usually any chance should include measures to avoid having to do it again | 15:30 |
| fungi | thanks. so that change is part of a planned series which doesn't fix anything or prevent something from becoming broken? i don't really have any opinion on whitespace and indentation in javascript, i'll leave it to others who are to review that | 15:32 |
| zbr | the reason why i started this was due to a comment in one review, i promised to sort spacing everywhere, so we follow the same rules. | 15:32 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Handle review requirements in canMerge https://review.opendev.org/749040 | 15:32 |
| *** iurygregory has joined #zuul | 15:42 | |
| tristanC | corvus: tobiash: would you have a moment to talk about zuul-runner, for example is there still concerns with the spec, and could i start rebasing the freeze_job topic? | 15:43 |
| corvus | tristanC: i think the latest updates to the spec have addressed my concerns, but let take a fresh look at it in a little bit. | 15:47 |
| tristanC | corvus: thanks in advance! | 15:48 |
| *** bhavikdbavishi has joined #zuul | 15:59 | |
| *** jcapitao has quit IRC | 16:00 | |
| *** ashbullock has quit IRC | 16:07 | |
| *** rlandy is now known as rlandy|afk | 16:12 | |
| *** bhavikdbavishi1 has joined #zuul | 16:16 | |
| *** bhavikdbavishi has quit IRC | 16:17 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 16:17 | |
| clarkb | I've discoverd that pip install has a --prefer-binary flag which will prefer older wheels if available over newer sdists as long as the wheel is still in the valid range of versions. I think that there may be a way to use that for nodepool arm64 builds using the openstack wheel mirror in a way that is reliable for pynacl /me is working on that this morning | 16:35 |
| fungi | oh, yep that option is potentially useful. i thought we looked into it before for something else and wound up not using it after all, but now i don't recall what or why | 16:36 |
| fungi | it's a fairly recent addition though, like in the last year | 16:36 |
| clarkb | ya the key is it means we can use slightly older pynacl from openstack's wheel mirror when openstack constraints lag behind | 16:38 |
| clarkb | seems like a reasonable compromise for nodepool not needing to follow openstack constraints but still get the benefits of those preexisting arm64 wheels | 16:38 |
| clarkb | the trick is how to fit this into python-builder's assemble script | 16:39 |
| clarkb | I think the easiest way to do that is via our pip.conf updates we already attempt to do in the arm64 work for nodepool images. Now to see if I can set that flag in the pip.conf | 16:43 |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Use arm64 wheels from openstack if available https://review.opendev.org/741942 | 16:48 |
| clarkb | I think ^ may do it. If people like that as a compromise until we've got richer wheel mirroring set up I can squash that change into its parent ot make it ready to merge (the parent is split out to show that the extra bits are necessary to build within the hour timeout) | 16:48 |
| clarkb | also I think even once we've got richer mirroring we'll want to keep the prefer-binary flag for the reason fungi just pointed out in #openstack-infra (because the wheel mirroring lags what is avaialble on pypi) | 16:49 |
| *** hashar has quit IRC | 16:58 | |
| *** mattd01 has quit IRC | 17:01 | |
| *** jpena is now known as jpena|off | 17:03 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Handle review requirements in canMerge https://review.opendev.org/749040 | 17:08 |
| fungi | the lag is usually no more than a few hours to a day, but it can make for a broad variation in job runtimes immediately after new releases of some libraries | 17:17 |
| *** armstrongs has joined #zuul | 17:39 | |
| clarkb | fungi: and with emulated arm it means job timeouts if building pynacl | 17:43 |
| clarkb | it seems like the other packages are fine now that cryptogrpahy and bcrypt have arm64 wheels on pypi | 17:43 |
| clarkb | they take a few seconds to build wheels for which leaves pynacl as the remaining problem build under emulation | 17:44 |
| *** mattd01 has joined #zuul | 17:55 | |
| *** bhavikdbavishi has quit IRC | 17:59 | |
| *** bhavikdbavishi has joined #zuul | 17:59 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: render-diff: fix typo in failure messsage https://review.opendev.org/749809 | 18:07 |
| clarkb | https://review.opendev.org/#/c/741942/ works with prefer-binary set. I think the only other thing I notice is that we're really close to the hour timeout so maybe I need to set the timeout to 80 minutes instead | 18:31 |
| clarkb | I'd love to know what others think about that change | 18:31 |
| *** vorotech has quit IRC | 18:34 | |
| *** mattd01 has left #zuul | 18:39 | |
| *** bhavikdbavishi has quit IRC | 18:47 | |
| *** sshnaidm is now known as sshnaidm|off | 18:48 | |
| openstackgerrit | Merged zuul/zuul-jobs master: render-diff: fix typo in failure messsage https://review.opendev.org/749809 | 19:14 |
| *** vishalmanchanda has quit IRC | 19:28 | |
| *** tosky has quit IRC | 19:45 | |
| *** rlandy|afk has quit IRC | 20:53 | |
| corvus | clarkb: lgtm -- what needs to happen to merge that? | 20:56 |
| clarkb | corvus: I need to squash it into its parent and bump the timeout by 20 minutes I'll do that now | 20:57 |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Build arm64 images using arm64 wheels from openstack if available https://review.opendev.org/741942 | 21:03 |
| clarkb | corvus: ^ that should be landable now (note it needs its parent too) | 21:03 |
| corvus | clarkb: +3 (carried +2 over rebase from tobiash) | 21:05 |
| corvus | setence word order bad but you get the idea | 21:05 |
| clarkb | thanks! exciting this could get us our first arm64 production image | 21:06 |
| clarkb | in opendev I mean | 21:06 |
| corvus | ++ | 21:06 |
| clarkb | hrm py38 job failure on those (should be unrelated since none of that runs on arm64). Looks like zk connection issues maybe | 21:28 |
| *** nils has quit IRC | 21:45 | |
| *** frenzyfriday has joined #zuul | 22:03 | |
| *** frenzyfriday has quit IRC | 22:12 | |
| *** hashar has joined #zuul | 22:16 | |
| openstackgerrit | Merged zuul/nodepool master: Update bindep and vhd-util install for arm64 https://review.opendev.org/749359 | 22:28 |
| corvus | clarkb: 1/2 | 22:36 |
| clarkb | yup and the other looks good so far | 22:37 |
| *** armstrongs has quit IRC | 22:37 | |
| openstackgerrit | Merged zuul/nodepool master: Build arm64 images using arm64 wheels from openstack if available https://review.opendev.org/741942 | 23:01 |
| clarkb | ianw: ^ I think that means we have an image wecan use on nb03 now | 23:02 |
| clarkb | any idea if achange to switch ut over exists yet? | 23:02 |
| *** Goneri has quit IRC | 23:16 | |
| corvus | clarkb: did we roll it out before and roll back? so it might be a revert revert we need to do | 23:33 |
| clarkb | corvus: I wasn't able to find it | 23:34 |
| clarkb | I found where we trimmed the puppet down to just nb03 but no flip flop | 23:34 |
| *** cloudnull has quit IRC | 23:47 | |
| *** frenzyfriday has joined #zuul | 23:52 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!