| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-01-31 | |||
| 00:01:26 | gmann | i added the test with admin_or_owner context to pass and other proejct to fail. but those tests fail as you can see in gate. | |
| 00:01:38 | melwitt | I guess my instinct would be to fix the bug first and then stack the new stuff on top, but that might be just me | |
| 00:02:24 | gmann | yeah that is what i was thinking. so that we can see the bug and fix first and then start the new policy defaults | |
| 00:02:25 | melwitt | I see. well, that's a good sign for the new test coverage I guess :) | |
| 00:02:32 | gmann | yeah | |
| 00:02:36 | melwitt | it's catching bugs | |
| 00:02:55 | gmann | i will file bug and put fix as separate change. | |
| 00:03:09 | melwitt | sounds great ++ | |
| 00:03:26 | melwitt | add me as reviewer when you have it ready | |
| 00:03:33 | gmann | sure, thanks | |
| 00:22:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix os-attach-interfaces policy to be admin_or_owner https://review.opendev.org/705135 | |
| 02:12:21 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix os-attach-interfaces policy to be admin_or_owner https://review.opendev.org/705135 | |
| 02:31:00 | gmann | melwitt: ^^ | |
| 02:34:41 | openstackgerrit | Merged openstack/nova stable/stein: Block rebuild when NUMA topology changed https://review.opendev.org/702972 | |
| 02:51:52 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove old policy enforcement in attach_interfaces https://review.opendev.org/705127 | |
| 04:26:53 | melwitt | gmann: ack thanks, will look tomorrow | |
| 09:46:39 | gibi | dansmith: I've investigated the obj_what_changed() way to detect if the pci requests need s refres during live migraton rollback but this idea fails for mutliple reasons. See my answer in https://review.opendev.org/#/c/702261/4/nova/compute/manager.py@8563 | |
| 10:17:29 | jawad_axd | Hi all. M writing once again here. Question is, iperf3 shows 9.89 Gbits/sec between hosts while instances on same network on these two hosts have iperf3 108 MBytes/sec from inside. How come bandwidth is so poor inside instances? I am using centos7.7, stein, linux bridge,vxlan setup. Any suggestions? | |
| 10:27:26 | gibi | jawad_axd: when you measure performance between the two hosts does the packets are going throigh the same linux bridge as the guests uses in the other measurement between guests? | |
| 10:32:50 | jawad_axd | Nope. Performance between physical hosts is using another bridge at physical NIC. Its just for reference point for guest performance. While guests using bridges created by neutron. | |
| 10:34:41 | gibi | jawad_axd: what I would do is to add layers from the guest-guest test to the host-host test one by one to see which layer causes the degradation | |
| 10:36:13 | stephenfin | bauzas: Could you take a look at these today? https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/remove-nova-network-ussuri | |
| 10:36:20 | bauzas | sure | |
| 10:36:31 | bauzas | I need to go to the gym but I can do it later | |
| 10:36:36 | stephenfin | ta | |
| 10:42:07 | gibi | bauzas: I'm +2 again on the placmenet audit patch https://review.opendev.org/#/c/670112/ | |
| 10:42:16 | bauzas | cool thanks ! | |
| 10:44:13 | gibi | stephenfin: I'm +2 on https://review.opendev.org/#/c/662522/16 but it need another rebase | |
| 12:58:14 | openstackgerrit | Lee Yarwood proposed openstack/nova master: virt: Provide block_device_info during rescue https://review.opendev.org/700811 | |
| 12:58:15 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Report COMPUTE_RESCUE_BFV and check during rescue https://review.opendev.org/701429 | |
| 12:58:15 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Add support for stable device rescue https://review.opendev.org/700812 | |
| 12:58:16 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Support boot from volume instance rescue https://review.opendev.org/701431 | |
| 12:58:16 | openstackgerrit | Lee Yarwood proposed openstack/nova master: api: Introduce microverion 2.82 allowing boot from volume rescue https://review.opendev.org/701430 | |
| 12:58:17 | openstackgerrit | Lee Yarwood proposed openstack/nova master: compute: Extract _get_bdm_image_metadata into nova.utils https://review.opendev.org/705212 | |
| 14:17:02 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reduce gen conflict in COMPUTE_STATUS_DISABLED handling https://review.opendev.org/704866 | |
| 14:17:04 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Deprecate base rules in favor of new rules https://review.opendev.org/701624 | |
| 14:22:22 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reduce gen conflict in COMPUTE_STATUS_DISABLED handling https://review.opendev.org/704866 | |
| 14:23:12 | gmann | cmurphy: updated the deprecation warning text. also replied on admin acrtion policy patch. | |
| 15:05:08 | dansmith | gibi: I replied just now | |
| 15:05:38 | dansmith | gibi: I think my original concerns are all still in place and "it's not as easy to check because of other bugs" doesn't really change my feeling on what is right | |
| 15:06:41 | dansmith | I shan't block on it, but I'm not loving it, nor really sure what the right thing to do is (other than fix those other things) | |
| 15:07:51 | dansmith | also, a decorator causing an instance lazy load is pretty gross :/ | |
| 15:11:17 | dansmith | and I'm not sure why flavor is being marked as dirty, because it seems like it shouldn't be from what I'm seeing... hmm.. | |
| 15:16:12 | openstackgerrit | Merged openstack/nova stable/stein: Disable NUMATopologyFilter on rebuild https://review.opendev.org/702973 | |
| 15:29:09 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing admin_password policies https://review.opendev.org/701629 | |
| 15:31:59 | gmann | melwitt: ^^ updated the admin password for user level enforcement tests. | |
| 15:32:21 | melwitt | gmann: cool thanks | |
| 15:33:51 | gmann | and i abandon this which were removing the user_id from policy target. test in 70629 should keep working for user enforcement through out the policy changes - https://review.opendev.org/#/c/701642/1 | |
| 15:34:27 | melwitt | k makes sense | |
| 15:43:28 | openstackgerrit | Merged openstack/nova stable/stein: FUP for in-place numa rebuild https://review.opendev.org/702974 | |
| 15:46:13 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Recalculate 'RequestSpec.numa_topology' on resize https://review.opendev.org/662522 | |
| 15:46:23 | melwitt | gmann: reviewing https://review.opendev.org/705135 I'm wondering if we should add a policy test for "fails with other project" (the testing that was originally missing the bug) | |
| 15:48:05 | gmann | melwitt: I am adding those in this where tests failing now and with this fix it should pass - https://review.opendev.org/#/c/705126/ | |
| 15:48:24 | gmann | i need to rebase that on top of fix to show the test passing. is that fine ? | |
| 15:48:37 | melwitt | yeah, was just about to ask that | |
| 15:49:00 | gmann | ok | |
| 15:49:41 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix os-attach-interfaces policy to be admin_or_owner https://review.opendev.org/705135 | |
| 15:49:52 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove old policy enforcement in attach_interfaces https://review.opendev.org/705127 | |
| 15:50:08 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing attach_interfaces policies https://review.opendev.org/705126 | |
| 15:50:17 | gmann | done. | |
| 15:50:40 | melwitt | thanks! | |
| 15:52:18 | efried | sean-k-mooney: if you're about, would you please confirm that https://review.opendev.org/#/c/702981/ is correct, and I'll send it | |
| 15:52:56 | sean-k-mooney | ill take a look now | |
| 15:56:28 | sean-k-mooney | efried: so an exception will be raised if the pagesize is not supported. so if we get to that line i think it means we dont have enough left of that size based on https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L660 im just reading futher | |
| 15:59:03 | efried | It's a debug log, so I'm not going to make a big deal out of it. I'll fast approve if it's correct. If it's an improvement, that'd be even better :) | |
| 15:59:28 | cmurphy | gmann: thanks, will review again today | |
| 15:59:40 | gmann | cmurphy: thanks | |
| 15:59:59 | sean-k-mooney | well it depens on how you interpert support. it cant suport the request. because its out of space. that is how that was intended to be interpreted | |
| 16:00:18 | sean-k-mooney | efried: im leaving a comment now but i think we can just change the message to | |
| 16:00:53 | sean-k-mooney | remove the or | |
| 16:01:40 | sean-k-mooney | e.g. "Host does not support requested memory pagesize as there are not enough free pages." | |
| 16:02:47 | efried | ack, thanks sean-k-mooney | |
| 16:04:13 | stephenfin | what other stable core can I annoy to merge this? https://review.opendev.org/#/c/663888/ | |
| 16:04:20 | stephenfin | it's been open for aaaaaaages | |
| 16:06:43 | sean-k-mooney | stephenfin: you can try annoying dansmith but im not sure that will end well for you :P | |
| 16:06:52 | sean-k-mooney | although technically no | |
| 16:07:05 | sean-k-mooney | since that would be 3 form redhat | |
| 16:08:21 | melwitt | I'd ping elod to review (I hope that's his nick) | |
| 16:08:34 | sean-k-mooney | tonyb: johnthetubaguy can you take a look at https://review.opendev.org/#/c/663888/ for stephenfin | |
| 16:18:26 | dansmith | sean-k-mooney: pretty sure we said that's not a problem on stable especially for clean backports, and especially for something like this | |
| 16:19:18 | sean-k-mooney | ya i think you are right | |
| 16:20:22 | stephenfin | dansmith: cheers | |
| 16:28:48 | openstackgerrit | Stephen Finucane proposed openstack/nova master: tests: Validate huge pages https://review.opendev.org/399653 | |
| 16:29:25 | stephenfin | sean-k-mooney, efried, bauzas: That ought to be useful if/when we start working on mempage modelling in placement ^ | |
| 16:31:06 | stephenfin | also artom who cares about functional tests ^ | |
| 16:32:07 | artom | stephenfin, yep, I'll take a look - and thanks for re-visiting NUMA func tests :) | |
| 16:32:27 | artom | There's one outstanding piece of dansmith's feedback I need to address though, about forcing servers on hosts | |
| 16:34:04 | sean-k-mooney | stephenfin: more func test for this stuff is always useful | |
| 16:34:46 | sean-k-mooney | if we can use that as a basis to also validate the placemnt reshape that will be needed even more so | |
| 16:37:39 | efried | wow, neat, a 3+yo patch | |
| 16:59:19 | openstackgerrit | Vladyslav Drok proposed openstack/nova master: Fix volume attachment rollback https://review.opendev.org/704847 | |
| 17:00:53 | artom | stephenfin, there, done | |
| 17:00:59 | stephenfin | wonderful | |
| 17:01:09 | stephenfin | 1:1 then I'll finish my comments on yours | |
| 17:01:18 | stephenfin | (it's 90% there) | |
| 17:03:02 | openstackgerrit | Elod Illes proposed openstack/nova master: DNM: test whether 'logs' content is gzipped https://review.opendev.org/705256 | |
| 17:10:29 | stephenfin | artom: done | |
| 17:11:06 | efried | Hey kids, it looks like blockdiag 2.0.0 breaks our docs build. I'm working on it. | |
| 17:11:07 | artom | stephenfin, thank you! I'll go do sean-k-mooney's downstream review, then circle back here | |
| 17:11:36 | artom | efried, would you say that it's... block-ing us? :D | |