| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-29 | |||
| 20:22:05 | bauzas | mriedem: agreed, all of that supposes a fully functional cloud | |
| 20:22:23 | bauzas | mriedem: yup, shelved instances was the hack | |
| 20:22:35 | bauzas | shelved *offloaded* | |
| 20:22:41 | mriedem | rather than just create and then stop in separate calls | |
| 20:22:58 | bauzas | but by the time the instance unshelves, you have no guarantee that it will succeed | |
| 20:23:15 | dansmith | bauzas: rebuild doesn't ask anything of the scheduler right? | |
| 20:23:32 | bauzas | dansmith: lemme verify, my guts say the contrary | |
| 20:23:51 | dansmith | only for evacuate, IIRC | |
| 20:24:25 | mriedem | the scheduler has to take into account image properties in the new image | |
| 20:24:26 | mriedem | for rebuild | |
| 20:24:50 | mriedem | i think we actually have a bug for that, | |
| 20:24:54 | mriedem | because we don't do that today | |
| 20:25:08 | dansmith | bauzas: host=instance.hosdt | |
| 20:25:34 | mriedem | that's the bug | |
| 20:25:49 | mriedem | the new image could have properties that don't fit the instance's existing host | |
| 20:25:57 | bauzas | dansmith: yeah you're right, just verified | |
| 20:26:06 | dansmith | mriedem: right, but rebuild is supposed to be in place | |
| 20:26:06 | bauzas | dansmith: and I think it's a miss | |
| 20:26:23 | mriedem | https://bugs.launchpad.net/nova/+bug/1667667 | |
| 20:26:23 | openstack | Launchpad bug 1667667 in OpenStack Compute (nova) "Rebuilding instance ignores image's property hw_disk_bus" [Low,Confirmed] | |
| 20:26:25 | bauzas | mriedem: resize != rebuild | |
| 20:26:31 | mriedem | bauzas: i know | |
| 20:26:46 | bauzas | mriedem: yeah I know you know, but it's not taking a new flavor, right? | |
| 20:26:55 | bauzas | oh, the image props | |
| 20:26:59 | dansmith | image | |
| 20:27:00 | dansmith | right | |
| 20:27:13 | bauzas | so yeah, we should call the scheduler too | |
| 20:27:15 | jaypipes | mriedem: np | |
| 20:27:33 | bauzas | either way, I think we should call it, just because we need to verify we can still fit onto the host | |
| 20:27:48 | dansmith | if we change hosts during a rebuild, we have more work to do right? | |
| 20:27:53 | dansmith | we have to handle volumes and ports and stuff | |
| 20:28:02 | bauzas | yeah | |
| 20:28:07 | dansmith | because rebuild assumes the original host is gone and is going to destroy on reboot | |
| 20:28:09 | dansmith | personally, | |
| 20:28:11 | bauzas | but we should restrict to the same host | |
| 20:28:20 | dansmith | I'd rather refuse to let you rebuild as another image if it's so incompatible that you can't rebuild in place | |
| 20:28:21 | bauzas | just verifying the scheduler once more | |
| 20:28:52 | bauzas | using requested_destination as saying "I want to test this one only" | |
| 20:29:03 | bauzas | anyway, I think we're diverting | |
| 20:29:29 | bauzas | I'm no longer involved with all those reservation crazy ideas, so I defer other Blazar folks to provide good usecases for that | |
| 20:29:33 | bauzas | but I bet they are | |
| 20:29:37 | bauzas | there are* | |
| 20:30:15 | bauzas | what I want is not a crazy hack or a new API into Nova | |
| 20:30:40 | bauzas | what I want is to see how the placement service could help their problems, that's it | |
| 20:30:56 | bauzas | and avoid seeing specs like https://review.openstack.org/#/c/389216/2 | |
| 20:31:17 | jaypipes_ | mriedem, bauzas: sorry, electrician turned off the power. :( | |
| 20:31:53 | bauzas | jaypipes_: heh, it's a trap | |
| 20:31:57 | dansmith | jaypipes_: don't be sorry, consider yourself lucky | |
| 20:39:40 | bauzas | heh, should I propose https://www.amazon.com/APC-Back-UPS-Protector-Charging-BE600M1/dp/B01FWAZEIU/ref=lp_764572_1_1?s=pc&ie=UTF8&qid=1490819957&sr=1-1 to jaypipes ? | |
| 20:49:13 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | |
| 21:04:16 | mriedem | dansmith: evacuate assumes the original host is gone, same code path to rebuild in the compute i know, | |
| 21:04:33 | mriedem | but i'm just pointing out the thing where you rebuild on a live host, and we don't validate the new image props still work on that host, | |
| 21:04:36 | mriedem | i agree we should just fail in that case | |
| 21:04:40 | mriedem | rather than try to rebuild on another host | |
| 21:05:14 | mriedem | i think we actually have a duplicate for that bug, i swear bauzas was on another bug just like that | |
| 21:06:12 | bauzas | mriedem: none I remember but okay :) | |
| 21:06:21 | bauzas | I rather have a long list of TODOs :) | |
| 21:07:37 | mriedem | bauzas: maybe get some sleep, and get up early, drink some nice french press coffee, a crepe, and tackle that todo list :) | |
| 21:08:01 | mriedem | oh i found the other bug | |
| 21:08:18 | bauzas | sleep is for weak people | |
| 21:08:47 | bauzas | enough spilled time | |
| 21:10:03 | mriedem | says the guy whose government mandates he only works a certain number of hours and has mandatory PTO | |
| 21:10:13 | mriedem | hi-o! | |
| 21:13:03 | bauzas | mriedem: I live close to Switzerland | |
| 21:16:25 | efried | Tu veux-dire que les suisses travaillent comme toi? Ou que tu travailles comme les suisses? ;-) | |
| 21:18:39 | openstackgerrit | Eric Fried proposed openstack/nova master: Glance download: only fsync files https://review.openstack.org/451094 | |
| 21:19:16 | efried | mdbooth ^^ Hopefully this (plus the fact that fsync isn't used anywhere else) goes toward assuaging your concerns about having to check everywhere. | |
| 21:19:26 | efried | thorst ^^ see whatcha think. | |
| 21:19:53 | thorst | efried: looking | |
| 21:22:41 | efried | Rats, forgot to remove imports. PS3 coming. | |
| 21:23:20 | openstackgerrit | Eric Fried proposed openstack/nova master: Glance download: only fsync files https://review.openstack.org/451094 | |
| 21:26:25 | bauzas | oh man, originally thought we had a Twitter account https://twitter.com/OpenStackNova | |
| 21:26:47 | bauzas | I should write to Trump asking for changing the name for Northern Virginia | |
| 22:54:53 | clarkb | hey so I'm poking at using UCA libvirt (because its newer and possibly gives us a place to start bisecting libvirt issues) and running into http://logs.openstack.org/92/451492/4/check/gate-devstack-dsvm-py35-updown-ubuntu-xenial-nv/a537722/logs/screen-n-cpu.txt.gz#_2017-03-29_22_44_08_090 does that mean nova doesn't actually support libvirt 2.5.0? python-libvirt is 3.1.0 | |
| 22:55:12 | clarkb | the job failing there is the py35 job, but the python2 job will eventually fail with the same error looks like | |
| 22:55:34 | clarkb | oh you know what, I wonder if this is a wheel build issue | |
| #openstack-nova - 2017-03-30 | |||
| 01:16:02 | openstackgerrit | christopher.uhler proposed openstack/nova master: Libvirt volume driver for Veritas HyperScale https://review.openstack.org/443951 | |
| 01:36:48 | openstackgerrit | Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_vmops https://review.openstack.org/451614 | |
| 02:25:21 | openstackgerrit | Gao Yuan proposed openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_vmops https://review.openstack.org/451614 | |
| 02:44:45 | gaoyuan | ／whois Kevin_Zheng | |
| 03:20:18 | openstackgerrit | Huan Xie proposed openstack/nova master: WIP: Fix race condition when neutron is using minimized poll https://review.openstack.org/444230 | |
| 03:22:06 | openstackgerrit | Alex Xu proposed openstack/nova master: [placement] Allow PUT and POST without bodies https://review.openstack.org/447625 | |
| 03:22:07 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201 | |
| 03:22:07 | openstackgerrit | Alex Xu proposed openstack/nova master: placement: Add Traits API to placement service https://review.openstack.org/376200 | |
| 03:22:08 | openstackgerrit | Alex Xu proposed openstack/nova master: Rename the model object ResourceProviderTraits to ResourceProviderTrait https://review.openstack.org/451635 | |
| 03:56:08 | openstackgerrit | Huan Xie proposed openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF https://review.openstack.org/451641 | |
| 04:52:48 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix wrong HTTP response codes https://review.openstack.org/451645 | |
| 04:55:26 | openstackgerrit | XieYingYun proposed openstack/nova master: Remove unnecessary setUp function in testcase https://review.openstack.org/451646 | |
| 05:34:30 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add description to policies in servers_migrations.py https://review.openstack.org/451487 | |
| 05:37:29 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove mox from nova/tests/unit/consoleauth/test_consoleauth.py https://review.openstack.org/448386 | |
| 05:42:02 | gmann | gaoyuan: ^^ | |
| 05:42:28 | gmann | gaoyuan: so on standard style, we do not have exact written doc as of now | |
| 05:42:55 | gmann | gaoyuan: but you can check old patches and already fixed tests to get the basic idea. | |
| 05:43:10 | gmann | gaoyuan: this is the original BP where mox removal things started - https://blueprints.launchpad.net/nova/+spec/remove-mox | |
| 06:07:02 | openstackgerrit | Huan Xie proposed openstack/nova master: XenAPI: Create linux bridge in dest host during live migration https://review.openstack.org/451657 | |
| 06:13:32 | openstackgerrit | jichenjc proposed openstack/nova master: Add resource classes description in api-ref https://review.openstack.org/451661 | |
| 06:16:47 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add description to policies in hypervisors.py https://review.openstack.org/451294 | |
| 06:19:23 | openstackgerrit | Alex Xu proposed openstack/nova-specs master: Deprecate multinic proxy API https://review.openstack.org/384261 | |
| 06:27:26 | openstackgerrit | lcsong proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |
| 06:44:27 | openstackgerrit | lcsong proposed openstack/nova master: Migrate VMs failed when anti-affinity group was removed from VMs. https://review.openstack.org/451664 | |