| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-12-13 | |||
| 15:57:34 | dansmith | well, service version check is pretty small and no external dependencies, but.. :) | |
| 15:57:42 | efried | the traits cyborg is requesting today are going into the device profile request groups, which isn't where this capability trait is. | |
| 15:58:12 | efried | And the place those request groups are being built from is kinda far from where the root_required will need to be put in. | |
| 15:58:21 | efried | which is why I said "not sure where that will go". | |
| 15:58:30 | dansmith | aight | |
| 15:58:43 | efried | but this is all very solvable, and moves the ball forward wrt using placement features for what we made them for. | |
| 15:58:51 | dansmith | ack | |
| 15:59:10 | efried | viz. string suffixes and rg/rp mappings | |
| 15:59:23 | dansmith | efried: could we do the microversion bump early in the series and get that merged so we don't add a piece to Sundar's puzzle? | |
| 15:59:31 | efried | for sure | |
| 15:59:42 | efried | I can probably work on that. | |
| 15:59:48 | efried | Sundar: ^ | |
| 15:59:56 | efried | I'll try to hack it up this afternoon. | |
| 15:59:56 | dansmith | that would be good, and would be moving the ball a little instead of increasing the weight of the ball | |
| 16:00:04 | Sundar | efried: dansmith: Thanks | |
| 16:00:35 | efried | sean-k-mooney: Are you about today? If not, I can fix up that func test on the image props sync patch and get that moving forward. | |
| 16:01:17 | sean-k-mooney | im around for about an hour but i wont get to it before i head to a team dinner | |
| 16:01:53 | efried | okay. It'll be trivial 1c changes to half a dozen test cases, so I'll still feel good about +2ing. | |
| 16:03:39 | sean-k-mooney | oh i forgot to update the notification samples | |
| 16:04:31 | sean-k-mooney | the notifiction really are a pain | |
| 16:07:22 | efried | yeah, I'm sure they're not worth the trouble. | |
| 16:08:15 | openstackgerrit | Eric Fried proposed openstack/nova master: Sync ImageMetaPropsPayload fields https://review.opendev.org/698613 | |
| 16:08:17 | efried | sean-k-mooney: ^ | |
| 16:08:40 | sean-k-mooney | :) i was just trying to run them locally to reporduce but ill go back to backporting. thanks | |
| 16:11:17 | openstackgerrit | Eric Fried proposed openstack/nova master: DRY: Build ImageMetaPropsPayload from ImageMetaProps https://review.opendev.org/698803 | |
| 16:11:22 | efried | sean-k-mooney: and ^ | |
| 16:11:31 | dansmith | mriedem: are you about to push a change to this? https://review.opendev.org/#/c/638046/56/nova/conductor/tasks/cross_cell_migrate.py | |
| 16:11:42 | efried | gibi, dansmith: That should be an easy win if you could take a quick look | |
| 16:11:50 | efried | https://review.opendev.org/698803 | |
| 16:11:53 | mriedem | dansmith: no those are fup'ed already https://review.opendev.org/#/c/698935/ | |
| 16:11:59 | dansmith | ah okay | |
| 16:14:08 | openstack | Launchpad bug 1856312 in oslo.config "RuntimeError during calling log_opts_values" [Undecided,New] | |
| 16:14:08 | mriedem | stephenfin: this is probably in your wheelhouse https://bugs.launchpad.net/oslo.config/+bug/1856312 | |
| 16:24:47 | stephenfin | mriedem: 1856312 | |
| 16:24:53 | stephenfin | whoops. https://review.opendev.org/698954 | |
| 16:25:04 | stephenfin | oh yeah | |
| 16:26:09 | dansmith | mriedem: check my comment here: https://review.opendev.org/#/c/638046/56 | |
| 16:33:43 | mriedem | dansmith: replied | |
| 16:36:21 | dansmith | mriedem: ack | |
| 16:49:15 | openstackgerrit | Sundar Nadathur proposed openstack/os-traits master: Add a trait that compute manager can handle accelerator requests. https://review.opendev.org/698961 | |
| 16:49:53 | finucannot | dansmith: I'm heading off, but can you make sure you're happy with https://review.opendev.org/698890 before you leave for the year? | |
| 16:50:18 | finucannot | gmann should probably look at it too, but I think the main concern about us breaking the DevStack default has been resolved now | |
| 16:50:34 | dansmith | finucannot: I skimmed it earlier, it's just reverting the stuff now that the devstack default is py3 yeah/ | |
| 16:50:45 | finucannot | 'zactly | |
| 16:51:10 | dansmith | ack fine with me | |
| 16:51:14 | finucannot | ta | |
| 16:51:16 | dansmith | but yeah, gmann | |
| 16:51:19 | mriedem | we might want to consider taking the boot_index part of this spec and doing it on its own https://specs.openstack.org/openstack/nova-specs/specs/train/approved/detach-boot-volume.html#proposed-change since there is some confusion going on in -sdks about knowing when a server is volume-backed or not | |
| 16:51:23 | finucannot | agreed | |
| 16:51:30 | mriedem | and twould be helpful if we exposed the boot_index of the attached volumes | |
| 16:51:43 | finucannot | dansmith: have a good one 🎄 | |
| 16:51:49 | dansmith | o/ | |
| 16:52:08 | gmann | dansmith: stephenfin +1 | |
| 16:53:07 | gmann | its m-1 also so is anyone is break its time foe them to merge the py2 drop patch. | |
| 16:53:21 | mriedem | dansmith: is there a standard way to check the type on a field? | |
| 16:53:33 | dansmith | mriedem: isinstance | |
| 16:53:55 | mriedem | heh yeah i was doing if isinstance(field, fields.ObjectField) but that didn't work | |
| 16:54:09 | mriedem | so i did value = getattr(target_obj, field) and if isinstance(value, fields.ObjectField): but that doens't work either | |
| 16:54:11 | dansmith | should work | |
| 16:54:48 | dansmith | mriedem: https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L773 | |
| 16:54:53 | mriedem | ok i'll tinker, got a call with HR in 5 minutes | |
| 16:54:56 | dansmith | oh I see what you're doing | |
| 16:55:03 | dansmith | the actual data isn't in the field, the field is in obj_fields | |
| 16:55:43 | mriedem | yeah http://paste.openstack.org/show/787566/ | |
| 16:55:56 | mriedem | oh nvm | |
| 16:55:57 | mriedem | i see | |
| 16:56:09 | mriedem | obj_fields yields the field name | |
| 16:56:44 | dansmith | obj_fields is a dict, | |
| 16:56:54 | dansmith | so you need to either get the field by name or iterate over the values() | |
| 16:57:19 | mriedem | return list(self.fields.keys()) + self.obj_extra_fields | |
| 16:57:19 | mriedem | def obj_fields(self): | |
| 16:57:19 | mriedem | @property | |
| 16:57:30 | mriedem | fields is the dict | |
| 16:57:32 | mriedem | so yeah i've got you | |
| 17:25:19 | mriedem | huh, i was trying to use the test_objects.MyObj for my test and that was failing the 'field in target_obj' checks but changed to Instance with a nested Flavor and it's working now, so go figure | |
| 17:26:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix accumulated non-docs nits for cross-cell-resize series https://review.opendev.org/698935 | |
| 17:37:40 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: avoid mushy wording around server.image description https://review.opendev.org/698976 | |
| 17:37:41 | openstackgerrit | Sundar Nadathur proposed openstack/os-traits master: Add a trait that compute manager can handle accelerator requests. https://review.opendev.org/698961 | |
| 17:39:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: avoid mushy wording around server.image description https://review.opendev.org/698976 | |
| 18:13:05 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move _update_pci_request_spec_with_allocated_interface_name https://review.opendev.org/696574 | |
| 18:14:43 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Support live migration with qos ports https://review.opendev.org/695905 | |
| 18:22:34 | gibi | efried_afk: I'm +2 on the vtpm spec (thanks to my nephew who let me read the diff while watching back to the future 3) | |
| 18:38:29 | openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs master: [ussuri][goal] Drop python 2.7 support https://review.opendev.org/698988 | |
| 19:14:29 | openstackgerrit | Merged openstack/nova master: Add confirm_snapshot_based_resize conductor RPC method https://review.opendev.org/637075 | |
| 19:18:03 | KeithMnemonic | dansmith I updated that change with my +1 after verifying the fix in Pike and added you as a reviewer. Thanks for your help! | |
| 19:29:16 | dansmith | ack | |
| 19:32:07 | efried_afk | gibi: thanks! | |
| 19:46:15 | openstackgerrit | Eric Fried proposed openstack/nova master: DRY: Build ImageMetaPropsPayload from ImageMetaProps https://review.opendev.org/698803 | |
| 19:54:59 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: define boot from volume in the glossary https://review.opendev.org/699009 | |
| 19:55:47 | efried | gmann: are you keeping up with your own drop-py27-support patches or are repo owners expected to pick them up? | |
| 19:56:17 | efried | Specifically https://review.opendev.org/#/c/698988/ -- I don't mind editing it, just lmk | |
| 19:56:27 | gmann | efried: owner first if they do not do then i push | |
| 19:57:19 | openstackgerrit | Eric Fried proposed openstack/nova-specs master: [ussuri][goal] Drop python 2.7 support https://review.opendev.org/698988 | |
| 19:57:47 | efried | gmann: okay, I updated ^ | |
| 19:57:51 | efried | and +2ed | |
| 19:58:17 | efried | oh | |
| 19:58:21 | gmann | efried: that might fail. requirement check job is adjusted for py2 requirement matches only | |
| 19:58:22 | efried | That was, apparently, wrong. | |
| 19:58:32 | efried | I don't know that the nova-specs repo has requirements check tho | |
| 19:58:37 | efried | I wouldn't expect it to. | |