Earlier  
Posted Nick Remark
#openstack-nova - 2017-12-22
17:40:54 bauzas haha
17:40:57 bauzas i'm back!
17:41:18 kashyap mriedem: Hmm, I actually modelled it just like other features, the class -- LibvirtConfigGuestFeatureAPIC() or LibvirtConfigGuestFeaturePAE()
17:41:38 bauzas donne moi du fromage savoureux
17:41:41 kashyap More appropriate one is - LibvirtConfigGuestFeatureKvmHidden(). Because it also as sub-element
17:44:25 kashyap No, I'm wrong, the KvmHidden() thing has its own element.
17:47:26 mriedem kashyap: more inline
17:47:32 mriedem for your subelement woes
17:47:52 mriedem your root element is the smm element,
17:48:00 mriedem you are adding a sub-element to that, which is wrong,
17:48:05 mriedem you need to set an attribute on the smm element
17:48:08 mriedem which is your root
17:48:26 mriedem xml is f'ing awesome
17:48:34 kashyap Yeah, I should model it something like LibvirtConfigGuestSMBIOS()
17:48:41 kashyap I know, XML...
17:49:02 kashyap From what I recall about why libvirt chose XML was...
17:50:02 kashyap The initial author of libvirt Daniel Veillard was on W3C standards, and 'libxml' maintainer...
17:50:11 kashyap Not sure JSON was there at that time yet
17:50:41 kashyap So now they're (we) stuck with it...
17:51:27 mriedem i worked on a project for 6 years dealing with https://www.oasis-open.org/committees/sdd/
17:51:30 mriedem so i'm familiar
17:52:06 mriedem you'd think i'd actually know something about tosca and heat and nfv stuff, but...
17:52:09 kashyap Wauw. Yeah, I knnow OSAIS standards; the 'virtio' documentation also is hosted there
17:52:38 kashyap But at the recent KVM Forum in Prague, I saw users of virtio, who want to implement devices complaining of the super dense documentation / standards.
17:52:55 mriedem maybe they would prefer CIM
17:52:58 kashyap Heh
17:53:46 kashyap mriedem: When you're back, I have a small future spec topic, for design discussion
17:54:00 kashyap If you're busy after lunch, and can't get to it; it's fine. This can wait
18:16:27 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Introduce SMM (System Management Mode) config class https://review.openstack.org/529611
18:39:07 f10d4 re
18:41:11 openstackgerrit Merged openstack/nova master: Add retry_on_deadlock decorator to action_event_start https://review.openstack.org/529672
19:09:41 openstackgerrit Merged openstack/nova master: Remove extensions module https://review.openstack.org/529206
19:09:46 openstackgerrit Merged openstack/nova master: Remove the LoadedExtensionInfo object https://review.openstack.org/529207
19:48:48 openstackgerrit Merged openstack/nova stable/newton: Use instance.project_id when creating request specs for old instances https://review.openstack.org/529389
21:28:36 mriedem melwitt: in https://review.openstack.org/#/c/334614/ the commit message says, "We cannot store the entire access_url in the database because it contains the token, but we can keep the base part of the url and generate the full access_url on demand."
21:28:45 mriedem but the spec talks quite a bit about storing a token in the db
21:28:53 mriedem "Because the new tokens will go in the database we need to consider cells v2."
21:31:32 mriedem maybe that's in a later change, the one that adds the ConsoleConnection object and db model?
21:37:20 mriedem maybe this just means we don't store the token in the url in the db because the token would make the url too long?
21:37:22 mriedem or it's not hashed?
22:12:59 efried_cya_jan mriedem It's 'powervm: recheck'. I'll send a note to esberglu to add it to the wiki
22:13:32 mriedem efried_cya_jan: already did
22:13:41 mriedem now get back to pantsless friday
22:13:45 efried_cya_jan mriedem As in, you sent the email?
22:13:49 mriedem i updated the wiki
22:13:55 efried_cya_jan oh, okay. Thanks :)
22:14:55 efried_cya_jan mriedem If you run 'recheck powervm' it'll recheck *everything*. It needs to be 'powervm:recheck' (with or without a space) to recheck just powervm.
22:16:41 efried_cya_jan mriedem I updated it.
22:30:37 mriedem melwitt: this will need a follow up https://review.openstack.org/#/c/320063/ but won't block on the stuff i mentioned in ther
22:30:38 mriedem *there
22:44:28 mriedem https://review.openstack.org/#/c/325381/ -1 on that one, and that's where i stop for the year
22:44:32 mriedem toodle-oo
23:42:05 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
#openstack-nova - 2017-12-23
00:16:22 openstackgerrit Merged openstack/nova master: Removed unused argument from LibvirtDriver._connect_volume https://review.openstack.org/529556
00:16:28 openstackgerrit Merged openstack/nova master: Remove unused argument from LibvirtDriver._disconnect_volume https://review.openstack.org/529585
00:23:43 mlavalle mriedem: is this something the nova team is working on https://review.openstack.org/#/c/529831?
00:23:52 mlavalle got corresponding patch on the nova side
00:24:03 mlavalle neutron side^^^
00:32:49 prometheanfire not sure if this is nova related, but I'm getting the following error in pike with memcache configured
00:32:52 prometheanfire https://gist.github.com/prometheanfire/6f8cc9737997a7f4bc0469023653acd7
00:33:33 prometheanfire I can connect to memcached just fine on the configured port
02:02:38 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support. https://review.openstack.org/514904
02:02:39 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273
02:02:39 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406
02:42:24 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Fix the formatting for 2.58 in the compute REST API history doc https://review.openstack.org/529902
04:16:49 openstackgerrit Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
04:34:50 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_vcpu_realtime_scheduler() https://review.openstack.org/527630
04:34:51 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_numa_node_memory() https://review.openstack.org/527631
05:42:08 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_vcpu_realtime_scheduler() https://review.openstack.org/527630
05:42:09 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_XXXpin_cpuset() https://review.openstack.org/527631
05:42:09 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: [libvirt] Add _get_numa_memnode() https://review.openstack.org/529906
05:52:52 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: disable numa feature when virt_type is not kvm https://review.openstack.org/465160
06:19:31 openstackgerrit Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
06:31:35 openstackgerrit Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486
06:51:58 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
06:52:44 openstackgerrit Yikun Jiang (Kero) proposed openstack/python-novaclient master: Microversion 2.58 - Instance actions list pagination https://review.openstack.org/528601
07:10:01 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations https://review.openstack.org/517273
08:46:45 openstackgerrit Chen Hanxiao proposed openstack/nova master: trivial: conf: libvirt: remove a redundant space https://review.openstack.org/529913
10:11:38 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/529924
14:25:53 openstackgerrit Matt Riedemann proposed openstack/nova master: block_device: fix propagation of instance AZ to Cinder https://review.openstack.org/366724
15:41:59 openstackgerrit Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385
15:43:19 openstackgerrit Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385
15:44:40 openstackgerrit Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/529924
16:03:54 ameeda can you please check my code here https://review.openstack.org/#/c/526900/ ?
16:51:23 openstackgerrit Merged openstack/nova master: trivial: conf: libvirt: remove a redundant space https://review.openstack.org/529913
16:51:29 openstackgerrit Merged openstack/nova master: Fix the formatting for 2.58 in the compute REST API history doc https://review.openstack.org/529902
16:52:20 openstackgerrit Merged openstack/nova master: Remove dead parameter from '_create_domain_and_network' https://review.openstack.org/512352
16:52:32 openstackgerrit Merged openstack/nova master: Provide example for placement last-modified header of now https://review.openstack.org/529354
16:52:47 openstackgerrit Merged openstack/nova master: hyper-v: Do not allow instances with pinned CPUs to spawn https://review.openstack.org/424116
16:52:53 openstackgerrit Merged openstack/nova master: Nits from Ic3ab7d60e4ac12b767fe70bef97b327545a86e74 https://review.openstack.org/523432
16:53:00 openstackgerrit Merged openstack/nova master: [placement] Fix getting placement request ID https://review.openstack.org/523606
17:15:32 openstackgerrit Merged openstack/nova master: libvirt: don't log snapshot success unless it actually happens https://review.openstack.org/529674
17:26:41 efried_cya_jan Anyone want to save me reading a hundred emails on the longer dev cycle? What's the TL;DR so far?
19:53:56 prometheanfire efried_cya_jan: longer is better, I think, I didn't read them all either
19:56:57 prometheanfire can someone tell me if it's nova or oslos that I should report this to? https://gist.github.com/prometheanfire/6f8cc9737997a7f4bc0469023653acd7
#openstack-nova - 2017-12-24
06:51:02 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/524795
09:12:50 ameeda JOIN
10:06:29 ameeda can you please review my code here https://review.openstack.org/#/c/526900/ ?
16:12:29 openstackgerrit Peter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver. https://review.openstack.org/140733
22:26:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add a warning in 'nova-manage cell_v2 delete_cell' https://review.openstack.org/513771

Earlier   Later