| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-11 | |||
| 15:45:27 | dtroyer | efried, sean-k-mooney: it is probably my github fail… I hit approve… but I've also been multi-tasking two meetings this whole time... | |
| 15:45:31 | stephenfin | when you've flown Ryanair, anything is a lovely airline | |
| 15:45:49 | melwitt | I find lufthansa's gate boarding "system" a total mess (in that there seems to be no system) but other than that, I've been happy enough with it | |
| 15:46:13 | sean-k-mooney | dtroyer: no worries. you have too also manually submit it if there is not a ci set up to do it | |
| 15:48:33 | efried | aspiers: fyi you can knock a patch out of the gate by rebasing it. I can re+W if you want to do that rather than waiting for the gate to fail. | |
| 15:49:18 | aspiers | efried: ah, nice idea thanks! | |
| 15:50:59 | bauzas | well, what I can just say about Lufthansa is how they're aggresive with carry-on luggage | |
| 15:51:05 | openstackgerrit | sean mooney proposed openstack/nova master: multi numa nfv testing job https://review.opendev.org/679656 | |
| 15:51:06 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] testing with new lable. https://review.opendev.org/680738 | |
| 15:51:35 | bauzas | every time I flew with them, my carry-on was taken on checked luggage, just because they felt they were running out of space in the cabin | |
| 15:51:59 | dtroyer | efried, sean-k-mooney: ok, for the record, just talked with Dolpher and his final change is ready to be merged into the Intel-OpenStack-CI-jobs repo, I should get that done after this meeting | |
| 15:52:07 | bauzas | I admit this was in Geneva, but come on, even on August when all people wear light stuff ? | |
| 15:52:12 | efried | dtroyer: nice | |
| 15:52:40 | sean-k-mooney | dtroyer: ok ill close my PR then | |
| 15:52:50 | sean-k-mooney | lets just go with the proper one | |
| 15:53:20 | sean-k-mooney | also i forgot to chnage the rul in the numa tests new patch incoming | |
| 15:54:56 | efried | melwitt: I wonder if you're hitting https://github.com/openstack/keystoneauth/blob/38cd5fc6c39c38a51c11683884caf9696ce5f367/keystoneauth1/exceptions/http.py#L411 which afaict will result in your exception containing bugger-all. | |
| 15:56:08 | openstackgerrit | sean mooney proposed openstack/nova master: multi numa nfv testing job https://review.opendev.org/679656 | |
| 15:56:09 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] testing with new lable. https://review.opendev.org/680738 | |
| 15:56:44 | efried | melwitt: ...which may be possible if the encoding is off or something. | |
| 15:59:04 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] test migration with pinning https://review.opendev.org/679754 | |
| 15:59:38 | melwitt | efried: thanks for the idea | |
| 16:01:12 | artom | Stupid sphinx errors line numbers that have nothing to do with anything | |
| 16:01:19 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] test non overlaping vcpu pin sets. https://review.opendev.org/679805 | |
| 16:02:06 | sean-k-mooney | artom: ok all the numa jobs should be using the old nodepool lable now and running agaist the patch before the functionl tests | |
| 16:05:50 | sean-k-mooney | maybe i missed one | |
| 16:06:29 | mriedem | artom: if it was reno, i use an online sphinx editor to validate those rather than wait to build them locally | |
| 16:07:08 | aspiers | efried: hrm, bit confused - now the failure has disappeared from the gate but there's another job for it still running in the gate's "integrated" queue which has no failures | |
| 16:07:30 | artom | mriedem, thanks for the suggestion, I think I'll do that | |
| 16:07:38 | artom | sean-k-mooney, cheers, thanks! | |
| 16:07:46 | aspiers | efried: might it have failed in a different queue in the gate? which gate queue(s) actually count for the V+2? | |
| 16:07:48 | sean-k-mooney | fixing the last one | |
| 16:08:02 | efried | aspiers: is it possible you were looking at your patch as it existed as part of some other patch's chain? | |
| 16:08:13 | efried | what's the change #? | |
| 16:08:21 | aspiers | efried: 681254 | |
| 16:08:34 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] test non overlaping vcpu pin sets. https://review.opendev.org/679805 | |
| 16:08:35 | aspiers | efried: the failure I saw was at the tip of a chain | |
| 16:09:34 | aspiers | weird, can't see it | |
| 16:09:41 | aspiers | the only failure was from the first time around | |
| 16:09:58 | aspiers | i.e. 2019-09-11T05:56:59 which was before my recheck | |
| 16:10:04 | aspiers | maybe I was hallucinating | |
| 16:10:12 | efried | yeah, it looks okay to me | |
| 16:10:35 | sean-k-mooney | donnyd: fyi i have 4 jobs queue 3 using the old lable and 1 using the new one so ill let you know how it goes. hopeful your router tweaks resolved the issue | |
| 16:14:30 | sean-k-mooney | stephenfin: efried is there anything i can do to help with either the PCPU in plamcenet seriese or pmem? | |
| 16:15:40 | efried | sean-k-mooney: I'm at a point where I'm willing to proxy your vote on https://review.opendev.org/#/c/678455/ -- once we've agreed whether it needs to be changed or not. | |
| 16:15:56 | efried | I'm not clear whether y'all came to an agreement above? | |
| 16:16:02 | stephenfin | sean-k-mooney: I'm on the vpmem one, so if you fancy looking at either https://review.opendev.org/#/c/674895/ | |
| 16:16:08 | stephenfin | efried: alex_xu is on it | |
| 16:16:18 | mriedem | TestInstanceNotificationSampleWithMultipleCompute.test_multiple_compute_actions just reset the gate again, | |
| 16:16:22 | mriedem | i think we have a regression | |
| 16:16:47 | stephenfin | We agreed it was better to keep the one guest NUMA node to one host NUMA node model, even if we don't do proper affinity for the VPMEM devices | |
| 16:17:12 | efried | and that's because qemu has some checks in place that don't allow us to do the slightly better thing? | |
| 16:17:13 | sean-k-mooney | stephenfin: so like pci passthoug with affinity disabled | |
| 16:17:25 | stephenfin | efried: Yup | |
| 16:17:30 | efried | okay cool. | |
| 16:17:42 | stephenfin | sean-k-mooney: Yeah, good analogy | |
| 16:18:01 | stephenfin | efried: I did have a comment on https://review.opendev.org/#/c/678448/20 though | |
| 16:18:36 | stephenfin | I'm not sure if it's worth respinning for but if we don't, we're stuck with that object name forever | |
| 16:19:32 | sean-k-mooney | whats SlugField | |
| 16:20:15 | sean-k-mooney | oh somting that is alpha numeric or has _ | |
| 16:20:27 | stephenfin | sean-k-mooney: That's the terminology I'm used to from Django. I thought it was more widespread than that, tbh | |
| 16:20:40 | stephenfin | yeah, exactly | |
| 16:20:44 | sean-k-mooney | not really but i have seen you use it before | |
| 16:21:16 | stephenfin | here we go https://en.wikipedia.org/wiki/Clean_URL#Slug | |
| 16:21:24 | efried | stephenfin: I thought slug was with hyphens, not underscores | |
| 16:21:33 | stephenfin | Good point. It is | |
| 16:21:44 | efried | oh, according to that wp article it's either. | |
| 16:22:02 | efried | but also lowercased | |
| 16:22:08 | efried | and also specific to URLs | |
| 16:22:15 | sean-k-mooney | ya this is specifcly only uppercase | |
| 16:22:27 | efried | in fact | |
| 16:22:27 | efried | I was beaten up about this by jaypipes at some point when I tried to name a method slugify | |
| 16:22:28 | sean-k-mooney | its basically what i know as CONST_CASE | |
| 16:22:40 | efried | I think it was the method now known in os-traits and os-resource-classes as normalize_name :) | |
| 16:22:41 | efried | so no | |
| 16:22:44 | efried | you can't call it a slug | |
| 16:22:53 | efried | because Jay will kick your ass | |
| 16:23:17 | efried | IMO it's definitely not worth respinning for this. | |
| 16:23:41 | efried | even if this object winds up single use | |
| 16:23:59 | stephenfin | Fair point. I was on the fence as it was | |
| 16:24:18 | sean-k-mooney | we can alwas do it later | |
| 16:24:33 | sean-k-mooney | if we have a common felid type this can jsut be an alias or inherit | |
| 16:24:41 | stephenfin | very true | |
| 16:24:47 | stephenfin | I switched to +2 anyway | |
| 16:24:51 | mriedem | https://bugs.launchpad.net/nova/+bug/1843615 | |
| 16:24:52 | openstack | Launchpad bug 1843615 in OpenStack Compute (nova) "TestInstanceNotificationSampleWithMultipleCompute.test_multiple_compute_actions intermittently failing since Sept 10, 2019" [Undecided,New] | |
| 16:25:09 | stephenfin | on account of it not being a slug and therefore probably not as reusable as I thought it would be | |
| 16:25:24 | sean-k-mooney | stephenfin: once question does the qutotat code need to be move down in the series | |
| 16:25:41 | stephenfin | sean-k-mooney: Yup, see my comment on same. I'll move it when I squash that fixup patch back | |
| 16:25:54 | stephenfin | except for the functional test. That has to wait til the end, of course | |
| 16:25:55 | sean-k-mooney | ah cool | |
| 16:26:12 | stephenfin | the squash back which will be happening any minute now | |
| 16:29:43 | openstackgerrit | Alex Xu proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456 | |
| 16:29:43 | openstackgerrit | Alex Xu proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455 | |
| 16:29:44 | openstackgerrit | Alex Xu proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470 | |
| 16:29:44 | openstackgerrit | Alex Xu proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640 | |
| 16:29:45 | openstackgerrit | Alex Xu proposed openstack/nova master: objects: use all_things_equal from objects.base https://review.opendev.org/681397 | |
| 16:30:39 | alex_xu | stephenfin: sean-k-mooney ^ done \o/ | |
| 16:31:09 | alex_xu | really small change actually | |
| 16:31:14 | alex_xu | and tested | |