| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-18 | |||
| 06:08:42 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640 | |
| 06:08:43 | openstackgerrit | Luyao Zhong proposed openstack/nova master: doc: attaching virtual persistent memory to guests https://review.opendev.org/680300 | |
| 06:08:43 | openstackgerrit | Luyao Zhong proposed openstack/nova master: objects: use all_things_equal from objects.base https://review.opendev.org/681397 | |
| 06:17:38 | openstackgerrit | garyk proposed openstack/nova master: Deconstruct the mother of all locks https://review.opendev.org/682242 | |
| 06:18:13 | openstackgerrit | Luyao Zhong proposed openstack/nova master: objects: use all_things_equal from objects.base https://review.opendev.org/681397 | |
| 08:01:50 | openstackgerrit | Yongli He proposed openstack/nova master: clean up orphan instances https://review.opendev.org/627765 | |
| 08:15:35 | bauzas | good morning Nova | |
| 08:19:38 | gibi | bauzas: good morning! | |
| 08:21:09 | openstackgerrit | Arthur Dayne proposed openstack/nova master: Fix block disk attachment failure https://review.opendev.org/682772 | |
| 08:24:46 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Stabilize unshelve notification sample tests https://review.opendev.org/676677 | |
| 08:51:38 | cshen | Good morning. I have a short question. This config "rpc_conn_pool_size = 30" in nova.conf on compute nodes , is this pool shared with all workers, or each worker has a rpc connection pool (size=30)? | |
| 08:57:22 | bauzas | cshen: good question | |
| 08:57:38 | bauzas | cshen: the rpc parameter you mention comes from oslo.rpc | |
| 08:57:52 | bauzas | cshen: while the notion of workers comes from oslo.service | |
| 08:58:46 | bauzas | cshen: since workers are fully seen as Linux processes, I'd guess the pool size is per worker | |
| 08:59:03 | bauzas | but I could be wrong | |
| 09:08:59 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rewrite host aggregate, availability zone docs https://review.opendev.org/667133 | |
| 09:09:21 | stephenfin | bauzas: You think you could take a look at that (unfortunately large) doc patch today/this week? ^ | |
| 09:09:31 | stephenfin | I mostly need a sanity check to make sure I'm not telling any lies | |
| 09:09:49 | bauzas | stephenfin: I surely can try to take a look on it :) | |
| 09:10:09 | stephenfin | aaaaand _another_ random job has failed for the cpu-resources series. FFS. | |
| 09:10:32 | stephenfin | This time nova-live-migration with something from devstack-gate, which I thought we weren't using anymore | |
| 09:10:43 | bauzas | oh, facepalm, dude | |
| 09:11:00 | bauzas | stephenfin: I haven't paid attention to the series tbh | |
| 09:11:10 | bauzas | I mean, once they were approved | |
| 09:12:20 | stephenfin | I count 13 rechecks on the base patch since approval :( | |
| 09:12:35 | stephenfin | and all of them for different reasons | |
| 09:12:48 | stephenfin | talk about a moving target | |
| 09:13:47 | brinzhang | stephenfin: these patch are not merged until now :( | |
| 09:14:51 | brinzhang | as you said, the failed reason are not same every time | |
| 09:15:46 | brinzhang | Is it because zuul's execution pressure is too great? | |
| 09:26:24 | stephenfin | I think it's just a lot of things combined | |
| 09:27:18 | stephenfin | TripleO hammering the gate, issues with a package, pressure of the gate, large series with many patches that means increased chances of hitting a bug, etc. | |
| 09:31:02 | cshen | bauzas: thanks for the answer. How many nova workers can I see on the compute node? | |
| 09:32:11 | cshen | bauzas: I can see only 1 nova process is running my compute node now. | |
| 09:59:45 | openstackgerrit | Merged openstack/nova stable/stein: Fix non-existent method of Mock https://review.opendev.org/676838 | |
| 09:59:55 | openstackgerrit | Merged openstack/nova stable/stein: Fix wrong assertions in unit tests https://review.opendev.org/677388 | |
| 10:46:08 | openstackgerrit | Merged openstack/nova stable/stein: Retrun 400 if invalid query parameters are specified https://review.opendev.org/676026 | |
| 11:27:53 | openstackgerrit | Arthur Dayne proposed openstack/nova master: Fix block disk attachment failure https://review.opendev.org/682772 | |
| 11:39:12 | aspiers | https://www.suse.com/c/improving-trust-in-the-cloud-with-openstack-and-amd-sev/ | |
| 11:39:38 | aspiers | https://blog.adamspiers.org/2019/09/13/improving-trust-in-the-cloud-with-openstack-and-amd-sev/ | |
| 11:40:12 | aspiers | which contain a special shout-out to efried and all the Red Hatters here for your help :) | |
| 11:40:21 | sean-k-mooney | aspiers: are you not ment to be on PTO | |
| 11:40:25 | aspiers | yes I am | |
| 11:40:32 | aspiers | but I wanted to get it out this week | |
| 11:40:40 | aspiers | sean-k-mooney: I tweaked that sentence on snooping based on your feedback, thanks | |
| 11:40:41 | sean-k-mooney | fair enough | |
| 11:40:44 | aspiers | hope it's more accurate now | |
| 11:42:55 | sean-k-mooney | aspiers yep and the footnote looks good too | |
| 11:43:01 | sean-k-mooney | enjoy | |
| 11:53:37 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Trigger real BuildAbortException during migrate with bandwidth https://review.opendev.org/682876 | |
| 11:54:41 | gibi | bauzas: a test improvement for the bandwidth functional tests ^^ | |
| 11:54:52 | bauzas | gibi: cool, will look | |
| 11:54:57 | gibi | bauzas: thanks | |
| 11:55:21 | gibi | bauzas: the next patch I will cook up is your request to test migration with pinned RPC version | |
| 12:35:29 | openstackgerrit | sean mooney proposed openstack/nova master: conf: Deprecate 'devname' field of '[pci] passthrough_whitelist' https://review.opendev.org/670585 | |
| 12:45:45 | efried | ++ thanks aspiers! | |
| 12:47:32 | efried | stephenfin, alex_xu: thanks for splitting out the series. In retrospect (hindsight 20/20 and all that) we should have done it long ago, but I didn't want to lose all the +Vs we already had up the series since the gate was so slow. | |
| 12:48:28 | openstackgerrit | Tao Yang proposed openstack/nova master: Add missing parameter https://review.opendev.org/682886 | |
| 12:56:15 | openstackgerrit | sean mooney proposed openstack/nova master: make config drives sticky bug 1835822 https://review.opendev.org/669738 | |
| 12:56:16 | openstack | bug 1835822 in OpenStack Compute (nova) "vms loose acess to config drive with CONF.force_config_drive=True after hard reboot" [Medium,In progress] https://launchpad.net/bugs/1835822 - Assigned to sean mooney (sean-k-mooney) | |
| 12:59:26 | mriedem | dansmith: if you're looking for an easy way to get back into things and be productive we need stable reviews https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:stable/stein | |
| 13:06:37 | shilpasd | dansmith: hi, want your review on spec for isolate aggregate https://review.opendev.org/#/c/675384/ | |
| 13:11:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: make config drives sticky bug 1835822 https://review.opendev.org/669738 | |
| 13:11:43 | openstack | bug 1835822 in OpenStack Compute (nova) "vms loose acess to config drive with CONF.force_config_drive=True after hard reboot" [Medium,In progress] https://launchpad.net/bugs/1835822 - Assigned to sean mooney (sean-k-mooney) | |
| 13:12:31 | mriedem | need another core on https://review.opendev.org/#/c/669738/ since it fixes a regression introduced in train | |
| 13:14:12 | sean-k-mooney | mriedem: thanks i though the api comment was for the flavor no the images i would have used the api otherwise | |
| 13:14:43 | sean-k-mooney | and ya i forgot to chagne the vm name | |
| 13:15:49 | sean-k-mooney | mdbooth: do we actully need the followup patch https://review.opendev.org/#/c/485930/10 i dont think we do as the orginal bug is actully already fixed but i think you wanted to refactor how this was done in general | |
| 13:16:40 | sean-k-mooney | i.e. once https://review.opendev.org/#/c/669738/ is merged i think we can just abandon the follow up patch unless we just want to do it for refactoring reasons | |
| 13:19:39 | sean-k-mooney | bauzas: gibi if ye are around care to review https://review.opendev.org/#/c/669738/ | |
| 13:33:47 | dansmith | mriedem: cool, thanks.. I love being spoon fed | |
| 13:33:53 | dansmith | shilpasd: ack | |
| 13:34:14 | efried | dansmith: you got that one, then? I can stop reading? | |
| 13:34:18 | mriedem | dansmith: i'll burp and change your dipee for $2 | |
| 13:34:25 | mriedem | *dipey? | |
| 13:34:32 | efried | I'm gonna go with dipey. | |
| 13:34:49 | dansmith | efried: what one? You're already +2 on the spec revision | |
| 13:35:41 | efried | dansmith: The config drive regression | |
| 13:36:04 | dansmith | efried: I was responding to mriedem asking me to hit the stable queue | |
| 13:37:38 | efried | okay, I was referring to 8:12:31 AM - mriedem: need another core on https://review.opendev.org/#/c/669738/ since it fixes a regression introduced in train | |
| 13:37:42 | efried | I'll keep reading | |
| 13:40:00 | dansmith | so, it looks like some of the stuff we +Wd before FF still haven't merged.. ISTR efried saying we'd reassess if things haven't merged by monday, and now it's wednesday | |
| 13:40:07 | dansmith | assume we're backing off from the recheck grind? | |
| 13:40:32 | efried | no, I hadn't planned to. | |
| 13:40:44 | bauzas | I think it's okay | |
| 13:40:53 | bauzas | at least until next Thursday... :D | |
| 13:40:59 | dansmith | how close to freeze are we going to keep doing that? | |
| 13:41:23 | efried | Ask me again on Monday | |
| 13:52:17 | mriedem | gibi: need to deal with some ironic-isms in https://review.opendev.org/#/c/666857/ | |
| 13:52:26 | gibi | mriedem: ack, looking | |
| 13:52:28 | mriedem | gibi: also noted that we'll get a warning on every comptue fresh start up | |
| 13:53:52 | gibi | mriedem: is it OK to start up the nova-compute with ironic setup if the ironic-api is not available? | |
| 13:54:51 | mriedem | gibi: that's what we've always had since ironic in devstack is a plugin and comes after nova | |
| 13:55:03 | mriedem | but i think in general it's good for the nova code to be resilient to when the hypervisor isn't available, | |
| 13:55:13 | gibi | mriedem: I see. Then I will log on that error but otherwise ignore it | |
| 13:55:16 | mriedem | e.g. in the libvirt driver case it auto-disables the service record so you can't schedule to that node | |
| 13:55:41 | mriedem | gibi: yeah i think log and return from the method is what i'd expect | |
| 13:55:49 | gibi | mriedem: ok | |
| 14:03:28 | shilpasd | dansmith: thanks for ack | |