| *** catintheroof has joined #openstack-searchlight | 00:08 | |
| *** yingjun has joined #openstack-searchlight | 00:38 | |
| *** catintheroof has quit IRC | 02:16 | |
| *** catintheroof has joined #openstack-searchlight | 02:16 | |
| *** catintheroof has quit IRC | 02:20 | |
| openstackgerrit | melissaml proposed openstack/python-searchlightclient: Replace six.iteritems() with .items() https://review.openstack.org/404000 | 03:23 |
|---|---|---|
| *** catintheroof has joined #openstack-searchlight | 03:25 | |
| openstackgerrit | melissaml proposed openstack/python-searchlightclient: Replace six.iteritems() with .items() https://review.openstack.org/404000 | 03:43 |
| *** catinthe_ has joined #openstack-searchlight | 04:23 | |
| *** catintheroof has quit IRC | 04:26 | |
| *** catinthe_ has quit IRC | 04:26 | |
| *** catintheroof has joined #openstack-searchlight | 04:27 | |
| *** catintheroof has quit IRC | 04:27 | |
| *** pcaruana has joined #openstack-searchlight | 07:21 | |
| *** yingjun has quit IRC | 09:41 | |
| *** yingjun has joined #openstack-searchlight | 09:41 | |
| *** yingjun_ has joined #openstack-searchlight | 09:41 | |
| *** yingjun_ has quit IRC | 09:45 | |
| *** yingjun has quit IRC | 09:45 | |
| *** openstackgerrit has quit IRC | 09:48 | |
| *** openstackgerrit has joined #openstack-searchlight | 09:49 | |
| *** yingjun has joined #openstack-searchlight | 10:35 | |
| *** yingjun has quit IRC | 10:38 | |
| *** yingjun has joined #openstack-searchlight | 10:38 | |
| *** yingjun has quit IRC | 10:43 | |
| *** catinthe_ has joined #openstack-searchlight | 12:37 | |
| *** yingjun has joined #openstack-searchlight | 13:25 | |
| *** yingjun has quit IRC | 13:53 | |
| *** yingjun has joined #openstack-searchlight | 13:54 | |
| *** yingjun has quit IRC | 14:14 | |
| *** yingjun has joined #openstack-searchlight | 14:15 | |
| *** yingjun has quit IRC | 14:19 | |
| openstackgerrit | Steve McLellan proposed openstack/searchlight: Add tools/test-setup.sh script to setup test environment https://review.openstack.org/399159 | 15:25 |
| AJaeger | sjmc7: that depends-on ^ is wrong. | 15:26 |
| AJaeger | We first need to merge this to decouple the two changes | 15:26 |
| AJaeger | if the project-config change merges first, you cannot merge anything at all until 399159 merges and we need coordination. If 339159 merges first, you can continue merging and it's decoupled | 15:27 |
| sjmc7 | AJaeger: yeah, I just realized that | 15:27 |
| sjmc7 | too early in the morning for me | 15:28 |
| sjmc7 | the infra change looks good; I have time today to fix the searchlight patch if something’s awry | 15:28 |
| AJaeger | sjmc7: So, let's go back to previous version (without depends-on), and once you have it in, I'll ask for project-concifg change. | 15:29 |
| openstackgerrit | Andreas Jaeger proposed openstack/searchlight: Add tools/test-setup.sh script to setup test environment https://review.openstack.org/399159 | 15:29 |
| AJaeger | removed depends-on ^ | 15:29 |
| sjmc7 | my worry is this won’t pass because it’ll stomp on what the infra stuff’s already doing | 15:30 |
| sjmc7 | but let’s see. if it passes i’ll get it approved this morning | 15:30 |
| AJaeger | sjmc7: we did this kind of change with nodepool already - in the order proposed. Worked fine... | 15:30 |
| sjmc7 | ok, cool | 15:30 |
| sjmc7 | i’ll find someone else to rubber stamp this | 15:30 |
| AJaeger | thanks, sjmc7 ! | 15:32 |
| sjmc7 | sure, thanks for all the work on this | 15:32 |
| *** TravT has joined #openstack-searchlight | 15:54 | |
| openstackgerrit | Merged openstack/searchlight: Add tools/test-setup.sh script to setup test environment https://review.openstack.org/399159 | 16:03 |
| sjmc7 | AJaeger: that patch is merged on our end | 16:05 |
| sjmc7 | you want me to get it merged on all the branches or wait til we can try it out end to end on master? | 16:05 |
| *** pcaruana has quit IRC | 16:36 | |
| AJaeger | sjmc7: let's merge project-config next... | 17:51 |
| AJaeger | I'm on it now | 17:52 |
| sjmc7 | ok, thanks | 17:52 |
| AJaeger | sjmc7: infra change has merged, I'm rechecking now changes | 19:22 |
| sjmc7 | awesome, thanks | 19:23 |
| openstackgerrit | Andreas Jaeger proposed openstack/searchlight: Dummy: Testing that infra change works https://review.openstack.org/404399 | 19:24 |
| sjmc7 | i miss being able to watch the jenkins console output live :) | 19:30 |
| AJaeger | You still can - go to status.openstack.org/zuul and hover over a change, you should get e.g. | 19:30 |
| AJaeger | telnet://[2001:4800:1ae1:18:f816:3eff:fe3f:6d8]:19885 | 19:31 |
| AJaeger | Now run "nc 2001:4800:1ae1:18:f816:3eff:fe3f:6d8 19885" - or telnet instead of nc | 19:31 |
| AJaeger | https://review.openstack.org/#/c/399183/ passed all tests | 19:31 |
| AJaeger | so, ready to merge ^ | 19:31 |
| sjmc7 | excellent. i’ll recheck the liberty one as well, check that the 1.x version works also | 19:32 |
| AJaeger | https://review.openstack.org/399182 is the liberty change - still failing. But liberty is nearly EOL, so you could ignore as well | 19:32 |
| AJaeger | master: https://review.openstack.org/404399 - passed tests | 19:32 |
| sjmc7 | yeah, that needs a separate change | 19:32 |
| sjmc7 | cool, thanks again! | 19:32 |
| AJaeger | newton: https://review.openstack.org/399184 works | 19:33 |
| AJaeger | so, mitaka, newton, master are fine | 19:33 |
| AJaeger | now over to the searchlight team ;) | 19:33 |
| sjmc7 | :) | 19:33 |
| sjmc7 | glad we could be a guinea pig | 19:34 |
| AJaeger | nodepool was the guinea pig, you were the second generation of it ;) | 19:34 |
| sjmc7 | :) probably for the best. going first is never a good idea | 19:34 |
| *** matt-borland has joined #openstack-searchlight | 20:15 | |
| AJaeger | sjmc7: I merged our two changes, hope that fix liberty now: https://review.openstack.org/399182 | 20:15 |
| sjmc7 | yeah, that should do it | 20:16 |
| AJaeger | So, https://review.openstack.org/#/c/399182/ works - just one failure in python34 | 20:19 |
| sjmc7 | ok. i can add a fix for that to the patch | 20:19 |
| AJaeger | your call now ;) | 20:19 |
| AJaeger | we're moving in the right direction, glad to see this | 20:20 |
| AJaeger | ok, I'm signing of from this channel. If anything is nevertheless broken, please tell me on #openstack-infra | 20:20 |
| sjmc7 | will do. thanks again! | 20:20 |
| AJaeger | you're welcome | 20:21 |
| AJaeger | bye | 20:21 |
| *** AJaeger has left #openstack-searchlight | 20:21 | |
| *** catinthe_ has quit IRC | 20:35 | |
| openstackgerrit | Merged openstack/searchlight: Show team and repo badges on README https://review.openstack.org/402443 | 21:25 |
| *** catintheroof has joined #openstack-searchlight | 21:34 | |
| *** catintheroof has quit IRC | 21:38 | |
| *** catintheroof has joined #openstack-searchlight | 21:47 | |
| openstackgerrit | Merged openstack/searchlight-ui: Imported Translations from Zanata https://review.openstack.org/402319 | 22:46 |
| openstackgerrit | Merged openstack/searchlight-ui: Show team and repo badges on README https://review.openstack.org/402445 | 22:47 |
| *** matt-borland has quit IRC | 23:03 | |
| *** catinthe_ has joined #openstack-searchlight | 23:03 | |
| *** catintheroof has quit IRC | 23:06 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!