Thursday, 2018-04-12

*** yamamoto has joined #openstack-dragonflow00:47
*** yamamoto has quit IRC00:52
*** openstackgerrit has quit IRC05:48
*** openstackgerrit has joined #openstack-dragonflow06:43
openstackgerritOmer Anson proposed openstack/dragonflow master: lockedobjects_db: Extract nested lock detection to a method  https://review.openstack.org/56079206:43
openstackgerritOmer Anson proposed openstack/dragonflow master: lockedobjects_db: Move lock logic to a context-manager method  https://review.openstack.org/56079306:44
openstackgerritOmer Anson proposed openstack/dragonflow master: locked_objects_db: Take function to get lock-id  https://review.openstack.org/56079406:44
oansonMorning06:44
*** yamamoto has joined #openstack-dragonflow07:05
oansonsnapiri, this is the nb_api's 'mainloop'07:38
oansonIt calls self._notification_cb, which is set by a call to 'register_notification_callback'07:38
oansonTurns out you need to also paste the link: https://github.com/openstack/dragonflow/blob/master/dragonflow/db/api_nb.py#L20307:39
oansonNow, you can see the mainloop is called from the DF local controller: 'https://github.com/openstack/dragonflow/blob/master/dragonflow/controller/df_local_controller.py#L121'07:39
openstackgerritOmer Anson proposed openstack/dragonflow master: Fix documentation compilation  https://review.openstack.org/56081507:46
*** yamamoto has quit IRC08:17
*** yamamoto has joined #openstack-dragonflow08:32
*** yamamoto has quit IRC08:36
openstackgerritMerged openstack/dragonflow master: Remove service_names dictionary from service.py  https://review.openstack.org/56031609:04
*** natanbro has joined #openstack-dragonflow09:25
*** yamamoto has joined #openstack-dragonflow09:46
*** yamamoto has quit IRC09:48
*** yamamoto has joined #openstack-dragonflow09:49
*** yamamoto has quit IRC09:52
*** yamamoto has joined #openstack-dragonflow09:54
*** yamamoto has quit IRC10:00
*** yamamoto has joined #openstack-dragonflow10:01
*** yamamoto has quit IRC10:56
*** yamamoto has joined #openstack-dragonflow11:02
*** yamamoto has quit IRC11:03
*** yamamoto has joined #openstack-dragonflow11:08
*** yamamoto has quit IRC11:11
*** yamamoto has joined #openstack-dragonflow11:11
*** yamamoto has quit IRC11:33
irenabsnapiri, ping12:18
irenaboanson, still here?12:19
oansonirenab, yep12:21
irenaboanson, I notices that job that installs kuryr with df is constantly failing, but cannot find any useful log12:24
irenaboanson, here for example https://review.openstack.org/#/c/560792/112:24
oansonI see it12:24
oansonGimme a second to skim the logs12:24
oansonirenab, 2018-04-12 06:54:54.657824 | primary | bash: /opt/stack/new/kuryr-kubernetes/devstack/gate_hook.sh: No such file or directory12:25
oansonat http://logs.openstack.org/92/560792/1/check/legacy-install-dsvm-dragonflow-kuryr-kubernetes/6d5635d/job-output.txt.gz#_2018-04-12_06_54_54_65782412:25
oansonProbably have to add kuryr-kubernetes to required-projects or some such12:26
irenabis this still moving to zuul3 issue?12:29
*** yamamoto has joined #openstack-dragonflow12:30
oansonirenab, not sure. I think they changed a bit how projects are cloned into the test environment.12:33
oansonI think that projects that aren't specifically defined, don't get cloned. And then things break.12:33
irenaboanson, do you know where this job is defined?12:34
oansonNot of the top of my head12:34
oanson(That wasn't a no. I'm looking for it (: )12:35
*** yamamoto has quit IRC12:35
oansonLooks like here: http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/zuul.d/zuul-legacy-jobs.yaml#n67412:36
oansonHmm, openstack/kuryr-kubernetes is in required-projects. Maybe it needs to be registered somewhere else as well12:36
irenabdo you know why is it in infra legacy jobs?12:38
oansonIt's also defined in PROJECTS environment variable: https://github.com/openstack-infra/openstack-zuul-jobs/blob/master/playbooks/legacy/install-dsvm-dragonflow-kuryr-kubernetes/run.yaml#L4412:38
oansonI guess no one converted it12:38
irenabmaybe this should be convereted and also defined in df?12:38
oansonYes and no. It should definitely be converted. And definitely added to both DF and kuryr-k8s .zuul.yaml files.12:39
oansonBut who owns the tests?12:39
oansonirenab, actually, as the core in both, I guess it's your decision :)12:39
irenabthis job is just for installation. In kuryr there is a tempest job12:40
irenabhttp://logs.openstack.org/13/560313/6/check/kuryr-kubernetes-tempest-dragonflow/ce53d25/12:40
irenabwhich actually also fails due to the FIP bug12:40
irenabI wonder if we need the install job at our (df) gate at all12:41
oansonIf we have the tempest job running in our gate, then no12:41
oansonThe tempest gate includes the installation12:42
oansonAnd too many gates means 1. too many statistical errors and 2. many resources going to waste (slowing down gates and review cycles)12:42
irenabagreed. But in kuryr there are k8s related tests, not sure we need this in DF12:42
oansonBetter to use an existing gate / test suite12:43
irenaboanson, for neutron, we run neutron tempest tests, right?12:43
oansonYes12:43
oansonActually, I think we run the general tempest test. We filter out anything not relevant12:44
irenabso probably we need to run kuryr-k8s test in df gate, and instead of the legacy install job, run the kuryr-tempest one12:44
oansonYes. We can always filter out tests we deem irrelevant. Although unless they fail for reasons not related to DF, I don't see a good reason to do that12:44
irenabI will check with snapiri on Sunday the tech details how to make the switch, no reason to keep unrelevant jobs in df gate12:45
oansonI think it should be a simple matter of updated .zuul.yaml12:45
oansonThe same way we have dragonflow-openstack-ansible-cross-repo12:46
irenaboanson, but this legacy job is not defined in the df zuul12:47
oansonNo. It would have to be removed from openstack-zuul-jobs12:47
*** yamamoto has joined #openstack-dragonflow12:47
oansonAnd from project-config12:48
irenabright, and the new one should be added in the DF zuul.yaml12:48
oansonYes12:49
*** yamamoto has quit IRC12:49
*** yamamoto has joined #openstack-dragonflow13:01
*** yamamoto has quit IRC13:01
*** yamamoto has joined #openstack-dragonflow13:05
*** yamamoto has quit IRC13:09
*** yamamoto has joined #openstack-dragonflow13:21
*** yamamoto has quit IRC13:24
*** yamamoto has joined #openstack-dragonflow13:43
*** yamamoto has quit IRC13:44
*** yamamoto has joined #openstack-dragonflow13:48
*** yamamoto has quit IRC13:49
*** yamamoto has joined #openstack-dragonflow13:49
*** mlavalle has joined #openstack-dragonflow13:54
openstackgerritOmer Anson proposed openstack/dragonflow master: New application framework  https://review.openstack.org/52935814:37
openstackgerritOmer Anson proposed openstack/dragonflow master: Connect new Datapath object to ryu base  https://review.openstack.org/52998014:37
openstackgerritOmer Anson proposed openstack/dragonflow master: Add dragonflow legacy node to new datapath  https://review.openstack.org/52998114:37
openstackgerritOmer Anson proposed openstack/dragonflow master: [WIP] Port portsec app to new framework  https://review.openstack.org/52998214:37
openstackgerritOmer Anson proposed openstack/dragonflow master: [WIP] Port SG app to new framework  https://review.openstack.org/53002814:37
openstackgerritOmer Anson proposed openstack/dragonflow master: locked_objects_db: Take function to get lock-id  https://review.openstack.org/56079414:51
openstackgerritOmer Anson proposed openstack/dragonflow master: A model instance update also sends an update on all referred instances  https://review.openstack.org/48019614:58
openstackgerritOmer Anson proposed openstack/dragonflow master: [WIP] Test selective proactive topology distribution  https://review.openstack.org/53883014:58
*** natanbro has quit IRC15:00
*** yamamoto has quit IRC16:47
*** yamamoto has joined #openstack-dragonflow16:58
*** yamamoto has quit IRC17:03
*** yamamoto has joined #openstack-dragonflow18:04
openstackgerritOmer Anson proposed openstack/dragonflow master: locked_objects_db: Take function to get lock-id  https://review.openstack.org/56079418:15
*** openstackgerrit has quit IRC18:19
*** lihi has quit IRC20:03
*** irenab has quit IRC20:04
*** snapiri has quit IRC20:04
*** oanson has quit IRC20:05
*** openstackgerrit has joined #openstack-dragonflow20:06
openstackgerritboden proposed openstack/dragonflow master: use rpc Connection rather than create_connection  https://review.openstack.org/56102920:06
*** snapiri has joined #openstack-dragonflow20:11
*** irenab has joined #openstack-dragonflow20:11
*** oanson has joined #openstack-dragonflow20:12
*** lihi has joined #openstack-dragonflow20:12
*** yamamoto has quit IRC20:21
*** yamamoto has joined #openstack-dragonflow20:22
*** openstackstatus has quit IRC21:27
*** openstack has joined #openstack-dragonflow21:31
*** ChanServ sets mode: +o openstack21:31
-openstackstatus- NOTICE: The Etherpad service at https://etherpad.openstack.org/ is being restarted to pick up the latest release version; browsers should see only a brief ~1min blip before reconnecting automatically to active pads23:39
*** mlavalle has quit IRC23:45

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!