| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-06 | |||
| 17:38:41 | openstackgerrit | Julian proposed openstack/nova: Transform instance.add_fixed_ip notification https://review.openstack.org/332876 | |
| 17:39:19 | leakypipes | sfinucan, sean-k-mooney: questions on commit message. | |
| 17:39:28 | melwitt | superdan: lettuce see | |
| 17:39:37 | superdan | melwitt: just talking out loud | |
| 17:40:11 | melwitt | oh, yeah. I was thinking maybe since listing does a /detail call, it's trying to pull some host info? that doesn't really make sense though | |
| 17:41:14 | sean-k-mooney | leakypipes: thanks jay ill take a look at it now. im just opening a bug for the windows issues that sfinucan change uncovered | |
| 17:42:05 | superdan | melwitt: it's just the test hitting the /hypervisors api afaict | |
| 17:42:34 | melwitt | yeah, I see that now. weird | |
| 17:46:48 | leakypipes | superdan: so did you figure out the substitution thing? | |
| 17:47:18 | superdan | leakypipes: no, I went with the + instead of * thing, and will change it if mriedem requires and/or tells me how | |
| 17:47:36 | superdan | leakypipes: but I'm hoping he'll just let it go as is | |
| 17:47:48 | leakypipes | superdan: but I see "%(int:service_id)s" | |
| 17:48:11 | leakypipes | superdan: should that just be "%(service_id)s" | |
| 17:48:13 | leakypipes | ? | |
| 17:49:06 | superdan | none of those work for me | |
| 17:54:05 | openstackgerrit | Gábor Antal proposed openstack/nova: Added instance.reboot.error to the legacy notifications https://review.openstack.org/411816 | |
| 17:54:06 | openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot.error notification https://review.openstack.org/411791 | |
| 17:55:11 | melwitt | superdan: I guess the only thing I notice is if you look at the test_results all of the tests failed with a timeout of some kind, but for some reason only the hypervisors api tests are showing any traces in the logs of a timeout | |
| 17:55:54 | melwitt | so I wonder if it's a side effect | |
| 17:56:09 | superdan | but.. look at the change.. it's just adding stuff to get_all instances | |
| 17:56:26 | superdan | seems like.. totally unrelated | |
| 17:57:06 | melwitt | I know. I was thinking maybe other tests that do a listing bombarded the queue so hard that it made all the other tests time out too? I dunno | |
| 17:57:36 | superdan | I dunno, or if we locked up the compute node somehow | |
| 17:57:46 | melwitt | hm, yeah | |
| 17:57:48 | superdan | the hypervisors one is the only one I saw with any tracebacks anywhere, | |
| 17:57:54 | superdan | so I was trying to follow that thread | |
| 17:59:08 | melwitt | there's a guru meditation report in the n-cpu.log | |
| 17:59:14 | melwitt | is that usual? | |
| 17:59:53 | superdan | any time we fail the run we do that I think | |
| 18:00:02 | superdan | also, I see debug in n-api that shows us listing cells prperly and stuff | |
| 18:00:08 | superdan | Found 2 cells: cell0,cell1 _get_instances_by_filters_all_cells | |
| 18:00:13 | superdan | which is what this thing changes | |
| 18:00:39 | superdan | although, the two patches before this don't run the cells job | |
| 18:00:43 | superdan | because they're test only | |
| 18:00:58 | superdan | so I wonder if this is just failing recently by chance? seems too unlikely, but... | |
| 18:01:29 | melwitt | yeah, I dunno. I wondered the same | |
| 18:04:11 | superdan | I see other runs that passed it a few hours ago, so.. must not be | |
| 18:04:14 | superdan | I just.. don't even | |
| 18:04:46 | melwitt | :( | |
| 18:10:59 | leakypipes | bauwser: cdent's patch adding microversion for member_of is now merged, FYI. | |
| 18:11:32 | leakypipes | bauwser: I'm sure you noticed the merge conflict already, eh? | |
| 18:12:20 | sean-k-mooney | leakypipes: by the way do you know who is in the bug triage team for os-vif? is it just danial? | |
| 18:12:40 | leakypipes | sean-k-mooney: dan berrange? | |
| 18:12:53 | leakypipes | sean-k-mooney: danpb no longer works on Nova or os-vif. | |
| 18:13:10 | leakypipes | sean-k-mooney: so it's just you, me, sfinucan and moshele. | |
| 18:13:21 | sean-k-mooney | leakypipes: ya i remember the ml post | |
| 18:13:41 | sfinucan | leakypipes: I'm in the bug triage team? :S :) | |
| 18:13:43 | leakypipes | sean-k-mooney: and since sfinucan isn't a member of os-vif-core I think we should probably consider adding him and mriedem :) | |
| 18:13:56 | leakypipes | sfinucan: I'm "volunteering" you :) | |
| 18:14:09 | sfinucan | leakypipes: Yay. Love being "volunteered" :) | |
| 18:14:12 | sean-k-mooney | leakypipes: looking at launchpad he is set as the driver and os-vif drivers is the maintaienr but im not in os-vif drivers | |
| 18:14:44 | leakypipes | sean-k-mooney: who are you referring to? danpb? | |
| 18:15:10 | jroll | leakypipes: that's "voluntold" | |
| 18:15:12 | sean-k-mooney | leakypipes: os-vif drivers is currently dan berrange, you and russel | |
| 18:15:17 | leakypipes | jroll: :) | |
| 18:15:26 | leakypipes | sean-k-mooney: sec, changing. | |
| 18:15:31 | sean-k-mooney | leakypipes: https://launchpad.net/~os-vif-drivers/+members | |
| 18:16:09 | openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot.error notification https://review.openstack.org/411791 | |
| 18:17:48 | leakypipes | sean-k-mooney, sfinucan, moshele: you are all now os-vif-drivers. | |
| 18:17:59 | sfinucan | leakypipes: Does it come with a pay rise? | |
| 18:18:12 | sean-k-mooney | leakypipes: thanks | |
| 18:18:16 | sfinucan | :) | |
| 18:18:17 | sean-k-mooney | sfinucan: it come with more work | |
| 18:18:24 | leakypipes | mriedem: you too. :) | |
| 18:18:27 | sean-k-mooney | sfinucan: same thing right | |
| 18:18:35 | sfinucan | sean-k-mooney: My favorite :) | |
| 18:18:57 | leakypipes | sean-k-mooney: feel like submitting the request to add sfinucan and mriedem to os-vif-core | |
| 18:18:58 | leakypipes | ? | |
| 18:19:24 | sean-k-mooney | in gerrit or to the mailing list | |
| 18:19:34 | sean-k-mooney | but im fine with adding both | |
| 18:21:21 | sean-k-mooney | leakypipes: in gerrit any member of a group can add others so i can add sfinucan and mriedem to the os-vif-core if we dont need to propose them for core membership on the mailing list first | |
| 18:21:47 | leakypipes | sean-k-mooney: no need to propose, no. just go ahead and add. | |
| 18:21:58 | sean-k-mooney | ok will do. | |
| 18:22:14 | leakypipes | sean-k-mooney: since danpb has stepped away indefinitely and both sfinucan and mriedem are familiar with the os-vif code. | |
| 18:23:23 | sean-k-mooney | sfinucan mriedem you are now both in os-vif-core enjoy the extra reviewing :) | |
| 18:23:33 | openstackgerrit | John Garbutt proposed openstack/nova: WIP: Add service_token for nova-cinder interaction https://review.openstack.org/397399 | |
| 18:23:48 | sfinucan | sean-k-mooney, leakypipes Cheers :) | |
| 18:24:03 | leakypipes | sfinucan: with great power... | |
| 18:24:33 | leakypipes | sfinucan: so does sean-k-mooney :) | |
| 18:24:36 | sfinucan | ...comes great electricity bill | |
| 18:24:43 | leakypipes | lol | |
| 18:24:52 | leakypipes | welcome back. | |
| 18:25:12 | sfinucan | Ctrl + W kills tabs in HexChat also. Who knew? | |
| 18:25:18 | leakypipes | :) | |
| 18:25:31 | openstackgerrit | John Garbutt proposed openstack/nova: WIP: Add service_token for nova-cinder interaction https://review.openstack.org/397399 | |
| 18:25:37 | sean-k-mooney | sfinucan: i opened a bug for the windows issue with os-vif https://bugs.launchpad.net/os-vif/+bug/1654616 i will try and adress it next week | |
| 18:25:37 | openstack | Launchpad bug 1654616 in os-vif "_plug_vif_windows uses linux net fuctions that do not work correctly on windows." [Undecided,New] | |
| 18:25:39 | leakypipes | sean-k-mooney: some MTU was set to 0? :( | |
| 18:26:00 | sean-k-mooney | leakypipes: that was the nova default until mitaka | |
| 18:26:15 | sean-k-mooney | leakypipes: and neutron default | |
| 18:26:25 | bauwser | leakypipes: oh no | |
| 18:26:31 | leakypipes | sean-k-mooney: I'm presuming it was meant to mean "default to whatever the PMTU is of the existing segment?" | |
| 18:26:32 | bauwser | leakypipes: okay, I'll rebase anyway | |
| 18:27:01 | leakypipes | bauwser: yeah, I have to as well :) | |
| 18:27:15 | leakypipes | bauwser: we're all competing for the API microversion mutex :) | |
| 18:27:40 | sean-k-mooney | leakypipes: no up to mitaka it was just 0 which in python ment "if mut" was false and nova and neutorn did not set one at all | |
| 18:27:46 | bauwser | I should write a git hook for that :) | |
| 18:27:50 | bauwser | leakypipes: ^ | |
| 18:27:51 | openstackgerrit | Stephen Finucane proposed openstack/os-vif: vif_plug_ovs: Always set MTU when plugging devices https://review.openstack.org/408779 | |
| 18:27:58 | leakypipes | sean-k-mooney: ah, ok, makes sense now. | |