Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-07
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] New objects to transmit NUMA config from dest to source https://review.openstack.org/634827
18:09:00 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Create claims when live migrating https://review.openstack.org/634606
18:09:01 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Update NUMA XML during live migration https://review.openstack.org/635229
18:09:01 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Send dest NUMA info inside LiveMigrateData https://review.openstack.org/634828
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
20:06:05 aspiers melwitt: I'll ask in #openstack-infra if it should have the group added
20:09:15 aspiers melwitt: ah sorry, I was looking at my master branch which was hopelessly out of date
20:09:22 melwitt aspiers: ok. if we end up adding it there, should also add it to os-vif, python-novaclient, os-resource-classes, osc-placement. and when placement switches to separate governance, the placement ones will need to be updated to the placement group
20:09:42 aspiers makes sense
20:30:15 openstackgerrit Merged openstack/nova master: Don't force evacuate/live migrate in notification sample tests https://review.openstack.org/634920
20:30:30 openstackgerrit Merged openstack/nova master: Isolate cell-targeting code in MigrationTask https://review.openstack.org/621310
20:38:46 openstackgerrit Merged openstack/nova master: Move resize.prep.start/end notifications to helper method https://review.openstack.org/633298
21:16:47 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtDestTask https://review.openstack.org/627890
21:16:48 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add PrepResizeAtSourceTask https://review.openstack.org/627891
21:16:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Add prep_snapshot_based_resize_at_source compute method https://review.openstack.org/634832
21:16:49 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add finish_snapshot_based_resize_at_dest compute method https://review.openstack.org/635080
21:16:49 openstackgerrit Matt Riedemann proposed openstack/nova master: Move finish_resize.(start|end) notifications to helper method https://review.openstack.org/635079
21:16:50 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add FinishResizeAtDestTask https://review.openstack.org/635646
21:16:57 openstackgerrit Merged openstack/nova master: Move resize.(start|end) notification sending to helper method https://review.openstack.org/634831
21:17:13 openstackgerrit Merged openstack/nova master: Extract compute API _create_image to compute.utils https://review.openstack.org/621311
21:39:31 openstackgerrit Jim Rollenhagen proposed openstack/nova master: ironic: partition compute services by conductor group https://review.openstack.org/635006
21:41:55 openstackgerrit Sam Morrison proposed openstack/nova-specs master: add spec "flavor-classes" https://review.openstack.org/635408
22:06:11 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Execute CrossCellMigrationTask from MigrationTask https://review.openstack.org/635668
22:15:39 artom Has anyone ever noticed how fragile and all over the place the live migration code is?
22:15:48 artom I'm 100% definitely sure I'm the first one to see that.
22:16:50 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Add migration param to check_can_live_migrate_destination https://review.openstack.org/634605
22:16:51 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] New objects to transmit NUMA config from dest to source https://review.openstack.org/634827
22:16:51 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Create claims when live migrating https://review.openstack.org/634606
22:16:52 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Update NUMA XML during live migration https://review.openstack.org/635229
22:16:52 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Send dest NUMA info inside LiveMigrateData https://review.openstack.org/634828
22:16:53 openstackgerrit Artom Lifshitz proposed openstack/nova master: [WIP] Introduce live migration claims https://review.openstack.org/635669
22:42:07 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Plumb allow_cross_cell_resize into compute API resize() https://review.openstack.org/635684
22:48:18 openstackgerrit Chris Friesen proposed openstack/nova master: fix up numa-topology live migration hypervisor check https://review.openstack.org/635350
23:23:08 openstackgerrit Jack Ding proposed openstack/nova master: Correct instance port binding for rebuilds https://review.openstack.org/603844
23:25:04 openstackgerrit Jack Ding proposed openstack/nova master: Improve libvirt image and snapshot handling https://review.openstack.org/616692
#openstack-nova - 2019-02-08
00:49:09 openstackgerrit Adam Spiers proposed openstack/os-traits master: Add HW_CPU_AMD_SEV trait https://review.openstack.org/635608
01:07:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Extend RequestGroup object for mapping https://review.openstack.org/619527
01:07:28 openstackgerrit Matt Riedemann proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268

Earlier   Later