Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-03-13
20:34:18 mriedem actually,
20:34:26 mriedem no,
20:34:33 dansmith mriedem: if I'm using default_availability_zone, I can change the target's affiliation with a restart right?
20:34:36 mriedem if RequestSpec.availability_zone is not None, the server *has* to migrate within the AZ,
20:34:56 mriedem meaning while the server is in VERIFY_RESIZE, it's in the same az on the dest host as the source host, so the admin can't rename the az
20:35:52 mriedem and if RequestSpec.availability_zone is None, then we'd end up updating the instance.availability_zone anyway
20:35:59 mriedem i mean with my optimization
20:36:10 dansmith yeah, but what about my CONF-based attack?
20:36:37 dansmith two hosts, both in default_availability_zone=foo, then I can change that conf to =bar on one and restart it and you'll do the wrong thing
20:36:49 dansmith (and instance.az will be wrong)
20:38:27 mriedem yeah i suppose
20:38:56 dansmith it just doesn't seem like enough optimization to worry about
20:39:01 dansmith to me anyway
20:39:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Update instance.availability_zone on revertResize https://review.openstack.org/643155
20:39:59 mriedem where is jaypipes when i want to hear someone say we should just delete availability zones as a concept from nova
20:40:20 dansmith pfft, he'll say that about anything
20:40:45 edleafe (uses my jaypipes imitation) Availability zones suck! Let's just get rid of them!
20:41:10 mriedem * suck! Let's just get rid of them!
20:41:11 mriedem volumes
20:41:12 mriedem quotas
20:41:14 mriedem everything
20:41:23 mriedem *root volumes
20:41:34 dansmith instances kinda suck too
20:41:37 dansmith we should nix those
20:48:08 openstackgerrit Merged openstack/nova master: Trivialfix for help description of images_type https://review.openstack.org/642942
20:48:18 openstackgerrit Merged openstack/nova stable/rocky: Fix an error when generating a host ID https://review.openstack.org/641173
20:48:27 openstackgerrit Merged openstack/nova stable/rocky: Move legacy-grenade-dsvm-neutron-multinode-live-migration in-tree https://review.openstack.org/640191
20:48:35 openstackgerrit Merged openstack/nova stable/rocky: Fix resource tracker updates during instance evacuation https://review.openstack.org/640982
21:00:08 openstackgerrit Merged openstack/nova master: Add missing libvirt exception during device detach https://review.openstack.org/641480
21:03:18 openstackgerrit Merged openstack/nova stable/queens: Make service all-cells min version helper use scatter-gather https://review.openstack.org/637594
21:03:24 openstackgerrit Merged openstack/nova master: api-ref: Add description for BDM volume_size https://review.openstack.org/641255
21:03:34 openstackgerrit Merged openstack/nova master: docs: Fix nits in remote console guide https://review.openstack.org/641098
22:11:00 gmann legacy base jobs to Bionic are merged and nova-next job failing, hold the recheck until this workaround is merged (in gate) - https://review.openstack.org/#/c/643129/3
22:11:08 gmann sent on ML too.
22:17:29 openstackgerrit Matt Riedemann proposed openstack/nova stable/rocky: Update instance.availability_zone during live migration https://review.openstack.org/643173
22:45:31 fungi melwitt: oops, i missed that https://review.openstack.org/642940 had a depends-on to the demonstration patch. that commit message needs to be amended to remove the depends-on line
22:45:51 fungi mriedem_afk: dansmith: ^
22:48:51 efried fungi, melwitt, mriedem_afk, dansmith: I'll take care of it, stand by...
22:49:07 fungi thanks!!!
22:49:07 openstackgerrit Eric Fried proposed openstack/nova master: Update nova-tox-functional-py35 parent job https://review.openstack.org/642940
22:49:14 fungi and sorry for missing that in review
22:49:35 efried fungi: done
22:49:43 efried I noticed that, but I thought it was intentional
22:49:54 efried like, I thought the other patch was going to be non-DNM'd soon
22:50:02 efried or something
22:50:10 efried I didn't read the backstory closely at all
22:50:19 fungi nope, it was entirely a demonstration. apologies for the confusion
22:50:51 fungi we couldn't actually show the effect of the change without a depends-on chain to an alternate universe base job
22:51:06 fungi at least not until the actual base job change merged
22:52:07 efried and with that, I'm outta here o/
22:52:07 efried ack
23:25:13 openstackgerrit Merged openstack/nova master: Update instance.availability_zone during live migration https://review.openstack.org/567701
23:25:24 openstackgerrit Merged openstack/nova master: Skip the ceph based live migration testing https://review.openstack.org/643122
23:40:27 gmann fungi: efried we have cross-dependency to unblock the gate. 643129 cannot merge which need 642940 and vice versa.
23:41:17 fungi possible we'll need to squash those
23:41:26 gmann fungi: can we get this forcefully in - 642940? or i can make temp patch to unblock either one
23:43:13 fungi they'll be properly tested if we squash the two commits together, but would need a nova-core on hand to approve the resulting revision of whichever change we squash into
23:43:54 gmann ok, let me squash first then i can catch someone alex_xu might be coming online soon
23:47:18 fungi i'm just veeery hesitant to bypass ci for a change where it's not strictly necessary
23:47:36 fungi pretty much every time i do, i break something a job would have caught
23:48:50 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Disable the tls-proxy in nova-next & fix nova-tox-functional-py35 parent https://review.openstack.org/643129
23:48:55 gmann done ^^
23:49:33 gmann efried: melwitt dansmith alex_xu bauzas stephenfin mriedem_afk ^^ if you are around.
23:50:02 openstackgerrit Merged openstack/nova master: Trivial: fix typo in reno https://review.openstack.org/643137
23:54:39 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Migrate legacy jobs to Ubuntu Bionic https://review.openstack.org/639017
#openstack-nova - 2019-03-14
00:03:58 melwitt gmann: I'll get it. thank you for all your work on this
00:04:49 gmann thanks.
00:13:18 fungi thanks melwitt!
00:19:51 openstackgerrit melanie witt proposed openstack/nova master: Re-enable Ceph in live migration testing https://review.openstack.org/643150
01:12:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Update instance.availability_zone on revertResize https://review.openstack.org/643155
01:12:46 openstack bug 1819963 in OpenStack Compute (nova) "Reverting a resize does not update the instance.availability_zone value to the source az" [Medium,In progress] https://launchpad.net/bugs/1819963 - Assigned to Matt Riedemann (mriedem)
01:12:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional recreate test for bug 1819963 https://review.openstack.org/643151
02:14:20 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Fix resetting non-persistent fields when saving obj https://review.openstack.org/643215
02:31:54 openstackgerrit huanhongda proposed openstack/nova stable/queens: Fix resource tracker updates during instance evacuation https://review.openstack.org/643219
02:44:00 openstackgerrit Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271
04:06:33 openstackgerrit Merged openstack/nova master: Disable the tls-proxy in nova-next & fix nova-tox-functional-py35 parent https://review.openstack.org/643129
04:14:40 openstackgerrit Takashi NATSUME proposed openstack/nova master: Override the 'get' method in DriverBlockDevice class https://review.openstack.org/638821
05:20:32 openstackgerrit Luyao Zhong proposed openstack/nova master: object: Add pmem_namespaces field to the NUMACell obj https://review.openstack.org/634547
05:20:33 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configures PMEM namespace when initiating libvirt driver https://review.openstack.org/634549
05:20:33 openstackgerrit Luyao Zhong proposed openstack/nova master: object: Add virtual_pmems fields to the InstanceNUMACell obj https://review.openstack.org/634548
05:20:34 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: report pmem namespaces resources by provider tree https://review.openstack.org/634551
05:20:34 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: Update PMEM namespaces info and usage https://review.openstack.org/634550
05:20:35 openstackgerrit Luyao Zhong proposed openstack/nova master: scheduler: translate virtual pmems request to placement request group https://review.openstack.org/634553
05:20:35 openstackgerrit Luyao Zhong proposed openstack/nova master: API: parse pmem related flavor extra spec https://review.openstack.org/634552
05:20:36 openstackgerrit Luyao Zhong proposed openstack/nova master: libvirt: boot VM with vpmems and cleanup vpmems when destroying https://review.openstack.org/634555
05:20:36 openstackgerrit Luyao Zhong proposed openstack/nova master: update _numa_fit_instance_cell for the support of virtual_pmems https://review.openstack.org/634554
05:20:37 openstackgerrit Luyao Zhong proposed openstack/nova master: support VM resizing with vpmem data migration https://review.openstack.org/634556
07:35:49 openstackgerrit zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403
07:47:49 openstackgerrit Boxiang Zhu proposed openstack/nova master: Raise error when force_raw_images and images_type are incompatible https://review.openstack.org/640271
08:34:56 openstackgerrit Yongli He proposed openstack/nova master: Clean up orphan instances https://review.openstack.org/627765
08:44:01 openstackgerrit Merged openstack/nova master: Update compute rpc version alias for stein https://review.openstack.org/642599
09:04:57 openstackgerrit Merged openstack/nova master: Migrate legacy jobs to Ubuntu Bionic https://review.openstack.org/639017
09:05:56 openstackgerrit Boxiang Zhu proposed openstack/nova master: Remove unsafe convert of lvm imagebackend https://review.openstack.org/642667
09:08:44 kashyap gibi: Morning, mind having a quick gander at this (simple change, but brings non-trivial perf benefit): https://review.openstack.org/#/c/640781/
09:12:04 openstackgerrit zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403
10:49:41 jangutter kashyap: 15x --- there's that story that nearly every programming task reduces to either a caching or string manipulation problem.
10:49:57 kashyap jangutter: Yeah...
10:50:26 kashyap It was a "one-worder" change, but I guess you read the full commit message
10:50:37 kashyap It took 2 days to dig into QEMU and talk to Block Layer folks
10:51:08 jangutter and the persistent nagging feeling that it can't be this simple.

Earlier   Later