| *** hamalq has quit IRC | 00:09 | |
| *** tetsuro has joined #openstack-nova | 00:27 | |
| *** lseki has quit IRC | 00:54 | |
| *** gouthamr_ has quit IRC | 00:59 | |
| *** gouthamr_ has joined #openstack-nova | 01:05 | |
| *** jdillaman has quit IRC | 01:11 | |
| *** Liang__ has joined #openstack-nova | 01:15 | |
| *** gouthamr_ has quit IRC | 01:25 | |
| *** gouthamr_ has joined #openstack-nova | 01:37 | |
| *** gyee has quit IRC | 01:38 | |
| *** Yumeng has joined #openstack-nova | 02:01 | |
| *** mkrai has joined #openstack-nova | 02:18 | |
| openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | 02:39 |
|---|---|---|
| *** mkrai has quit IRC | 02:49 | |
| openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | 02:50 |
| openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | 02:50 |
| *** gouthamr_ has quit IRC | 02:52 | |
| *** tetsuro has quit IRC | 02:55 | |
| *** brinzhang has joined #openstack-nova | 02:55 | |
| *** mkrai has joined #openstack-nova | 02:56 | |
| *** gouthamr_ has joined #openstack-nova | 02:56 | |
| *** swp20 has joined #openstack-nova | 02:57 | |
| *** rcernin has quit IRC | 03:00 | |
| *** jmlowe has quit IRC | 03:01 | |
| *** rcernin has joined #openstack-nova | 03:03 | |
| *** yaawang has quit IRC | 03:10 | |
| *** yaawang has joined #openstack-nova | 03:10 | |
| *** brinzhang_ has joined #openstack-nova | 03:11 | |
| *** songwenping_ has joined #openstack-nova | 03:12 | |
| *** brinzhang has quit IRC | 03:15 | |
| *** swp20 has quit IRC | 03:15 | |
| *** gouthamr_ has quit IRC | 03:26 | |
| *** gouthamr_ has joined #openstack-nova | 03:32 | |
| openstackgerrit | XiaolinZhao proposed openstack/nova master: Add UEFI support for architecture MIPS https://review.opendev.org/740151 | 03:35 |
| *** ociuhandu has joined #openstack-nova | 03:56 | |
| *** gouthamr_ has quit IRC | 04:00 | |
| *** ociuhandu has quit IRC | 04:01 | |
| *** markvoelker has joined #openstack-nova | 04:08 | |
| *** gouthamr_ has joined #openstack-nova | 04:11 | |
| *** ratailor has joined #openstack-nova | 04:22 | |
| *** gouthamr_ has quit IRC | 04:29 | |
| *** gouthamr_ has joined #openstack-nova | 04:30 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #openstack-nova | 04:33 | |
| openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | 04:34 |
| *** tetsuro has joined #openstack-nova | 04:34 | |
| *** gouthamr_ has quit IRC | 04:35 | |
| *** gouthamr_ has joined #openstack-nova | 04:40 | |
| *** Luzi has joined #openstack-nova | 05:12 | |
| *** udesale has joined #openstack-nova | 05:13 | |
| *** brinzhang0 has joined #openstack-nova | 05:13 | |
| *** markvoelker has quit IRC | 05:13 | |
| *** songwenping__ has joined #openstack-nova | 05:13 | |
| *** markvoelker has joined #openstack-nova | 05:14 | |
| *** songwenping_ has quit IRC | 05:16 | |
| *** brinzhang_ has quit IRC | 05:16 | |
| *** songwenping__ has quit IRC | 05:16 | |
| *** swp20 has joined #openstack-nova | 05:18 | |
| *** markvoelker has quit IRC | 05:18 | |
| *** Luzi has quit IRC | 05:19 | |
| *** zzzeek has quit IRC | 05:19 | |
| *** Luzi has joined #openstack-nova | 05:19 | |
| *** zzzeek has joined #openstack-nova | 05:23 | |
| *** links has joined #openstack-nova | 05:43 | |
| *** mkrai has quit IRC | 05:50 | |
| *** songwenping_ has joined #openstack-nova | 05:53 | |
| *** swp20 has quit IRC | 05:57 | |
| *** brinzhang_ has joined #openstack-nova | 05:59 | |
| *** songwenping__ has joined #openstack-nova | 06:01 | |
| *** brinzhang0 has quit IRC | 06:03 | |
| *** songwenping_ has quit IRC | 06:04 | |
| *** vishalmanchanda has joined #openstack-nova | 06:05 | |
| *** ralonsoh has joined #openstack-nova | 06:10 | |
| *** dklyle has quit IRC | 06:16 | |
| *** yaawang has quit IRC | 06:32 | |
| *** yaawang has joined #openstack-nova | 06:33 | |
| *** slaweq has joined #openstack-nova | 06:39 | |
| *** xek_ has joined #openstack-nova | 06:44 | |
| *** CeeMac has joined #openstack-nova | 06:44 | |
| *** brinzhang0 has joined #openstack-nova | 06:48 | |
| *** gouthamr_ has quit IRC | 06:50 | |
| *** brinzhang_ has quit IRC | 06:51 | |
| *** gouthamr_ has joined #openstack-nova | 06:52 | |
| *** knikolla has quit IRC | 06:54 | |
| *** slaweq has quit IRC | 06:55 | |
| *** knikolla has joined #openstack-nova | 06:56 | |
| *** csatari has quit IRC | 06:58 | |
| *** csatari has joined #openstack-nova | 07:01 | |
| *** gouthamr_ has quit IRC | 07:04 | |
| openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | 07:05 |
| *** gouthamr_ has joined #openstack-nova | 07:05 | |
| gibi | good morning nova | 07:09 |
| openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | 07:11 |
| *** gouthamr_ has quit IRC | 07:15 | |
| *** ttsiouts has joined #openstack-nova | 07:17 | |
| *** tesseract has joined #openstack-nova | 07:18 | |
| *** gouthamr_ has joined #openstack-nova | 07:22 | |
| *** tony_su has joined #openstack-nova | 07:22 | |
| tony_su | gibi: sean-k-mooney: I am still taking the provider-config-file job and fully supporting its merging into upstream. | 07:24 |
| gibi | tony_su: cool. So I will move the series to a runway slot and we will try to focus on it in the next two weeks | 07:25 |
| gibi | are you available in the next to week to iterate on the series based on our feedback? | 07:26 |
| openstackgerrit | Elancheran S proposed openstack/nova master: Removes the delta file once image is extracted https://review.opendev.org/734421 | 07:27 |
| tony_su | yes, definitely. | 07:27 |
| tony_su | gibi: yes, definitely. | 07:28 |
| gibi | tony_su: cool | 07:28 |
| gibi | thanks | 07:28 |
| *** xek_ has quit IRC | 07:38 | |
| *** markvoelker has joined #openstack-nova | 07:43 | |
| *** gouthamr_ has quit IRC | 07:46 | |
| *** ttsiouts has quit IRC | 07:47 | |
| *** slaweq has joined #openstack-nova | 07:47 | |
| *** markvoelker has quit IRC | 07:48 | |
| huaqiang | gibi: hi can you review https://review.opendev.org/728480? | 07:49 |
| *** alex_xu has quit IRC | 07:50 | |
| *** gouthamr_ has joined #openstack-nova | 07:52 | |
| *** tony_su has quit IRC | 07:52 | |
| openstackgerrit | Jie Li proposed openstack/nova-specs master: add volume backed server rebuild spec https://review.opendev.org/739349 | 07:55 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: doc: correct the link to user/flavor.rst https://review.opendev.org/740176 | 07:55 |
| gibi | huaqiang: is this in the middle of the series? | 07:57 |
| *** gouthamr_ has quit IRC | 07:57 | |
| huaqiang | gibi: do you mean https://review.opendev.org/739349 | 07:57 |
| huaqiang | for https://review.opendev.org/739349, yes it is, it belongs to bp/mixed-instance | 07:58 |
| gibi | I mean https://review.opendev.org/#/c/728480/ | 07:58 |
| gibi | I can review https://review.opendev.org/#/c/728480/ but I reviewd the ancestors of that patch in the series already | 07:58 |
| huaqiang | my mistake | 07:59 |
| huaqiang | https://review.opendev.org/#/c/728480/ belongs to bp/mixed-instances | 07:59 |
| huaqiang | Stephen suggests to have your review, | 07:59 |
| huaqiang | let me check | 08:00 |
| *** udesale_ has joined #openstack-nova | 08:00 | |
| *** udesale has quit IRC | 08:03 | |
| *** gmann has quit IRC | 08:03 | |
| *** fyx has quit IRC | 08:04 | |
| *** fyx has joined #openstack-nova | 08:04 | |
| *** gmann has joined #openstack-nova | 08:06 | |
| *** ttsiouts has joined #openstack-nova | 08:06 | |
| gibi | huaqiang: sure I will try to look deeper into the series. I have feedback in https://review.opendev.org/#/c/714701 | 08:07 |
| gibi | and in https://review.opendev.org/#/c/714703 | 08:07 |
| gibi | and also alex_xu has feedback in that series | 08:08 |
| *** gouthamr_ has joined #openstack-nova | 08:09 | |
| *** iurygregory has quit IRC | 08:10 | |
| brinzhang0 | gibi: do you have time to review the nova-cyborg patch? https://review.opendev.org/#/c/716186/13 | 08:10 |
| *** udesale_ has quit IRC | 08:10 | |
| brinzhang0 | gibi: and the cyborg evacuate support patch https://review.opendev.org/#/c/715326/15 | 08:11 |
| brinzhang0 | the cyborg shelve/unshelve support depends on these patch, and these pataches are already to review, thanks | 08:11 |
| huaqiang | gibi: sorry about that, I missed your comments even I go through some of my abandoned changes. | 08:15 |
| huaqiang | gibi: can you have a look again :) | 08:15 |
| huaqiang | gibi: get your message, thanks | 08:16 |
| *** songwenping_ has joined #openstack-nova | 08:17 | |
| *** songwenping__ has quit IRC | 08:20 | |
| openstackgerrit | Wenping Song proposed openstack/nova-specs master: Add no user token when get Cyborg client https://review.opendev.org/740184 | 08:20 |
| openstackgerrit | Wenping Song proposed openstack/nova-specs master: Add no user token when get Cyborg client https://review.opendev.org/740184 | 08:22 |
| *** iurygregory has joined #openstack-nova | 08:24 | |
| *** songwenping__ has joined #openstack-nova | 08:25 | |
| *** tobiash has quit IRC | 08:26 | |
| *** ttsiouts_ has joined #openstack-nova | 08:27 | |
| *** songwenping_ has quit IRC | 08:28 | |
| *** maciejjozefczyk has joined #openstack-nova | 08:28 | |
| gibi | brinzhang0: if the cyborg patches are ready then please add them to the runway queue. At the momement I will focus on the 3 series that is currently in the 3 slots. | 08:29 |
| *** ttsiouts_ has quit IRC | 08:29 | |
| gibi | huaqiang: ack I will look | 08:29 |
| *** ttsiouts has quit IRC | 08:30 | |
| brinzhang0 | gibi: ok, I will add these to the runway queue | 08:30 |
| gibi | brinzhang0: thanks | 08:31 |
| brinzhang0 | gibi: np | 08:31 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Fix os_CODENAME detection and repo refresh during ceph tests https://review.opendev.org/739608 | 08:32 |
| gibi | brinzhang0: also summer vacation time will soon arrives. I will take 3 weeks of during july - august | 08:32 |
| gibi | s/of/off/ | 08:32 |
| gibi | I guess others from EU will do similar things | 08:32 |
| brinzhang0 | gibi: have a holiday ^ | 08:32 |
| gibi | thanks :) | 08:32 |
| lyarwood | elod: https://review.opendev.org/#/c/739608/ should finally fix stable/rocky | 08:34 |
| *** derekh has joined #openstack-nova | 08:36 | |
| *** tetsuro has quit IRC | 08:38 | |
| *** benj_ has quit IRC | 08:41 | |
| stephenfin | gibi, elod, lyarwood: Can raise on the team meeting, but what do we think about adding the Backport-Candidate flag to Gerrit like oslo has done? https://review.opendev.org/#/c/740068/ | 08:44 |
| *** k_mouza has joined #openstack-nova | 08:44 | |
| lyarwood | stephenfin: OOoOOOoO shiny | 08:46 |
| lyarwood | stephenfin: +1 from me if we can query on that flag | 08:46 |
| lyarwood | that we must be able to | 08:47 |
| stephenfin | https://review.opendev.org/#/q/label:Backport-Candidate%253D%252B2 | 08:47 |
| stephenfin | Yup | 08:47 |
| stephenfin | (label:Backport-Candidate=+2) | 08:47 |
| gibi | stephenfin: how does this relate to the way we target bugs to series in launchpad? | 08:48 |
| gibi | will we do duble accounting? | 08:48 |
| stephenfin | gibi: what do you mean? | 08:48 |
| stephenfin | oh, as in setting the affected series for a reported bug in LP? | 08:49 |
| gibi | in launchpad I can go to a bug and target it to multpile series (Ussurit, Train...) this way showing that the bugfix needs to be backported to those releases | 08:49 |
| gibi | this also shows _where_ to backport | 08:50 |
| stephenfin | that would sound like double accounting, yes. I guess we could either check both, or decide on which one we'd prefer | 08:50 |
| stephenfin | iiuc we only do that launchpad tracking for some bugs, so we could use the backport-candidate feature for everything, and then be more specific in launchpad if necessary | 08:50 |
| * stephenfin tends to keep backporting things as far back as they can reasonably go | 08:51 | |
| gibi | correct, there are non-bugs that we backport | 08:53 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: Fix os_CODENAME detection and repo refresh during ceph tests https://review.opendev.org/740193 | 08:53 |
| gibi | and launchpad does not help there | 08:53 |
| gibi | do you have a good working example for a patch that is backported but would be strange to track it with an LP bug? | 08:54 |
| *** songwenping_ has joined #openstack-nova | 08:54 | |
| stephenfin | I wasn't even being that specific. I meant bug reporters often don't set target versions in Launchpad, and I don't tend to set them when reviewing bugfixes | 08:55 |
| stephenfin | But the ability to say "this looks like a fix we probably want to backport" from Gerrit does sound like something I'd use, if only because it doesn't involve context switching | 08:56 |
| stephenfin | so tl;dr: this feels like something we'd collectively use far more, if that makes sense? | 08:57 |
| *** songwenping__ has quit IRC | 08:57 | |
| *** benj_ has joined #openstack-nova | 08:58 | |
| gibi | stephenfin: I see, so the benefit of the new solution is that the recording of the information is closer to the place where the information is created. That definitely helps | 09:01 |
| stephenfin | Yeah, that's a better way of saying what I'm trying to say :) | 09:01 |
| xiaolin | lyarwood: Do you have time to review https://review.opendev.org/#/c/740151/ , I don not know how to change the branch and proposed a new one against branch master | 09:02 |
| gibi | we can use the gerrit flag, to force a discusson of the question "how far to be backported?" and then the answer can be recorded in LP by assigning series | 09:02 |
| lyarwood | xiaolin: I'll take a look today yes | 09:02 |
| *** tobiash has joined #openstack-nova | 09:03 | |
| gibi | I'm not against the new flag, I feel it has some cost of re-learning the gerrit interface due to the extra flag a the reply button, but I see the benefit of the flag | 09:03 |
| xiaolin | lyarwood: thanks | 09:03 |
| *** songwenping__ has joined #openstack-nova | 09:07 | |
| stephenfin | gibi: I'm already doing that for oslo so not a massive deal for me, personally :) | 09:10 |
| *** songwenping_ has quit IRC | 09:10 | |
| *** tosky has joined #openstack-nova | 09:10 | |
| gibi | yeah, I can adapt too :) | 09:10 |
| *** swp20 has joined #openstack-nova | 09:11 | |
| *** udesale has joined #openstack-nova | 09:12 | |
| *** songwenping__ has quit IRC | 09:13 | |
| *** rcernin has quit IRC | 09:23 | |
| *** dtantsur|afk is now known as dtantsur | 09:29 | |
| elod | stephenfin: that could help, yes, it's easy to see if a patch should be backported, in that way | 09:31 |
| elod | stephenfin: i usually check the bugreport as well if a patch was marked to be backported to certain branches, too | 09:31 |
| elod | stephenfin: unfortunately most of the time that detail is not filled in | 09:32 |
| elod | stephenfin: so maybe it's worth adding that option :) | 09:34 |
| *** hrw has joined #openstack-nova | 09:38 | |
| hrw | morning | 09:38 |
| hrw | https://1d9789842ee1597bfc07-adfe1c2e7945e3f1ebfe34ebeb2d7062.ssl.cf2.rackcdn.com/739800/2/check/kayobe-overcloud-centos8/c7c8c01/primary/kolla/nova/nova-compute.txt shows new nova issue | 09:38 |
| hrw | 2020-07-08 16:44:28.078 6 ERROR nova File "/usr/lib/python3.6/site-packages/nova/virt/libvirt/utils.py", line 65, in <module> | 09:39 |
| hrw | 2020-07-08 16:44:28.078 6 ERROR nova 'avx512vbmi': os_traits.HW_CPU_X86_AVX512VBMI, | 09:39 |
| hrw | 2020-07-08 16:44:28.078 6 ERROR nova AttributeError: module 'os_traits' has no attribute 'HW_CPU_X86_AVX512VBMI' | 09:39 |
| *** songwenping_ has joined #openstack-nova | 09:39 | |
| *** xek has joined #openstack-nova | 09:40 | |
| *** swp20 has quit IRC | 09:42 | |
| frickler | hrw: guess you either need an intermediate os_traits release or use it from git instead pypi | 09:43 |
| *** markvoelker has joined #openstack-nova | 09:44 | |
| hrw | or update to 2.4.0 | 09:45 |
| *** k_mouza has quit IRC | 09:47 | |
| *** markvoelker has quit IRC | 09:48 | |
| lyarwood | yeah FWIW tripleo hit this as well and they just needed to pull in the new os-traits release | 09:48 |
| lyarwood | that's AFAIK anyway | 09:48 |
| hrw | weird as image says 2.4.0 os_traits. | 09:49 |
| hrw | something to debug then | 09:49 |
| bauzas | stephenfin: jumping late on your Backport-Candidate question, could you please explain the usage ? | 09:52 |
| lyarwood | hrw: do you have a link to the build logs showing that? | 09:53 |
| bauzas | stephenfin: if it's for telling "I think it's a good change for backporting it", then anyone can just bakcport it and leave the stable cores to look at it | 09:53 |
| bauzas | I don't really see what it's helping | 09:54 |
| hrw | lyarwood: https://1d9789842ee1597bfc07-adfe1c2e7945e3f1ebfe34ebeb2d7062.ssl.cf2.rackcdn.com/739800/2/check/kayobe-overcloud-centos8/c7c8c01/ is zuul job log dir | 09:54 |
| hrw | lyarwood: docker kolla/centos-binary-nova-compute:master image used | 09:55 |
| bauzas | stephenfin: and like gibi said, we already have both launchpad tags and series for stable | 09:55 |
| hrw | yaawang: I did docker run image, python3, import os_traits and HW_CPU_X86_AVX512VBMI was there so wondering what is going on | 09:55 |
| stephenfin | bauzas: You can just do the backport, but that means you have to remember to do it there and then | 09:56 |
| stephenfin | which might be easier said than done, particularly if there are merge conflicts | 09:56 |
| *** k_mouza has joined #openstack-nova | 09:56 | |
| stephenfin | this is just a way for (core) reviewers to say "I think this should be backported" | 09:57 |
| stephenfin | or, conversely, I think this should not be backported | 09:57 |
| stephenfin | and then the actual backport can take place later | 09:57 |
| *** jraju__ has joined #openstack-nova | 09:57 | |
| hrw | stephenfin: 'Backport Candidate' options in gerrit you discuss? | 09:57 |
| stephenfin | hrw: correct | 09:58 |
| hrw | I like them. Use them in kolla | 09:58 |
| *** links has quit IRC | 09:58 | |
| stephenfin | So do I. Maybe you can answer the questions gibi and bauzas have? Namely, what it gives over setting the series metadata in Launchpad | 09:59 |
| bauzas | mmm, I see your usecase then | 10:00 |
| bauzas | it would be for people wanting to know what to backport ? | 10:01 |
| bauzas | but honestly, who ? | 10:01 |
| hrw | stephenfin bauzas: when I have fix for stable/* I have to get it into master and then u->t->s->r if needed. so I send for master and then comment with "backport needed" and say which branches | 10:01 |
| hrw | so later it is visible which patches need backporting | 10:01 |
| openstackgerrit | XiaolinZhao proposed openstack/nova master: Add USB keyboard to support for MIPS architecture https://review.opendev.org/740213 | 10:01 |
| bauzas | and who would say +1/-1 ? stable cores or master cores ? | 10:01 |
| hrw | bauzas: in kolla we have just cores | 10:02 |
| bauzas | either way, not saying I disagree, just it looks to me something we don't really *need* | 10:02 |
| stephenfin | bauzas: I mean, if I was given a list of stuff to backport and promised reviews then I'd be happy to skim the backport candidate list occasionally | 10:02 |
| stephenfin | and it's backport candidate - not actual backport - so normal cores | 10:02 |
| stephenfin | though we could conceivably do both | 10:02 |
| hrw | bauzas: as you have master and stable cores then I would say that master core suggests backport and stable core may accept/reject | 10:03 |
| stephenfin | yeah, that ^ | 10:03 |
| bauzas | okay I see, it's then just yet another process... | 10:03 |
| *** jmlowe has joined #openstack-nova | 10:03 | |
| bauzas | some people would use it, some not | 10:04 |
| stephenfin | Yes and no. It's just another checkbox in the review dropdown | 10:04 |
| stephenfin | Personally, everyone is using it in oslo land because it's really low friction | 10:04 |
| stephenfin | Certainly less than going to Launchpad and setting metadata there (which I don't currently do because I didn't know I was supposed to) | 10:04 |
| hrw | xiaolin: nice to see that some other arch follows aarch64 steps | 10:05 |
| *** xek has quit IRC | 10:05 | |
| stephenfin | bauzas: If it means we remember to backport even a few more bugfixes then I think it's a good thing, personally | 10:06 |
| stephenfin | lyarwood: Have you been looking at https://review.opendev.org/739349 ? | 10:06 |
| bauzas | okay okay, let's discuss tonight anyway | 10:06 |
| stephenfin | seems tangentially related to stuff you've been working on recently | 10:06 |
| lyarwood | stephenfin: on my list | 10:09 |
| xiaolin | hrw: thanks : ) We just started | 10:10 |
| *** its-surya has joined #openstack-nova | 10:11 | |
| hrw | xiaolin: check https://review.opendev.org/#/c/538003/ as you need to add test cases | 10:11 |
| *** maciejjozefczyk has quit IRC | 10:13 | |
| *** maciejjozefczyk has joined #openstack-nova | 10:14 | |
| *** songwenping__ has joined #openstack-nova | 10:14 | |
| *** ociuhandu has joined #openstack-nova | 10:16 | |
| xiaolin | hrw: Thanks :) I will add test cases | 10:17 |
| *** songwenping_ has quit IRC | 10:17 | |
| hrw | xiaolin: it took some revisions before I got them right so prefer to point you at start | 10:21 |
| *** jmlowe has quit IRC | 10:24 | |
| *** jmlowe has joined #openstack-nova | 10:24 | |
| *** Liang__ has quit IRC | 10:28 | |
| *** xek has joined #openstack-nova | 10:45 | |
| xiaolin | hrw: Thanks so much | 10:48 |
| xiaolin | hrw: do you have time to review https://review.opendev.org/#/c/740151/ | 10:50 |
| hrw | sure | 10:52 |
| hrw | xiaolin: you do not use pflash on mips? | 10:54 |
| elod | lyarwood: thanks for the patch! \o/[4~ I've fast-approved it >>> https://review.opendev.org/#/c/739608/ | 10:55 |
| hrw | xiaolin: done | 10:56 |
| lyarwood | elod: awesome thanks :) | 10:56 |
| xiaolin | hrw: our KVM only support rom now | 10:57 |
| lyarwood | elod: https://review.opendev.org/#/c/740193/ should also fix stable/queens btw | 10:57 |
| hrw | xiaolin: comment comment then | 10:59 |
| elod | lyarwood: thanks! let's wait for Rocky patch to merge then I'll +2+W that too :) | 10:59 |
| sean-k-mooney | lyarwood: what branches are teh migration test broken on | 10:59 |
| sean-k-mooney | ah also rocky? | 10:59 |
| sean-k-mooney | i notice many of the rechecks i kicked off for some of my backports failed yesterday on the live migration job | 11:00 |
| sean-k-mooney | havent had time to look into it yet | 11:00 |
| sean-k-mooney | ERROR] /tmp/devstack-plugin-ceph/devstack/lib/ceph:1029 Supported for Ubuntu Xenial, Bionic or Focal. Not supported for other releases. | 11:01 |
| sean-k-mooney | thats what ye are fixing? | 11:01 |
| lyarwood | yes | 11:01 |
| lyarwood | just taken a while as there were other issues with the ceph plugin we had to fix first | 11:02 |
| sean-k-mooney | cool ill recheck those after it merges so | 11:02 |
| lyarwood | https://review.opendev.org/#/c/739694/ | 11:02 |
| sean-k-mooney | are tehre any other issue i should be aware of for rocky and queens that i should wait for | 11:02 |
| *** tkajinam has quit IRC | 11:02 | |
| lyarwood | sean-k-mooney: rocky should be GREEN once that lands | 11:03 |
| lyarwood | sean-k-mooney: and queens once https://review.opendev.org/#/c/740193/ lands | 11:03 |
| lyarwood | AFAIK | 11:03 |
| sean-k-mooney | ok i have a number of backport against rock and queens that im trying to get off my outgouing queue but i can wait another while | 11:03 |
| lyarwood | yeah shouldn't be long now | 11:04 |
| lyarwood | I also have a load of stuff I want to rebase once these land | 11:04 |
| sean-k-mooney | oh will i need to rebase i can but then i have too update the commit ids in train which i can do but effort :P | 11:04 |
| lyarwood | sean-k-mooney: and this is why I hate the cherry-pick lines ;) | 11:05 |
| lyarwood | sean-k-mooney: and yes you need to rebase | 11:05 |
| sean-k-mooney | hehe stephenfin aggrees with you | 11:05 |
| stephenfin | self-inflicted pain | 11:05 |
| lyarwood | yarp | 11:06 |
| sean-k-mooney | i normally just avoid backporting acrooss more then one or two branches but its still less painful to do it upstream then downstream | 11:06 |
| sean-k-mooney | if i do it donwstream i also have to clone the bug across multiple releases | 11:06 |
| elod | actually i think the rebase is only needed if there's a merge conflict :-o | 11:10 |
| elod | otherwise zuul takes care of it | 11:10 |
| sean-k-mooney | elod: that is my understandign too | 11:11 |
| sean-k-mooney | although sometiems we get weird behavior for example as with the new cherry-pick checker but normally the zuul speculative merge is suffiecent | 11:12 |
| gibi | cores: can someone send this through? https://review.opendev.org/#/c/725163/ It would unblock the python-novaclient gate | 11:12 |
| sean-k-mooney | and also get once step closer to no more legacy jobs | 11:13 |
| elod | sean-k-mooney: the new cherry-pick checker had a problem if the patch's parent was not the HEAD of the branch, but that is fixed already (in ussuri and train, where the check is merged so far) | 11:13 |
| sean-k-mooney | elod: yep i saw the discussion on the topic | 11:14 |
| sean-k-mooney | downstream our branches are named differently so we are figuring out how to handel that | 11:14 |
| elod | oh :S | 11:15 |
| sean-k-mooney | i think we are going to trun it off and replace it with a different one | 11:15 |
| sean-k-mooney | so that if we import change to the script it wont confliect with our downstream changes | 11:15 |
| *** songwenping__ has quit IRC | 11:15 | |
| sean-k-mooney | lyarwood: did you propsoe a patch to do that by the way? | 11:16 |
| *** songwenping__ has joined #openstack-nova | 11:16 | |
| *** dave-mccowan has joined #openstack-nova | 11:17 | |
| sean-k-mooney | elod: right now it just another reason to do more backports upstream since they can actully merge :) | 11:17 |
| sean-k-mooney | well pass pep8 | 11:17 |
| elod | sean-k-mooney: that's at least a good thing ;) | 11:17 |
| lyarwood | sean-k-mooney: yeah we disabled it downstream for now | 11:18 |
| sean-k-mooney | cool | 11:18 |
| lyarwood | https://review.gerrithub.io/c/redhat-openstack/octario/+/497097 | 11:18 |
| sean-k-mooney | oh we just comment it out | 11:19 |
| sean-k-mooney | that works | 11:19 |
| *** markvoelker has joined #openstack-nova | 11:19 | |
| sean-k-mooney | its not the most horrid thing we do to the tox.ini | 11:19 |
| elod | then I just quietly advertise that the cherry-pick check script is backported to stein and waiting for review :X o:) | 11:22 |
| *** markvoelker has quit IRC | 11:24 | |
| *** songwenping_ has joined #openstack-nova | 11:25 | |
| *** songwenping__ has quit IRC | 11:28 | |
| *** songwenping__ has joined #openstack-nova | 11:42 | |
| *** songwenping_ has quit IRC | 11:45 | |
| sean-k-mooney | im not a functional progaming person normally but its nice whe a refactoring turns a function that previously required mocks to test into a pure function of its inputs | 11:46 |
| *** jungleboyj_ is now known as jungleboyj | 11:46 | |
| *** songwenping_ has joined #openstack-nova | 11:52 | |
| *** songwenping__ has quit IRC | 11:55 | |
| *** bhagyashris is now known as bhagyashris|brb | 12:04 | |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.PY2 and six.PY3 https://review.opendev.org/726512 | 12:07 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.add_metaclass https://review.opendev.org/726565 | 12:08 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.reraise https://review.opendev.org/726898 | 12:08 |
| *** songwenping__ has joined #openstack-nova | 12:14 | |
| *** xek has quit IRC | 12:14 | |
| *** songwenping_ has quit IRC | 12:17 | |
| gibi | stephenfin: I'm pretty close to +A https://review.opendev.org/#/c/714701/12/nova/virt/hardware.py@1956 but it would be nice to get a nod from you too | 12:21 |
| stephenfin | looking | 12:21 |
| sean-k-mooney | isnt that stephens pathc | 12:21 |
| gibi | sean-k-mooney: the explanation is not from Stephen | 12:22 |
| sean-k-mooney | ah | 12:22 |
| sean-k-mooney | i was like the fact he wrote it proably means he thinks its correct | 12:22 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224 | 12:23 |
| stephenfin | gibi: Yeah, so numa_topology is set to None on line 1813 https://review.opendev.org/#/c/714701/12/nova/virt/hardware.py@a1813 | 12:23 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757 | 12:23 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777 | 12:23 |
| stephenfin | and it's not set to anything else until it's returned on line 1915 | 12:23 |
| stephenfin | in the old code, this is | 12:23 |
| *** raildo has joined #openstack-nova | 12:23 | |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094 | 12:23 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109 | 12:24 |
| gibi | stephenfin: ack, thanks | 12:24 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117 | 12:24 |
| gibi | stephenfin: if you are at it, I have a similar question in https://review.opendev.org/#/c/714703/11/nova/virt/hardware.py@a1148 | 12:25 |
| stephenfin | iirc no because it's set by something else later, but let me double check :) | 12:26 |
| sean-k-mooney | that if looks correct to me too by the way | 12:26 |
| sean-k-mooney | although i proably woudl have fliped it and returned none rather then the else | 12:26 |
| openstackgerrit | Merged openstack/nova stable/rocky: Fix os_CODENAME detection and repo refresh during ceph tests https://review.opendev.org/739608 | 12:27 |
| lyarwood | \o/ | 12:30 |
| openstackgerrit | Wenping Song proposed openstack/nova-specs master: Add no user token when get Cyborg client https://review.opendev.org/740184 | 12:30 |
| stephenfin | gibi: I need to pull this down to figure it out, but I'm in the middle of fixing up some vTPM stuff. I'll get to it soon as I'm done with that :) | 12:31 |
| gibi | stephenfin: no worries I can switch from vtpm to the pcpu series in the meantime | 12:31 |
| gibi | I mean the opposite | 12:32 |
| stephenfin | gibi: Cool. This one's the one I'm fixing by the way. I got my logic wrong :( Comments left https://review.opendev.org/#/c/739208/3 | 12:32 |
| stephenfin | I'm going to get rid of the '_create_domain_from_guest' function entirely since it's not doing what we think (all it does is resume the guest) | 12:33 |
| stephenfin | that file needs to be broken up asap :( | 12:33 |
| sean-k-mooney | stephenfin: while you are hear quick question about the type annotation, do you only use 'module.classname' for cases where it is not in scope locally where you are using the annotation | 12:34 |
| sean-k-mooney | there is no reason to do that if its in scope correct | 12:34 |
| stephenfin | sean-k-mooney: I have a great answer for that prepared for gibi :D | 12:34 |
| gibi | stephenfin: I missed that logic error too. good catch | 12:34 |
| stephenfin | sean-k-mooney: https://review.opendev.org/#/c/730382/6/nova/virt/libvirt/driver.py@3586 | 12:34 |
| sean-k-mooney | i was reading the upstream docs and i cant find any reference to the '' syntax that is why i asked | 12:34 |
| sean-k-mooney | as i would have assume that triggered type inferance and would treat it as a sting | 12:35 |
| stephenfin | Nope, it just prevents Python (the interpreter) from parsing it | 12:35 |
| stephenfin | Only mypy will do so | 12:35 |
| sean-k-mooney | ah ok | 12:36 |
| * gibi glad that he is not the only one got confused by type hinting and mypy | 12:36 | |
| stephenfin | sean-k-mooney: https://www.python.org/dev/peps/pep-0563/ | 12:36 |
| sean-k-mooney | reading your responce now | 12:36 |
| sean-k-mooney | gibi: its relitivly new and still evolving | 12:36 |
| stephenfin | sean-k-mooney: and also https://mypy.readthedocs.io/en/latest/kinds_of_types.html#class-name-forward-references | 12:36 |
| *** bhagyashris|brb is now known as bhagyashris | 12:37 | |
| sean-k-mooney | ah i skimed the kinds_of_types section a few mintues ago but missed that | 12:37 |
| stephenfin | Yeah, lots of rough edges. I like what it promises though, plus the fact we can introduce it in stages | 12:37 |
| * stephenfin has been using it for Sphinx | 12:37 | |
| sean-k-mooney | ah its a forwared refernce that is what i was asking | 12:37 |
| sean-k-mooney | its like when you do struct MyStruct{}; in c/c++ to avoid importing MyStruct.h | 12:38 |
| * gibi likes sean-k-mooney's c++ metaphors | 12:40 | |
| sean-k-mooney | i almost never write c++ but i watch a tone of talks ans podcasts on c++ | 12:41 |
| sean-k-mooney | i hate c but to be honest after 7+ years of python at this point im now coming to like it as much or more then c++ | 12:42 |
| sean-k-mooney | type hits certenly remove one of my gripes with python | 12:42 |
| gibi | I wrote c++ last time 7 years ago still I appreciate it as a language. I would go with Rust if I have to choose today | 12:46 |
| *** raildo has quit IRC | 12:47 | |
| sean-k-mooney | i dont like the syntax of rust but i can appreciate some of the design chosies regarding the borrow checker. though the clang based tooling that has been wrrent for c++ is quickly approchign the same level of checking | 12:48 |
| *** nweinber has joined #openstack-nova | 12:48 | |
| sean-k-mooney | languates that use let to intoduce variables like basic just feel strange to me | 12:49 |
| sean-k-mooney | it took me a very long time to get used to type on the right had side of an identifier too | 12:49 |
| *** raildo has joined #openstack-nova | 12:50 | |
| sean-k-mooney | its becomeing the norm now but the reason new lanagues do that is its eaier to write the parser it has nothing to do with human readabilty | 12:50 |
| *** lseki has joined #openstack-nova | 12:51 | |
| gibi | stephenfin: you did a nice job splitting up the vtpm series. thank you for that | 12:55 |
| stephenfin | np | 12:56 |
| stephenfin | thanks for the reviews :) | 12:56 |
| stephenfin | be sure to let me know if I can return the favour | 12:57 |
| gibi | sure :) | 12:57 |
| *** alex_xu has joined #openstack-nova | 12:58 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Reject boot request for unsupported images https://review.opendev.org/708608 | 13:05 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Add functional test for bug 1550919 https://review.opendev.org/732707 | 13:05 |
| openstack | bug 1550919 in OpenStack Compute (nova) rocky "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Lee Yarwood (lyarwood) | 13:05 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732708 | 13:05 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: compute: Allow snapshots to be created from PAUSED volume backed instances https://review.opendev.org/729177 | 13:07 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: libvirt: check job status for VIR_DOMAIN_EVENT_SUSPENDED_MIGRATED event https://review.opendev.org/711233 | 13:07 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Remove 'test_cold_migrate_with_physnet_fails' test https://review.opendev.org/703115 | 13:08 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Block rebuild when NUMA topology changed https://review.opendev.org/703116 | 13:08 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Disable NUMATopologyFilter on rebuild https://review.opendev.org/703117 | 13:08 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: FUP for in-place numa rebuild https://review.opendev.org/703118 | 13:08 |
| lyarwood | sean-k-mooney: ^ rebased for you btw | 13:09 |
| *** songwenping_ has joined #openstack-nova | 13:13 | |
| sean-k-mooney | oh cool thanks | 13:16 |
| *** songwenping__ has quit IRC | 13:17 | |
| *** xek has joined #openstack-nova | 13:17 | |
| *** mtreinish has quit IRC | 13:33 | |
| *** mtreinish has joined #openstack-nova | 13:33 | |
| *** songwenping__ has joined #openstack-nova | 13:35 | |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: objects: Introduce the 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/716267 | 13:37 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: libvirt: apply mixed instance CPU policy https://review.opendev.org/728481 | 13:37 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: Validate CPU pinning configuration for mixed instance https://review.opendev.org/716268 | 13:37 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: Calculate the CPU usage for mixed instance https://review.opendev.org/716269 | 13:37 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: scheduler: mixed instance CPU resource translating https://review.opendev.org/728482 | 13:37 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: flavor: Enable 'hw:cpu_dedicated_mask' for creating a mixed instance https://review.opendev.org/714706 | 13:37 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: compute: bump nova-compute service version for mixed instance https://review.opendev.org/716271 | 13:37 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: hardware: create 'mixed' instance for realtime and shared CPUs https://review.opendev.org/728485 | 13:37 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: hardware: refactor numa_get_constraints https://review.opendev.org/718294 | 13:37 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936 | 13:37 |
| openstackgerrit | Huaqiang Wang proposed openstack/nova master: release note: add new features introduces by use-pcpu-and-vcpu-in-one-instance https://review.opendev.org/736997 | 13:37 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Simplify '_create_domain' function https://review.opendev.org/739208 | 13:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Add vTPM config support https://review.opendev.org/739209 | 13:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: scheduler: Request vTPM trait based on flavor or image https://review.opendev.org/739210 | 13:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: crypto: Add support for creating, destroying vTPM secrets https://review.opendev.org/739211 | 13:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: manager: Prevent compute startup on invalid vTPM config https://review.opendev.org/739212 | 13:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add emulated TPM support to Nova https://review.opendev.org/631363 | 13:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934 | 13:38 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add docs for vTPM support https://review.opendev.org/739213 | 13:38 |
| *** songwenping_ has quit IRC | 13:38 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | 13:44 |
| stephenfin | gibi: Have effectively rewritten https://review.opendev.org/#/c/739208/ again from learnings on the previous PS. Hopefully it makes sense | 13:48 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP/DNM zuul: Attempt to migrate and break up the nova-live-migration job https://review.opendev.org/711604 | 13:57 |
| *** ratailor has quit IRC | 14:04 | |
| openstackgerrit | Merged openstack/nova master: hardware: Invert order of NUMA topology generation https://review.opendev.org/714701 | 14:12 |
| *** mriedem has joined #openstack-nova | 14:16 | |
| *** lbragstad_ has joined #openstack-nova | 14:29 | |
| *** lbragstad has quit IRC | 14:29 | |
| *** lchabert has quit IRC | 14:29 | |
| *** dklyle has joined #openstack-nova | 14:30 | |
| *** lchabert has joined #openstack-nova | 14:32 | |
| *** mlavalle has joined #openstack-nova | 14:50 | |
| *** bhagyashris is now known as bhagyashris|dinn | 14:52 | |
| *** vishalmanchanda has quit IRC | 14:54 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: DNM: testing os-vif gate on ubuntu focal https://review.opendev.org/738130 | 14:56 |
| *** Luzi has quit IRC | 14:56 | |
| *** jraju__ has quit IRC | 14:57 | |
| *** mnasiadka has joined #openstack-nova | 14:58 | |
| openstackgerrit | Merged openstack/nova stable/queens: Fix os_CODENAME detection and repo refresh during ceph tests https://review.opendev.org/740193 | 15:11 |
| *** ganso has quit IRC | 15:13 | |
| *** ganso has joined #openstack-nova | 15:17 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: migrate testing to ubuntu focal https://review.opendev.org/738126 | 15:22 |
| *** k_mouza has quit IRC | 15:31 | |
| *** k_mouza has joined #openstack-nova | 15:37 | |
| openstackgerrit | Merged openstack/python-novaclient master: Switch legacy Zuul jobs to native Zuul v3 jobs https://review.opendev.org/725163 | 15:42 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/stein: Remove stale nested backport from InstancePCIRequests https://review.opendev.org/738199 | 15:44 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: libvirt: Mark e1000e VIF as supported https://review.opendev.org/736962 | 15:44 |
| gibi | stephenfin: https://review.opendev.org/#/c/739208/ looks good to me | 15:48 |
| *** bhagyashris|dinn is now known as bhagyashris | 15:49 | |
| *** gyee has joined #openstack-nova | 15:50 | |
| gibi | nova meeting starts in 9 minutes on #openstack-meeting-3 | 15:51 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Improve documentation of (unpin|pin)_cpus_with_siblings() https://review.opendev.org/740327 | 15:56 |
| artom | sean-k-mooney, stephenfin ^^ quick drive-by comment patch while I work on the downstream bug | 15:56 |
| sean-k-mooney | there are docs for that :P | 16:00 |
| sean-k-mooney | so on master if cpus & sib: should alwasy be empty/false when you have hw:cpu_thread_policy=isolate | 16:02 |
| sean-k-mooney | we change the meaning in train | 16:02 |
| sean-k-mooney | before it ment select host without SMT or claim both siblings if SMT is enabled on the host | 16:03 |
| sean-k-mooney | form train on its just select a host without SMT | 16:03 |
| artom | sean-k-mooney, *shrug* feel free to -1 :) | 16:03 |
| sean-k-mooney | so i dont think there is a valid path to use pin_cpus_with_siblings anymore unless we are allowing the old behavior if you dont report PCPUs | 16:04 |
| sean-k-mooney | stephenfin: are we supporting ^ | 16:04 |
| stephenfin | sean-k-mooney: https://review.opendev.org/#/c/705666/ | 16:05 |
| sean-k-mooney | hehe i even reviewd it i tought we spoke about this before | 16:05 |
| sean-k-mooney | so we can remove those in victoria right | 16:06 |
| stephenfin | yup | 16:07 |
| sean-k-mooney | should thos be in the runway with the mixed cpus changes or just treated as seperate cleanups | 16:07 |
| stephenfin | just got to find the time | 16:07 |
| stephenfin | separate | 16:07 |
| stephenfin | let's land the feature first | 16:07 |
| sean-k-mooney | ok | 16:07 |
| *** udesale_ has joined #openstack-nova | 16:08 | |
| *** udesale has quit IRC | 16:10 | |
| *** hamalq has joined #openstack-nova | 16:18 | |
| *** k_mouza has quit IRC | 16:19 | |
| *** hamalq_ has joined #openstack-nova | 16:19 | |
| *** hamalq has quit IRC | 16:22 | |
| *** ociuhandu_ has joined #openstack-nova | 16:23 | |
| openstackgerrit | sean mooney proposed openstack/nova master: lookup nic feature by pci address https://review.opendev.org/739131 | 16:24 |
| sean-k-mooney | alex_xu: i need to test this on my sriov servers so ill set -w on it for now but ^ should adress your comment | 16:24 |
| sean-k-mooney | it should be much more effiecnt then we had before | 16:25 |
| *** ociuhandu has quit IRC | 16:27 | |
| *** udesale_ has quit IRC | 16:27 | |
| *** ociuhandu_ has quit IRC | 16:28 | |
| gmann | sean-k-mooney: there are more deps need to bump lower-constraint, I am getting nova job green first and then same we can apply on os-vif- https://review.opendev.org/#/c/738126/8 | 16:31 |
| gmann | i think i need to create Focal env and test in one shot. let's see if there are more failure | 16:33 |
| sean-k-mooney | i have folal devn envs but i have run devstack so they are not clean anymore | 16:44 |
| sean-k-mooney | i can proabnly test the pathcs locally if you need me too i just need to boot a clean vm but not today | 16:44 |
| sean-k-mooney | gmann: i assme infra have no plance to add the deps to the base images so we either need to fix them in bindeps in each project or bump the minor verions | 16:45 |
| sean-k-mooney | gmann: looks like https://review.opendev.org/#/c/738130/ passed for os-vif | 16:46 |
| gmann | sean-k-mooney: cool. | 16:46 |
| gmann | infra side, not at lest min version as that is project side things | 16:47 |
| *** andreaf has quit IRC | 16:48 | |
| *** mnasiadka_ has joined #openstack-nova | 16:48 | |
| sean-k-mooney | i was more thinking of the libffi-dev issue | 16:48 |
| sean-k-mooney | but ok | 16:48 |
| gmann | let me update cmt msg and remove depends-on as 738130 has lower-constraint update also | 16:48 |
| *** yankcrime has quit IRC | 16:49 | |
| sean-k-mooney | sure. the kuryr-kubernetes-tempest job is not an os-vif job by the way so we will need to fix kuryr-kubernetes before that will pass | 16:49 |
| *** dasp_ has joined #openstack-nova | 16:49 | |
| gmann | yeah | 16:49 |
| sean-k-mooney | its uses os-vif form git however so we run it to make sure we dont break them | 16:50 |
| sean-k-mooney | which reminds me with the upcoming change in devstack to use ovn by default - os-vif-ovs will be come an ovn job but we need to make sure that - os-vif-ovs-iptables keeps using ml2/ovs | 16:51 |
| sean-k-mooney | thats unrealted but it just occured to me | 16:51 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Re-enable live snapshot for paused instances https://review.opendev.org/740334 | 16:53 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Remove workaround for really old QEMU https://review.opendev.org/740335 | 16:53 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: WIP: Attempt to restore some sanity to snapshot https://review.opendev.org/740336 | 16:53 |
| gmann | yeah, till now we added ovn as n-v job on devstack and check its stability but yes we need to adjust few bits for ovs specific jobs | 16:53 |
| gmann | let me try kuryr-kubernetes migration also and test all things together. | 16:54 |
| *** coreycb has quit IRC | 16:55 | |
| *** mnasiadka has quit IRC | 16:55 | |
| *** rm_work has quit IRC | 16:55 | |
| *** nicolasbock has quit IRC | 16:55 | |
| *** mordred has quit IRC | 16:55 | |
| *** dasp has quit IRC | 16:55 | |
| *** mnasiadka_ is now known as mnasiadka | 16:55 | |
| sean-k-mooney | given that job install k8s it might be more work then other to get working | 16:55 |
| sean-k-mooney | it will pull in docker and k8s which both shoudl be availabel on ubuntu 20.04 but there may be issue with the devstack plugin even before you get to the kuyry part | 16:56 |
| sean-k-mooney | or it might just work | 16:56 |
| gmann | yeah, i remember the bionic migration case. but let's try | 16:56 |
| *** rouk has quit IRC | 16:56 | |
| *** masterpe has quit IRC | 16:56 | |
| sean-k-mooney | ill admit its the job i know least about on the os-vif gate | 16:57 |
| gmann | any reason it was n-v ? | 16:57 |
| sean-k-mooney | yes it used to be quite unstable | 16:58 |
| sean-k-mooney | well the old job did at least then the old job was deleted | 16:58 |
| *** coreycb has joined #openstack-nova | 16:58 | |
| *** rm_work has joined #openstack-nova | 16:58 | |
| *** nicolasbock has joined #openstack-nova | 16:58 | |
| sean-k-mooney | its been fine form most of last cycle but we have had relitivly low volume of patches so it never seamed worth the effort to make it voting | 16:58 |
| sean-k-mooney | i still check its results before mreging stuff but dont want to have it block someting it there gate is broken | 16:59 |
| *** derekh has quit IRC | 16:59 | |
| sean-k-mooney | we did plan to make it voting evnetually it has just never happened. | 17:00 |
| *** masterpe has joined #openstack-nova | 17:00 | |
| *** yankcrime has joined #openstack-nova | 17:00 | |
| *** mordred has joined #openstack-nova | 17:07 | |
| gmann | ok, I created Focal env, testing locally | 17:08 |
| *** songwenping_ has joined #openstack-nova | 17:16 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Allow 'hw:cpu_realtime_mask' to be omitted https://review.opendev.org/737026 | 17:17 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Tweak the 'cpu_realtime_mask' handling slightly https://review.opendev.org/461456 | 17:17 |
| *** songwenping__ has quit IRC | 17:19 | |
| *** lbragstad_ is now known as lbragstad | 17:21 | |
| bauzas | gibi: sorry, I was off my keyboard, and I'll be on PTO from tomorrow to Wednesday morning | 17:23 |
| * bauzas going to some resort | 17:23 | |
| *** slaweq has quit IRC | 17:28 | |
| *** slaweq has joined #openstack-nova | 17:31 | |
| *** slaweq has quit IRC | 17:36 | |
| *** dtantsur is now known as dtantsur|afk | 17:45 | |
| *** ralonsoh has quit IRC | 17:57 | |
| *** JamesBenson has joined #openstack-nova | 18:18 | |
| *** tesseract has quit IRC | 18:27 | |
| *** its-surya has quit IRC | 18:35 | |
| *** xek has quit IRC | 18:57 | |
| openstackgerrit | Merged openstack/python-novaclient master: use stevedore to load extensions https://review.opendev.org/739331 | 19:03 |
| *** songwenping__ has joined #openstack-nova | 19:13 | |
| *** songwenping_ has quit IRC | 19:16 | |
| *** tosky has quit IRC | 19:26 | |
| *** damien_r has quit IRC | 19:50 | |
| *** damien_r has joined #openstack-nova | 19:50 | |
| *** JamesBenson has quit IRC | 20:06 | |
| *** xek has joined #openstack-nova | 20:07 | |
| *** ociuhandu has joined #openstack-nova | 20:12 | |
| *** Yumeng has quit IRC | 20:13 | |
| *** damien_r has quit IRC | 20:24 | |
| *** ociuhandu has quit IRC | 20:31 | |
| *** slaweq has joined #openstack-nova | 20:33 | |
| *** xek has quit IRC | 20:36 | |
| *** ociuhandu has joined #openstack-nova | 20:47 | |
| *** nweinber has quit IRC | 20:56 | |
| *** ociuhandu has quit IRC | 20:59 | |
| openstackgerrit | Merged openstack/nova stable/ussuri: hardware: Raise useful error for invalid mempage size https://review.opendev.org/736073 | 21:05 |
| eandersson | bauzas if you guys have time could you look at getting https://review.opendev.org/#/c/703116/ merged? would love to push that patch out | 21:07 |
| eandersson | Worst case I can just cherry-pick it, but a lot nicer to just be able to rebase :D | 21:07 |
| eandersson | https://review.opendev.org/#/q/topic:bug/1763766+(status:open) | 21:11 |
| *** damien_r has joined #openstack-nova | 21:11 | |
| *** maciejjozefczyk has quit IRC | 21:12 | |
| *** raildo has quit IRC | 21:22 | |
| *** songwenping_ has joined #openstack-nova | 21:26 | |
| *** songwenping__ has quit IRC | 21:29 | |
| openstackgerrit | Merged openstack/nova stable/train: Guard against missing image cache directory https://review.opendev.org/738455 | 21:39 |
| *** eharney has quit IRC | 21:42 | |
| *** slaweq has quit IRC | 22:21 | |
| *** damien_r has quit IRC | 22:33 | |
| *** slaweq has joined #openstack-nova | 22:38 | |
| *** jamesdenton has quit IRC | 22:40 | |
| *** hrw has quit IRC | 22:43 | |
| *** damien_r has joined #openstack-nova | 22:44 | |
| *** hrw has joined #openstack-nova | 22:53 | |
| *** tkajinam has joined #openstack-nova | 23:00 | |
| *** mlavalle has quit IRC | 23:06 | |
| *** mriedem has left #openstack-nova | 23:07 | |
| *** damien_r has quit IRC | 23:11 | |
| *** damien_r has joined #openstack-nova | 23:17 | |
| *** damien_r has quit IRC | 23:22 | |
| openstackgerrit | Merged openstack/nova stable/queens: Reset the cell cache for database access in Service https://review.opendev.org/720596 | 23:44 |
| *** gyee has quit IRC | 23:56 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!