| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-23 | |||
| 15:43:10 | dansmith | and the gold one shows up in placement as CUSTOM_CSUTOM_GOLD for some reason | |
| 15:51:48 | openstackgerrit | John Garbutt proposed openstack/nova master: Add test so we remember why CUSTOM_ prefix added https://review.openstack.org/513001 | |
| 15:52:56 | johnthetubaguy | dansmith: I turned the change upside down ^ | |
| 15:53:48 | dansmith | johnthetubaguy: +2 thanks | |
| 15:55:40 | openstackgerrit | Murali Annamneni proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643 | |
| 16:14:59 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: config drive https://review.openstack.org/409404 | |
| 16:15:17 | openstackgerrit | Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 16:19:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: db: add flavors.description column https://review.openstack.org/514371 | |
| 16:27:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954 | |
| 16:29:17 | openstackgerrit | Stephen Finucane proposed openstack/nova master: xenapi: cached images should be cleaned up by time https://review.openstack.org/465954 | |
| 16:36:05 | openstackgerrit | Murali Annamneni proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643 | |
| 16:46:12 | mriedem | huh, it's odd that we allow flavors.name = None in the object but name is not nullable in the db | |
| 16:57:10 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257 | |
| 17:07:11 | efried | mriedem How come https://review.openstack.org/#/c/510244/ isn't showing up in https://specs.openstack.org/openstack/nova-specs/specs/queens/ ? | |
| 17:08:40 | mriedem | busted docs job? | |
| 17:09:12 | mriedem | looks like there is no publish job | |
| 17:09:46 | efried | Is this a post-zuulv3 thing? | |
| 17:09:53 | mriedem | i believe so | |
| 17:10:10 | efried | Other specs are in that list, but I haven't dug in to see if any were ones that merged after the zuulv3 cutover | |
| 17:10:44 | efried | Should I follow up in -infra? Or -docs? | |
| 17:10:55 | efried | or you got this? | |
| 17:11:09 | mriedem | find out where "openstack-specs-jobs" is defined | |
| 17:11:42 | mriedem | looks like it's in openstack-zuul-jobs | |
| 17:11:57 | mriedem | there is a post job "publish-openstack-specs" | |
| 17:12:01 | mriedem | that should publish the latest | |
| 17:12:05 | mriedem | not sure if that's broken or what | |
| 17:12:49 | mriedem | efried: you should follow up in -infra | |
| 17:12:56 | efried | mriedem ack | |
| 17:19:48 | mriedem | dansmith: want to push this through the specs repo to trigger the publish job? https://review.openstack.org/#/c/456756/ | |
| 17:28:29 | dansmith | mriedem: done | |
| 17:40:36 | openstackgerrit | Merged openstack/nova-specs master: Add a new section: "Upgrade impact" to the template https://review.openstack.org/456756 | |
| 18:07:06 | mriedem | efried: ^ should have flushed a post job | |
| 18:07:21 | efried | mriedem Not yet https://specs.openstack.org/openstack/nova-specs/specs/queens/index.html | |
| 18:07:25 | efried | been watching. | |
| 18:28:16 | mriedem | dansmith: remember this test? https://github.com/openstack/nova/blob/master/nova/tests/unit/db/test_sqlalchemy_migration.py#L379 | |
| 18:28:50 | mriedem | it's causing me some problems with the flavor.description stuff because it doesn't run the schema change that adds the description column, but it creates a flavor using the object, which has a flavor attribute now, | |
| 18:29:00 | mriedem | so trying to think of non terrible ways to resolve that in the test | |
| 18:29:28 | dansmith | mriedem: create the flavor with the model directly | |
| 18:30:07 | mriedem | yeah was just thinking about bypassing the object and using nova.objects.Flavor._flavor_crate | |
| 18:30:10 | mriedem | *create | |
| 18:30:26 | dansmith | yeah | |
| 18:36:18 | mriedem | ah that doesn't work, something to do with the sqla model | |
| 18:40:49 | mriedem | i could manually add the flavors.description column in the test itself, which isn't fun but not sure what else can be done | |
| 18:48:15 | dansmith | what fails when using it directly? | |
| 18:56:37 | mriedem | dansmith: same thing as via the object | |
| 18:56:38 | mriedem | one)] | |
| 18:56:38 | mriedem | oslo_db.exception.DBError: (sqlite3.OperationalError) table flavors has no column named description [SQL: u'INSERT INTO flavors (created_at, updated_at, name, memory_mb, vcpus, root_gb, ephemeral_gb, flavorid, swap, rxtx_factor, vcpu_weight, disabled, is_public, description) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)'] [parameters: ('2017-10-23 18:35:19.692208', None, 'foo', 123, 1, 1, None, 'm1.foo', 0, 1.0, None, 0, | |
| 18:57:00 | dansmith | is it not nullable? | |
| 18:57:07 | mriedem | it is | |
| 18:57:22 | mriedem | in that test, | |
| 18:57:33 | mriedem | we haven't run the table schema change to add the column to the flavors table | |
| 18:57:35 | dansmith | and you're not setting it on the model rght? | |
| 18:57:36 | dansmith | right | |
| 18:57:38 | mriedem | right | |
| 18:57:40 | dansmith | huh | |
| 18:57:47 | dansmith | I would have thought that would work | |
| 18:57:47 | mriedem | i think the model is defaulting it to None? | |
| 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. | |