Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-10-04
19:46:56 dansmith efried: he was there, but doing the "sit in the back of the room looking like he wants to kill everyone" thing he does
19:47:05 sean-k-mooney that came up in related to neutron and rack scale design before
19:47:57 efried oh, see, if he had been out of the room actually killing someone, that might have been a good excuse
19:48:08 dansmith depends on who
19:48:22 efried comes back in dripping blood, "Okay, now what decisions did y'all make without me?"
19:48:50 jaypipes dansmith: s/kill someone/kill himself/ .... but yeah. that.
19:49:04 sean-k-mooney jaypipes: arent you glad you rejoined chat
19:49:34 jaypipes are we ready to hangout?
19:50:11 dansmith https://hangouts.google.com/call/7dw4HGf7o0A9UahxcR3ZAAEE
19:50:18 dansmith we can watch jaypipes make that look until it's time
19:50:30 melwitt I'm gonna lurk
19:53:07 efried cfriesen, sean-k-mooney, jackding:
19:53:43 cdent dansmith, efried, jaypipes: i'm unable to join, but as I said before: the combined might of the brains involved ought to be able to get it right, but I do hope you will be summarizing to email or whatever please and thank you.
19:54:26 sean-k-mooney efried: sorry was making tea. joining
19:55:54 melwitt bauzas: it's hangout time ^
19:58:50 bauzas melwitt: thanks
21:30:23 openstackgerrit iain MacDonnell proposed openstack/nova master: Handle online_data_migrations exceptions https://review.openstack.org/608091
21:30:55 imacdonn ^ dansmith: please review
21:32:48 openstackgerrit Merged openstack/nova stable/ocata: Re-use existing ComputeNode on ironic rebalance https://review.openstack.org/607626
21:51:05 cfriesen dansmith: jaypipes: efried: sean-k-mooney: how does this look for a recap email? http://paste.openstack.org/raw/731538/
21:52:23 cfriesen and now I know what efried meant by getting spanked by Sigyn
21:54:40 efried - If we do get down to libvirt, and the trait is set, and the driver for whatever reason determines it's not capable of flipping the switch, it should fail.
21:54:40 efried - This approach only makes sense for simple booleans. Anything requiring more complex configuration will likely need additional extra_spec and/or config and/or unicorn dust.
21:54:40 efried cfriesen: You could mention:
21:55:00 efried and "HPET timer" is redundant :P
21:55:03 efried otherwise, lgtm.
21:55:29 cfriesen efried: cool, will add that.
21:56:38 efried cfriesen: And you/jack are going to take over the spec, yah?
21:57:48 cfriesen efried: yep
21:58:03 efried coo
21:59:32 efried imacdonn: Where does the log go when you run that?
21:59:46 imacdonn efried: nova-manage.log
22:00:10 imacdonn efried: or, I guess, wherever logging puts the log named "nova-manage"
22:00:18 efried hmph. Where is that set up?
22:00:32 imacdonn efried: schema migrations stuff already gets logged there
22:00:35 efried I mean, I assume you tried this out locally and were able to see those logs
22:00:41 efried okay
22:00:42 imacdonn efried: affirm
22:01:41 jaypipes cfriesen: yes, nice work on that ML post. ++
22:01:52 efried imacdonn: oic, it gets set up in main()
22:02:29 imacdonn efried: ah, yeah
22:05:35 openstackgerrit Merged openstack/nova master: Don't emit warning when ironic properties are zero https://review.openstack.org/605754
22:05:41 openstackgerrit Merged openstack/nova stable/pike: Fix service list for disabled compute using MC driver https://review.openstack.org/592337
22:06:28 efried cfriesen: When you're spiffing up that spec, I forgot to put blueprint: support-hpet-on-guest in the commit message
22:07:18 efried interestingly, lp seems to have picked it up anyway. Maybe based on the topic?
22:07:34 cfriesen I noticed that too. gotta be the topic
22:07:43 cfriesen but yes, will add it in
22:08:02 efried okay, folks, I'm done like toast. ō/
22:08:26 imacdonn you're fried? fnar-fnar
22:10:09 cfriesen jaypipes: so would we need to propose changes in os-traits adding new traits for HPET and TPM?
22:13:45 jaypipes cfriesen: go for it :)
22:13:59 cfriesen jaypipes: just wondering what the "proper" way to do this is
22:16:20 cfriesen is anything in nova using os-traits yet?
22:26:39 cfriesen what do you guys think of using HW_SYSTEM_X86_* and HW_SYSTEM_* for instance traits that aren't cpu/gpu/nic specific, like HPET and TPM?
22:27:04 cfriesen and uefi
22:41:50 cfriesen if we have a trait that is requested in the flavor and forbidden in the image, do we error out currently?
23:21:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992
23:22:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993
23:23:09 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153
23:23:23 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104
23:23:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (4) https://review.openstack.org/574106
23:23:52 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (5) https://review.openstack.org/574110
23:33:29 openstackgerrit Merged openstack/nova master: Enable nested allocation candidates in scheduler https://review.openstack.org/585672
23:38:26 openstackgerrit Merged openstack/nova master: Use provider tree in virt FakeDriver https://review.openstack.org/604083
23:52:20 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add API ref guideline for examples https://review.openstack.org/604060
23:52:30 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add API ref guideline for body text https://review.openstack.org/605628
#openstack-nova - 2018-10-05
00:22:49 openstackgerrit Merged openstack/nova master: Fix logging parameter in _populate_pci_mac_address https://review.openstack.org/607628
00:45:53 openstackgerrit Merged openstack/nova master: Not set instance to ERROR if set_admin_password failed https://review.openstack.org/555160
03:14:11 openstackgerrit Merged openstack/nova master: Handle missing marker during online data migration https://review.openstack.org/605164
03:26:50 openstackgerrit Merged openstack/nova master: Placement: Remove usage of get_legacy_facade() https://review.openstack.org/607336
06:05:51 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove an unnecessary duplicate flag https://review.openstack.org/608162
06:34:28 openstackgerrit Vlad Gusev proposed openstack/nova stable/rocky: Not set instance to ERROR if set_admin_password failed https://review.openstack.org/608165
06:42:25 giblet happy Friday nova
07:33:02 mrch_ .
07:39:40 PapaOurs good Friday novaers
08:08:58 openstackgerrit Vlad Gusev proposed openstack/nova stable/queens: Not set instance to ERROR if set_admin_password failed https://review.openstack.org/608179
08:09:50 openstackgerrit Vlad Gusev proposed openstack/nova stable/pike: Not set instance to ERROR if set_admin_password failed https://review.openstack.org/608180
08:26:15 openstackgerrit Merged openstack/nova master: Refactor allocation checking in functional tests https://review.openstack.org/607287
09:01:17 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: libvirt: Remove reference to transient domain when detaching devices https://review.openstack.org/608186
09:02:30 openstackgerrit Sylvain Bauza proposed openstack/nova master: libvirt: implement reshaper for vgpu https://review.openstack.org/599208
09:13:03 PapaOurs giblet: off the discussions we had, I just rebased ^
09:18:40 giblet PapaOurs: ack
10:03:48 openstackgerrit Merged openstack/nova stable/queens: libvirt: Use os.stat and os.path.getsize for RAW disk inspection https://review.openstack.org/604295
12:01:37 openstackgerrit Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova list when a cell is down https://review.openstack.org/567785
12:01:37 openstackgerrit Surya Seetharaman proposed openstack/nova master: Modify get_by_cell_and_project() to get_not_deleted_by_cell_and_project() https://review.openstack.org/607663
12:01:38 openstackgerrit Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova show when a cell is down https://review.openstack.org/591658
12:01:38 openstackgerrit Surya Seetharaman proposed openstack/nova master: [WIP] Refactor scatter-gather utility to return exception objects https://review.openstack.org/607934
12:01:39 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion bump for handling-down-cell https://review.openstack.org/591657
12:01:39 openstackgerrit Surya Seetharaman proposed openstack/nova master: Return a minimal construct for nova service-list when a cell is down https://review.openstack.org/584829
12:20:21 leakypipes cdent, efried, sean-k-mooney: morning sunshines. so, quick question... where should we put the HPET trait in os-traits namespace hierarchy? I was thinking either COMPUTE_HPET or HW_HPET but could also see a case for a new namespace... something like HW_TIME_HPET or just TIME_HPET. Thoughts?
12:20:47 leakypipes for the record, the COMPUTE_ namespace prefix is where we put the virt driver capabilities.
12:21:01 leakypipes and HW_ namespace is where hardware-specific things go.
12:21:21 leakypipes with sub-namespaces like HW_CPU_ and HW_GPU_ etc
12:23:53 sean-k-mooney leakypipes: i would be fine with HW_HPET or HW_TIME_HPET
12:24:14 cdent leakypipes: are we wanting to mean "this host provides an HPET"? If so I'd go HW_HPET or HW_TIME_HPET (in case we ever want a HW_TIME_PIT or some such)
12:25:15 sean-k-mooney leakypipes: the spec was suggesting HW_GUEST_HPET_CAPABLE i think
12:26:54 sean-k-mooney actully it was HPET_GUEST_CAPABLE.. im not sure to be honest. when i see HW_HPET i assume that means the host has a HPET available and eneabled
12:27:27 leakypipes sean-k-mooney: well, to continue my point about traits being capabilities... the _CAPABLE is redundant.
12:27:42 leakypipes sean-k-mooney: and again, the trait decorates the provider (the host) not the guest.
12:27:54 sean-k-mooney so maybe we want 2 traits HW_GUEST_HPET(enable guest HPET) and HW_HPET(find host with hardware hpet)

Earlier   Later