| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-14 | |||
| 14:19:32 | gibi | mriedem: would you be OK with making progress null and signal that with a reno? | |
| 14:20:04 | efried | gibi jaypipes added a comment. | |
| 14:20:10 | mriedem | what is this holding up? | |
| 14:20:12 | mriedem | why do we care? | |
| 14:21:29 | gibi | it is come up as part of the bugfix https://review.openstack.org/#/c/421981/ but it doesn't hold it up any more | |
| 14:22:31 | gibi | mriedem: so we can ignore the problem if we want | |
| 14:24:08 | mriedem | gibi: jaypipes: efried: commented with my 2 cents, | |
| 14:24:14 | mriedem | ultimately i say abandon | |
| 14:24:20 | mriedem | this isn't worth the time spent discussing it | |
| 14:25:13 | mriedem | especially not when we could be bikeshedding over flavor extra spec validation! | |
| 14:25:41 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup the plethora of libvirt live migration options https://review.openstack.org/471834 | |
| 14:25:50 | gibi | mriedem: I got your point. This is a side track really | |
| 14:26:02 | gibi | mriedem: so I'm not against droping it | |
| 14:27:54 | sdatko | hey mriedem | |
| 14:28:20 | openstackgerrit | Maciej Kucia proposed openstack/nova-specs master: WIP Extended PCI alias https://review.openstack.org/474218 | |
| 14:28:37 | sdatko | do you remember, by change, i had issues with testing db_api in current Nova/Master? | |
| 14:28:52 | mriedem | sdatko: i don't | |
| 14:29:32 | sdatko | you recommened me to use nova.objects instead of db.flavor_create /etc. | |
| 14:30:11 | mriedem | efried: more on notifications if you're interested https://docs.openstack.org/developer/nova/notifications.html | |
| 14:30:36 | mriedem | sdatko: ok, i vaguely remember you messing around with things in the database | |
| 14:30:50 | mriedem | and me trying to discourage that | |
| 14:30:55 | gibi | Kevin_Zheng, mriedem: about state vs status we can a status based on https://github.com/openstack/nova/blob/master/nova/api/openstack/common.py#L113 I'm not against it especially if you have a use case needing it | |
| 14:30:55 | sdatko | yep :) | |
| 14:31:20 | mriedem | gibi: i'm not against it really either, as long as i understand the use case first | |
| 14:32:13 | gibi | mriedem: then I guess Kevin_Zheng can put up a patch with the use case in the commit message :) | |
| 14:32:48 | sdatko | mriedem: so i am trying to get all flavorIDs on specific host | |
| 14:33:18 | mriedem | gibi: you know how you're creating the new instance create payload? didn't we do that once already with some other action payload? i was thinking related to snapshots | |
| 14:33:22 | mriedem | but i might be misremembering | |
| 14:33:50 | mriedem | like, | |
| 14:33:57 | edleafe | alex_xu: around? | |
| 14:34:02 | mriedem | we needed to add the snapshot image id to the payload but didn't want to add it to every action payload | |
| 14:34:30 | mriedem | sdatko: flavors are not tied to a compute host, the instance is | |
| 14:34:33 | mriedem | and the instance has a flavor | |
| 14:34:37 | mriedem | associated with it | |
| 14:34:42 | mriedem | embedded really | |
| 14:34:52 | mriedem | nova.objects.Instance.flavor field | |
| 14:35:47 | sdatko | mriedem: in nova.db.sqlalchemy.api i have function that returns model_query(context, models.Instance.instancE_typE_id).filter(models.Instance.host == host).filter(models.Instance.deleted == 0).distinct(models.Instance.instance_type_id).all() | |
| 14:35:53 | mriedem | gibi: ah this https://review.openstack.org/#/c/453077/ | |
| 14:36:13 | gibi | mriedem: there are actions with specific payload like https://github.com/openstack/nova/blob/master/nova/notifications/objects/instance.py#L121 | |
| 14:36:40 | mriedem | gibi: this is what i was thinking of https://review.openstack.org/#/c/453077/2/doc/notification_samples/instance-snapshot-end.json@71 | |
| 14:36:45 | sdatko | and it seems to work; however, i am trying to write unit test for that, but i receive "oslo_db.exception.DBNonExistentTable: (sqlite3.OperationalError) no such table: instances" | |
| 14:36:48 | mriedem | we added a more specific payload, | |
| 14:36:53 | mriedem | and i was checking to see what we did with the version | |
| 14:37:03 | mriedem | so that we are being consistent with InstanceCreatePayload | |
| 14:37:18 | gibi | mriedem: ahh I see. You are askin about the version number | |
| 14:37:23 | mriedem | yes | |
| 14:37:29 | mriedem | i want to be consistent when we rename payloads | |
| 14:37:36 | mriedem | consistent in what we do about the version i mean | |
| 14:37:40 | mriedem | does it stay the same or increment | |
| 14:37:51 | openstackgerrit | Maciej Kucia proposed openstack/nova-specs master: WIP Extended PCI alias https://review.openstack.org/474218 | |
| 14:38:03 | mriedem | gibi: looks like it's being consistent and not incrementing the version https://review.openstack.org/#/c/453077/9/doc/notification_samples/instance-snapshot-end.json | |
| 14:38:37 | gibi | mriedem: in case of snapshot_id we did increase as not just renamed the class but added the new field | |
| 14:38:42 | Kevin_Zheng | mriedem gibi I will ask for detailed usecase and see if it is persuasive enough | |
| 14:38:48 | gibi | mriedem: but in the create case we had a separate patch to rename | |
| 14:39:01 | gibi | mriedem: so there we kept the version | |
| 14:39:02 | mriedem | gibi: then isn't this wrong? https://review.openstack.org/#/c/453077/9/doc/notification_samples/instance-snapshot-start.json@72 | |
| 14:39:06 | openstackgerrit | Maciej Kucia proposed openstack/nova-specs master: WIP Extended PCI alias https://review.openstack.org/474218 | |
| 14:39:06 | mriedem | shouldn't that be 1.3? | |
| 14:39:28 | gibi | mriedem: yes, that is wrong | |
| 14:39:39 | mriedem | cool :) | |
| 14:39:41 | mriedem | -1! | |
| 14:39:54 | mriedem | Kevin_Zheng: thanks | |
| 14:39:54 | gibi | mriedem: thanks for founding it :) | |
| 14:41:09 | mriedem | sdatko: is this for something you're planning on contributing to nova in the community? or just working on separately downstream? because if it's the latter, i don't really have time to help with forking nova. | |
| 14:41:39 | gibi | mriedem: I will be more explicit in the https://review.openstack.org/#/c/453667/ as well about keeping the same version in case of pure rename and bumping it if also content is changed in the same step | |
| 14:42:08 | mriedem | gibi: ok i was going to ask that we document this in the devref | |
| 14:42:56 | sdatko | mriedem: nothing very important, i am just trying to get into Nova | |
| 14:44:07 | sdatko | playing on DevStack :-) Ok, maybe some other time | |
| 14:44:15 | sdatko | thanks! | |
| 14:44:45 | mriedem | sdatko: ok, i figured with the types of questions you're asking you are trying to implement some out of tree feature | |
| 14:45:20 | mriedem | sdatko: ultimately it depends on your use case, but i think if you're just trying to figure out which flavors are being used on which hosts, that can all be done via the REST API / client side tools | |
| 14:45:40 | mriedem | if you're working on a scheduler filter or something like that, then it's a different story | |
| 14:49:53 | kashyap | mriedem: Forgot to respond here, as I was asleep - about your persistent domain question | |
| 14:50:02 | kashyap | mriedem: But you answered yourself, and you were right | |
| 14:50:58 | mriedem | kashyap: yeah now just need to know why it takes 5 minutes to remove an ethernet interface device from a guest | |
| 14:51:12 | mriedem | was hoping danpb could help with some of that | |
| 14:51:17 | kashyap | mriedem: That sounds like an awful lot of time | |
| 14:54:13 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667 | |
| 14:54:13 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077 | |
| 14:54:31 | gibi | mriedem: Fixed the versioning problem in the snapshot_id patch ^^ | |
| 14:54:45 | mriedem | kashyap: cool, comments 14 and on are the details https://bugs.launchpad.net/nova/+bug/1696125 | |
| 14:54:45 | openstack | Launchpad bug 1696125 in OpenStack Compute (nova) "Detach interface failed - timeout waiting to detach tap device in linuxbridge job (pike)" [High,Confirmed] - Assigned to Matt Riedemann (mriedem) | |
| 14:54:56 | mriedem | gibi: cool | |
| 14:58:26 | mriedem | gibi: ok +2 on the instance create payload change https://review.openstack.org/#/c/463001/ now | |
| 14:58:44 | gibi | mriedem: thanks for the review! | |
| 14:59:42 | gibi | mriedem: and thanks for the feedback on the doc. I will fix it soon | |
| 15:03:35 | openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800 | |
| 15:03:43 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667 | |
| 15:03:48 | mriedem | gibi: ok also some issues in https://review.openstack.org/#/c/453077/ | |
| 15:03:53 | mriedem | d'oh | |
| 15:04:08 | dansmith | dtantsur: is there an easy way to detect that I'm on the primary node or not? | |
| 15:04:16 | dansmith | actually | |
| 15:04:33 | dansmith | we should probably only run this if we're on a node where there is something like nova-api configured, so I'll do that | |
| 15:05:15 | mriedem | dansmith: that's what devstack/stack.sh checks for | |
| 15:05:17 | mriedem | so yeah | |
| 15:05:26 | dansmith | yeah | |
| 15:05:54 | gibi | mriedem: I have to run now, but will get back to the snapshot patch tomorrow morning. thanks again for the review! | |
| 15:06:18 | mriedem | yw | |
| 15:07:40 | mriedem | man we don't even go over conductor for compute_node_statistics | |
| 15:08:22 | dansmith | mriedem: well, it's api only, so we wouldn't | |