| *** gouthamr has joined #zuul | 00:03 | |
| *** mmedvede_ has joined #zuul | 00:24 | |
| *** mmedvede has quit IRC | 00:24 | |
| *** mmedvede_ is now known as mmedvede | 00:24 | |
| *** gcb_ has quit IRC | 01:00 | |
| *** harlowja has quit IRC | 01:06 | |
| tristanC | SpamapS: dmsimard: ok, fixing the last issue and adding support for custom security-group as using the default one is not recommended | 01:52 |
|---|---|---|
| *** hughsaunders has quit IRC | 02:29 | |
| dmsimard | lots more fixes actually :D | 03:18 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: Implement an Amazon EC2 driver https://review.openstack.org/535558 | 03:21 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool master: Implement an OpenShift resource provider https://review.openstack.org/570667 | 04:52 |
| openstackgerrit | Jarek Lukow proposed openstack-infra/zuul master: Add missing parentheses to title invocation https://review.openstack.org/595361 | 05:09 |
| *** dmellado has joined #zuul | 05:27 | |
| *** nchakrab has joined #zuul | 06:24 | |
| *** pcaruana has joined #zuul | 06:33 | |
| *** hashar has joined #zuul | 06:45 | |
| *** AJaeger has quit IRC | 06:58 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: WIP web: rewrite interface in react https://review.openstack.org/591604 | 07:15 |
| *** jpena|off is now known as jpena | 07:36 | |
| *** electrofelix has joined #zuul | 07:40 | |
| odyssey4me | corvus: yeah, sorry - we've not done anything like that either | 08:37 |
| *** jpena is now known as jpena|lunch | 11:03 | |
| *** ssbarnea has joined #zuul | 11:26 | |
| *** jpena|lunch is now known as jpena | 12:09 | |
| *** pcaruana has quit IRC | 12:16 | |
| *** pcaruana has joined #zuul | 12:16 | |
| *** nchakrab has quit IRC | 12:29 | |
| *** rlandy has joined #zuul | 12:31 | |
| *** samccann has joined #zuul | 13:43 | |
| *** ianychoi has quit IRC | 13:59 | |
| *** ianychoi has joined #zuul | 14:00 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Add write-inventory role https://review.openstack.org/595378 | 15:27 |
| *** pcaruana has quit IRC | 15:36 | |
| *** harlowja has joined #zuul | 16:17 | |
| *** jpena is now known as jpena|off | 16:25 | |
| SpamapS | tristanC: the aws nodepool driver isn't working for me, it's exploding with the same error about not being able to set the pool attribute. If you have a few minutes later today, I could use some tips. | 16:28 |
| jlk | mordred: relevant to your interests, Ansible folks are looking for feedback on https://github.com/ansible/ansible/pull/42163 | 16:29 |
| *** harlowja has quit IRC | 16:40 | |
| tristanC | SpamapS: seems like you are missing the edits of L69 of https://review.openstack.org/#/c/535558/7..8/nodepool/driver/aws/config.py | 17:05 |
| tristanC | SpamapS: are you using PS8? | 17:05 |
| SpamapS | I thought I was | 17:06 |
| SpamapS | but I'll double check | 17:06 |
| SpamapS | It's possible there was some disconnect between my image build/push/deploy | 17:06 |
| tristanC | fwiw, we ran some zuul-jobs on aws with PS8, you need to make sure the default vps authorize inbound ssh, or you can create a new vps and set the security-group-id and subnet-id in the pool settings | 17:08 |
| SpamapS | tristanC: I actually did that... have screwed that up too many times on openstack. :) | 17:10 |
| pabelanger | can I loop back tox with sudo discussion from yesterday, https://review.openstack.org/593150/ gives us a new tox-with-sudo job. Which people can use or we even create an additional tox-functional job | 17:18 |
| corvus | jlk: ah that's interesting. we are using command to run ansible-playbook often enough that mordred was considering writing a module. i'm sure he'll want to discuss with you, but he is in some combination of travel/conference/vacation until the second week of december. | 17:19 |
| corvus | jlk: wow. i mean second week of *september* | 17:20 |
| pabelanger | wow, that would have been an awesome vacation | 17:20 |
| corvus | i'm apparently ready for a long vacation myself :) | 17:20 |
| pabelanger | ++ | 17:21 |
| *** hashar is now known as hasharAway | 17:22 | |
| Shrews | pffft... like he'd be able to stay away until december | 17:44 |
| Shrews | i have my doubts about september | 17:44 |
| *** electrofelix has quit IRC | 17:48 | |
| *** sshnaidm is now known as sshnaidm|off | 17:54 | |
| *** weshay is now known as weshay|rover | 17:57 | |
| SpamapS | jlk: I commented. Looks pretty lame IMO. | 18:16 |
| SpamapS | If it did *anything* to help you deliver inventory/playbooks/roles I'd feel differently | 18:16 |
| SpamapS | but it's just a silly wrapper on the anisible-playbook CLI. | 18:17 |
| jlk | nod | 18:38 |
| jlk | that's the kind of feedback they're looking for | 18:38 |
| jlk | We can discuss what we WANT out of something like that, and reject the current implementation | 18:38 |
| jlk | corvus: I think we'll both be at the Ansiblefest in Austin, which is where we planned to talk about such things. | 18:38 |
| pabelanger | SpamapS: so far, I've relied on tox to manage nested ansible dependancy from ansible, then wrapper to ansible-galaxy to install required modules. But agree, would be nice to have something native in ansible | 18:41 |
| *** pcaruana has joined #zuul | 18:52 | |
| *** pcaruana has quit IRC | 19:09 | |
| rcarrillocruz | https://ansible-runner.readthedocs.io/en/latest/ | 19:47 |
| rcarrillocruz | Not a module, but nevertheless a neat interface to ansible | 19:47 |
| rcarrillocruz | In case folks are not aware | 19:47 |
| *** panda has quit IRC | 20:05 | |
| dmsimard | SpamapS: you're trying out the aws driver too ? | 20:08 |
| dmsimard | There's some subtle but sort of annoying things to handle that are missing.. openstack is much better in some regards | 20:09 |
| dmsimard | For example, some flavors in aws come with an ebs volume and boto doesn't enable deletion of the volume on instance termination -- but "delete on termination" is an argument of the volume, not the instance.. so you need to override the blockdevice mapping https://boto3.readthedocs.io/en/latest/reference/services/ec2.html#EC2.ServiceResource.create_instances | 20:10 |
| dmsimard | and then there's the size of the volume too, that's a property of the volume, not the flavor | 20:11 |
| dmsimard | so I don't think we can get away without providing support for customizing those | 20:11 |
| *** elyezer_ has joined #zuul | 20:26 | |
| *** elyezer has quit IRC | 20:29 | |
| *** samccann has quit IRC | 21:03 | |
| *** panda has joined #zuul | 21:34 | |
| SpamapS | dmsimard: I am! | 21:57 |
| SpamapS | and in fact my pods were still running ,7 not ,8 | 21:57 |
| dmsimard | hacking on a #9 right now :p | 21:57 |
| SpamapS | ok good cause ,8 fails in a different way | 21:58 |
| SpamapS | http://paste.openstack.org/show/728713/ | 21:59 |
| dmsimard | SpamapS: the pool needs the security-group-id and subnet-id parameters | 21:59 |
| SpamapS | but my config may be wrong | 21:59 |
| SpamapS | I see yeah | 21:59 |
| dmsimard | SpamapS: what's your config like ? the example config here is good but misses security-group-id and subnet-id https://review.openstack.org/#/c/535558/8/nodepool/tests/fixtures/config_validate/good.yaml | 21:59 |
| SpamapS | the key-name is the replacement of the zuul-public-key | 21:59 |
| dmsimard | and yeah that | 22:00 |
| SpamapS | fairly straight forward to fix.. kk | 22:00 |
| * SpamapS is doing two crazy things at once | 22:00 | |
| SpamapS | 1) deploying the aws driver | 22:00 |
| SpamapS | 2) deploying zuul into an EKS k8s cluster | 22:00 |
| SpamapS | the latter is going well now | 22:01 |
| SpamapS | the former has been challenging. ;) | 22:01 |
| dmsimard | sounds like fun | 22:01 |
| SpamapS | we need to talk about your definition of fun... | 22:05 |
| dmsimard | :D | 22:06 |
| dmsimard | What I've got seems to not be horribly broken, need to ask tristanC if he minds that I push new patchsets | 22:07 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul-jobs master: Add write-inventory role https://review.openstack.org/595905 | 22:39 |
| *** threestrands has joined #zuul | 22:53 | |
| *** rlandy is now known as rlandy|brb | 23:24 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!