Wednesday, 2018-08-22

*** edmondsw has quit IRC02:45
*** edmondsw has joined #openstack-powervm05:34
efriededmondsw: Did you see:12:46
efried6:20:15 PM - tonyb: edmondsw, mujahidali: 'sup with DIB?12:46
*** esberglu has quit IRC13:20
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm master: import zuul job settings from project-config  https://review.openstack.org/59498413:28
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm master: add python 3.6 unit test job  https://review.openstack.org/59498513:28
openstackgerritDoug Hellmann proposed openstack/networking-powervm master: import zuul job settings from project-config  https://review.openstack.org/59498613:28
openstackgerritDoug Hellmann proposed openstack/nova-powervm master: import zuul job settings from project-config  https://review.openstack.org/59498713:28
openstackgerritDoug Hellmann proposed openstack/nova-powervm master: add python 3.6 unit test job  https://review.openstack.org/59498813:28
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm stable/ocata: import zuul job settings from project-config  https://review.openstack.org/59498913:28
openstackgerritDoug Hellmann proposed openstack/networking-powervm stable/ocata: import zuul job settings from project-config  https://review.openstack.org/59499013:28
openstackgerritDoug Hellmann proposed openstack/nova-powervm stable/ocata: import zuul job settings from project-config  https://review.openstack.org/59499113:28
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm stable/pike: import zuul job settings from project-config  https://review.openstack.org/59499213:28
openstackgerritDoug Hellmann proposed openstack/networking-powervm stable/pike: import zuul job settings from project-config  https://review.openstack.org/59499313:28
openstackgerritDoug Hellmann proposed openstack/nova-powervm stable/pike: import zuul job settings from project-config  https://review.openstack.org/59499413:28
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm stable/queens: import zuul job settings from project-config  https://review.openstack.org/59499513:28
openstackgerritDoug Hellmann proposed openstack/networking-powervm stable/queens: import zuul job settings from project-config  https://review.openstack.org/59499613:28
openstackgerritDoug Hellmann proposed openstack/nova-powervm stable/queens: import zuul job settings from project-config  https://review.openstack.org/59499713:28
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm stable/rocky: import zuul job settings from project-config  https://review.openstack.org/59499813:28
openstackgerritDoug Hellmann proposed openstack/networking-powervm stable/rocky: import zuul job settings from project-config  https://review.openstack.org/59499913:28
openstackgerritDoug Hellmann proposed openstack/nova-powervm stable/rocky: import zuul job settings from project-config  https://review.openstack.org/59500013:28
edmondswtonyb we're talking about upgrading nodepool for our CI, and esberglu said that will require us to use diskimage-builder14:05
edmondswhe also said that you'd once offered to help us understand that, or something along those lines14:05
edmondswefried thanks, I did not... somehow my bouncer missed that14:06
edmondswefried I'll take first stab at the dhellmann patches above and then let you know14:13
efriededmondsw: Roger that, and thanks.14:14
*** efried is now known as efried_goatin14:14
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm master: import zuul job settings from project-config  https://review.openstack.org/59498414:51
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm master: add python 3.6 unit test job  https://review.openstack.org/59498514:51
openstackgerritDoug Hellmann proposed openstack/networking-powervm master: import zuul job settings from project-config  https://review.openstack.org/59498614:51
openstackgerritDoug Hellmann proposed openstack/nova-powervm master: import zuul job settings from project-config  https://review.openstack.org/59498714:51
openstackgerritDoug Hellmann proposed openstack/nova-powervm master: add python 3.6 unit test job  https://review.openstack.org/59498814:51
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm stable/ocata: import zuul job settings from project-config  https://review.openstack.org/59498914:51
openstackgerritDoug Hellmann proposed openstack/networking-powervm stable/ocata: import zuul job settings from project-config  https://review.openstack.org/59499014:51
openstackgerritDoug Hellmann proposed openstack/nova-powervm stable/ocata: import zuul job settings from project-config  https://review.openstack.org/59499114:51
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm stable/pike: import zuul job settings from project-config  https://review.openstack.org/59499214:52
openstackgerritDoug Hellmann proposed openstack/networking-powervm stable/pike: import zuul job settings from project-config  https://review.openstack.org/59499314:52
openstackgerritDoug Hellmann proposed openstack/nova-powervm stable/pike: import zuul job settings from project-config  https://review.openstack.org/59499414:52
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm stable/queens: import zuul job settings from project-config  https://review.openstack.org/59499514:52
openstackgerritDoug Hellmann proposed openstack/networking-powervm stable/queens: import zuul job settings from project-config  https://review.openstack.org/59499614:52
openstackgerritDoug Hellmann proposed openstack/nova-powervm stable/queens: import zuul job settings from project-config  https://review.openstack.org/59499714:52
openstackgerritDoug Hellmann proposed openstack/ceilometer-powervm stable/rocky: import zuul job settings from project-config  https://review.openstack.org/59499814:52
openstackgerritDoug Hellmann proposed openstack/networking-powervm stable/rocky: import zuul job settings from project-config  https://review.openstack.org/59499914:52
openstackgerritDoug Hellmann proposed openstack/nova-powervm stable/rocky: import zuul job settings from project-config  https://review.openstack.org/59500014:52
*** mmedvede is now known as mmedvede_14:55
*** mmedvede_ is now known as mmedvede14:55
*** tonyb has quit IRC15:04
*** efried_goatin is now known as efried15:06
*** openstackgerrit has quit IRC15:31
*** efried has quit IRC15:36
*** efried has joined #openstack-powervm15:36
*** openstackgerrit has joined #openstack-powervm15:39
openstackgerritMatthew Edmonds proposed openstack/nova-powervm master: Clean up .gitignore  https://review.openstack.org/59454615:39
openstackgerritMatthew Edmonds proposed openstack/networking-powervm master: add python 3.6 unit test job  https://review.openstack.org/59523315:48
edmondswefried alright ready for you to look at those15:50
edmondswwe want to merge the "import zuul..." ones first, and then the "add python 3.6..." are based on them (master-only)15:50
edmondswany questions, let me know15:50
efriedack15:51
edmondswsee also https://review.openstack.org/#/c/595004/ , which will merge after these15:51
edmondswat least after the "import zuul" ones15:51
efriedack15:51
edmondswand note that one of the "add python" ones was proposed by me (with dhellmann's consent) because his script didn't handle the neutron special case15:52
*** efried is now known as efried_rollin16:14
*** esberglu has joined #openstack-powervm16:41
*** esberglu has quit IRC16:45
*** esberglu has joined #openstack-powervm17:15
*** esberglu has quit IRC17:19
*** mmedvede has quit IRC17:25
*** mmedvede has joined #openstack-powervm17:59
*** ChanServ has quit IRC18:16
*** ChanServ has joined #openstack-powervm18:22
*** barjavel.freenode.net sets mode: +o ChanServ18:22
*** efried_rollin is now known as efried19:47
*** mdrabe has quit IRC20:22
*** mdrabe has joined #openstack-powervm20:24
*** tonyb has joined #openstack-powervm20:26
tonybedmondsw: Ahh okay cool.  Well I can certainly help with that.  once esberglu has had a look y'all know where I am20:32
edmondswtonyb yep, thanks in advance!20:34
tonyb\o/20:37
efriededmondsw: Help me understand how to review these project config changes.20:52
edmondswsure20:53
edmondswfirst, CI doesn't matter because the files changed only apply to zuul20:53
efriedI'm comparing the removals in https://review.openstack.org/#/c/595004/3/zuul.d/projects.yaml with the additions in e.g. https://review.openstack.org/#/c/594999/2/.zuul.yaml20:53
efriedand they're the same, so that's... it?20:53
edmondswyep, pretty much20:53
edmondswfor the "import zuul" changes... those are just moving from project-config to our repo, no change20:54
edmondswthen the "add python 3.6" is obviously adding a new job for checking 3.620:54
edmondswthe only trick there is that the networking-powervm jobs tack on -neutron to the job name20:55
edmondswhaving these in our repo gives us the flexibility to run different jobs on different branches (e.g. only testing 3.6 on master)20:55
efriedyeah, I happened to start with the neutron one, which probably confused me more than it needed to.20:58
efriedWho are we going to get to +A the ones of these that you submitted?21:03
openstackgerritMerged openstack/networking-powervm stable/rocky: import zuul job settings from project-config  https://review.openstack.org/59499921:08
efriededmondsw: Okay, +A'd all except the two you authored. We need a second core for those. mdrabe?21:10
efriedhttps://review.openstack.org/59454621:10
efriedhttps://review.openstack.org/59523321:10
edmondswefried I only authored one21:10
efriedthe .gitignore one21:10
edmondswoh, the .gitignore?21:10
edmondswyeah21:10
efriednot related to this effort, but still.21:11
edmondswsure21:11
edmondswtx21:11
edmondswI'll do similar .gitignore changes for the other *-powervm projects, just wanted to throw it out for one first21:11
openstackgerritMerged openstack/ceilometer-powervm stable/queens: import zuul job settings from project-config  https://review.openstack.org/59499521:13
openstackgerritMerged openstack/networking-powervm master: import zuul job settings from project-config  https://review.openstack.org/59498621:15
openstackgerritMerged openstack/networking-powervm stable/pike: import zuul job settings from project-config  https://review.openstack.org/59499321:16
openstackgerritMerged openstack/networking-powervm stable/queens: import zuul job settings from project-config  https://review.openstack.org/59499621:30
openstackgerritMerged openstack/networking-powervm stable/ocata: import zuul job settings from project-config  https://review.openstack.org/59499021:30
openstackgerritMerged openstack/nova-powervm stable/ocata: import zuul job settings from project-config  https://review.openstack.org/59499121:31
openstackgerritMerged openstack/nova-powervm stable/rocky: import zuul job settings from project-config  https://review.openstack.org/59500021:32
openstackgerritMerged openstack/nova-powervm stable/queens: import zuul job settings from project-config  https://review.openstack.org/59499721:32
openstackgerritMerged openstack/nova-powervm stable/pike: import zuul job settings from project-config  https://review.openstack.org/59499421:32
openstackgerritMerged openstack/nova-powervm master: import zuul job settings from project-config  https://review.openstack.org/59498721:32
openstackgerritMerged openstack/nova-powervm master: add python 3.6 unit test job  https://review.openstack.org/59498821:39
openstackgerritMerged openstack/ceilometer-powervm stable/rocky: import zuul job settings from project-config  https://review.openstack.org/59499821:40
openstackgerritMerged openstack/ceilometer-powervm stable/pike: import zuul job settings from project-config  https://review.openstack.org/59499221:40
openstackgerritMerged openstack/ceilometer-powervm master: import zuul job settings from project-config  https://review.openstack.org/59498421:40
openstackgerritMerged openstack/ceilometer-powervm master: add python 3.6 unit test job  https://review.openstack.org/59498521:40
openstackgerritMerged openstack/ceilometer-powervm stable/ocata: import zuul job settings from project-config  https://review.openstack.org/59498921:40
efriededmondsw: I think we're ready for https://review.openstack.org/#/c/595004/ to merge now. Wanna add your +1?21:50
edmondswlooking...21:51
efriedI added a helpful link21:51
edmondswefried done21:52
edmondswefried maybe just fast-approve the one that I authored for networking-powervm? I don't think mdrabe is a core there21:53
efriedokay.21:53
edmondswmaybe he should be...21:54
efriedI would have thought so.21:54
openstackgerritMerged openstack/networking-powervm master: add python 3.6 unit test job  https://review.openstack.org/59523322:16
*** ChanServ has quit IRC22:49
*** ChanServ has joined #openstack-powervm23:03
*** barjavel.freenode.net sets mode: +o ChanServ23:03

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