Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-01-18
14:53:55 mriedem https://github.com/openstack-dev/devstack/commit/f165ad49206f520e8fc3aac535e77f82ea410ea4
14:54:04 mriedem but i think grenade runs in singleconductor mode
14:57:00 dansmith well, that means we probably need to tweak it so it gets a neutered config like in devstack
14:57:21 mriedem that is devstack, but yes on the neutering
14:57:45 dansmith I mean a devstack job
15:09:20 mriedem checking logstash yeah it's all grenade jobs
15:25:34 kashyap What is this "badge" thing you speak of? Fedora project does these "badges" for fun (e.g. if your build failed 20 times, you get a badge; you write a doc, you get a "Junior Editor" badge, etc.)
15:25:45 kashyap So recently I got an "adult frog" badge :D -- https://badges.fedoraproject.org/badge/adult-frog
15:26:20 kashyap Anyway. Enough with the diggression.
15:27:07 mriedem so if someone licks your back now they hallucinate?
15:27:45 kashyap Heh, just learnt about "licking toads"
15:28:19 kashyap Here are all the "fun" badges I accumulated :D: https://badges.fedoraproject.org/user/kashyapc
15:28:35 kashyap mriedem: On that review; just saw your remarks, will address the nits.
15:44:31 mriedem will see if this cleans up the logs in grenade runs https://review.openstack.org/631811
15:50:14 melwitt
16:22:13 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Import IP implementation modules outside privsep context https://review.openstack.org/629025
16:43:04 jroll mriedem | jroll: ironic has a task engine thing right? does that use taskflow? <- yes, no
16:45:42 openstackgerrit sean mooney proposed openstack/os-vif master: do not always plug ovs ports. https://review.openstack.org/631829
16:52:46 openstackgerrit sean mooney proposed openstack/os-vif master: Cleanup device at the end of 'test_iproute_object_closes_correctly' test https://review.openstack.org/629112
16:52:47 openstackgerrit sean mooney proposed openstack/os-vif master: Import IP implementation modules outside privsep context https://review.openstack.org/629025
16:55:49 mriedem jaypipes: are you going to be able to jump on this qos minimum bandwidth call in 5 minutes?
16:55:58 mriedem also, isn't it friday?
16:56:01 mriedem no superdan
16:56:03 mriedem no leakypipes
16:56:10 mriedem no fried_rice
16:57:09 lbragstad but where's hans_moleman?
16:57:39 fried_rice Thanks mriedem
16:58:02 cdent I assumed that casual nick friday was shut down as part of the federal shutdown
16:58:17 hansmoleman i've got a table full of fast food that says otherwise
16:58:40 cdent self-funded? It's only legit if its self-funded
16:59:10 gibi bandwidth hangouts https://hangouts.google.com/call/oZAfCFV3XaH3IxaA0-ITAEEI
16:59:10 hansmoleman it's funded via my non-profit yes
16:59:12 sean-k-mooney hi can i get some eye on the 3 os-vif change i just pushed i want to fast track them to do a release of os-vif today to unblock the gate
16:59:30 cdent hansmoleman: the depth you are able to carry a reference is impressive
16:59:40 jaypipes mrhillsman: yes
17:00:00 jaypipes doink, sorry mrhillsman, I meant hansmoleman
17:00:47 hansmoleman https://etherpad.openstack.org/p/bandwidth-way-forward
17:01:08 mrhillsman np
17:01:47 hansmoleman sean-k-mooney: we're on this call if you care
17:02:03 sean-k-mooney oh sorry yes i should join continue with out me
17:02:42 sean-k-mooney ill be there in a sec
17:03:06 melwitt guess no one cares about jgwentworth
17:04:34 hansmoleman so this is the functional test stub https://review.openstack.org/#/c/567268/43/nova/tests/functional/test_servers.py@5711
17:04:47 hansmoleman to enable the code in functional tests even though the API doesn't allow the new stuff
17:05:10 dansmith hansmoleman: I think that grenade patch makes sense at first glance
17:08:39 cdent gerrit being restarted?
17:08:51 cdent guess so, better now
17:08:55 fried_rice They did threaten that that was going to happen about now.
17:09:00 fried_rice upgrading
17:09:27 fried_rice cdent: http://lists.openstack.org/pipermail/openstack-discuss/2019-January/001833.html
17:09:33 cdent my ability to keep track of things is broken
17:09:46 cdent thanks
17:10:00 fried_rice Crowdsourced thing tracking --^
17:25:31 openstackgerrit Paul-Emile Element proposed openstack/nova master: Add emulated TPM support to Nova https://review.openstack.org/631363
17:30:02 openstackgerrit Elod Illes proposed openstack/nova stable/ocata: Make supports_direct_io work on 4096b sector size https://review.openstack.org/631843
17:36:22 leakypipes mlavalle: mind having a look at my comment on https://review.openstack.org/#/c/631684/1/nova/network/neutronv2/api.py?
17:36:47 mlavalle leakypipes: sure, I'll look at it
17:38:52 leakypipes mlavalle: gracias
17:39:07 mlavalle de nada
17:47:25 openstackgerrit sean mooney proposed openstack/os-vif master: do not always plug ovs ports. https://review.openstack.org/631829
17:56:11 openstackgerrit sean mooney proposed openstack/os-vif master: Cleanup device at the end of 'test_iproute_object_closes_correctly' test https://review.openstack.org/629112
17:56:12 openstackgerrit sean mooney proposed openstack/os-vif master: Import IP implementation modules outside privsep context https://review.openstack.org/629025
18:10:44 gibi melwitt sean-k-mooney hansmoleman leakypipes mlavalle: summary of the hangouts call about the bandwidth feature is on the ML http://lists.openstack.org/pipermail/openstack-discuss/2019-January/001879.html
18:11:08 mlavalle gibi: thanks. have a nice weekend
18:11:24 melwitt gibi: thanks, have a good weekend o/
18:11:33 gibi same to you
18:19:47 openstackgerrit Merged openstack/nova master: Remove args(os=False) in monkey_patch https://review.openstack.org/568999
18:28:45 sean-k-mooney melwitt: leakypipes i have submited a patch to release os-vif 1.14.0 to unblock the gates. https://review.openstack.org/#/c/631860/1 it depends on 3 patches that still need to be merged. when zuul comments back will ye be around later to review them?
18:30:56 melwitt sean-k-mooney: we usually use Depends-On in the release patch to hold it for specific things (so it won't pass CI until ready)
18:31:19 sean-k-mooney the release repo jobs dont actully support depends-on
18:31:28 sean-k-mooney i usually add it out of habit
18:31:35 sean-k-mooney but it will fail the validate job
18:31:41 sean-k-mooney until the depencys are merged
18:31:47 melwitt on the first patch, you +2ed your own patch
18:32:10 melwitt yeah, I meant validate job will hold it
18:32:22 sean-k-mooney ya i generally avoid that but im not goint to +w so im going to leave that to others.
18:32:45 sean-k-mooney if we want to have 2 +2 + mine that is fine too
18:36:35 melwitt sean-k-mooney: ok, if os-vif has a single +2 policy, or a self-approval-allowed policy, I haven't been aware of it
18:37:05 sean-k-mooney melwitt: but yes point taken i normally take more time but the neutron folks been pushing to fix this.
18:37:54 leakypipes sean-k-mooney: +2 on bottom patch.
18:38:42 sean-k-mooney self approval + one other has been what we did for quite a while before added nova-core to os-vif as there were 3 active reviews so usually the patch was written by one of those 3
18:38:57 sean-k-mooney but as i said i generally avoid it
18:39:43 melwitt ok, I wasn't aware of that. thanks
18:45:25 sean-k-mooney leakypipes: since i need to resping the 3 change will i fix the typos in the first while im at it?
18:48:40 sean-k-mooney melwitt: by the way if you want to change to 2 independet +2s for os-vif we can. since nova-core has been added there are more peole that can review but its still mainly myself stephen and jay that do in pratice https://www.stackalytics.com/?module=os-vif
18:48:49 leakypipes sean-k-mooney: up to you. I'm happy to fast-+2 after
18:49:57 sean-k-mooney ok i might as well. it will delay the check jobs but we will have to wait for the gate jobs after anyway so it wont be much of a delay
18:50:06 melwitt sean-k-mooney: no, wasn't implying I want a change. I just didn't know the policy was different than the usual
19:07:40 leakypipes fried_rice: k, I'm +2 on all of https://review.openstack.org/#/q/topic:static-rt-in-compute-mgr
19:30:51 sean-k-mooney by the way for Depends-on are we ment to use ther gerrit url or change id now with zuul v3
19:32:40 sean-k-mooney i think Depends-On: https://review.openstack.org/#/c/629025/8 is prefered over Depends-on: I1852038b8d40d4aa52c4f8e661bc78e54e0fc966 correct?
19:37:29 melwitt sean-k-mooney: yes, supposed to use gerrit url now i.e. https://review.openstack.org/629025 but you don't need to change it. I was just mentioning FYI for next time to make it easier to see what changes your release request depends on
19:39:07 sean-k-mooney melwitt: well i need to update the sha so ill add it. thanks. i was 90% sure but could not find it documented
20:06:36 openstackgerrit sean mooney proposed openstack/os-vif master: do not always plug ovs ports. https://review.openstack.org/631829
20:06:37 openstackgerrit sean mooney proposed openstack/os-vif master: Cleanup device at the end of 'test_iproute_object_closes_correctly' test https://review.openstack.org/629112
20:06:38 openstackgerrit sean mooney proposed openstack/os-vif master: Import IP implementation modules outside privsep context https://review.openstack.org/629025
20:07:57 hansmoleman dansmith: yeah https://review.openstack.org/#/c/631811/ fixed that tracing once we upgrade
20:08:47 dansmith mriedem: cool, +1d
20:21:12 hogepodge melwitt: mriedem: Do you know if there's a workaround to this bug? https://bugs.launchpad.net/nova/+bug/1755756
20:21:14 openstack Launchpad bug 1755756 in OpenStack Compute (nova) "ImageUnacceptable: Image is unacceptable: 'qemu-img info' parsing failed" [Undecided,New] - Assigned to pandatt (pandatt)

Earlier   Later