| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-02-03 | |||
| 09:31:50 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Compose accelerator PCI devices into domain XML in libvirt driver. https://review.opendev.org/631245 | |
| 09:31:50 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Pass accelerator requests to each virt driver from compute manager. https://review.opendev.org/698581 | |
| 09:31:51 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable hard/soft reboot with accelerators. https://review.opendev.org/697940 | |
| 09:31:51 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | |
| 09:31:52 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554 | |
| 09:31:52 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable start/stop of instances with accelerators. https://review.opendev.org/699553 | |
| 09:31:53 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999 | |
| 09:31:53 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227 | |
| 09:52:40 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227 | |
| 09:52:40 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554 | |
| 09:52:41 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999 | |
| 10:49:16 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove extra instance.save() calls related to qos SRIOV ports https://review.opendev.org/702261 | |
| 10:50:48 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use common server create function for qos func tests https://review.opendev.org/701353 | |
| 10:52:20 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enable live migration with qos ports https://review.opendev.org/699066 | |
| 10:55:26 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support unshelve with qos ports https://review.opendev.org/704759 | |
| 11:28:47 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reduce gen conflict in COMPUTE_STATUS_DISABLED handling https://review.opendev.org/704866 | |
| 12:36:04 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support unshelve with qos ports https://review.opendev.org/704759 | |
| 13:01:47 | openstackgerrit | Huachang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936 | |
| 13:01:48 | openstackgerrit | Huachang Wang proposed openstack/nova master: Create 'mixed' instance from PCPU and VCPU resources https://review.opendev.org/705437 | |
| 13:05:42 | openstackgerrit | Huachang Wang proposed openstack/nova master: [WIP] metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936 | |
| 13:05:42 | openstackgerrit | Huachang Wang proposed openstack/nova master: [WIP] Create 'mixed' instance from PCPU and VCPU resources https://review.opendev.org/705437 | |
| 14:02:23 | openstack | Launchpad bug 1860739 in OpenStack Compute (nova) "Cinder backup failed for restore volume snapshot (for instance snapshot)" [Undecided,Confirmed] | |
| 14:02:23 | rlljorge | Hi folks, some chance some one look for : https://bugs.launchpad.net/nova/+bug/1860739 | |
| 14:07:03 | stephenfin | bauzas: Think you could keep marching through this today? https://review.opendev.org/#/c/697155/ | |
| 14:07:17 | bauzas | stephenfin: lemme try | |
| 14:07:32 | stephenfin | ta. lmk if you've any questions | |
| 14:08:52 | sean-k-mooney | stephenfin: do you need me to take a look at that seriese too or will i leave it to bauzas et al | |
| 14:09:19 | sean-k-mooney | thats the nova net begone series right | |
| 14:09:20 | bauzas | sean-k-mooney: you can surely review them if you want | |
| 14:09:37 | bauzas | I'm not really a nova-net expert so having more eyes is better | |
| 14:09:42 | stephenfin | I will never turn down reviewers | |
| 14:10:01 | bauzas | any review is helpful FWIW | |
| 14:10:25 | sean-k-mooney | well basically im just going to double check the commit message and the code make sense and you are not breaking the neutorn path | |
| 14:10:37 | bauzas | even if I (sometimes/always/somehow/pick the one you prefer) disagree with the review and associated comments | |
| 14:11:51 | sean-k-mooney | stephenfin: by the way once this is complete i want to do a pass over os-vif to delete the nova-net only bits | |
| 14:13:17 | sean-k-mooney | we have a few object fields that i think are only there for nova-net so i want to deprecate them and then likely remove them early in V although we could maybe do it in u. time will tell. | |
| 14:14:11 | sean-k-mooney | things like this https://github.com/openstack/os-vif/blob/master/os_vif/objects/network.py#L33 | |
| 14:14:36 | sean-k-mooney | the multi_host flag is for nova networks and i dont think we ever use it in os-vif | |
| 14:15:30 | sean-k-mooney | ok we technically do but only in the linux bridge driver and even so only with nova-networks https://github.com/openstack/os-vif/blob/1e4cf6160efc7fb4fd8279bd2a666e4d72f4fb6c/vif_plug_linux_bridge/linux_bridge.py#L101 | |
| 14:17:58 | sean-k-mooney | dansmith: by the way is your ephemeral object class merged in ovo yet? | |
| 14:18:30 | sean-k-mooney | i would like to use it to clean up this crap https://github.com/openstack/os-vif/blob/master/os_vif/objects/network.py#L41-L47 but i might just copy the base class for now like we did in nova if it is not merged | |
| 14:21:08 | francoisp | hello lyarwood, stephenfin, doing code rework for https://review.opendev.org/#/c/669674/ and I have a question for you: | |
| 14:21:09 | francoisp | I understand we want do do away with the config option, and stick with a hard-coded value for max number of re-tries. | |
| 14:21:09 | francoisp | "If something is failing to complete after 5 loops, surely that's an issue that bumping the value will only mask?" | |
| 14:21:09 | francoisp | stephenfin you put in a comment at nova/conf/compute.py, line 871: | |
| 14:21:10 | francoisp | are you suggesting we use a lower value (ex.: max 3 retries) ? | |
| 14:22:37 | stephenfin | no, I'm only questioning the need to make this configurable rather than hard code it | |
| 14:22:55 | stephenfin | 5 is fine. I'm don't think we'd want to go bigger than that though | |
| 14:22:57 | stephenfin | francoisp: ^ | |
| 14:23:00 | efried | sean-k-mooney: https://review.opendev.org/#/c/702049/ | |
| 14:23:25 | sean-k-mooney | efried: cool thanks that is the one i was looking for | |
| 14:23:39 | sean-k-mooney | efried: so ya ill just bump the min version and use that | |
| 14:23:48 | sean-k-mooney | ill wait till after m2 however to do that | |
| 14:23:49 | efried | That's in nova, no bump | |
| 14:24:01 | sean-k-mooney | oh sorry i taught that was the ovo one | |
| 14:24:21 | sean-k-mooney | os-vif does not import form nova so i either need to copy that or wait for it to merge in ovo | |
| 14:24:54 | sean-k-mooney | this is the ovo review https://review.opendev.org/#/c/701795/ | |
| 14:24:57 | efried | oh, yeah, the ovo one hasn't merged yet https://review.opendev.org/#/c/701795/ | |
| 14:25:00 | efried | yeah | |
| 14:26:23 | sean-k-mooney | looks like dansmith's -w was to wait for concensus on the nova review but since we have merged it i think we are good provided this version matches the nova one | |
| 14:27:45 | sean-k-mooney | efried: ill proably submit a relase patch for os-vif on friday ahead of m2 next week and ill add you once its up | |
| 14:27:45 | dansmith | sean-k-mooney: I was waiting for us to land it in nova and be happy with that definition | |
| 14:28:10 | dansmith | I haven't circled back since I got back from being out, and I thought you were going to think of better names... :) | |
| 14:28:58 | sean-k-mooney | we could. i know i suggested that but i havent really found one that is better | |
| 14:30:36 | sean-k-mooney | ill likely just submit the patches to os-vif to use it and leave the bottom one with -w until your happy with it then | |
| 14:33:44 | gibi | dansmith: hi! updated and responded in https://review.opendev.org/#/c/702261/ | |
| 14:33:47 | ignaziocassano1 | Hello All, anyone could help me on openstack queens cinder with unity iscsi driver ? Often when a vm is migrated or a volume is detached , nova compute log resports errors with the command multipath -f comman (map in use) | |
| 14:35:24 | sean-k-mooney | that ^ sound vaguly familar but maybe lyarwood would know if that is a know issue that is maybe fixed | |
| 14:36:00 | dansmith | gibi: ack, will look post-caffeination | |
| 14:36:05 | gibi | dansmith: thanks | |
| 14:37:15 | efried | sean-k-mooney: Am I to understand that blockdiag accepted your fix? | |
| 14:37:15 | lyarwood | sean-k-mooney: I think that came up on the ML recently and I asked for logs | |
| 14:37:23 | lyarwood | ignaziocassano1: can you write that up in a bug with logs? | |
| 14:37:44 | sean-k-mooney | efried: they merged a similar one that does the same thing but also updates the change log | |
| 14:37:48 | sean-k-mooney | so basically yes | |
| 14:38:12 | efried | sean-k-mooney: okay, cool, so we should blacklist 2.0.0 in u-c and accept, what, 2.0.1 once they cut it? | |
| 14:38:28 | sean-k-mooney | ya basically | |
| 14:38:49 | sean-k-mooney | we can blacklist it now if we want too but i dont know what the eta is on the 2.0.1 release is | |
| 14:39:15 | efried | Well, we know 2.0.0 is busted, but it also doesn't seem to be affecting anyone but us, so... | |
| 14:39:22 | sean-k-mooney | oh its already done | |
| 14:40:56 | sean-k-mooney | https://github.com/blockdiag/blockdiag/compare/2.0.0...2.0.1 | |
| 14:40:58 | ignaziocassano1 | lyqrwood: I am using centos 7.....how can open a bug ? Which site must I use ? | |
| 14:41:20 | sean-k-mooney | efried: want me to submit the patch to the requiremetns repo | |
| 14:42:00 | lyarwood | ignaziocassano1: https://bugs.launchpad.net/os-brick/+filebug - if you could include DEBUG logs showing the volume attachment and attempt to detach that would be super useful | |
| 14:42:17 | ignaziocassano1 | thanks | |
| 14:44:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unused nova-network objects https://review.opendev.org/697156 | |
| 14:44:27 | openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-net: Remove unused parameters https://review.opendev.org/703974 | |
| 14:45:20 | efried | sean-k-mooney: sure | |
| 14:45:51 | sean-k-mooney | ok ill do it shortly once i finish my current review | |
| 14:47:09 | sean-k-mooney | efried: did we already merge stephens fix | |
| 14:47:19 | efried | yes | |
| 14:47:36 | efried | sean-k-mooney: https://review.opendev.org/#/c/705263/ | |
| 14:48:06 | sean-k-mooney | cool im just going to blacklist it in gr we could raise the min version in nova if we want to removed the monkey patching seperatly | |
| 14:48:36 | sean-k-mooney | im not going to raise the lower constratin in the requirements repo. | |
| 14:50:00 | ignaziocassano1 | lyarwood: I opened a bug. Did you know if I will receive email when someone will work on it ? | |
| 14:50:33 | lyarwood | ignaziocassano1: can you share the bug URL here? | |
| 14:50:46 | openstack | Launchpad bug 1861691 in os-brick "openstack queens emc iscsi errors migrating volume and or detaching volumes" [Undecided,New] | |
| 14:50:46 | ignaziocassano1 | https://bugs.launchpad.net/os-brick/+bug/1861691 | |
| 14:51:01 | ignaziocassano1 | I attached the nova-compute log | |
| 14:51:44 | ignaziocassano1 | mmm I must change the description | |