Wednesday, 2021-04-28

*** hrw has quit IRC00:10
*** hrw has joined #opendev00:12
*** CeeMac has quit IRC00:12
*** hrw has quit IRC00:46
*** hrw has joined #opendev00:47
*** pongboom has quit IRC00:49
openstackgerritMerged opendev/glean master: Fix Gentoo "is" comparisons  https://review.opendev.org/c/opendev/glean/+/78812600:58
kevinzianw: Sorry yesterday I do not check the irc.  Is there anything I can help?01:30
ianwkevinz: we were wondering if it would be possible (maybe even better?) to upload .raw images to linaro, instead of qcow2?01:31
kevinzianw: Yes, I think it should be fine01:31
ianwkevinz: ok, we can propose a change to switch it; should be a bit friendlier on the builder as it won't have to also do qcow201:33
kevinzianw: OK, cool01:35
ianwkevinz: other than that, i did a bit of ping testing.  there still seems to be the odd packet loss01:35
ianwI was just doing with "ping -D -O -q -<4 / 6> zk02.openstack.org01:36
ianwboth ipv6 and ipv4 dropped some packets; not many, but i guess enough to be annoying to long-running things like image uploads and zk connections01:36
kevinzianw: Could I ping zk02.openstack.org from our router? I need to check if there are some issue with infra netowrk01:37
ianwkevinz: sure.  actually clarkb is in the process of changing them to zk04, zk05, zk06 as part of upgrades01:38
ianwbut i think any of them exhibit the same issue01:40
ianw2021-04-28 00:03:04,652 INFO kazoo.client: Zookeeper connection established, state: CONNECTED01:40
ianw2021-04-28 00:26:30,509 INFO kazoo.client: Zookeeper connection established, state: CONNECTED01:40
ianw2021-04-28 00:58:03,900 INFO kazoo.client: Zookeeper connection established, state: CONNECTED01:40
ianw2021-04-28 01:04:29,465 INFO kazoo.client: Zookeeper connection established, state: CONNECTED01:40
ianw2021-04-28 01:25:56,801 INFO kazoo.client: Zookeeper connection established, state: CONNECTED01:40
ianwthat's the dropped connections from midnight01:41
kevinzOK01:44
kevinzianw: how long period can you observe packet loss?01:45
ianwi guess i left it running for 3851 packet, i think it does one a second, so about an hour01:47
*** hamalq has quit IRC01:52
*** hamalq has joined #opendev01:52
*** priteau has quit IRC02:03
kevinzianw: OK, I'm trying on our router02:15
*** redrobot has quit IRC02:45
*** hemanth_n has joined #opendev02:49
openstackgerritMerged opendev/glean master: Update hacking and fix pep8 violations  https://review.opendev.org/c/opendev/glean/+/78229602:54
*** ykarel has joined #opendev03:41
*** redrobot has joined #opendev03:50
*** vishalmanchanda has joined #opendev04:40
*** mkowalski has quit IRC04:42
*** mkowalski has joined #opendev04:42
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Remove dib-nodepool-functional-openstack-ubuntu-bionic  https://review.opendev.org/c/openstack/diskimage-builder/+/78840405:06
*** ykarel has quit IRC05:09
*** marios has joined #opendev05:20
*** ralonsoh has joined #opendev05:27
*** sboyron_ has joined #opendev05:57
*** slaweq has joined #opendev06:00
*** sboyron has joined #opendev06:01
*** sboyron_ has quit IRC06:01
*** lpetrut has joined #opendev06:16
*** ysandeep|away is now known as ysandeep06:17
*** eolivare has joined #opendev06:25
*** ykarel has joined #opendev06:56
*** amoralej|off is now known as amoralej06:58
*** hashar has joined #opendev07:00
*** fressi has joined #opendev07:02
*** hamalq has quit IRC07:03
*** ykarel is now known as ykarel|pto07:13
*** andrewbonney has joined #opendev07:15
*** ykarel|pto has quit IRC07:21
*** jpena|off is now known as jpena07:21
*** rpittau|afk is now known as rpittau07:36
*** hamalq has joined #opendev07:39
*** ysandeep is now known as ysandeep|lunch08:02
*** hamalq has quit IRC08:02
*** hamalq has joined #opendev08:09
*** brinzhang0 has joined #opendev08:39
*** mordred has quit IRC08:41
*** brinzhang_ has quit IRC08:42
*** mordred has joined #opendev08:50
kevinzianw: checking from our router, 2.5 hours without any packet loss for IPV4.  Will check the network issue from our internal cluster08:50
*** mkowalski has quit IRC09:09
*** mkowalski has joined #opendev09:11
*** dhellmann has quit IRC09:17
*** dhellmann has joined #opendev09:17
*** priteau has joined #opendev09:22
*** ysandeep|lunch is now known as ysandeep09:26
*** hashar has quit IRC09:35
*** hamalq has quit IRC09:44
*** tosky has joined #opendev09:45
*** vishalmanchanda has quit IRC10:00
*** vishalmanchanda has joined #opendev10:11
*** whoami-rajat_ has joined #opendev10:22
*** dtantsur is now known as dtantsur|brb10:32
*** brinzhang_ has joined #opendev10:59
*** brinzhang0 has quit IRC11:02
*** jpena is now known as jpena|lunch11:31
*** dtantsur|brb is now known as dtantsur11:41
fungiianw: clarkb (when you wake up): this is an interesting failure condition: https://zuul.opendev.org/t/openstack/build/f5858c54e9e2473baf9634a26ee8242e12:02
fungi"Ensure Zookeeper not running" fails on a dib-nodepool-functional-openstack-centos-8-stream-src job12:02
*** ysandeep is now known as ysandeep|afk12:12
*** jpena|lunch is now known as jpena12:19
*** amoralej is now known as amoralej|lunch12:22
*** whoami-rajat_ is now known as whoami-rajat12:32
*** hemanth_n has quit IRC12:32
openstackgerritdaniel.pawlik proposed opendev/lodgeit master: Fixed issue related to new Pillow lib  https://review.opendev.org/c/opendev/lodgeit/+/78773412:40
dpawlikfungi, clarkb: hey, could you check later please ^^12:40
*** lpetrut has quit IRC12:44
*** ysandeep|afk is now known as ysandeep12:56
*** iurygregory has quit IRC12:59
fungidpawlik: frickler left you some comments13:09
dpawlikack, thanks fungi and frickler13:12
*** amoralej|lunch is now known as amoralej13:15
*** toomer has quit IRC13:19
*** toomer has joined #opendev13:19
*** fdegir has quit IRC13:23
*** fdegir has joined #opendev13:24
openstackgerritdaniel.pawlik proposed opendev/lodgeit master: Fix issue related to new Pillow lib; change hash string encode  https://review.opendev.org/c/opendev/lodgeit/+/78773413:28
*** iurygregory has joined #opendev13:35
*** mkowalski has quit IRC13:40
*** mkowalski has joined #opendev13:41
*** hrw has quit IRC13:52
*** zoharm has joined #opendev14:14
*** fressi has quit IRC14:18
openstackgerritdaniel.pawlik proposed opendev/lodgeit master: Fix issue related to new Pillow lib; change hash string encode  https://review.opendev.org/c/opendev/lodgeit/+/78773414:39
clarkbfungi: I'll take a look. Making some tea now14:48
clarkbfungi: I plan to try and catch up on the openstack logging stuff too (can move that into the dib side if no one else has)14:48
*** zoharm has quit IRC14:59
*** hashar has joined #opendev15:03
fricklermnaser: do you have any update on the v6 situation for vexxhost yet?15:12
clarkbfungi: I suspect that `pkill -f zookeeper` returns >0 if zookeeper is not running and that is why we ignore errors on that task15:18
clarkbfungi: ianw: the actual error appears to be 'manifest for zuul/nodepool-launcher:siblings not found: manifest unknown: manifest unknown'15:19
clarkbwhich implies the parent job that does image builds didn't work?15:19
*** mlavalle has joined #opendev15:22
clarkbcorvus: looks like the intermediate registry is having socket/ssl problems? `sudo docker logs --since 2021-04-28T00:00:00 registry-docker_registry_1` I'm not quite sure how to parse what that is telling us yet15:25
clarkbbut it seems to be causing jobs like the dib functional jobs to timeout15:25
clarkbseems like maybe we aren't catching some exception that we should be and that caused the whole thing to crash? however the actual process and the container seem to be running15:26
clarkb(I'm inclined to restart it but wanted to give you a chance to look at the error first in case we want to do further debugging)15:27
clarkbfwiw the tracebacks don't seem to bubble up into our application, they stop within cherrypy so not clear if we should be handling that ourselves or if this is a cherrypy bug that needs fixing15:32
*** d34dh0r53 has quit IRC15:48
*** ysandeep is now known as ysandeep|away15:56
clarkbok I'm going to go ahead and restart it now. I recordred the tracebacks in my homedir on that host15:57
openstackgerritMerged opendev/lodgeit master: Fix issue related to new Pillow lib; change hash string encode  https://review.opendev.org/c/opendev/lodgeit/+/78773415:58
*** d34dh0r53 has joined #opendev15:59
*** marios is now known as marios|out16:13
*** ysandeep|away is now known as ysandeep16:14
*** jpena is now known as jpena|off16:18
yoctozeptoI love it how gerrit says it's running dirty16:18
clarkbbecause we do our own builds (there is probably a way to fix that)16:19
*** dtantsur is now known as dtantsur|afk16:22
*** d34dh0r53 has quit IRC16:22
*** marios|out has quit IRC16:30
*** d34dh0r53 has joined #opendev16:30
corvusclarkb: interesting; i'm unfamiliar with those errors, and they don't mean anything to m....16:37
clarkbcorvus: that was the end of the log too so I'm pretty sure it was functionally fatal but the process kept going16:38
clarkbI suspect a bug in cherrypy16:38
clarkbmaybe we should rebuild the registry and pick up newer cherrypy if available?16:39
*** rpittau is now known as rpittau|afk16:39
openstackgerritGhanshyam proposed opendev/irc-meetings master: Update TC office hours time for Xena cycle  https://review.opendev.org/c/opendev/irc-meetings/+/78855216:39
corvusclarkb: maybe; i just happened to be writing 2 changes to the registry; we could merge those :)16:39
clarkb++ I can take a look shortly16:40
*** amoralej is now known as amoralej|off16:42
openstackgerritClark Boylan proposed openstack/diskimage-builder master: debian-minimal: Set bullseye version  https://review.opendev.org/c/openstack/diskimage-builder/+/78766516:48
openstackgerritClark Boylan proposed openstack/diskimage-builder master: Collect openstack logs  https://review.opendev.org/c/openstack/diskimage-builder/+/78855316:48
clarkbfungi: ianw  ^ that moves the openstack log collection into dib whcih should make that mergeable16:48
*** hamalq has joined #opendev17:04
*** hamalq has quit IRC17:05
*** hamalq has joined #opendev17:06
openstackgerritJeremy Stanley proposed openstack/diskimage-builder master: debian-minimal: bullseye: /updates -> -security  https://review.opendev.org/c/openstack/diskimage-builder/+/78745417:07
fungirackspace opened tickets to let us know that logstash-worker06.openstack.org and lists.katacontainers.io were both on hardware which experienced a failure17:11
fungilists.k.i seems to be up and running the various python processes mailman expects17:12
*** ysandeep is now known as ysandeep|away17:14
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (4)  https://review.opendev.org/c/openstack/project-config/+/78855517:22
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (5)  https://review.opendev.org/c/openstack/project-config/+/78855617:28
*** lbragstad has quit IRC17:37
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (6)  https://review.opendev.org/c/openstack/project-config/+/78855717:37
*** lbragstad has joined #opendev17:38
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (7)  https://review.opendev.org/c/openstack/project-config/+/78855817:42
*** gothicserpent has quit IRC17:43
*** eolivare has quit IRC17:44
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (8)  https://review.opendev.org/c/openstack/project-config/+/78856117:49
*** gothicserpent has joined #opendev17:49
openstackgerritKevin Putnam proposed openstack/project-config master: Adds r/stx.5.0 to the list of branches to promote for starlingx docs.  https://review.opendev.org/c/openstack/project-config/+/78856217:52
*** ralonsoh has quit IRC17:53
*** andrewbonney has quit IRC18:00
openstackgerritRadosÅ‚aw Piliszek proposed opendev/gerritbot master: Add branch to all the remaining event messages  https://review.opendev.org/c/opendev/gerritbot/+/78856518:18
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (9)  https://review.opendev.org/c/openstack/project-config/+/78856718:26
*** ifest has joined #opendev18:38
openstackgerritRadosÅ‚aw Piliszek proposed opendev/gerritbot master: Add branch to all the remaining event messages  https://review.opendev.org/c/opendev/gerritbot/+/78856518:39
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (10)  https://review.opendev.org/c/openstack/project-config/+/78856918:46
yoctozeptoinfra-root: how costly would it be to make gerritbot run with the fix above? ^ :-)18:51
fungiyoctozepto: do you mean in beers, or bitcoin? ;)18:52
clarkbyoctozepto: I'm not sure I agree with the premise. Many repos are single branch18:52
yoctozeptofungi: I mean in using your precious time18:53
yoctozeptoclarkb: other messages show branch18:53
clarkbthe biggest cost is you'll be losing message bytes in an already constrained system18:53
* yoctozepto needs to rethink the way he builds his questions18:53
clarkbyoctozepto: ya I'm not saying I'm against it. Just want to point out that the branch isn't necessarily useful even if we emit it18:53
fungiright, as it stands the messages are already a bit longish and in many cases may exceed the max message length for irc and get truncated/rejected/other breakage18:54
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (11)  https://review.opendev.org/c/openstack/project-config/+/78857118:54
clarkband the biggest "cost" is irc messages are limited to 512 bytes including the message prefix stuff so you'd be cutting into that18:54
yoctozepto512 bytes is a lot, long commit titles are bad anyway18:55
openstackgerritKevin Putnam proposed openstack/project-config master: Adds r/stx.5.0 to the list of branches to promote for starlingx docs.  https://review.opendev.org/c/openstack/project-config/+/78856218:56
clarkbyup18:56
clarkbmy hunch is that this was simply overlooked and not intentional18:56
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (12)  https://review.opendev.org/c/openstack/project-config/+/78857218:56
yoctozeptoclarkb: I checked git history and it was no branch at first18:57
fungi512 bytes aren't really a *lot* when you consider that includes the channel name, bot nick, and whatever fields are being included in the message, along with multi-byte utf-8 encoding of some chanracters... at a minimum the change subject is what should wind up truncated to compensate, but right now it's not the last field (the change url is) so that's not trivial18:57
yoctozeptothen it was added to the other messages18:57
clarkbfungi: ya but the vast majority seem to come in well under 51218:58
clarkbjust based on what gerritbot does emit18:58
fungiwe might also be able to save some bytes by converting to "old-style" gerrit urls (stripping the project name)18:58
yoctozeptooh, that would be nice18:58
yoctozeptoas the project name is already embedded18:59
yoctozeptowe can also likely change the order so that the message goes last18:59
yoctozeptobut that's optional18:59
fungithe counterargument there is that old-style gerrit urls are no longer guaranteed to be unique, i think? but in practice they still seem to be (for now anyway)19:00
clarkbfungi: they are unique, there is a change number counter somewhere19:00
clarkbI think in all-projects19:00
*** hashar has quit IRC19:01
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (13)  https://review.opendev.org/c/openstack/project-config/+/78857319:02
*** ifest has quit IRC19:03
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (14)  https://review.opendev.org/c/openstack/project-config/+/78857419:04
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (15)  https://review.opendev.org/c/openstack/project-config/+/78857719:08
clarkbsuccess! https://zuul.opendev.org/t/openstack/build/c7c698c95f2b4590ba95f14a6c97ffc0/log/nodepool/openstack/screen-n-api.txt?severity=3#1803 and by that I mean it failed19:11
clarkbhttps://zuul.opendev.org/t/openstack/build/c7c698c95f2b4590ba95f14a6c97ffc0/log/nodepool/openstack/screen-keystone.txt?severity=0#8308 keystone doesn't seem to provide much more info though19:12
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (16)  https://review.opendev.org/c/openstack/project-config/+/78857819:13
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (17)  https://review.opendev.org/c/openstack/project-config/+/78858019:17
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (18)  https://review.opendev.org/c/openstack/project-config/+/78858119:19
openstackgerritMerged openstack/project-config master: Adds r/stx.5.0 to the list of branches to promote for starlingx docs.  https://review.opendev.org/c/openstack/project-config/+/78856219:21
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (2)  https://review.opendev.org/c/openstack/project-config/+/78673919:22
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (19)  https://review.opendev.org/c/openstack/project-config/+/78858419:27
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (3)  https://review.opendev.org/c/openstack/project-config/+/78674019:28
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (4)  https://review.opendev.org/c/openstack/project-config/+/78855519:29
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (17)  https://review.opendev.org/c/openstack/project-config/+/78858019:33
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (16)  https://review.opendev.org/c/openstack/project-config/+/78857819:33
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (18)  https://review.opendev.org/c/openstack/project-config/+/78858119:33
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (5)  https://review.opendev.org/c/openstack/project-config/+/78855619:40
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (6)  https://review.opendev.org/c/openstack/project-config/+/78855719:40
*** vishalmanchanda has quit IRC19:40
*** sboyron has quit IRC19:41
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (7)  https://review.opendev.org/c/openstack/project-config/+/78855819:44
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (8)  https://review.opendev.org/c/openstack/project-config/+/78856119:44
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (9)  https://review.opendev.org/c/openstack/project-config/+/78856719:44
clarkbianw: fungi: to tl;dr the dib functional job failure it seems that the domain is not being supplied with the request. However, we're sourcing the devstack openrc which does set the domain env vars. I've asked about this in #openstack-qa but no responses yet. I think it is possible that this may be a nova issue as it seems to be happening via nova's validate_networks() call and other requests prior19:47
clarkbto the server create work fine using the same credentials19:48
clarkbno clear cause yet though19:48
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (10)  https://review.opendev.org/c/openstack/project-config/+/78856919:51
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (11)  https://review.opendev.org/c/openstack/project-config/+/78857119:51
clarkbhrm maybe it is happening after validate_networks19:52
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (12)  https://review.opendev.org/c/openstack/project-config/+/78857219:52
clarkb_refresh_neutron_extensions_cache() is in the traceback19:52
clarkbI wonder a stale cache / timing issue is why it doesn't fail consistently19:52
*** lbragstad_ has joined #opendev20:01
*** lbragstad has quit IRC20:03
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (13)  https://review.opendev.org/c/openstack/project-config/+/78857320:06
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (14)  https://review.opendev.org/c/openstack/project-config/+/78857420:06
openstackgerritElod Illes proposed openstack/project-config master: Move projects under meta-config acl (15)  https://review.opendev.org/c/openstack/project-config/+/78857720:07
fungiclarkb: openstacksdk regression maybe?20:50
fungithe cliaas work hasn't merged yet, right?20:51
*** whoami-rajat has quit IRC20:52
*** brinzhang0 has joined #opendev20:59
*** brinzhang_ has quit IRC21:02
*** mkowalski has quit IRC21:11
*** mkowalski has joined #opendev21:11
openstackgerritKevin Putnam proposed openstack/project-config master: Adds docs_branch_path value needed for promoting release branches.  https://review.opendev.org/c/openstack/project-config/+/78859321:18
*** slaweq has quit IRC21:25
ianwclarkb: o/ ... just catching up.  so we don't have issues with the intermediate registry any more?  i feel like one of my jobs hit that21:33
fungiianw: in theory the restart has cleared that problem21:39
ianwfungi: i got a response overnight on the aarch64 glibc question in #centos-stream: "<carlwgeorge> ianw: it will be released soon"21:46
ianwso make of that what you will21:46
clarkbfungi: ianw  correct looks like the registry restart fixed it21:58
clarkbas far as I could tell it had crashed (tracebacks in my homedir on the host)21:58
clarkbbut the process was still running so docker didn't restart it21:59
clarkbok it has been about 24 hours since the zk cluster upgrade. I haven't seen any indication we will need to rollback22:00
clarkbinfra-root ^ ya'll good with me cleaning up the 3 old hosts now?22:00
ianw++22:03
corvus++22:05
clarkb#status log Deleted zk01-zk03.openstack.org as they have been replaced with zk04-06.opendev.org22:07
openstackstatusclarkb: finished logging22:07
clarkbianw: fwiw I think https://review.opendev.org/c/openstack/diskimage-builder/+/788553/1 is working proeprly if we can just get it to land now :)22:14
clarkbI'll abandon the nodepool change that did similar22:14
ianwhaha yeah, i want to take more of a look in a little after school drop off22:16
ianwi guess we could put a speculative hold on all functional jobs and run it through a few times, hopefully catching one22:16
clarkbya that may be the next thing to do. Then if the issue is persistent we could hand it off to nova if necessary22:17
*** tosky has quit IRC22:48
corvusianw: re your q in #zuul23:21
ianwVariant <Job dib-nodepool-functional-openstack-centos-8-stream-src branches: {MatchAny:{ImpliedBranchMatcher:master}} source: openstack/diskimage-builder/.zuul.d/jobs.yaml@master#167> did not match <Change 0x7ffa649ab340 openstack/openstacksdk 788420,1>23:21
corvus2021-04-28 21:45:33,181 DEBUG zuul.layout: [e: 75d0d82962bd45cc9a168a39c1144cee] Variant <Job dib-nodepool-functional-openstack-centos-8-stream-src branches: {MatchAny:{ImpliedBranchMatcher:master}} source: openstack/diskimage-builder/.zuul.d/jobs.yaml@master#167> did not match <Change 0x7ffa649ab340 openstack/openstacksdk 788420,1>23:21
ianwi think it must be a branch matcher23:21
corvusianw: also 2021-04-28 21:45:33,181 DEBUG zuul.layout: [e: 75d0d82962bd45cc9a168a39c1144cee] No matching parents for job dib-nodepool-functional-openstack-centos-8-stream-src and change <Change 0x7ffa649ab340 openstack/openstacksdk 788420,1>23:22
corvusianw: and that change is on stable/victoria, so likely some branch mismatching going on23:23
ianwyeah, the job hierarchy is long23:23
ianwdib-nodepool-functional-openstack-centos-8-stream-src -> dib-nodepool-functional-src-base -> nodepool-functional-container-openstack-siblings-base23:24
ianw-> nodepool-functional-container-openstack-base23:26
corvusianw: dib is a multi-branch job23:28
corvusianw: which is why it's getting an implied branch matcher attached to it23:28
clarkbbecause the feature/v2 branch exists? I think that may have served its usefulness and we can tag it and then delete it?23:29
corvusianw: assuming you don't want to remove the feature/v2 branch, you may need to set a pragma to tell zuul not to do that23:29
corvusianw: https://zuul-ci.org/docs/zuul/reference/pragma_def.html#attr-pragma.implied-branch-matchers23:29
corvusclarkb: ya23:29
corvusclarkb: delete would be my recommendation, but you know me, i love deleting branches ;)23:29
clarkbha, in this case I'm fairly certain the v2 work completed a while back23:29
ianwahh.  yeah v2 was a branch point where we could backport image building changes but keep python2 compat, if we needed such a thing23:30
ianwi'd agree that ship has sailed23:30
ianwnow i think about it, this job is going to clone devstack master anyway @ https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/ensure-devstack/tasks/main.yaml#L923:37
clarkbianw: ya that was one of the things i checked when lookingat the nova 500 errors. Has me wondering if we tried current stable if we'd get better results (granted current stable just released so unlikely)23:39
ianwso we'd be testing master nodepool with stable/* openstacksdk against a master devstack23:39
ianwthinking that through; i don't think there's any guarantee that master nodepool stays compatible with stable/* openstacksdk.  it could easily use new features23:42
clarkbianw: we have historically been forced to not do that though because production clouds tend to lag significantly23:43
clarkboh you mean for openstacksdk specifically? ya I suppose that is possible23:43
corvusmaster sdk should work with old clouds23:43
clarkbcorvus: yup I misread23:43
ianwi don't think we'd notice in the siblings image build, because we a) install nodepool b) loop around our listed siblings and install them.  we're not checking in b) that the version of openstacksdk we're installing from the zuul chekcout satisfies nodepool requirements23:43
ianwyeah i'm trying to think through what this testing is providing on the openstacksdk side23:45
ianwhttps://opendev.org/zuul/nodepool/src/branch/master/playbooks/nodepool-functional-openstack/pre.yaml#L1823:51
ianwnon containers does the same thing as well.  installs nodepool in a venv, then loops through the required projects to install them one by one23:52
ianwso if nodepool sets openstacksdk>=X , then the openstacksdk uses this test and installs stable/foo that's a breakage23:53
clarkbis the idea to use nodepools less trivial set of tasks as a reasonable sanity check for sdk?23:54
clarkbI think the sdk does have a test suite that tries to do a number of actions against a real cloud23:55
ianwyeah, i should say i'm thinking through openstacksdk's usage on stable branches, not master23:59
*** mlavalle has quit IRC23:59

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!