Friday, 2020-09-18

openstackgerritMerged zuul/nodepool master: Use rax.dfw mirror for arm64 wheels  https://review.opendev.org/75255300:10
*** hamalq has quit IRC00:30
*** rfolco|ruck has joined #zuul00:38
*** rfolco|ruck has quit IRC00:42
*** zbr7 has joined #zuul01:16
*** Diabelko has joined #zuul01:19
*** zbr has quit IRC01:19
*** systemd has quit IRC01:19
*** zbr7 is now known as zbr01:19
*** saneax has quit IRC01:19
*** zenkuro has quit IRC01:24
*** rfolco|ruck has joined #zuul01:36
*** hamalq has joined #zuul01:37
*** rfolco|ruck has quit IRC01:41
*** hamalq_ has joined #zuul01:41
*** hamalq has quit IRC01:44
*** hamalq_ has quit IRC03:18
*** ianychoi has quit IRC04:29
*** evrardjp has quit IRC04:33
*** evrardjp has joined #zuul04:33
*** saneax has joined #zuul05:19
*** wuchunyang has joined #zuul05:29
*** wuchunyang has quit IRC05:33
*** vishalmanchanda has joined #zuul05:34
*** wuchunyang has joined #zuul05:41
*** hamalq has joined #zuul05:44
openstackgerritDmitriy Rabotyagov (noonedeadpunk) proposed zuul/zuul-jobs master: Add support to use stow for ensure-python  https://review.opendev.org/75161105:56
openstackgerritTobias Henkel proposed zuul/zuul master: Change Pipeline.getAllItems to generator  https://review.opendev.org/75249306:08
*** hamalq has quit IRC06:29
*** hamalq has joined #zuul06:32
openstackgerritTobias Henkel proposed zuul/zuul master: Change Pipeline.getAllItems to generator  https://review.opendev.org/75249306:40
openstackgerritTobias Henkel proposed zuul/zuul master: Remove an unneeded api call when creating check_runs  https://review.opendev.org/75205606:41
openstackgerritTobias Henkel proposed zuul/zuul master: Save superfluous api requests in check run reporting  https://review.opendev.org/75207906:41
*** hamalq has quit IRC06:44
*** zbr5 has joined #zuul06:45
openstackgerritJan Kubovy proposed zuul/zuul master: Add logging and monitoring of reporting timings  https://review.opendev.org/75244306:47
*** zbr5 has quit IRC06:49
*** zbr has quit IRC06:49
*** hamalq has joined #zuul06:52
*** gundalow has quit IRC06:52
*** guilhermesp has quit IRC06:53
*** mwhahaha has quit IRC06:53
*** johnsom has quit IRC06:53
*** Open10K8S has quit IRC06:53
*** ChrisShort has quit IRC06:53
*** rpittau|afk has quit IRC06:53
*** stevthedev has quit IRC06:53
*** donnyd has quit IRC06:53
*** aprice has quit IRC06:53
*** iamweswilson has quit IRC06:53
openstackgerritzbr proposed zuul/zuul-jobs master: WIP: Update ensure-docker for new release  https://review.opendev.org/75263006:54
*** Open10K8S has joined #zuul06:55
*** zbr has joined #zuul06:55
*** mwhahaha has joined #zuul06:56
*** aprice has joined #zuul06:56
*** ChrisShort has joined #zuul06:56
*** guilhermesp has joined #zuul06:56
*** stevthedev has joined #zuul06:56
*** iamweswilson has joined #zuul06:56
*** rpittau|afk has joined #zuul06:56
*** hashar has joined #zuul06:56
*** donnyd has joined #zuul06:57
*** johnsom has joined #zuul06:57
*** armstrongs has joined #zuul07:03
*** jfoufas1 has joined #zuul07:05
*** jfoufas1 has quit IRC07:05
*** hamalq has quit IRC07:05
*** jfoufas1 has joined #zuul07:06
*** armstrongs has quit IRC07:07
*** gundalow has joined #zuul07:10
*** jcapitao has joined #zuul07:10
*** tosky has joined #zuul07:25
openstackgerritFelix Edel proposed zuul/zuul master: Make reporting asynchronous  https://review.opendev.org/69125307:38
*** jpena|off is now known as jpena07:49
*** holser has joined #zuul08:09
*** nils has joined #zuul08:09
openstackgerritzbr proposed zuul/zuul-jobs master: Update ensure-docker for new releases  https://review.opendev.org/75263008:43
*** mordred has quit IRC08:52
*** masterpe has quit IRC08:52
*** Eighth_Doctor has quit IRC08:53
openstackgerritMatthieu Huin proposed zuul/zuul-client master: Fix integration with Zuul before release  https://review.opendev.org/75129109:13
*** zenkuro has joined #zuul09:15
openstackgerritMatthieu Huin proposed zuul/zuul-website master: Add link to zuul-client documentation  https://review.opendev.org/75131209:16
*** Eighth_Doctor has joined #zuul09:16
*** Eighth_Doctor has quit IRC09:19
*** masterpe has joined #zuul09:25
openstackgerritMatthieu Huin proposed zuul/zuul master: Add zuul-client testing  https://review.opendev.org/75203909:35
*** vishalmanchanda has quit IRC09:43
openstackgerritMatthieu Huin proposed zuul/zuul master: Add zuul-client testing  https://review.opendev.org/75203909:44
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] Add zuul-client to requirements  https://review.opendev.org/75238509:45
*** Eighth_Doctor has joined #zuul09:49
*** mordred has joined #zuul09:49
*** Diabelko is now known as systemd09:53
*** vishalmanchanda has joined #zuul09:57
openstackgerritMatthieu Huin proposed zuul/zuul-client master: Add cross testing with Zuul  https://review.opendev.org/75126409:59
*** wuchunyang has quit IRC10:06
*** jcapitao is now known as jcapitao_lunch10:16
openstackgerritMatthieu Huin proposed zuul/zuul-client master: Add cross testing with Zuul  https://review.opendev.org/75126410:23
openstackgerritJan Kubovy proposed zuul/zuul master: Remove superfluous flushes and queries from SQL reporter  https://review.opendev.org/75266410:33
openstackgerritJan Kubovy proposed zuul/zuul master: Add logging and monitoring of reporting timings  https://review.opendev.org/75244310:47
*** hashar is now known as hasharAway10:47
*** saneax has quit IRC11:28
*** jpena is now known as jpena|lunch11:39
*** holser has quit IRC11:46
*** holser has joined #zuul11:51
*** hasharAway is now known as hashar11:51
*** bhavikdbavishi has joined #zuul11:53
*** jcapitao_lunch is now known as jcapitao11:59
*** mattd01 has joined #zuul12:00
*** rlandy has joined #zuul12:07
zbrclarkb: apparently the bionic-arm is bit fractured (alongside focal one), https://review.opendev.org/#/c/752630/12:09
zbrthat is new, yesterday they were working12:09
mhucorvus, hello, I've answered your comments re: https://review.opendev.org/#/q/status:open+topic:zuul-client_0.0.2 - the order of review should be https://review.opendev.org/#/c/751291/ (fix zuul integration bugs), then https://review.opendev.org/#/c/752039/ (add test_zuulcient in zuul unit tests), then https://review.opendev.org/#/c/751264/ (add zuul functional testing in zuul-client)12:20
mhuthen we can tag zuul-client 0.0.2 to fix the PyPI package, and proceed with https://review.opendev.org/#/c/752385/ (use zuul-client lib in zuul)12:21
*** saneax has joined #zuul12:22
*** jpena|lunch is now known as jpena12:37
*** Goneri has joined #zuul12:42
*** saneax has quit IRC12:45
*** bhavikdbavishi has quit IRC12:57
*** jfoufas1 has quit IRC13:03
openstackgerritMatthieu Huin proposed zuul/zuul-client master: Add build-info subcommand  https://review.opendev.org/75107013:04
*** saneax has joined #zuul13:10
*** saneax has quit IRC13:17
*** saneax has joined #zuul13:18
*** tosky_ has joined #zuul13:18
*** tosky is now known as Guest8304713:18
*** tosky_ is now known as tosky13:18
zenkurohi, Im curious if people here know is there dedicated guide on running devstack on centos8?13:18
fungizenkuro: that sounds like a question for the #openstack-qa channel. it's not really related to zuul13:19
zenkurofungi: yes, but CI people sometimes run devstack on centos... sorry for disturbance, I will ask at QA13:20
*** saneax has quit IRC13:38
openstackgerritFelix Edel proposed zuul/zuul master: Make reporting asynchronous  https://review.opendev.org/69125313:40
*** hashar has quit IRC14:12
*** hashar has joined #zuul14:12
clarkbzbr: the mirror had trouble in the linaro cloud. I would recheck for now14:49
*** ianychoi has joined #zuul14:59
zbrclarkb: ok, rechecked again.14:59
zbrclarkb: what do you think about https://review.opendev.org/#/c/748682/ ? -- i think we reached a scalability issue with how we run/trigger test jobs on zuul-jobs.15:02
zbri have the feeling that will never merge because is almost impossible to get 100 jobs to be green at the same time.15:03
zbrone of the failure is known, we talked about it yesterday, but the issue is still here.15:04
zbrwe cannot improve the code/testing because other broken jobs are preventing us from merging the changes.15:04
zbrit creates a vicious cycle15:05
clarkbzuul os running all those tests pre merge which means we can improve them?15:05
fungimost changes to zuul-jobs don't trigger every role job15:05
zbrif you are lucky is only one or two simple to fix one, but in some cases you uncover long chain15:06
zbrlets open pandora box: https://review.opendev.org/#/c/748606/ :D15:07
zbrwe do know that "zuul-jobs-test-fetch-sphinx-tarball-with-zuul-output" is broken and i do not know any way to fix it.15:08
zbri do not blame ianw for adding that comment, still... what can we do?15:08
clarkbto be clear that test was broken and no one knew about it because we didnt have need to run it. When it was last run it worked15:08
clarkbso the ladt time we made changes to the role it was properly tested15:09
zbrthat being one of the reasons i started working on enabling periodic testing15:09
zbryep, code rots, roles too.15:09
clarkbit is an unfortunate situation that our testing relied on unwanted behavior from zuul15:10
zbrmy view here is that using xfail would be ok for the moment, until we have a better solution.15:10
clarkbanother option is to not modify the role until we cant test it15:11
clarkbor modify the role using the existing base-test dance15:12
clarkbI'm not sure what the best option is but there are options15:12
zbrdue to the nature of this chance it would be easier to re-enable it in a follow-up.15:13
zbrzuul-maint: is anyone against enabling periodic-weekly for testing zuul-jobs? https://review.opendev.org/#/c/748706/15:30
zbrwe already have the mailing list and I said i will keep an eye on it.15:31
clarkbI guess even if people dont subsvribe to emails they can check the zuul builds dashboard15:33
clarkbso is accessible on demand that way15:33
tristanCzbr: looking at https://review.opendev.org/748682 , i worry this will cause unreasonable load. Is it necessary to test all distro every week?15:34
clarkbI think that is why it is weekly not daily in a new pipeline15:35
clarkbit should also trogger during a quiet period15:35
zbryep, that is why is weekly, we need schedule to discover what gets rotten. but we could later tune its recurrence if we want/need.15:36
zbrtristanC: if we test only some distros we provide not much protection, one not-tested distro may get rotten and when someone would have to touch a role that builds it, he will find the issue, blocking his patch.15:37
zbrbut i guess that if we find some distros to be unreliable, we could endup removing support for them (and their jobs).15:38
zbrtristanC: while the number of jobs seems quite big, all are <10min, i bet the cumulative load cost is less than a single tripleo-ci buildset :D15:41
zbrin the future I could try to implement a feature to skip running a scheduled job if it did already run recently, like travis does.15:43
zbrwhich could address concerns of wasting compute resources15:43
clarkbhrm failures with the nodepool azure driver? ImportError: No module named 'msrestazure'16:01
clarkbazure-mgmt-compute and azure-mgmt-network were released a few hours ago16:06
clarkbI'm looking into what the fix is16:06
*** jcapitao has quit IRC16:08
fungithose jobs are also very quick. it's not like the hundreds of periodic jobs openstack schedules daily many of which run 2+ hours16:18
clarkblooks like azure's sdk libs are cool with non backward compat changes16:18
clarkbI've got a patch I'm trying to test with at least our azure tests to ensure things work16:19
clarkbwow it looks like they've even broken themselves internally. The azure.common module isn't compatible with the changes to the sub modules16:20
fungimugsie may find that interesting16:21
*** hashar has quit IRC16:21
mugsiethey did it again?16:21
mugsie-_-16:21
fungiif only there were a ci system capable of testing changes across multiple repos ;)16:22
*** saneax has joined #zuul16:22
mugsie:D16:22
openstackgerritClark Boylan proposed zuul/nodepool master: Cap azure dep versions due to breaking changes  https://review.opendev.org/75272416:23
clarkbI had started with a change to update the excepton to the new exception type. I also do a s/credentials/credential/ but then the use of azure.common functions was still broken so now we get ^16:23
*** vishalmanchanda has quit IRC16:23
clarkbmugsie: looks like azure.common may not explicitly dep on msrestazure which causes an import error now that the other bits don't pull that in. Possible the azure.common fix is as simple as including that as a dep and making a release but I'm not sure16:24
mugsieclarkb: cool, I will ping some people here after my flood of US/EU crossover meetings :D16:26
*** sanjayu_ has joined #zuul16:31
clarkbit looks like azure tries to have stable releases and unstable releases except the unstable releases get proper versions in pypi so they are pip installed16:32
clarkbI think thatmeans our cap is actually the correct fix here, but I'm not sure that is a very user friendly release mechanism16:32
*** saneax has quit IRC16:34
fungithat's what pip/pypi added prerelease versioning to handle. if a version is a prerelease it won't be installed unless requested16:36
*** jpena is now known as jpena|off16:56
*** hamalq has joined #zuul17:01
*** iurygregory has quit IRC17:10
*** mattd01 has quit IRC17:12
clarkbhttps://review.opendev.org/752724 passes unittests now. Getting that approved to fix nodepool testing would be great17:13
clarkb(its still running check though)17:13
corvusclarkb: version differences seem weird; out of curiosity, why aren't the caps <15.0.0 for all?17:18
corvus(but also +2)17:19
clarkbcorvus: they don't release them in sync with each other. https://azure.github.io/azure-sdk/releases/latest/all/python.html is a giant page of libs and their disparate versioning17:19
clarkb(also its not up to date today due to the new releases)17:20
clarkbI capped at less than today's release versions17:20
*** sanjayu_ has quit IRC17:29
*** mattd01 has joined #zuul18:12
*** irclogbot_3 has quit IRC18:19
*** irclogbot_0 has joined #zuul18:25
*** irclogbot_0 has quit IRC18:31
*** irclogbot_0 has joined #zuul18:37
openstackgerritClark Boylan proposed zuul/nodepool master: Stop using fedora-30  https://review.opendev.org/75274318:37
*** mattd01 has left #zuul18:52
clarkblooks like fedora has dropped zookeper19:10
clarkbtristanC: ^ any thoughts on the best way to run it on fedora-32 for the openshift job?19:11
clarkbalso if we're supplying it ourselves maybe we should switch that to centos8?19:11
clarkbdoesn't look like epel has it so we either install from the upstream tarball (what I do on suse) or run the container?19:20
tristanCclarkb: perhaps we could use a ensure-zookeeper to do that, or switch to using a container19:32
clarkblooking more closely at the job we run oc on centos and nodepool on fedora19:50
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: wip: ensure-zookeeper: add role to setup zookeeper  https://review.opendev.org/75275719:52
clarkbtristanC: should I update my change to use ^ or do you plan to do that?19:55
openstackgerritTristan Cacqueray proposed zuul/nodepool master: wip: replace zookeeper package by role for openshift test  https://review.opendev.org/75275819:56
tristanCclarkb: well let's see how that goes ^19:56
openstackgerritMerged zuul/nodepool master: Cap azure dep versions due to breaking changes  https://review.opendev.org/75272419:58
clarkbafter looking at the job more closely I think other options available to us are running the launcher on ubuntu and installing oc similar to how we install zk in your change. Or to use the built docker images in that job which includes oc19:58
tristanCclarkb: we might want a common ensure-zookeeper role in zuul-jobs to re-use the setup of zk-ca20:00
clarkbya I think the role is useful either way20:00
clarkbjust brainstorming options there20:06
clarkbzbr: I've approved the periodic-weekly change. FOr https://review.opendev.org/#/c/748682/4 I wonder if we need to split that out so that we don't include the jobs that currently fail20:12
clarkbWhen we address those jobs they can be added to periodic-weekly too. Its a weird chicken and egg because we're modifying the file that define the jobs20:13
clarkbwhich forces them to run20:13
openstackgerritMerged zuul/project-config master: Assure periodic-weekly emails to zuul-jobs-failures  https://review.opendev.org/74870620:14
*** veecue has joined #zuul20:33
*** veecue has quit IRC20:36
zbrtoo late now but imho we should make them nv, that is the scope of nv, to still run a give time to fix them without blocking other changes.21:00
zbri prefer to see few them as nv until they are either fixed or we decide to remove them. i see this as a natural process.21:02
tristanCzbr: I guess it makes sense to run the existing test periodically, though i now wonder if they are appropriate in zuul-jobs after all. What are zuul-jobs-failures@lists.zuul-ci.org readers expected to do when a job fails because upstream-x doesn't work on distro-y ?21:08
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: wip: ensure-zookeeper: add role to setup zookeeper  https://review.opendev.org/75275721:15
*** rlandy has quit IRC21:18
tristanCzbr: perhaps before using a periodic pipeline to run every test weekly, we could recheck https://review.opendev.org/748682 manually and see if that is useful21:26
*** Goneri has quit IRC21:39
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: wip: ensure-zookeeper: add role to setup zookeeper  https://review.opendev.org/75275721:42
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: ensure-zookeeper: add role to setup zookeeper  https://review.opendev.org/75275722:11
tristanCclarkb: topic:ensure-zk seems to work, feel free to squash the nodepool change, otherwise i'll follow-up on monday22:18
clarkbtristanC: I think to start the idea is just to have the info22:22
clarkbtristanC: we'll get that both in the zuul db and email22:22
clarkband from there we can see what needs to be done22:22
clarkbre ensure-zk ya I'll take a look at using it momentarily22:22
openstackgerritClark Boylan proposed zuul/nodepool master: Stop using fedora-30  https://review.opendev.org/75274322:31
clarkbtristanC: ^ squashed22:31
openstackgerritClark Boylan proposed zuul/nodepool master: Use centos-8 to run openshift cluster in testing  https://review.opendev.org/75276922:32
clarkbI'll review ensure-zookeeper shortly too22:32
*** hamalq has quit IRC23:02
*** EmilienM has quit IRC23:06
*** EmilienM has joined #zuul23:06
*** sanjayu_ has joined #zuul23:10
openstackgerritClark Boylan proposed zuul/zuul-jobs master: ensure-zookeeper: add role to setup zookeeper  https://review.opendev.org/75275723:27
*** tosky has quit IRC23:31
*** sanjayu_ has quit IRC23:44
*** Goneri has joined #zuul23:44
*** sanjayu_ has joined #zuul23:44
*** hamalq has joined #zuul23:48

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