| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-23 | |||
| 12:41:56 | moshele | mriedem: as I see it we need to skip such config for vnic_type direct and physical_direct, but I will check to be sure ... | |
| 12:42:50 | ykarel | jroll, flavors created like: https://logs.rdoproject.org/openstack-periodic/git.openstack.org/openstack-infra/tripleo-ci/master/legacy-periodic-tripleo-ci-centos-7-ovb-1ctlr_1comp-featureset002-master-upload/9a25ebd/logs/undercloud/home/zuul/overcloud_prep_images.log.txt.gz#_2018-08-21_13_14_01 | |
| 12:42:57 | ykarel | jroll, is that wrong ? | |
| 12:43:27 | moshele | sahid, mriedem: I have to go I will update you guys next week | |
| 12:44:37 | jroll | ykarel: hm, that looks right. looks like maybe there's a bug in nova that doesn't override flavor.ram with flavor.properties['resources:MEMORY_MB']? this definitely used to work | |
| 12:44:49 | jroll | ykarel: I don't have time to dig into that code right now though, sorry | |
| 12:44:53 | ykarel | jroll, what can be reason for phys_ram=0:- phys_ram=0MB used_ram=0MB phys_disk=0GB used_disk=0GB total_vcpus=0 used_vcpus=0 pci_stats=[] | |
| 12:45:09 | openstackgerrit | Merged openstack/nova master: [placement] Regex consts for placement schema https://review.openstack.org/591863 | |
| 12:45:12 | ykarel | jroll, no issue, ^^ | |
| 12:45:22 | jroll | ykarel: baremetal resources no longer report ram/cpu/disk, we only rely on resource classes | |
| 12:45:44 | ykarel | jroll, in devstack ironic i saw phys_ram=384 | |
| 12:46:21 | jroll | ykarel: on master? I'd like to see that :) | |
| 12:46:51 | ykarel | jroll, yes http://logs.openstack.org/42/588942/1/check/ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/4e2e0d5/logs/subnode-2/screen-n-cpu.txt.gz#_Aug_22_03_30_57_227367 | |
| 12:47:29 | mriedem | ok so moshele said rx/tx isn't support for vnic_type direct* is what i heard | |
| 12:47:30 | jroll | ykarel: that's a stable/pike patch | |
| 12:47:36 | ykarel | :( | |
| 12:47:42 | sean-k-mooney | jroll: the code works the otherway in nova. we calulate teh resources dictionary from flavor if not set not the other way around | |
| 12:47:55 | ykarel | jroll, okk, will check some master one :) | |
| 12:48:44 | jaypipes | mriedem: weird how vif plugging succeeds before the libvirt XML file is even defined...? | |
| 12:48:57 | jroll | sean-k-mooney: we explicitly agreed at some point that those resource classes in the flavor's properties would override the direct flavor properties. so I say it's a bug, if nova agrees that it isn't a bug, then ironic has docs bugs and ironic flavors have a UX problem :( | |
| 12:49:14 | jroll | sean-k-mooney: and the code did work this way in the past | |
| 12:49:32 | jaypipes | sean-k-mooney: I trust whatever jroll says and remembers. | |
| 12:50:09 | jroll | jaypipes: I'm putting that on my wall and will remember it in the future :D | |
| 12:50:14 | sean-k-mooney | im not that familar with the ironic code path so your proably right | |
| 12:50:27 | ykarel | jroll, correct in master phys_ram=0, jroll ok will try to understand placement now :) and see i can find anything there | |
| 12:50:38 | ykarel | then | |
| 12:50:44 | jaypipes | jroll: deal. :) | |
| 12:51:21 | mriedem | i also remember the flavor override thing | |
| 12:51:27 | ykarel | jroll, can this be the possible cause:- Details: There are no compute resource providers in the Placement | | |
| 12:51:28 | ykarel | | service but there are 2 compute nodes in the deployment. | | |
| 12:51:28 | ykarel | | This means no compute nodes are reporting into the | | |
| 12:51:28 | ykarel | | Placement service and need to be upgraded and/or fixed. | | |
| 12:51:28 | ykarel | | See | |
| 12:51:33 | mriedem | not sure if it was in a spec though, if not - probably in some old ptg etherpad | |
| 12:51:35 | jroll | whoops | |
| 12:52:04 | mriedem | i'm pretty sure edleafe had included the flavor overwrite stuff in the utility method for this... | |
| 12:52:28 | jroll | ykarel: the cause is a bug in the code that does overrides in the flavor, I'm sure of it :) | |
| 12:52:46 | jroll | in the short term you can remove the top-level ram/cpu/disk properties from your flavor to work around it | |
| 12:52:47 | mriedem | i'd be in this path https://github.com/openstack/nova/blob/master/nova/scheduler/utils.py#L367 | |
| 12:52:54 | jroll | which is fine for CI, but not awesome for users | |
| 12:53:14 | ykarel | jroll, ack will try that | |
| 12:53:25 | ykarel | and wait for the fix in nova | |
| 12:53:30 | mriedem | https://specs.openstack.org/openstack/nova-specs/specs/pike/implemented/custom-resource-classes-in-flavors.html | |
| 12:54:15 | mriedem | jroll: this isn't regressed in rocky right? it's just a master thing | |
| 12:54:20 | jroll | mriedem: I don't know | |
| 12:54:37 | jroll | mriedem: everything I know about the bug is here in channel | |
| 12:54:55 | openstackgerrit | melanie witt proposed openstack/nova master: Correct the release notes related to nova-consoleauth https://review.openstack.org/595455 | |
| 12:55:00 | jroll | mriedem: it could have regressed in rocky, but was masked by the fact we were still reporting cpu/ram/disk for the ironic nodes | |
| 12:55:47 | mriedem | https://github.com/openstack/nova/blob/cc436c2b2a2dad974c4d28871851a456ebd80e48/nova/scheduler/utils.py#L220 | |
| 12:58:52 | jroll | man this code is not super easy to understand | |
| 12:58:54 | mriedem | this is the old code https://review.openstack.org/#/c/515223/12/nova/scheduler/utils.py@a224 | |
| 12:59:01 | mriedem | it wasn't before either | |
| 12:59:41 | Tomatosoup1 | hello | |
| 13:01:08 | Tomatosoup1 | i would like to set up SNAT network on one of the openstack VM, how may i do it? docs are mentioning SNAT but only in theoretical way | |
| 13:01:10 | jroll | mriedem: so that first link, are you saying it should work as advertised? | |
| 13:01:57 | mriedem | w/o a functional test showing it works, idk | |
| 13:02:05 | mriedem | if it was regressed, it's been broken since queens | |
| 13:02:10 | mriedem | with https://review.openstack.org/#/c/515223/ | |
| 13:02:19 | jroll | awesome | |
| 13:02:43 | mriedem | :) this is why i was easing into the "remove the old flavor support for baremetal" pool | |
| 13:03:07 | jroll | yeah, glad we didn't kill it at the tail end of rocky | |
| 13:03:21 | mriedem | you remember talking about it though right? | |
| 13:03:24 | mriedem | like 2 weeks ago | |
| 13:03:44 | jroll | about removing it? or about this bug? | |
| 13:03:52 | mriedem | removing it | |
| 13:03:55 | jroll | yeah | |
| 13:04:13 | jroll | I can say "you win" if that's what you're looking for :) | |
| 13:04:16 | mriedem | YES | |
| 13:04:56 | mriedem | i would take some eggs and toast | |
| 13:05:14 | jroll | okay, I shall go make them | |
| 13:05:51 | mriedem | thanks babe | |
| 13:06:53 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Merge keypair extension response into server view builder https://review.openstack.org/584748 | |
| 13:06:54 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Merge server usage extension response into server view builder https://review.openstack.org/585262 | |
| 13:06:55 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Merge security groups extension response into server view builder https://review.openstack.org/585475 | |
| 13:06:56 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Merge extended_status extension response into server view builder https://review.openstack.org/592092 | |
| 13:07:48 | mriedem | tssurya: fyi ^ i'm trying to keep gmann's series straight and rebased so we can flush it through | |
| 13:08:19 | mriedem | Tomatosoup1: see channel topic please | |
| 13:09:02 | Tomatosoup1 | mriedem, sorry! | |
| 13:09:04 | tssurya | mriedem: noted thanks :) I need extended_volumes also, probably we will have it when he is back | |
| 13:09:53 | sean-k-mooney | are there any known bugs with pysaml2? my devstack is failing to install it. im wondering do we need a version cap | |
| 13:12:18 | mriedem | tssurya: he's out for another week, | |
| 13:12:30 | mriedem | these changes are pretty mechanical so could probably just do it yourself | |
| 13:12:33 | mriedem | following the pattern, | |
| 13:12:38 | mriedem | the hardest part would be the unit test stuff | |
| 13:12:57 | tssurya | ah okay, I didn't want to step into his stuff, but if that's okay I can get it up and update them | |
| 13:13:01 | mriedem | sean-k-mooney: upper-constraints should restrict it in devstack | |
| 13:18:32 | edmondsw | in my devstack I'm seeing a lot of "DEBUG nova.notifications.objects.base [None req-af08f64a-756e-4b43-8357-fd879d3bbbc9 None None] Defaulting the value of the field 'numa_membw_values' to None in MetricPayload due to 'Cannot load 'numa_membw_values' in the base class' {{(pid=145045) populate_schema /opt/stack/nova/nova/notifications/objects/base.py:142}}" | |
| 13:18:45 | edmondsw | but I can't find anything that would ever actually set numa_membw_values, outside of tests | |
| 13:19:26 | edmondsw | should that be removed? What am I missing? | |
| 13:22:41 | edmondsw | or set to Null here? https://github.com/openstack/nova/blob/baf858b03d55273f92fc115ea5c8f71c0bbb6582/nova/compute/monitors/cpu/virt_driver.py#L42 | |
| 13:23:13 | edmondsw | if it's only ever null, though... | |
| 13:24:57 | sean-k-mooney | mriedem: its trying to install 4.6.0 and thats failing on my system. | |
| 13:25:10 | sean-k-mooney | ill pin locally to 4.5 for now | |
| 13:29:14 | sean-k-mooney | edmondsw: numa_membw_values being memory bandwidth? i think that is only supported skylake or newer intel chips and on skylake its broken becaue but memory reand and memory write counters were actully the read value | |
| 13:29:47 | edmondsw | sean-k-mooney unclear from the code I've found | |
| 13:30:29 | edmondsw | I wonder if this was added with the intention of having a followup that used it and maybe that just never happened | |
| 13:31:09 | sean-k-mooney | perhaps that or there is patch in review that has not landed yet | |
| 13:37:31 | stephenfin | mriedem: Does we need to wait for blueprint approval for [1] or can I start flushing them through? [1] https://review.openstack.org/#/q/topic:bp/api-extensions-merge-stein+status:open | |
| 13:38:37 | stephenfin | mriedem: Wait, it's specless and approved. Ignore me | |
| 13:39:09 | zigo | mriedem: cdent: I now know why I missed the webob version, it's because I forgot the EPOCH ... | |