| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-04 | |||
| 19:38:02 | sean-k-mooney | i assumed it died on the vine at the ptg but i have not been following | |
| 19:38:10 | efried | no, it was agreed upon at the ptg | |
| 19:38:15 | efried | so I put up a patch for it, sec... | |
| 19:38:25 | efried | sean-k-mooney: https://review.openstack.org/#/c/602160/ | |
| 19:39:40 | sean-k-mooney | hehe i like the doc string | |
| 19:40:11 | sean-k-mooney | traited is a ajative that is not used often enough | |
| 19:40:39 | efried | Given Jay's (albeit reluctant) capitulation on the ML, we *might* still get away with using custom traits for ownership for a little while until we have that represented in some native key/value form (Jay's preference being an actual column in the resource provider table). | |
| 19:40:54 | efried | but I don't see the standard traits flying. | |
| 19:41:30 | jaypipes | ok, I'm back now.. sorry about that | |
| 19:41:39 | sean-k-mooney | was that not the original propsoal. add an owner colume to the resouce providers | |
| 19:41:50 | dansmith | sean-k-mooney: it was, and it was NAKd | |
| 19:41:59 | dansmith | it was for a different reason though | |
| 19:43:00 | dansmith | the consumer type/owner has to not be just "cinder", "nova" if we want to support the multi-service possibility, | |
| 19:43:16 | dansmith | which means you really only know "is this mine or not" if you're looking at that field | |
| 19:43:25 | dansmith | which is probably okay | |
| 19:43:50 | sean-k-mooney | oh but that would be an ower of an allocation not an ower of the resouce provider | |
| 19:44:09 | dansmith | ah you're right I'm confusing these two things | |
| 19:45:09 | efried | We do have to assume well-behaved API client code. And I don't think we want to, like, make the providers only appear to their owners, that would be crazy. | |
| 19:45:28 | efried | so yeah, load up a provider record, look at the owner field, if it ain't me, don't dork with it. | |
| 19:45:41 | sean-k-mooney | for owner of the resouce provder we disucssed 3 options. a trait. a nameing schem or a new db/object filed. the first two did not need placement change and the trait could be standardised so we suggeted that one | |
| 19:45:49 | sean-k-mooney | at least i think that was the rough logic | |
| 19:46:11 | efried | ++ | |
| 19:46:24 | cdent | could (not saying it is a good idea, just being a completist) also use an aggregate | |
| 19:46:24 | dansmith | yep | |
| 19:46:27 | efried | But Jay was out of the room or something. | |
| 19:46:48 | sean-k-mooney | cdent: yes that could also work | |
| 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 | cfriesen: You could mention: | |
| 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 | - 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: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_ | . | |