| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-23 | |||
| 18:58:08 | dansmith | so you could easily just set without the orm | |
| 18:58:20 | dansmith | i.e. table.insert(...) | |
| 18:58:31 | mriedem | yeah... | |
| 18:58:33 | mriedem | will try that | |
| 19:04:32 | mriedem | issue the 2nd: Flavor.save() isn't remotable, so preferences on either making that remotable or adding a new save_updates() remotable or something? Flavor.save() calls off to remotables for updating projects and extra_specs | |
| 19:04:46 | mriedem | not sure if we can add remotable for Flavor.save() now or not | |
| 19:04:51 | mriedem | guessing that's bad form | |
| 19:08:06 | openstackgerrit | Eric Fried proposed openstack/nova master: Include /resource_providers/uuid/allocations link https://review.openstack.org/499826 | |
| 19:09:32 | dansmith | mriedem: computes are the only things that really need things to be remotable, and they can't talk to conductors that could do any flavor saving | |
| 19:15:47 | mriedem | table.insert().execute(values) works btw - easier than the other hacks i was thinking of | |
| 19:15:48 | mriedem | thanks | |
| 19:16:32 | dansmith | cool | |
| 19:44:32 | mriedem | can i use "彩虹和独角兽" as a description in tests? | |
| 19:45:16 | mikal | yes | |
| 19:45:21 | mriedem | i don't even know if we encode those first | |
| 19:45:30 | openstackgerrit | Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664 | |
| 19:45:30 | openstackgerrit | Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541 | |
| 19:45:31 | openstackgerrit | Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354 | |
| 19:45:31 | openstackgerrit | Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351 | |
| 19:45:32 | openstackgerrit | Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398 | |
| 19:46:05 | mikal | sdague: I done fixed that release note for you | |
| 19:51:42 | dansmith | mriedem: niiiice | |
| 19:52:10 | mriedem | rainbows and unicorns are somehow more magical in mandarin | |
| 20:01:03 | sdague | mikal: cool, I'll circle in a minute | |
| 20:04:11 | efried | Anyone know if we're supposed to handle queryparams with multiple values in the placement API? | |
| 20:04:35 | efried | E.g. ?resources=VCPU:1,MEMORY_MB:2048&resources=DISK_GB:5 | |
| 20:05:06 | efried | edleafe dansmith jaypipes cdent ^ ? | |
| 20:05:27 | mikal | sdague: yeah, no rush | |
| 20:05:32 | efried | The code doesn't appear to be set up to do so, but I could be reading it wrong fo sho. | |
| 20:06:33 | edleafe | efried: my understanding is that each resource requires its own resource= | |
| 20:07:45 | efried | edleafe If that's the case, I'm *definitely* reading the code wrong. Let me grab some links... | |
| 20:09:34 | efried | edleafe We grab the 'resources' qparam value here: https://github.com/openstack/nova/blob/a067f8c64654a058e04e41389019bd6ff5841cdc/nova/api/openstack/placement/handlers/allocation_candidate.py#L166 | |
| 20:09:47 | efried | edleafe Then we start chewing it up here: https://github.com/openstack/nova/blob/a067f8c64654a058e04e41389019bd6ff5841cdc/nova/api/openstack/placement/util.py#L223-L230 | |
| 20:10:00 | efried | The latter chunk appears to be expecting a string, never a list | |
| 20:10:39 | efried | And if I'm understanding MultiDict correctly, you'll get a list if you do GET['resources'] and the qstring was like my example above. | |
| 20:10:53 | edleafe | efried: yeah, I'm looking at normalize_resources_qs_param(), and it does look like you can do what you asked | |
| 20:11:13 | efried | edleafe can or can't? | |
| 20:12:03 | edleafe | can: multiple RCs per resource=. I don't see where multiple resource= is supported | |
| 20:12:23 | efried | edleafe Right, in my example DISK_GB was specified in a separate resources= | |
| 20:13:07 | edleafe | I would have to run a few tests with the combinations to say for sure. | |
| 20:13:23 | efried | edleafe oh, interestingly, when you say MultiDict[key] you don't get a list. You get the last one. | |
| 20:13:48 | efried | edleafe You have to use getall to get the list. | |
| 20:14:00 | efried | So: not supported. Good to know. | |
| 20:14:35 | edleafe | too bad cdent isn't here. I remember the back-and-forth we had, but not why we settled on the current approach. He might remember that. | |
| 20:16:42 | efried | edleafe I will be sure to follow up with him on that at some point; but for now it's enough for me to know where we stand. | |
| 21:07:44 | mriedem | dansmith: just noticed something strange - not sure if this was intentional or not, | |
| 21:07:57 | mriedem | but was writing a test where i create an instance, | |
| 21:08:14 | mriedem | then i set instance.flavor and assert the only changed field is the flavor | |
| 21:08:15 | mriedem | self.assertEqual(['flavor'], list(instance.obj_what_changed())) | |
| 21:08:33 | mriedem | that failed with: testtools.matchers._impl.MismatchError: ['flavor'] != ['ec2_ids', 'flavor'] | |
| 21:09:07 | mriedem | it looks like on instance.create() we load ec2_ids and then reset the field, but because we aren't recursively resetting the EC2Ids object, it still shows up in the obj_what_changed() results | |
| 21:10:49 | dansmith | ah might be | |
| 21:10:51 | mriedem | so on a subsequent instance.save() we'll call _save_ec2_ids but that's a no-op, so it's not like it hurts anything, | |
| 21:10:55 | mriedem | but it's unnecessary processing | |
| 21:13:39 | trozet | hi all. I was wondering if someone can help me with an issue. Ocata + 5 ironic nodes. I go to deploy instances (via tripleO) and see Starting with 0 host(s) get_filtered_objects /usr/lib/python2.7/site-packages/nova/filters.py:70 | |
| 21:13:55 | trozet | all ironic nodes are in hypervisor-list | |
| 21:18:00 | efried | trozet I think you want to try the #openstack channel. | |
| 21:31:11 | mriedem | hmm, shouldn't any model update result in an auto-update to the updated_at field? | |
| 21:31:20 | mriedem | given: updated_at = Column(DateTime, onupdate=lambda: timeutils.utcnow()) | |
| 21:35:34 | mriedem | ha, nvm | |
| 21:35:49 | mriedem | something something test assertions doing what you wanted them to do | |
| 21:40:36 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add Flavor.description attribute https://review.openstack.org/514371 | |
| 21:47:46 | mwynne | Hi guys. I'm trying to resize an instance but it just goes to an ACTIVE state and never resizes. | |
| 21:47:57 | mriedem | mwynne: check compute logs for errors | |
| 21:48:14 | mwynne | compute logs on the compute node? | |
| 21:48:28 | mwynne | Is there an easy way for me to tell which compute node the VM is on? | |
| 21:56:26 | melwitt | mwynne: I think if you do a 'nova show' with admin credentials it will show what compute node it's on | |
| 22:08:43 | mwynne | melwitt: hostId? | |
| 22:10:02 | melwitt | mwynne: no, hypervisor_hostname | |
| 22:19:41 | mwynne | melwitt: I don't see that field in a nova show | |
| 22:19:53 | mwynne | melwitt: I'm running newton. That's probably it? | |
| 22:19:59 | melwitt | mwynne: it only shows it if you use admin credentials | |
| 22:46:38 | openstackgerrit | Eric Fried proposed openstack/nova master: Parse numbered & unnumbered resources & traits https://review.openstack.org/514091 | |
| 22:46:39 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Numbered groupings to GET /allocation_candidates https://review.openstack.org/514092 | |
| 22:46:55 | efried | edleafe The 'Parse' one should be ready to go ^ | |
| 22:47:02 | efried | ...assuming we can agree on the approach. | |
| 22:53:15 | efried | fungi mriedem_away FYI that publish job kicked in, https://specs.openstack.org/openstack/nova-specs/specs/queens/index.html appears to be fully populated now. Thanks for the help. | |
| 22:54:13 | fungi | efried: any time! | |
| 22:54:19 | fungi | glad it's working finally | |
| 22:57:46 | efried | alex_xu We could slot https://review.openstack.org/514091 and L186 & L190 of https://review.openstack.org/#/c/514092/2/nova/api/openstack/placement/handlers/allocation_candidate.py in front of your series. All of that is backward compatible, so you can have the microversion change all to yourself :) | |
| 23:54:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add Flavor.description attribute https://review.openstack.org/514371 | |
| #openstack-nova - 2017-10-24 | |||
| 00:09:20 | mwynne | melwitt: Ah, you're right. Thanks a lot :) | |
| 01:58:57 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix test_get_volume_config method https://review.openstack.org/489467 | |
| 01:59:51 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | |
| 02:00:03 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | |
| 02:02:09 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Update document related to host aggregate https://review.openstack.org/514499 | |
| 02:51:27 | openstackgerrit | zhangyanxian proposed openstack/nova-specs master: Sriov-bond https://review.openstack.org/463526 | |
| 05:33:32 | openstackgerrit | priyaduggirala proposed openstack/nova master: Rename parameters in call() of nova/image/glance.py https://review.openstack.org/508533 | |
| 06:19:46 | openstackgerrit | Merged openstack/nova master: hyperv: report disk_available_least field https://review.openstack.org/504904 | |
| 06:19:52 | openstackgerrit | Merged openstack/nova stable/ocata: Add live.migration.force.complete to the legacy notification whitelist https://review.openstack.org/508878 | |
| 07:52:44 | openstackgerrit | Michael Still proposed openstack/nova master: Move the idmapshift binary into privsep. https://review.openstack.org/495541 | |
| 07:52:45 | openstackgerrit | Michael Still proposed openstack/nova master: Move nbd commands to privsep. https://review.openstack.org/500351 | |
| 07:52:45 | openstackgerrit | Michael Still proposed openstack/nova master: Move loopback setup and removal to privsep. https://review.openstack.org/495664 | |
| 07:52:46 | openstackgerrit | Michael Still proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398 | |
| 07:52:46 | openstackgerrit | Michael Still proposed openstack/nova master: Move kpartx calls to privsep. https://review.openstack.org/500354 | |
| 08:40:03 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | |
| 08:41:02 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | |
| 08:47:02 | bauzas | jianghuaw: around ? | |
| 08:47:16 | bauzas | jianghuaw: I'm about to write a change for adding the new conf opt | |
| 08:47:23 | jianghuaw | hi bauzas | |