*** yamamoto has joined #openstack-dragonflow | 00:47 | |
*** yamamoto has quit IRC | 00:52 | |
*** openstackgerrit has quit IRC | 05:48 | |
*** openstackgerrit has joined #openstack-dragonflow | 06:43 | |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: lockedobjects_db: Extract nested lock detection to a method https://review.openstack.org/560792 | 06:43 |
---|---|---|
openstackgerrit | Omer Anson proposed openstack/dragonflow master: lockedobjects_db: Move lock logic to a context-manager method https://review.openstack.org/560793 | 06:44 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: locked_objects_db: Take function to get lock-id https://review.openstack.org/560794 | 06:44 |
oanson | Morning | 06:44 |
*** yamamoto has joined #openstack-dragonflow | 07:05 | |
oanson | snapiri, this is the nb_api's 'mainloop' | 07:38 |
oanson | It calls self._notification_cb, which is set by a call to 'register_notification_callback' | 07:38 |
oanson | Turns out you need to also paste the link: https://github.com/openstack/dragonflow/blob/master/dragonflow/db/api_nb.py#L203 | 07:39 |
oanson | Now, 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 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: Fix documentation compilation https://review.openstack.org/560815 | 07:46 |
*** yamamoto has quit IRC | 08:17 | |
*** yamamoto has joined #openstack-dragonflow | 08:32 | |
*** yamamoto has quit IRC | 08:36 | |
openstackgerrit | Merged openstack/dragonflow master: Remove service_names dictionary from service.py https://review.openstack.org/560316 | 09:04 |
*** natanbro has joined #openstack-dragonflow | 09:25 | |
*** yamamoto has joined #openstack-dragonflow | 09:46 | |
*** yamamoto has quit IRC | 09:48 | |
*** yamamoto has joined #openstack-dragonflow | 09:49 | |
*** yamamoto has quit IRC | 09:52 | |
*** yamamoto has joined #openstack-dragonflow | 09:54 | |
*** yamamoto has quit IRC | 10:00 | |
*** yamamoto has joined #openstack-dragonflow | 10:01 | |
*** yamamoto has quit IRC | 10:56 | |
*** yamamoto has joined #openstack-dragonflow | 11:02 | |
*** yamamoto has quit IRC | 11:03 | |
*** yamamoto has joined #openstack-dragonflow | 11:08 | |
*** yamamoto has quit IRC | 11:11 | |
*** yamamoto has joined #openstack-dragonflow | 11:11 | |
*** yamamoto has quit IRC | 11:33 | |
irenab | snapiri, ping | 12:18 |
irenab | oanson, still here? | 12:19 |
oanson | irenab, yep | 12:21 |
irenab | oanson, I notices that job that installs kuryr with df is constantly failing, but cannot find any useful log | 12:24 |
irenab | oanson, here for example https://review.openstack.org/#/c/560792/1 | 12:24 |
oanson | I see it | 12:24 |
oanson | Gimme a second to skim the logs | 12:24 |
oanson | irenab, 2018-04-12 06:54:54.657824 | primary | bash: /opt/stack/new/kuryr-kubernetes/devstack/gate_hook.sh: No such file or directory | 12:25 |
oanson | at 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_657824 | 12:25 |
oanson | Probably have to add kuryr-kubernetes to required-projects or some such | 12:26 |
irenab | is this still moving to zuul3 issue? | 12:29 |
*** yamamoto has joined #openstack-dragonflow | 12:30 | |
oanson | irenab, not sure. I think they changed a bit how projects are cloned into the test environment. | 12:33 |
oanson | I think that projects that aren't specifically defined, don't get cloned. And then things break. | 12:33 |
irenab | oanson, do you know where this job is defined? | 12:34 |
oanson | Not of the top of my head | 12:34 |
oanson | (That wasn't a no. I'm looking for it (: ) | 12:35 |
*** yamamoto has quit IRC | 12:35 | |
oanson | Looks like here: http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/zuul.d/zuul-legacy-jobs.yaml#n674 | 12:36 |
oanson | Hmm, openstack/kuryr-kubernetes is in required-projects. Maybe it needs to be registered somewhere else as well | 12:36 |
irenab | do you know why is it in infra legacy jobs? | 12:38 |
oanson | It'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#L44 | 12:38 |
oanson | I guess no one converted it | 12:38 |
irenab | maybe this should be convereted and also defined in df? | 12:38 |
oanson | Yes and no. It should definitely be converted. And definitely added to both DF and kuryr-k8s .zuul.yaml files. | 12:39 |
oanson | But who owns the tests? | 12:39 |
oanson | irenab, actually, as the core in both, I guess it's your decision :) | 12:39 |
irenab | this job is just for installation. In kuryr there is a tempest job | 12:40 |
irenab | http://logs.openstack.org/13/560313/6/check/kuryr-kubernetes-tempest-dragonflow/ce53d25/ | 12:40 |
irenab | which actually also fails due to the FIP bug | 12:40 |
irenab | I wonder if we need the install job at our (df) gate at all | 12:41 |
oanson | If we have the tempest job running in our gate, then no | 12:41 |
oanson | The tempest gate includes the installation | 12:42 |
oanson | And too many gates means 1. too many statistical errors and 2. many resources going to waste (slowing down gates and review cycles) | 12:42 |
irenab | agreed. But in kuryr there are k8s related tests, not sure we need this in DF | 12:42 |
oanson | Better to use an existing gate / test suite | 12:43 |
irenab | oanson, for neutron, we run neutron tempest tests, right? | 12:43 |
oanson | Yes | 12:43 |
oanson | Actually, I think we run the general tempest test. We filter out anything not relevant | 12:44 |
irenab | so probably we need to run kuryr-k8s test in df gate, and instead of the legacy install job, run the kuryr-tempest one | 12:44 |
oanson | Yes. 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 that | 12:44 |
irenab | I will check with snapiri on Sunday the tech details how to make the switch, no reason to keep unrelevant jobs in df gate | 12:45 |
oanson | I think it should be a simple matter of updated .zuul.yaml | 12:45 |
oanson | The same way we have dragonflow-openstack-ansible-cross-repo | 12:46 |
irenab | oanson, but this legacy job is not defined in the df zuul | 12:47 |
oanson | No. It would have to be removed from openstack-zuul-jobs | 12:47 |
*** yamamoto has joined #openstack-dragonflow | 12:47 | |
oanson | And from project-config | 12:48 |
irenab | right, and the new one should be added in the DF zuul.yaml | 12:48 |
oanson | Yes | 12:49 |
*** yamamoto has quit IRC | 12:49 | |
*** yamamoto has joined #openstack-dragonflow | 13:01 | |
*** yamamoto has quit IRC | 13:01 | |
*** yamamoto has joined #openstack-dragonflow | 13:05 | |
*** yamamoto has quit IRC | 13:09 | |
*** yamamoto has joined #openstack-dragonflow | 13:21 | |
*** yamamoto has quit IRC | 13:24 | |
*** yamamoto has joined #openstack-dragonflow | 13:43 | |
*** yamamoto has quit IRC | 13:44 | |
*** yamamoto has joined #openstack-dragonflow | 13:48 | |
*** yamamoto has quit IRC | 13:49 | |
*** yamamoto has joined #openstack-dragonflow | 13:49 | |
*** mlavalle has joined #openstack-dragonflow | 13:54 | |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: New application framework https://review.openstack.org/529358 | 14:37 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: Connect new Datapath object to ryu base https://review.openstack.org/529980 | 14:37 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: Add dragonflow legacy node to new datapath https://review.openstack.org/529981 | 14:37 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: [WIP] Port portsec app to new framework https://review.openstack.org/529982 | 14:37 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: [WIP] Port SG app to new framework https://review.openstack.org/530028 | 14:37 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: locked_objects_db: Take function to get lock-id https://review.openstack.org/560794 | 14:51 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: A model instance update also sends an update on all referred instances https://review.openstack.org/480196 | 14:58 |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: [WIP] Test selective proactive topology distribution https://review.openstack.org/538830 | 14:58 |
*** natanbro has quit IRC | 15:00 | |
*** yamamoto has quit IRC | 16:47 | |
*** yamamoto has joined #openstack-dragonflow | 16:58 | |
*** yamamoto has quit IRC | 17:03 | |
*** yamamoto has joined #openstack-dragonflow | 18:04 | |
openstackgerrit | Omer Anson proposed openstack/dragonflow master: locked_objects_db: Take function to get lock-id https://review.openstack.org/560794 | 18:15 |
*** openstackgerrit has quit IRC | 18:19 | |
*** lihi has quit IRC | 20:03 | |
*** irenab has quit IRC | 20:04 | |
*** snapiri has quit IRC | 20:04 | |
*** oanson has quit IRC | 20:05 | |
*** openstackgerrit has joined #openstack-dragonflow | 20:06 | |
openstackgerrit | boden proposed openstack/dragonflow master: use rpc Connection rather than create_connection https://review.openstack.org/561029 | 20:06 |
*** snapiri has joined #openstack-dragonflow | 20:11 | |
*** irenab has joined #openstack-dragonflow | 20:11 | |
*** oanson has joined #openstack-dragonflow | 20:12 | |
*** lihi has joined #openstack-dragonflow | 20:12 | |
*** yamamoto has quit IRC | 20:21 | |
*** yamamoto has joined #openstack-dragonflow | 20:22 | |
*** openstackstatus has quit IRC | 21:27 | |
*** openstack has joined #openstack-dragonflow | 21:31 | |
*** ChanServ sets mode: +o openstack | 21: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 pads | 23:39 | |
*** mlavalle has quit IRC | 23:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!