Earlier  
Posted Nick Remark
#openstack-nova - 2019-02-07
12:32:44 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion 2.68: Handles Down Cells Documentation https://review.openstack.org/635147
12:34:10 stephenfin gibi_off: Done
12:34:26 gibi_off stephenfin: tahnks
12:34:28 gibi_off thanks
13:33:31 openstackgerrit Matt Riedemann proposed openstack/nova master: doc: specify --os-compute-api-version when setting flavor description https://review.openstack.org/635508
13:33:33 mriedem is someone chairing the nova meeting today?
13:33:50 mriedem gibi: ^? if you're out i can run it
13:33:53 cdent earlier in the week gibi said he would
13:37:57 gibi mriedem: I can run it
13:39:08 bauzas good afternoon Nova
13:39:19 bauzas here and now beings my upstream work for the day
13:40:51 gibi mriedem: the untriaged bug number is pretty fluid, I've updated the meeting agenda an hour ago ;)
13:41:22 mriedem heh
13:42:09 mriedem it's been hovering around 58 all week
13:46:23 mriedem bauzas: i'm interested, are you now contractually obligated to state when you switch from downstream to upstream work because your time is specifically sliced down to a rigid percentage?
13:46:48 openstackgerrit Surya Seetharaman proposed openstack/nova master: Plumbing required in servers ViewBuilder to construct partial results https://review.openstack.org/635146
13:46:49 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion 2.68: Handles Down Cells https://review.openstack.org/591657
13:46:49 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion 2.68: Handles Down Cells Documentation https://review.openstack.org/635147
13:47:03 bauzas mriedem: no, just that priority work is *always* about customers or other downstream stuff
13:47:27 bauzas mriedem: so before this week, I was not having a lot of time for upstream
13:47:42 bauzas so when folks were pinging me, they weren't knowing when I was there
13:48:09 bauzas that's why this week, I try to make sure I have time for upstream by saying when I work for it
13:48:38 bauzas even if I have other priorities (and be sure I have some...), nova will have at least 2 or 3 hours from me
13:50:42 gibi nova meeting starts in 10 minutes in #openstack-meeting
14:11:21 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix deprecation warning for threadgroup.add_timer https://review.openstack.org/635516
14:11:24 mriedem gibi: bauzas: ^
14:11:39 bauzas mriedem: ack
14:17:22 openstackgerrit Merged openstack/nova master: Fix InstanceMapping to always default queued_for_delete=False https://review.openstack.org/635185
14:17:30 openstackgerrit Merged openstack/nova master: Ignore sqla-migrate inspect.getargspec deprecation warnings https://review.openstack.org/634451
14:23:27 bauzas mriedem: comment on https://review.openstack.org/#/c/635516/1
14:37:39 mriedem bauzas: good catch
14:38:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix deprecation warning for threadgroup.add_timer https://review.openstack.org/635516
14:38:12 mriedem there you go
14:54:13 mriedem bauzas: want to just approve since gibi was +2?
14:54:20 bauzas mriedem: fair
15:12:47 openstackgerrit Surya Seetharaman proposed openstack/nova master: [POC]: Limit _get_computes_for_cells to only go into the required cells https://review.openstack.org/635532
15:13:29 mriedem tssurya: you want to backport https://review.openstack.org/#/c/635185/ to stable/rocky?
15:14:17 openstackgerrit Corey Bryant proposed openstack/nova master: xenapi/agent: whitelist deprecated warnings from OpenSSL 1.1.1+ https://review.openstack.org/635533
15:14:35 tssurya mriedem: yea on it
15:17:20 openstackgerrit Surya Seetharaman proposed openstack/nova stable/rocky: Fix InstanceMapping to always default queued_for_delete=False https://review.openstack.org/635534
15:19:01 openstackgerrit Chris Friesen proposed openstack/nova master: fix up numa-topology live migration hypervisor check https://review.openstack.org/635350
15:35:26 openstackgerrit Yury Kulazhenkov proposed openstack/nova master: rename ScaleIO driver to VxFlex Os https://review.openstack.org/634866
15:56:54 openstackgerrit Merged openstack/nova stable/queens: Explicitly fail if trying to attach SR-IOV port https://review.openstack.org/607729
16:08:53 openstackgerrit Merged openstack/nova stable/queens: Handle tags in _bury_in_cell0 https://review.openstack.org/624824
16:09:00 openstackgerrit Merged openstack/nova master: api-ref: fix link to flavor extra specs docs https://review.openstack.org/635252
16:12:53 melwitt o/
16:43:40 openstackgerrit Stephen Finucane proposed openstack/nova master: API: Remove evacuate/live-migrate 'force' parameter https://review.openstack.org/634600
16:44:16 openstackgerrit Matt Riedemann proposed openstack/nova master: Change live_migration_wait_for_vif_plug=True by default https://review.openstack.org/635360
16:47:15 openstackgerrit Matt Riedemann proposed openstack/nova master: Replace ansible --sudo with --become in live_migration/hooks scripts https://review.openstack.org/635308
17:13:02 openstackgerrit Matt Riedemann proposed openstack/nova master: Set the conductor indirection API when running nova-metadata under uwsgi https://review.openstack.org/635577
17:17:24 openstackgerrit Stephen Finucane proposed openstack/python-novaclient master: Microversion 2.68: Remove 'forced' live migrations, evacuations https://review.openstack.org/635131
17:19:30 mriedem dansmith: a day late but the bottom 3 here should be ready to go https://review.openstack.org/#/c/621310/ - just refactoring code
17:19:35 mriedem that i need to touch or re-use later
17:19:45 mriedem but is admittedly crufty in it's current form anyway
17:19:59 dansmith ack
17:23:00 dansmith mriedem: on the third one, are you not going from three notifications to two?
17:23:23 dansmith er actually three to four?
17:23:46 mriedem don't think so, 4 to 4
17:23:58 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Rework the PCI passthrough guides https://review.openstack.org/635243
17:24:26 mriedem existing unit tests should have spanked me if i f'ed that up
17:24:31 dansmith oh, yeah, you're right I see, i was thinking you removed one,
17:24:46 dansmith but got all confused
17:24:49 dansmith I gotcha
17:27:26 mriedem what's gross?
17:27:46 mriedem oh the wrapping
17:45:46 cfriesen sean-k-mooney: if you look at the compute_nodes table in the DB, do you see hypervisor_type showing "qemu" or "QEMU" ?
17:46:31 sean-k-mooney QEMU i belive
17:49:15 cfriesen that's what I see too. But obj_fields.HVType.QEMU is "qemu", so we can't directly compare it
17:49:53 sean-k-mooney well the hypvervior_type field in the compute is declared as a string
17:50:01 sean-k-mooney not a HVType field
17:52:35 cfriesen what do you think...compare against the "QEMU" string, or do a lowercase comparison against obj_fields.HVType.QEMU ?
17:53:06 cfriesen stephenfin: ^
17:58:18 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Remove args/kwargs from discover_extensions() https://review.openstack.org/635589
17:58:52 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Remove args/kwargs from discover_extensions() https://review.openstack.org/635589
18:09:00 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Create claims when live migrating https://review.openstack.org/634606
18:09:00 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] New objects to transmit NUMA config from dest to source https://review.openstack.org/634827
18:09:01 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Send dest NUMA info inside LiveMigrateData https://review.openstack.org/634828
18:09:01 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Update NUMA XML during live migration https://review.openstack.org/635229
18:19:28 zzzeek jaypipes: ping
18:33:08 zzzeek jaypipes: OK, I just (re?) discovered this innodb behavior: https://blog.pythian.com/understanding-mysql-isolation-levels-repeatable-read/ . blah
19:28:14 openstackgerrit Adam Spiers proposed openstack/os-traits master: Add HW_CPU_AMD_SEV trait https://review.openstack.org/635608
19:30:30 aspiers Are changes to os-traits supposed to auto-update the whiteboard of the corresponding nova blueprint?
19:30:58 aspiers because this hasn't happened for https://blueprints.launchpad.net/nova/+spec/amd-sev-libvirt-support, unless I'm just too impatient
19:43:23 melwitt aspiers: I would have thought so but must not be the case if it didn't update by now. this is the code that updates blueprints, that infra uses: https://github.com/openstack-infra/jeepyb/blob/master/jeepyb/cmd/update_blueprint.py#L79 and it must not be using the 'nova' project (which makes sense bc it's os-traits),
19:43:45 aspiers melwitt: thanks!
19:44:10 melwitt and it looks like this is probably how the linkage works for nova-specs, the nova-specs project has the group 'nova' so it probably finds it that way https://github.com/openstack-infra/project-config/blob/15e227684db3a31a005e2ce14cab8a29057774f7/gerrit/projects.yaml#L5431
19:44:30 melwitt because I do know that gerrit reviews to nova-specs will link to the blueprint
19:45:12 melwitt in order for blueprint update to work for os-traits and friends, they'd have to have 'nova' in 'groups' in project-config, is my guess
19:46:24 aspiers yes, I saw it working for nova-specs
19:46:28 melwitt this line in the update_blueprint code looks like it probably takes the 'groups' from projects to use https://github.com/openstack-infra/jeepyb/blob/master/jeepyb/cmd/update_blueprint.py#L76
19:47:52 aspiers yep, which takes it from the registry in utils.py
19:48:02 melwitt TIL :)
19:48:38 aspiers funnily enough, TIL what TIL means :)
19:48:44 aspiers I saw it on Twitter this morning
19:48:52 aspiers I guess I am behind the times ...
19:49:59 melwitt heh
19:56:01 aspiers melwitt: os-traits has "acl-config: /home/gerrit2/acls/openstack/nova.config" in its project-config, but not "nova" in "groups"
19:56:26 aspiers not sure why
19:58:14 aspiers Well, I've added it to the whiteboard manually. Hopefully jeepyb won't remove it on the next update.
20:05:33 melwitt aspiers: I see it using placement.config https://github.com/openstack-infra/project-config/blob/master/gerrit/projects.yaml#L6004-L6007 but yeah, no 'nova' in 'groups'. probably just no one noticed it not updating related blueprints

Earlier   Later