| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-02-08 | |||
| 20:06:50 | fried_rice | aspiers: ack, I've had that spec on my list for yonks. hope to get around to reading it eventually ... | |
| 20:06:55 | aspiers | fried_rice: thanks a lot for all the info, I'll take a look | |
| 20:07:04 | aspiers | hehe | |
| 20:07:39 | aspiers | OK, I really REALLY need dinner now. bbl ;-) | |
| 20:07:40 | cfriesen | melwitt: what was the exact cutoff? the vTPM stuff went in on Jan 8, and there was a minor change on Jan 5 | |
| 20:08:05 | cfriesen | anyways, I'll propose a release | |
| 20:08:53 | melwitt | not sure. smcginnis, do you know how the auto release works? we noticed there wasn't an automatic release proposed for os-traits at s-2 | |
| 20:16:55 | melwitt | yeah, I don't find any auto-proposed releases for s-2 like this one from s-1 https://review.openstack.org/612843 | |
| 20:58:40 | smcginnis | melwitt, cfriesen: Were there more than 1 or 2 non-zuul, functional changes? It's possible it was accidentally skipped if it didn't look like there were really changes to get out. | |
| 20:59:29 | cfriesen | smcginnis: one actual change went in on jan 8, so maybe that was it. | |
| 21:00:30 | smcginnis | Actually, looks like that didn't merge until the 11th, so it was most likely after we checked. | |
| 21:00:33 | smcginnis | https://review.openstack.org/#/c/629299/ | |
| 21:01:27 | melwitt | smcginnis: there was one functional change, I think. I tried a gerrit search for project:openstack/releases message:"stein milestone-2" and didn't find any other auto-generated releases | |
| 21:01:51 | melwitt | (unless the commit message format changed) | |
| 21:02:16 | smcginnis | I can't remember if we used "stein milestone-2" in the commit messages for those. | |
| 21:05:19 | melwitt | ok, I was curious if any other project got an auto-proposed release for s-2 but didn't know how to find | |
| 21:06:29 | melwitt | it was nice having it for s-1 anyway :) | |
| 21:10:25 | melwitt | mriedem: can you sanity check me, is this backward incompatible, the 'only_contrib' removal? it looks to me like it's removed but not backward incompat https://github.com/openstack/python-novaclient/compare/11.1.0...master | |
| 21:13:58 | mriedem | no it's not, so a minor version bump at least | |
| 21:14:06 | mriedem | that warning has been there since pike i think | |
| 21:14:29 | mriedem | i also don't know if that discover_extensions() thing is an external interface on the client at all | |
| 21:14:47 | mriedem | so not backward compatible but also meh | |
| 21:16:05 | melwitt | yeah, you know, that is ringing a bell that some project was calling discover_extensions() externally | |
| 21:18:10 | melwitt | looks like lots of stuff actually. heat, cinder, neutron | |
| 21:18:49 | melwitt | karbor, masakari, osc | |
| 21:19:47 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add FinishResizeAtDestTask https://review.openstack.org/635646 | |
| 21:19:48 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Execute CrossCellMigrationTask from MigrationTask https://review.openstack.org/635668 | |
| 21:19:48 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Plumb allow_cross_cell_resize into compute API resize() https://review.openstack.org/635684 | |
| 21:20:41 | mriedem | yeah i guess http://codesearch.openstack.org/?q=%5C.discover_extensions&i=nope&files=&repos= | |
| 21:20:49 | mriedem | not sure why, it doesn't do anything anymore as far as i know | |
| 21:20:59 | mriedem | as in novaclient doesn't have 'extensions' anymore | |
| 21:22:01 | melwitt | yeah. probably been there forever | |
| 21:22:32 | mriedem | dansmith: if i fix that issue in tssurya's patch quick are you ok with approving any of those bottom changes yet today or just wait for monday? | |
| 21:22:53 | openstackgerrit | Corey Bryant proposed openstack/nova master: xenapi/agent: Change openssl error handling https://review.openstack.org/635533 | |
| 21:23:13 | dansmith | mriedem: I stopped when I got to that and asked you about it, then got distracted | |
| 21:23:27 | dansmith | mriedem: but yeah, if you fix that quick I'll hit that one at least and see what the next one looks like | |
| 21:23:41 | mriedem | ok | |
| 21:27:20 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Plumbing for ignoring list_records_by_skipping_down_cells https://review.openstack.org/635121 | |
| 21:27:21 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Plumbing for allowing the all-tenants filter with down cells https://review.openstack.org/635145 | |
| 21:27:22 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Plumbing required in servers ViewBuilder to construct partial results https://review.openstack.org/635146 | |
| 21:27:22 | openstackgerrit | Matt Riedemann proposed openstack/nova master: API microversion 2.68: Handles Down Cells https://review.openstack.org/591657 | |
| 21:27:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: API microversion 2.68: Handles Down Cells Documentation https://review.openstack.org/635147 | |
| 21:27:32 | dansmith | damit mriedem | |
| 21:28:11 | mriedem | sheesh | |
| 21:28:48 | mriedem | full rebasearro | |
| 21:28:54 | mriedem | *rebasearoo | |
| 21:29:10 | melwitt | actually I think this should be a major bump because of the removal of those contrib modules. in case anyone was using them? | |
| 21:29:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Modify InstanceMappingList.get_not_deleted_by_cell_and_project() https://review.openstack.org/635120 | |
| 21:29:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Plumbing for ignoring list_records_by_skipping_down_cells https://review.openstack.org/635121 | |
| 21:29:17 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Plumbing for allowing the all-tenants filter with down cells https://review.openstack.org/635145 | |
| 21:29:17 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Plumbing required in servers ViewBuilder to construct partial results https://review.openstack.org/635146 | |
| 21:29:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: API microversion 2.68: Handles Down Cells https://review.openstack.org/591657 | |
| 21:29:18 | openstackgerrit | Matt Riedemann proposed openstack/nova master: API microversion 2.68: Handles Down Cells Documentation https://review.openstack.org/635147 | |
| 21:29:27 | mriedem | melwitt: whatever you want | |
| 21:29:41 | melwitt | going to play it safe | |
| 21:30:25 | dansmith | mriedem: okay hit the bottom two, have the third up for when I finish something | |
| 21:30:58 | mriedem | cool | |
| 21:31:11 | mriedem | time for me to start my think friday | |
| 21:32:55 | dansmith | oh man, forgot about think friday | |
| 21:34:37 | melwitt | os-vif has been released a billion times so I don't need to do that | |
| 22:09:05 | openstackgerrit | Matt Riedemann proposed openstack/nova master: API: Remove evacuate/live-migrate 'force' parameter https://review.openstack.org/634600 | |
| 22:17:27 | dansmith | mriedem: okay got N-3 | |
| 22:18:09 | dansmith | the actual microversion patch is like half the size now, which is oh so much better | |
| 22:19:09 | mriedem | yeah | |
| 22:19:31 | mriedem | and most of it is samples | |
| 22:20:01 | dansmith | yeah | |
| 22:20:32 | mriedem | welp it's 4:20, too late to start think friday now | |
| 22:37:58 | fried_rice | o ovo wizards, remind me what the idiom is for getting extra specs out of a flavor obj | |
| 22:38:16 | fried_rice | if 'extra_specs' in flavor: | |
| 22:38:16 | fried_rice | extra_specs = flavor.extra_specs | |
| 22:38:16 | fried_rice | ? | |
| 22:38:36 | mriedem | anyone want to drop the nova-multiattach job from nova changes? https://review.openstack.org/#/c/606981/ | |
| 22:38:50 | mriedem | fried_rice: yes, but i think it's always there | |
| 22:39:11 | mriedem | return cls._from_db_object(context, cls(context), db_flavor, | |
| 22:39:11 | mriedem | expected_attrs=['extra_specs']) | |
| 22:39:12 | fried_rice | apparently not, at least in test: http://logs.openstack.org/43/631243/8/check/openstack-tox-py27/e61ba3b/testr_results.html.gz | |
| 22:39:28 | mriedem | unit tests not withstanding | |
| 22:39:30 | fried_rice | maybe the test is just borked. | |
| 22:39:45 | mriedem | yeah in real runtime code the flavor would have the extra specs loaded on get from the db | |
| 22:40:02 | fried_rice | can it be None? | |
| 22:40:16 | mriedem | is the field marked as nullable=True? | |
| 22:40:19 | fried_rice | ... | |
| 22:40:27 | mriedem | 631243 | |
| 22:40:28 | mriedem | oops | |
| 22:40:33 | mriedem | 'extra_specs': fields.DictOfStringsField(), | |
| 22:40:33 | mriedem | no | |
| 22:40:35 | mriedem | so it's not nullable | |
| 22:40:43 | mriedem | if it's set, it's not None | |
| 22:40:49 | mriedem | and cannot be set to None | |
| 22:41:02 | fried_rice | (Whether it is or not, I've learned not to trust my intuition when it comes to db objs and ovos) | |
| 22:41:18 | mriedem | you're usually pretty safe unless it's RequestSpec | |
| 22:41:23 | mriedem | then you're in for a world of pain | |
| 22:41:27 | fried_rice | okay, so it's always legal to say flavor.extra_specs.get('some_key') and not have to dork with checking whether extra_specs is a) present, b) a dict? | |
| 22:41:38 | mriedem | it's alwasy a dict | |
| 22:41:45 | fried_rice | ight. | |
| 22:41:47 | mriedem | in runtime code it should be present | |
| 22:43:00 | mriedem | yeah this is wonky https://review.openstack.org/#/c/631243/8/nova/objects/flavor.py@231 | |
| 22:43:14 | mriedem | that could just be self.extra_specs.get('accel:device_profile') | |
| 22:43:16 | fried_rice | yeah, that's what I'm ripping on. | |
| 22:43:21 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add detection of SEV support from QEMU/AMD-SP/libvirt on AMD hosts https://review.openstack.org/633855 | |