| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-11 | |||
| 20:13:42 | mriedem | keys in the port resource body | |
| 20:14:00 | efried | okay, so the first one is borked. I shall -1 accordingly. Thanks. | |
| 20:21:27 | cdent | all this fiddling suggests some kind of heal integration test would be a nice thing | |
| 20:21:38 | cdent | if one could be conceived | |
| 20:22:19 | efried | I think mriedem wrote one recently. Though I sort of doubt it went anywhere near this deep. | |
| 20:22:27 | mriedem | gibi has a patch for that eventually | |
| 20:22:34 | mriedem | fiddle me timbers | |
| 20:22:43 | efried | cdent: https://review.opendev.org/#/c/667994/ | |
| 20:22:50 | mriedem | https://review.opendev.org/#/c/669879/ | |
| 20:22:52 | mriedem | drats | |
| 20:23:26 | efried | different patches | |
| 20:23:42 | cdent | yeah, the depth is apparently not quite depthy? | |
| 20:23:42 | mriedem | oh right, gibi's builds on mine | |
| 20:24:00 | efried | cdent: look at gibi's followon, it does a little better | |
| 20:24:03 | mriedem | i didn't dig into his actual test changes | |
| 20:24:39 | mriedem | if i can get https://review.opendev.org/#/c/670196/ then eventually we could break apart the post_test_hook mega script into a smaller series of exercises | |
| 20:24:43 | mriedem | so it's more manageable | |
| 20:25:25 | cdent | that would be nice | |
| 20:25:45 | cdent | gibi's follow does look like an improvement. | |
| 20:26:17 | efried | good call cdent. I've commented accordingly in gibi's patch. | |
| 20:27:14 | cdent | i've been on the hook this week for a bunch of internal bug handling, and omg, i can't even | |
| 20:27:21 | cdent | everything is horrible, everywhere | |
| 20:27:33 | cdent | i had a nice afternoon fishing, though | |
| 20:27:49 | efried | like, fishing in the water for fish? | |
| 20:28:04 | efried | or something job-related that's virtually like fishing? | |
| 20:28:15 | efried | (i.e. excruciatingly boring and unrewarding) | |
| 20:30:20 | cdent | https://tank-binaries.s3.amazonaws.com/08e47f8678ed462d95e5a273d927c77a.jpe | |
| 20:30:45 | cdent | sister's sister's family down for the week, invited for some deep sea fishing | |
| 20:30:57 | cdent | that was my dinner | |
| 20:34:15 | melwitt | efried: sorry, I missed your earlier message bc I guess my znc server disconnected.. I was referring to the wording like, "replaces the fixed_ip attribute when you specify it in the request body" which I took to mean it will replace fixed_ip if you specify it in the body but will not erase the fixed_ip if the field is not specified in the request body | |
| 20:56:19 | openstackgerrit | Eric Fried proposed openstack/os-resource-classes master: Propose FPGA and PGPU resource classes https://review.opendev.org/657464 | |
| 21:01:14 | efried | melwitt: okay, cool, thanks for pointing that out. | |
| 21:02:23 | efried | ...though that makes it by no means obvious that similar behavior applies to the whole API. | |
| 21:03:15 | melwitt | yeah, I find the doc not comprehensive. was just guessing based on those bits | |
| 21:03:41 | melwitt | also, I'm not advocating guessing. was just thinking aloud | |
| 21:04:31 | efried | Ima open a doc bug for that | |
| 21:05:45 | efried | I just skimmed the front matter too, and there doesn't appear to be anything there. | |
| 21:07:37 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for node.list https://review.opendev.org/656027 | |
| 21:07:37 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899 | |
| 21:07:38 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for setting instance id https://review.opendev.org/659690 | |
| 21:07:38 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for validating instance and node https://review.opendev.org/656028 | |
| 21:07:39 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for getting network metadata from node https://review.opendev.org/670213 | |
| 21:07:39 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Use SDK instead of ironicclient for add/remove instance info from node https://review.opendev.org/659691 | |
| 21:18:26 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test for resize crash compute restart revert https://review.opendev.org/670393 | |
| 21:18:30 | mriedem | artom: sean-k-mooney: ^ weee | |
| 21:23:07 | openstack | Launchpad bug 1836263 in neutron "doc: PUT /ports/{port_id} updates selectively" [Undecided,New] | |
| 21:23:07 | efried | melwitt, mriedem, cdent: FYI https://bugs.launchpad.net/neutron/+bug/1836263 | |
| 21:24:06 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Convert nova-next to a zuul v3 job https://review.opendev.org/670196 | |
| 21:27:38 | cdent | I read that as mowes, as in I'm gonna mowe some tasty chicken | |
| #openstack-nova - 2019-07-12 | |||
| 02:48:14 | openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | |
| 02:54:49 | openstackgerrit | Merged openstack/python-novaclient master: Add host and hypervisor_hostname to create servers https://review.opendev.org/647671 | |
| 04:20:18 | openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | |
| 04:26:28 | openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | |
| 04:56:42 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Retrun 400 if invalid query parameters are specified https://review.opendev.org/670440 | |
| 04:57:20 | openstackgerrit | ya.wang proposed openstack/nova master: vCPU mdoels selection https://review.opendev.org/670298 | |
| 04:57:20 | openstackgerrit | ya.wang proposed openstack/nova master: Add method 'all_required_traits' to scheduler utils https://review.opendev.org/670297 | |
| 04:57:21 | openstackgerrit | ya.wang proposed openstack/nova master: Support report multi CPU model traits https://review.opendev.org/670300 | |
| 04:57:21 | openstackgerrit | ya.wang proposed openstack/nova master: Add compatibility checks for CPU mode and CPU models and extra flags https://review.opendev.org/670299 | |
| 04:57:22 | openstackgerrit | ya.wang proposed openstack/nova master: Add release note https://review.opendev.org/670441 | |
| 07:43:01 | openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | |
| 10:24:25 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tox: Keeping going with docs https://review.opendev.org/670332 | |
| 10:24:36 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rewrite host aggregate, availability zone docs https://review.opendev.org/667133 | |
| 11:01:58 | stephenfin | efried: Could you take a look at https://review.opendev.org/#/c/666630/ today? At first glance it's not your traditional wheelhouse but it's really just a Python thing | |
| 11:02:21 | stephenfin | I'll ask mriedem too when he's about, since he surely has an opinion | |
| 11:09:40 | openstackgerrit | guangchao.pan proposed openstack/nova master: Add a name for the automatically created volume https://review.opendev.org/660096 | |
| 11:48:30 | cdent | stephenfin: I think mriedem has head out on vacation | |
| 11:55:04 | sean-k-mooney | cdent: ya i think so too | |
| 11:55:23 | sean-k-mooney | cdent: stephenfin is away for lunch at the moment | |
| 12:55:41 | sean-k-mooney | dansmith: are you happy with th latest version of https://review.opendev.org/#/c/667177/10 | |
| 12:56:04 | sean-k-mooney | well as happy as you can be about the fact we have to deal with those network events in nova | |
| 13:06:11 | openstack | Launchpad bug 1835958 in OpenStack Compute (nova) "Nova sync power state on large clusters causes poor performance" [Undecided,New] | |
| 13:06:11 | mriedem | cdent: i left comments on https://bugs.launchpad.net/nova/+bug/1835958 but i'm not sure what "sync" this person is talking about | |
| 13:06:26 | mriedem | there is no sync that i'm aware of that happens during interface attach/detach for example | |
| 13:07:12 | cdent | I think he's misdescribing the problem | |
| 13:07:49 | mriedem | if it's just "_sync_power_states task takes a long time for vcenter" then yes that's a known issue | |
| 13:07:52 | mriedem | and i linked in some things | |
| 13:08:05 | cdent | aren't you supposed to be on vacation? | |
| 13:08:07 | cdent | thanks for that | |
| 13:08:13 | mriedem | we head out at noon | |
| 13:08:34 | cdent | guns at high noon | |
| 13:09:12 | mriedem | although it's my god given right, i'm not sure they want me bringing a gun on the plan | |
| 13:09:14 | mriedem | *plane | |
| 13:10:39 | artom | What God giveth, the TSA taketh away | |
| 13:13:53 | mriedem | artom: https://review.opendev.org/#/c/668631/5 | |
| 13:15:47 | artom | Whoa, yeah. How the hell did it pass integration tests then? | |
| 13:16:32 | mriedem | b/c you're not using a spec'ed mock, | |
| 13:16:38 | mriedem | and we don't test restarting crashed computes in tempest | |
| 13:16:49 | artom | Oh right, it's in _init_host, and we never have instances stuck in migrating in the gate | |
| 13:16:51 | mriedem | that's kind of hard to do in a single node parallel test worker environment | |
| 13:17:05 | mriedem | this is why i wanted to write that functional test patch | |
| 13:17:14 | mriedem | b/c unit tests aren't sufficient for that complicated _init_instance mess | |
| 13:17:59 | artom | Yep, I think to be safe I'll rebase on top of your patch, once the patch on the bottom merges | |
| 13:19:14 | mriedem | artom: note that you do'nt need my functional test patch for backports, just fyi for when the time comes | |
| 13:19:25 | mriedem | although - you're not backporting the interface change anyway, that's why we did it | |
| 13:19:26 | mriedem | so nvm | |
| 13:19:34 | openstack | Launchpad bug 1836105 in OpenStack Compute (nova) "Instance does not start - Error during following call to agent: ovs-vsctl" [Undecided,New] | |
| 13:19:34 | mriedem | sean-k-mooney: if anyone can debug this it's probably you https://bugs.launchpad.net/nova/+bug/1836105 | |
| 13:19:39 | mriedem | *debug/triage | |
| 13:20:09 | sean-k-mooney | ill take a look. | |
| 13:20:21 | sean-k-mooney | i have been putting off fixing some dpdk specific bugs | |