Thursday, 2018-11-15

SpamapSugh00:53
SpamapSneed to spend some time debugging ec2+nodepool00:53
SpamapSjust found 7 leaked instances00:53
SpamapShttp://paste.openstack.org/show/734843/00:58
SpamapSI think that's the culprit00:58
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/nodepool master: WIP: Implement an Amazon EC2 driver  https://review.openstack.org/53555801:02
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/nodepool master: AWS driver fix to not explode on common images  https://review.openstack.org/60136101:02
SpamapSsomething very broken in that driver too01:16
SpamapSI say max-servers: 5, but it just keeps adding servers until the cloud refuses.01:16
SpamapSdmsimard: tristanC ^^ have you all been playing with that driver much since we last talked about it?01:19
* SpamapS will try and whip it in to shape01:19
dmsimardSpamapS: I have not03:43
tristanCSpamapS: me neither, and my 12 month period amazon account is now over...03:46
*** bjackman has quit IRC03:58
*** bjackman has joined #zuul04:17
SpamapShah ok04:50
*** chandankumar has joined #zuul04:59
*** chandankumar is now known as chkumar|ruck05:03
*** bhavikdbavishi has joined #zuul05:05
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/nodepool master: Amazon EC2 driver  https://review.openstack.org/53555805:09
SpamapS^^ tests fixed, schema normalized closer to the openstack one.05:10
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/zuul-jobs master: Add a markdownlint job and role  https://review.openstack.org/60769105:16
openstackgerritClint 'SpamapS' Byrum proposed openstack-infra/zuul-jobs master: Add a markdownlint job and role  https://review.openstack.org/60769105:35
*** rlandy has quit IRC05:48
*** gundalow has quit IRC05:58
*** samccann has quit IRC06:00
*** corvus has quit IRC06:00
*** gundalow has joined #zuul06:00
*** samccann has joined #zuul06:00
*** threestrands has quit IRC06:01
*** pcaruana has joined #zuul07:19
*** goern has joined #zuul07:24
*** bhavikdbavishi has quit IRC07:53
*** quiquell is now known as quiquell|bbl08:10
*** sshnaidm|pto is now known as sshnaidm08:15
*** jpena|off is now known as jpena08:35
*** bhavikdbavishi has joined #zuul08:39
*** bhavikdbavishi has quit IRC08:40
*** bhavikdbavishi has joined #zuul08:40
*** ParsectiX has joined #zuul08:47
*** ParsectiX has quit IRC08:48
*** ParsectiX has joined #zuul08:48
*** bhavikdbavishi1 has joined #zuul08:59
*** bhavikdbavishi has quit IRC08:59
*** bhavikdbavishi1 is now known as bhavikdbavishi08:59
*** bjackman has quit IRC09:05
*** bhavikdbavishi has quit IRC09:15
*** hashar has joined #zuul09:22
*** sshnaidm is now known as sshnaidm|afk09:26
*** corvus has joined #zuul09:56
*** sshnaidm|afk is now known as sshnaidm10:02
*** bjackman has joined #zuul10:09
*** sdake has quit IRC10:11
*** sdake has joined #zuul10:14
*** kklimonda has joined #zuul10:17
*** ParsectiX has quit IRC10:19
*** ParsectiX has joined #zuul10:38
*** ParsectiX has quit IRC11:00
*** ParsectiX has joined #zuul11:04
*** panda|rover|off is now known as panda|rover11:08
*** dkehn has quit IRC11:32
*** bhavikdbavishi has joined #zuul11:39
*** ParsectiX has quit IRC11:47
*** hashar has quit IRC11:50
*** ParsectiX has joined #zuul11:55
*** ParsectiX has quit IRC11:56
*** electrofelix has joined #zuul12:00
*** jpena is now known as jpena|lunch12:02
tobiash@zuulers here in Berlin, have to leave now. It was great to see you again and hopefully cya next time in Denver :)12:08
*** quiquell|bbl is now known as quiquell12:09
openstackgerritQuique Llorente proposed openstack-infra/zuul master: Prevent exception at executor  https://review.openstack.org/61798412:14
tristanCtobiash: have a safe trip home!12:17
quiquellfungi: ^12:17
quiquelltristanC: Hello there, found small issue at cloning on executors12:17
quiquelltristanC: Maybe you can take a look at the review12:17
tristanCquiquell: i think the parameter is exist_ok12:20
tobiashquiquell, corvus: I think there was a fix race, you both have identical patches in review ^ ;)12:20
tobiashtristanC: thanks, u212:21
quiquelltristanC: Damn, fixing...12:23
quiquelltristanC: I test locally exist_ok then write it wrong :-)12:23
openstackgerritQuique Llorente proposed openstack-infra/zuul master: Prevent exception at executor  https://review.openstack.org/61798412:23
quiquelltristanC, tobiash: Done12:23
*** bjackman has quit IRC12:28
openstackgerritQuique Llorente proposed openstack-infra/zuul master: Prevent exception at executor  https://review.openstack.org/61798412:33
quiquellRefactored another makedirs in the same file12:34
*** hashar has joined #zuul12:34
AJaegertobiash: great seeing you here! Safe travels home!12:41
quiquelltristanC: Do you know if there are any option to disable host key verificatoin at executors ?12:43
pabelangertobiash: o/ always great to meet up12:55
*** rlandy has joined #zuul12:56
*** ParsectiX has joined #zuul12:59
*** bhavikdbavishi has quit IRC13:08
*** chkumar|ruck is now known as chkumar|ruck|brb13:15
*** chkumar|ruck|brb is now known as chkumar|ruck13:18
*** ianychoi has joined #zuul13:24
*** dkehn has joined #zuul13:35
*** jpena|lunch is now known as jpena13:42
openstackgerritPaul Belanger proposed openstack-infra/zuul master: WIP Add support for zones in executors  https://review.openstack.org/54919713:47
pabelangerrcarrillocruz: corvus: just a rebase, going to start adding tests now ^13:48
pabelangerand docs13:48
quiquellHi there14:00
quiquellI am facing this Timeout exception waiting for the logger. Please check connectivity to [38.145.34.145:19885]14:00
quiquellPlaying with zuul at my machine14:00
quiquellIt means that I have to open some kind of port at the security group at my cloud provider ?14:01
*** ParsectiX has quit IRC14:01
clarkbquiquell: the way console logging works is the job ansible runs a plugin hook thing (I forget actual term) on port 19885 that the executor reads from14:01
clarkbquiquell: so ya need to have that open to the executor14:01
quiquellclarkb: executor or node ?14:02
*** chkumar|ruck has quit IRC14:02
clarkbexecutor opens tcp connection to port 19885 on the test node14:02
quiquellclarkb: So the port has to be accesible from the node targeted by ansible, is that it ?14:02
quiquellclarkb: I mean nodes configured at nodepool14:03
clarkbit has to be opened on the node targetted by ansible14:03
*** ParsectiX has joined #zuul14:03
quiquellclarkb: Cool thanks so much, do you know any mort port needed there ?14:03
clarkbquiquell: I think ssh and 19885 are it and thinking about this more it might make sense to hvae zuul tunnel that connection over ssh since ssh is already assumed to be open14:04
clarkbmordred: ^ fyi14:04
quiquellclarkb: Humm that's a good one, It's not done because performance ?14:04
clarkbquiquell: or just an oversight? I'm not sure14:04
quiquellclarkb: Me neither I am noob here :-), but like it14:05
Diabelkopabelanger: here14:15
pabelangerDiabelko: o/14:15
*** hashar has quit IRC14:20
tobiashclarkb: I just saw a picture of zuul socks... Nice!14:23
AJaegertobiash: please send a photo ;)14:26
corvusclarkb, quiquell: we should totally tunnel the port.  mordred's new idea for how to do logginng will obsolete that (he actually wants to tunnel a unix domain socket), but i think we can implement tunelling the tcp port now fairly easily.14:34
quiquellcorvus: It's difficult to implement, I would like to put some reviews for that14:35
quiquell?14:35
quiquellWas a question forgot the question mark ?14:35
corvusquiquell: i think the tunnel needs to be set up when the ssh agent is started; i think mordred's unix domain work may do something similar; we may be able to copy that.14:36
quiquellcorvus: ack, thanks for info though14:37
corvusi would characterize this as moderate difficulty -- the code involved can be a little hard to follow, but it's not a complex problem14:37
corvusmostly requires looking into the log callback plugin and zuul_console14:37
quiquellcorvus: 80 read 20 write relation14:40
*** ParsectiX has quit IRC15:08
quiquellcorvus, clarkb: Do you know when do we create  /etc/ci/mirror_info.sh ?15:08
quiquellHumm this is more infra than zuul isue15:09
*** ParsectiX has joined #zuul15:37
*** ParsectiX has quit IRC16:04
*** ParsectiX has joined #zuul16:04
*** ParsectiX has quit IRC16:11
pabelangerquiquell: yah, it is in openstack-zuul-jobs mirror-info role16:18
pabelangerIIRC we do it in base job for openstack16:18
quiquellpabelanger: Yap found it thanks16:18
*** bjackman has joined #zuul16:26
*** chandankumar has joined #zuul16:26
*** chandankumar has quit IRC16:30
*** bjackman has quit IRC16:36
*** bhavikdbavishi has joined #zuul16:42
*** quiquell has quit IRC16:46
*** bhavikdbavishi has quit IRC16:46
*** bjackman has joined #zuul16:51
*** bjackman has quit IRC16:58
*** j^2 has joined #zuul17:14
*** j^2 has quit IRC17:16
*** j^2 has joined #zuul17:16
*** sshnaidm is now known as sshnaidm|afk17:37
*** chandankumar has joined #zuul17:44
dmsimardWould there be a way for Zuul to automatically enqueue a change that depended on a change that merged ?17:48
dmsimardPretend there's changes A and B, B has been approved and depends on A. A hasn't been merged yet -- if I approve A, it doesn't kick the jobs for B17:49
dmsimard(hopefully that makes sense)17:49
dmsimardA concrete example would be https://review.openstack.org/#/c/618018/ which I had to re-approve17:51
pabelangerdmsimard: do they share the same change queue for gate?17:59
pabelangersounds like they don't17:59
*** panda|rover is now known as panda|rover|off18:05
*** zigo has quit IRC18:06
dmsimardpabelanger: I didn't set one so I guess they're in their own queues -- I'll check that out, thanks18:15
*** chandankumar has quit IRC18:17
*** jpena is now known as jpena|off18:34
*** j^2 has quit IRC18:59
*** hashar has joined #zuul19:19
*** electrofelix has quit IRC19:31
SpamapSSeems like free CI services are popping up every day19:43
SpamapShttps://meta.sr.ht/19:43
SpamapSdmsimard: isn't what you're asking handled by this: https://zuul-ci.org/docs/zuul/admin/drivers/zuul.html?highlight=parent%20change%20enqueued  ?19:46
*** sshnaidm|afk is now known as sshnaidm|off20:00
*** pcaruana has quit IRC20:14
*** pcaruana has joined #zuul20:27
manjeetscorvus, Hi ping quick question regarding nodepool20:46
manjeetshttps://github.com/openstack-infra/zuul/blob/master/doc/source/admin/examples/etc_nodepool/nodepool.yaml20:46
manjeetsinstead of using some provider cloud, would I be able to use a pool of baremetal machines (ip list)20:47
*** j^2 has joined #zuul21:35
*** hashar has quit IRC21:45
*** pcaruana has quit IRC21:55
ianwmanjeets: i think what you want is the static driver -> https://zuul-ci.org/docs/zuul/admin/nodepool_static.html22:38
manjeetscool so using ip as name + ssh keys might work23:09
manjeetsthanks ianw23:09
*** j^2 has quit IRC23:22
ianwmanjeets: yes, although do you plan to re-use the nodes?23:39
manjeetsianw, so idea is to have a cleanup script for cleanning the node after test run as part of job itself ?23:40
manjeetswould that work ?23:40
ianwmanjeets: i think it depends on what you're doing; i think yes that can work, but you can also envisage scenarios where jobs go wrong a destroy machines or do things the cleanup doesn't handle23:43
ianwfor example, well before secret support, we used static nodes with secret keys deployed on them for things like signing23:44
ianwif it's complete general purpose workloads, however, maybe you want to investigate something like using ironic to manage your baremetal and working via that23:45
manjeetsianw we need baremetal nodes for hardware testing, other thing that could be done is run cleanup before starting a job and after job, but something crazy can happen in b/w23:45
manjeetsin that case I think using cloud with ironic could be a option too ?23:46
ianwtbh, it sounds like that's probably where you want to end up.  it seems like it would be nice for your environment to be managing your hardware like that23:47
ianwbut, certainly no reason not to start with static nodes and work from there23:47
manjeetsianw I'm kind of learning the whole pipeline, the initial goal is get at run of job atleast once on baremetal and then if that works fine, we just need to change the nodepool config to replace static nodes with ironic cloud ? if I understand it right23:49
ianwwell yeah, i mean there's a lot going on so minimising your moving parts to start is probably helpful :)23:50
ianwif you get the zuul part up and running, taking the changes and running the job as you want on a static node that seems like a good start23:51
ianwthen really it shouldn't be too different to get that plugged into a cloud that's managing your hardware via ironic23:52

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