Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-12-09
18:40:19 mnaser we do
18:41:27 mnaser task state gets updated inside nova.compute.api -- does that mean that technically there could be 2 concurrent api requests?
18:42:06 sean-k-mooney you can have two concurrent request to destroy yes at the api level
18:42:27 sean-k-mooney but the window would be quite short
18:42:27 mnaser so i guess in that case if two api requests came at the same time, we can end up with two requests in the compute level
18:42:52 sean-k-mooney yes
18:43:32 sean-k-mooney although we normaly do a check of the taskstate wehn ever we updat it so one would fail
18:49:57 mordred mriedem, efried: sdk release was cut including your ironic fix
18:51:05 efried ack, thx
18:51:08 mnaser sean-k-mooney: _record_action_start is called, ill check logs
18:54:09 efried sean-k-mooney: theoretically...
18:54:17 efried if a rebuild is done with the same image...
18:54:36 efried couldn't I have changed the metadata on that image since my instance was originally booted?
18:57:21 mnaser sean-k-mooney: sigh, i wonder if we call _destroy before starting up an instance which is why it says "Instance destroyed successfully."
18:58:02 mnaser sean-k-mooney: yes thats exactly what happens, we call destroy for power on because we call hard reboot
19:05:12 dustinc [1] https://blueprints.launchpad.net/nova/+spec/provider-config-file
19:05:12 dustinc disable the entire feature by default instead
19:05:12 dustinc dansmith: I was working on docs/etc for [1] and was thinking that it might be a good idea to have a master on/off switch for the feature with a default of off...the reason I was thinking that is because as it is specced right now, the directory is the only config option and a cautious operator would probably want to make sure the default directory is present with permissions set..it would maybe be easier to just
19:05:16 dustinc what do you think?
19:05:56 dansmith dustinc: how is a conf knob turned off different from the directory/file(s) being missing?
19:06:37 dustinc in theory if not present someone else could create the directory and place files
19:07:18 efried I was also thinking maybe instead of having the directory conf option default to something sane, have it default to None, and that's the "off" switch. You have to turn it on explicitly by setting it to something.
19:07:23 dansmith dustinc: in /etc/nova? not anyone without sufficient privileges
19:09:47 dustinc dansmith: if that's sufficient then maybe leave it as it is then..
19:11:47 dustinc thanks
19:14:32 mnaser sean-k-mooney: off the top of your head? i could swear there was a nova option which skipped waiting for port plugs when starting up an instance
19:14:37 mnaser am i imagining things
19:15:49 dansmith mnaser: vif_plugging_timeout=0 and vif_plugging_fatal=False
19:15:58 dansmith mnaser: but don't do it except for debugging
19:16:01 mnaser dansmith: ah yes, ok that makes sense
19:16:27 mnaser dansmith: yep, i don't want to either, we're just trying to figure out why rdocloud can bring instances up so quickly (yet this new cloud is taking so long the tripleo-ci b its are timing out)
19:16:54 mnaser so given its taking ~1-2 minutes to turn up an instance, because a lot of it is unplugging (in cleanup) and plugging (in startup)
19:17:02 dustinc dansmith: dont_be_slow=true
19:36:16 openstackgerrit Alexandre arents proposed openstack/nova stable/rocky: Do not update root_device_name during guest config https://review.opendev.org/696353
20:21:55 sean-k-mooney mnaser: i see dansmith alreay answered your question. are the new ci and rdocloud both using hte same netwrok backend.
20:22:13 mnaser sean-k-mooney: afaik that runs ovs and so do we, dont have too much details except ocata vs stein
20:22:37 sean-k-mooney i think rdocloud is running ml2/ovs too
20:23:44 sean-k-mooney efried: if a rebuild is done with the same iamge we do not go to the schduler and use the cached copy of the image mentadata
20:24:08 sean-k-mooney efried: we only go to the schduler if the image changes
20:34:43 efried interesting
20:38:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Add sequence diagrams for cross-cell-resize https://review.opendev.org/698051
20:38:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Flesh out docs for cross-cell resize/cold migrate https://review.opendev.org/696212
20:38:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement cleanup_instance_network_on_host for neutron API https://review.opendev.org/697162
20:38:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Simplify FinishResizeAtDestTask event handling https://review.opendev.org/695337
20:38:47 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Implement reschedule logic for cross-cell resize/migrate https://review.opendev.org/696213
20:38:47 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add negative test to delete server during cross-cell resize claim https://review.opendev.org/688832
20:38:50 sean-k-mooney resource usage is not ment to change on a rebuild so since the flavor does not change and the image does not change you dont need to check.
20:40:19 mriedem except for silly ol numa
20:40:27 sean-k-mooney yep
20:40:30 mriedem which is why eric finds himself in this situation
20:41:20 sean-k-mooney well even in the numa case if you dont change the image then your fine but ya people want to do that for some reason :)
20:41:49 mordred changing the image doesn't seem like a rebuild to me - it seems like creating a new server
20:41:55 mordred but - you know - what do I know?
20:42:54 sean-k-mooney :) well we are stuck with the terms we have
20:43:11 sean-k-mooney atleast its less confusing then evacuate
21:36:09 openstackgerrit Mykola Yakovliev proposed openstack/nova master: Validate aggregate IDs before querying database https://review.opendev.org/698094
21:38:01 openstackgerrit Merged openstack/nova master: Handle ServiceNotFound in DbDriver._report_state https://review.opendev.org/697301
22:08:46 openstackgerrit Merged openstack/nova master: VMware: disk_io_limits settings are not reflected when resize https://review.opendev.org/680296
22:21:10 openstack bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Medium,In progress] https://launchpad.net/bugs/1829479 - Assigned to Matt Riedemann (mriedem)
22:21:10 openstackgerrit Merged openstack/nova stable/stein: Add functional recreate test for bug 1829479 and bug 1817833 https://review.opendev.org/695932
22:21:11 openstack bug 1817833 in OpenStack Compute (nova) "Check compute_id existence when nova-compute reports info to placement" [Medium,In progress] https://launchpad.net/bugs/1817833 - Assigned to Matt Riedemann (mriedem)
22:21:14 openstackgerrit Merged openstack/nova stable/stein: Add functional recreate test for bug 1852610 https://review.opendev.org/695935
22:21:15 openstack bug 1852610 in OpenStack Compute (nova) stein "API allows source compute service/node deletion while instances are pending a resize confirm/revert" [Undecided,In progress] https://launchpad.net/bugs/1852610 - Assigned to Matt Riedemann (mriedem)
22:21:20 openstackgerrit Merged openstack/nova stable/stein: Add functional recreate revert resize test for bug 1852610 https://review.opendev.org/695938
22:59:14 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add NodeOwnerFilter https://review.opendev.org/697331
23:00:41 efried sean-k-mooney: (I hope you don't see this until tomorrow) Reviewed the series, have a question on the affinity patch: why do we need the notification payload change?
23:02:03 efried and with that, I'm outta here o/
23:12:03 openstackgerrit Merged openstack/nova stable/stein: Block deleting compute services with in-progress migrations https://review.opendev.org/695940
23:13:09 openstack bug 1829479 in OpenStack Compute (nova) "The allocation table has residual records when instance is evacuated and the source physical node is removed" [Medium,In progress] https://launchpad.net/bugs/1829479 - Assigned to Matt Riedemann (mriedem)
23:13:09 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Add functional recreate test for bug 1829479 and bug 1817833 https://review.opendev.org/698106
23:13:10 openstack bug 1817833 in OpenStack Compute (nova) "Check compute_id existence when nova-compute reports info to placement" [Medium,In progress] https://launchpad.net/bugs/1817833 - Assigned to Matt Riedemann (mriedem)
23:21:14 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Add functional recreate test for bug 1852610 https://review.opendev.org/698108
23:21:15 openstack bug 1852610 in OpenStack Compute (nova) rocky "API allows source compute service/node deletion while instances are pending a resize confirm/revert" [Undecided,New] https://launchpad.net/bugs/1852610
23:22:50 openstack bug 1852610 in OpenStack Compute (nova) rocky "API allows source compute service/node deletion while instances are pending a resize confirm/revert" [Undecided,New] https://launchpad.net/bugs/1852610
23:22:50 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Add functional recreate test for bug 1852610 https://review.opendev.org/698108
23:31:18 openstack bug 1852610 in OpenStack Compute (nova) rocky "API allows source compute service/node deletion while instances are pending a resize confirm/revert" [Undecided,New] https://launchpad.net/bugs/1852610
23:31:18 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Add functional recreate revert resize test for bug 1852610 https://review.opendev.org/698110
23:35:38 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Block deleting compute services with in-progress migrations https://review.opendev.org/698113
23:49:08 openstackgerrit Merged openstack/nova master: Block rebuild when NUMA topology changed https://review.opendev.org/687957
#openstack-nova - 2019-12-10
01:49:51 openstackgerrit xulei proposed openstack/nova master: Optimization for nova-api _checks_for_create_and_rebuild https://review.opendev.org/698125
03:05:46 openstackgerrit Merged openstack/nova stable/stein: compute: Use long_rpc_timeout in reserve_block_device_name https://review.opendev.org/696955
03:37:09 openstackgerrit Eric Xie proposed openstack/nova master: Report trait 'COMPUTE_IMAGE_TYPE_PLOOP' https://review.opendev.org/698132
05:41:05 openstackgerrit XiaojueGuan proposed openstack/nova master: Fix error out with an UnboundException when in build_and_run_instance https://review.opendev.org/698140
06:35:37 openstackgerrit xulei proposed openstack/nova master: Optimization for nova-api _checks_for_create_and_rebuild https://review.opendev.org/698125
06:40:26 openstackgerrit xulei proposed openstack/nova master: Optimization for nova-api _checks_for_create_and_rebuild https://review.opendev.org/698125
09:00:06 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_QEMU_FILE_BACKED_DISCARD_VERSION https://review.opendev.org/695915
09:00:07 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Remove MIN_{LIBVIRT,QEMU}_FILE_BACKED_VERSION https://review.opendev.org/695942
09:32:50 luyao efried, stephenfin : I invite you to look at spec for live migration with vpmem if you get time. https://review.opendev.org/#/c/695863
10:00:23 shilpasd bauzas: thanks for review https://review.opendev.org/#/c/650188 >> NFS spec
10:05:31 bauzas shilpasd: I'll try to look more
10:05:43 bauzas still working downstream unfortunately
10:07:01 shilpasd bauzas: ok, thank you
10:07:14 shilpasd attending nova weekly to have more review from nova core
10:08:07 shilpasd this just for FYI
10:12:38 openstackgerrit Eric Xie proposed openstack/nova master: Report trait 'COMPUTE_IMAGE_TYPE_PLOOP' https://review.opendev.org/698132
11:01:36 openstackgerrit Stephen Finucane proposed openstack/nova master: Switch to uses_virtio to enable iommu driver for AMD SEV https://review.opendev.org/696697
11:01:37 openstackgerrit Stephen Finucane proposed openstack/nova master: Create a controller for qga when SEV is used https://review.opendev.org/693072
11:01:37 openstackgerrit Stephen Finucane proposed openstack/nova master: Also enable iommu for virtio controllers and video in libvirt https://review.opendev.org/684825
11:07:02 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Convert remaining API tests to use neutron https://review.opendev.org/696509
11:07:02 openstackgerrit Stephen Finucane proposed openstack/nova master: nova-net: Drop nova-network-base security group tests https://review.opendev.org/696508

Earlier   Later