| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-23 | |||
| 15:31:30 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Move blkid calls to privsep. https://review.openstack.org/500398 | |
| 15:31:31 | johnthetubaguy | dansmith: yep, agreed. | |
| 15:31:36 | dansmith | johnthetubaguy: yeah I know, but it's not equivalent | |
| 15:32:01 | dansmith | johnthetubaguy: it'd be like us using the uuid, as CUSTOM_$UUID and then people wanting to set their uuid to CUSTOM_$UUID... they're not the same field | |
| 15:33:55 | johnthetubaguy | it just seems to me like 80% of folks are setting the resource classes to something they can request in a flavor, they care little about the translation, even if they probably should | |
| 15:34:36 | johnthetubaguy | but I guess its dangerous to assume they are the same thing, when you use them, because they may not be | |
| 15:34:41 | dansmith | and integers are like floats, but usually without the decimal point? | |
| 15:36:21 | johnthetubaguy | I was thinking about following up with a new microversion that forces all new resource classes to follow the placement patterns, etc. | |
| 15:36:54 | johnthetubaguy | but that is kinda aweful in its own way | |
| 15:36:59 | dansmith | you have to migrate all their node names currently | |
| 15:37:57 | dansmith | and will you enforce CUSTOM_ as a prefix? you kinda have to, but then you have chicken and egg... | |
| 15:38:26 | dansmith | the ironic docs are exceedingly clear on this point | |
| 15:39:07 | edleafe | With hindsight, maybe prefixing standard classes with STD_ would have been simpler | |
| 15:39:41 | dansmith | I don't think that changes this particular thing, but I agree it'd be nice to have consistent namespacing instead of one magic one | |
| 15:39:43 | openstackgerrit | priyaduggirala proposed openstack/nova master: Rename parameters in call() of nova/image/glance.py https://review.openstack.org/508533 | |
| 15:42:22 | dansmith | another argument for operator confusion would be: | |
| 15:42:33 | dansmith | I named one class CUSTOM_GOLD and another SILVER | |
| 15:42:48 | dansmith | both show up in placement as CUSTOM_$thing, yet one tool can never find the silver ones for some reason | |
| 15:42:49 | dansmith | or | |
| 15:42:58 | dansmith | I have CUSTOM_SILVER and CSUTOM_GOLD | |
| 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 | |