| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-01-15 | |||
| 15:57:38 | dansmith | Sundar: no, return_value will work as many times as you want: https://pastebin.com/2Wdgb64J | |
| 15:58:20 | sean-k-mooney | if you want to have a limited number of retruns or different retrun i think you use sideffect with an iterable | |
| 15:58:42 | Sundar | Ah, my bad, I still have it as side_effect. Will fix that | |
| 15:58:53 | dansmith | ... | |
| 16:01:04 | Sundar | I think it should work with these changes. This was the only pending point, AFAICS. Are you good with updating the patch set once this is resolved? | |
| 16:01:11 | Sundar | dansmith: ^ | |
| 16:01:45 | dansmith | Sundar: unless you've replied to other comments I had, I think so | |
| 16:02:56 | Sundar | I pretty much agreed to other changes, even if it is a matter of preference. | |
| 16:03:24 | aarents | lyarwood: I replied to your remark https://review.opendev.org/#/c/696084 maybe kashyap can confirm that implemeting finish_unshelved as you suggest is the way to go | |
| 16:23:03 | openstackgerrit | Sundar Nadathur proposed openstack/os-traits master: Add a trait that compute manager can handle accelerator requests. https://review.opendev.org/698961 | |
| 16:43:53 | openstack | bug 1859766 in OpenStack Compute (nova) stein "functional tests intermittently fails with "ReadOnlyFieldError: Cannot modify readonly field uuid"" [Undecided,New] https://launchpad.net/bugs/1859766 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 16:43:53 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/stein: DNM: try to trace bug 1859766 https://review.opendev.org/702700 | |
| 17:27:53 | openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Ussuri https://review.opendev.org/693844 | |
| 17:28:41 | kashyap | aarents: Sorry, I wasn't ignoring that change, just been buried. | |
| 17:46:06 | stephenfin | efried_afk: not sure if you're gone for the day, but https://review.opendev.org/#/c/697537/ could probably be revisited now that x-cell resize has merged | |
| 17:47:03 | stephenfin | and melwitt, I think I've clarified my (new) intent in https://review.opendev.org/#/c/696745/, which should assuage some of your concerns hopefully | |
| 17:47:08 | stephenfin | with that | |
| 17:47:32 | melwitt | stephenfin: ack | |
| 18:02:52 | openstack | bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms) | |
| 18:02:52 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404 | |
| 18:03:46 | artom | dansmith, could I ask you to look at that 2-patch series ^^ when you get a chance? stephenfin's the other reviewer, but your +1 or +W'less +2 can still carry weight and help the evential non-RH approver (efried, I'm guessing) | |
| 18:12:05 | dansmith | artom: yeah, although I'm a couple layers deep on the stack and have a call in 18 minutes | |
| 18:12:27 | dansmith | artom: meaning, don't let me forget | |
| 18:14:15 | fungi | anybody remember how far back nova started providing configdrive by default without needing to manually enable it in the config? | |
| 18:14:29 | fungi | (i mean providing it if the client requests it at boot) | |
| 18:14:35 | dansmith | a long time | |
| 18:15:06 | fungi | cool, config reference only goes back as far as pike but seemed to indicate it was available by default at least that far back | |
| 18:15:10 | fungi | thanks dansmith! | |
| 18:15:27 | sean-k-mooney | fungi: its been availabel via the api | |
| 18:15:27 | dansmith | yeah I'm pretty sure it's been around for a real (real) long time | |
| 18:15:55 | sean-k-mooney | its not on by defualt unless you request it or set the force config drive options in the config | |
| 18:16:12 | sean-k-mooney | it might predate micorverions | |
| 18:16:23 | fungi | yep, thanks. trying to help a user in #zuul figure out whether they have to do anything fancy in nova's config to make configdrive available to nodepool | |
| 18:16:25 | dansmith | it definitely predates microversions | |
| 18:16:42 | fungi | sounds like no, just make sure nodepool asks for a configdrive | |
| 18:17:03 | dansmith | well, isn't there a way to configure it to *not* honor the user's configdrive request? | |
| 18:17:47 | dansmith | ah, nevermind, looks like they can always ask for it | |
| 18:17:48 | dansmith | so yeah I think you're good | |
| 18:18:45 | sean-k-mooney | fungi: the only thing they shoudl have to do is make sure the iso vfat tools are installed so nova can actully create the config drive | |
| 18:18:57 | sean-k-mooney | but they will be on any normal install | |
| 18:19:14 | fungi | right, already suggested that to them | |
| 18:19:24 | fungi | either genisofs or mkisofs depending on the vintage | |
| 18:19:36 | sean-k-mooney | yep | |
| 18:19:49 | fungi | i have a feeling they just need to adjust their nodepool launcher config | |
| 18:20:58 | fungi | they provided a `openstack show server` paste which indicated config_drive was not True and i expect that comes down to the boot parameters it provided | |
| 18:22:22 | sean-k-mooney | right. it could be forced in there config but nodepool has a setting for that too right | |
| 18:23:01 | sean-k-mooney | you can use https://zuul-ci.org/docs/nodepool/configuration.html#attr-providers.[openstack].diskimages.config-drive | |
| 18:24:04 | sean-k-mooney | actully it looks like node pool does not use the boot parmater since that is on the image and instead is using the image metadata right? | |
| 18:24:57 | sean-k-mooney | there are 3 ways to trun on cofig drive, the force_config_drive config option, the api boot request or the image metadata | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999 | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554 | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553 | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable hard/soft reboot with accelerators. https://review.opendev.org/697940 | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Compose accelerator PCI devices into domain XML in libvirt driver. https://review.opendev.org/631245 | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Pass accelerator requests to each virt driver from compute manager. https://review.opendev.org/698581 | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Define Cyborg ARQ binding notification event. https://review.opendev.org/692707 | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243 | |
| 18:31:46 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242 | |
| 18:39:56 | openstackgerrit | waleed mousa proposed openstack/os-vif master: [Follow Up] OVS DPDK port representors support https://review.opendev.org/665965 | |
| 18:43:29 | artom | dansmith, sorry, was on a 1:1 | |
| 18:43:36 | artom | dansmith, thanks! I'll be sure to hound your ass then | |
| 19:01:40 | efried | #success With the merging of https://review.opendev.org/#/c/696518/ mox is finally purged from nova \o/ | |
| 19:02:02 | sean-k-mooney | cool | |
| 19:02:06 | efried | takashin, stephenfin: ^ I closed out the bp | |
| 19:02:20 | efried | hm, no response from successbot | |
| 19:02:30 | efried | shrug | |
| 19:02:43 | sean-k-mooney | it was so amazed that its finally done it need a minute | |
| 19:03:19 | sean-k-mooney | or its borked | |
| 19:06:03 | dansmith | maybe it is opinionated now and doesn't agree it's a successbot-worthy moment? | |
| 19:06:59 | sean-k-mooney | in that case can we get it to do code reviews for us too | |
| 19:07:42 | sean-k-mooney | its tallens are clearly being wasted | |
| 19:11:10 | openstackgerrit | Merged openstack/nova master: libvirt: Remove MIN_LIBVIRT_KVM_AARCH64_VERSION https://review.opendev.org/695901 | |
| 19:16:00 | efried | stephenfin: done, and the one a couple further up the series. | |
| 19:19:00 | fungi | openstackstatus is not present in here. checking to see if it's fallen victim to a netsplit or something, hold on | |
| 19:22:40 | fungi | efried: okay, try again! | |
| 19:22:53 | efried | #success With the merging of https://review.opendev.org/#/c/696518/ mox is finally purged from nova \o/ | |
| 19:22:54 | openstackstatus | efried: Added success to Success page (https://wiki.openstack.org/wiki/Successes) | |
| 19:22:58 | efried | nice, thanks fungi! | |
| 19:23:05 | fungi | it was disconnected for the past 50 minutes due to some shakeup in freenode | |
| 19:23:11 | fungi | but should be fine now | |
| 19:23:21 | efried | you da person | |
| 19:23:24 | fungi | efried: thanks for communicating the moxtermination! | |
| 19:23:44 | fungi | i think we've all been looking forward to it for a *very* long time | |
| 19:24:23 | fungi | heh | |
| 19:25:21 | sean-k-mooney | fungi: while you are here. do you know what the plan will be regarding ubuntu 20.04 i assume we will move to it sometime in V | |
| 19:26:30 | sean-k-mooney | i guess that decsion wont be made untill closer to the start of V but just said i would ask | |
| 19:26:45 | fungi | sean-k-mooney: it's slated to be released around ussuri rc1 last i looked, so we'll likely have images available before ussuri releases but maybe not as soon as master reopens for victoria commits | |
| 19:27:29 | sean-k-mooney | is it actully victoria. i did not see a poll for the name although i might have missed it | |
| 19:28:17 | fungi | aiui, the tc is ready to declare ubuntu 20.04 a release platform for victoria though: https://review.opendev.org/693743 | |
| 19:29:28 | sean-k-mooney | ok cool we were discussing a kerenl 5.4 feature downstream earlier and the reason i asked is i think 20.04 will be based on 5.4 | |
| 19:30:46 | fungi | sean-k-mooney: yeah, voting on the v cycle name happened in early december, collecting ideas started in mid-november | |
| 19:30:49 | fungi | http://lists.openstack.org/pipermail/openstack-discuss/2019-November/010880.html | |
| 19:30:55 | fungi | http://lists.openstack.org/pipermail/openstack-discuss/2020-January/011947.html | |
| 19:31:19 | fungi | announcement was ^ last week after legal was done with trademark searches and risk eval | |
| 19:31:19 | sean-k-mooney | ah ok i saw the tread on the proposal but i obviously missed the vote likely when i went on vaction | |
| 19:32:07 | fungi | and the w cycle naming process is about to get underway: https://review.opendev.org/702414 | |
| 19:36:37 | sean-k-mooney | cool thanks for the update :) | |
| 19:38:19 | sean-k-mooney | efried: dansmith i currently have sundars latest update stacking. ill likely test it out tomorow however since im about to call it a day | |