Odd_Blokerharper: Did you see my reply to your comment: https://github.com/canonical/cloud-init/pull/453/files#r444872750 ?18:06
Odd_Bloke(That's going to conflict with the next networking PR I want to put up, so I'd like to resolve it sooner rather than later, if possible. :)18:08
rharperOdd_Bloke: sorry, not yet;  today's crazy at the house; I'll look at it shortly18:08
meenaOdd_Bloke: i think i'll need to see a sample PR doing one of the functions before i can sensibly contribute…18:50
meenai expect that startig mid-august when my daughter's in child-care, so i hope i'll have more… brain left18:52
Odd_Blokemeena: I'm not proposing it yet (because it conflicts with the PR I'm discussing with Ryan), but I've pushed up an example that you can take a look at: https://github.com/OddBloke/cloud-init/compare/net...net318:58
Odd_Blokemeena: Obviously that hasn't been reviewed at all, so I would still hold off on using it as a template for now.18:58
meenaOdd_Bloke: i'm surprised that is_physical isn't used anywhere in cloudinit/net/__init__.py https://github.com/OddBloke/cloud-init/compare/net...net3#diff-ce643f3a459f3fe8a4a910905190bf7719:00
meenaalso, wow, this is… https://github.com/OddBloke/cloud-init/compare/net...net3#diff-a708ab28fe2a7127c07c0800289b6bd4R68-R76 wow.19:02
meenaOdd_Bloke: so, basically, after the first thing goes thru, most consumers will have a distro parameter where it's needed and for those that don't, we can use that first PR as example.19:05
rharperOdd_Bloke: commented on the PR and the bug19:05
meenarharper: which ones?19:07
meenaah yeah, i keep thinking that all important PRs / issues are the ones in which i'm "participating" in on github19:14
rharperhehe, it doesn't help that there are a lot of threads on this refactor;  thank you for working on it19:18
Odd_Blokerharper: Thanks!  Would you be able to give me a +1 there too?19:18
rharperoh, right,19:21
meenarharper: if by working on it, make Odd_Bloke work on it, then, yes, you're perfectly welcome.19:26
meenaif i had managed to make him do this a year or two ago, we could've been long done already19:29
blackboxswfalcojr: or lucasmoura for SRU review if you get a chance https://github.com/cloud-init/ubuntu-sru/pull/134. either one of you19:30
blackboxswit's netplan vs eni prioritization on x b e f19:30
blackboxswand I'm hitting SRU review queue19:31
blackboxswfalcojr: and lucasmoura and if you approve any of the PRs in ubuntu-sru or qa-scripts or uss-tableflip repos, I think you have commit rights, so feel free to squash merge that stuff in github UI if you want.19:32
blackboxswthanks for all the reviews19:32
blackboxswman all those merges feel good though19:34
Odd_Blokemeena: OK, you can do some work and review https://github.com/canonical/cloud-init/pull/457 ;)19:43
Odd_Bloke(If you have time, ofc!)19:43
meenai have time, i'll checkout your pr, and see if i can cotribute a BSD function too ;)19:44
Odd_BlokeThanks! :)19:47
meenaOdd_Bloke: somebody in #freebsd helped me come up with a way to list all the physical interfaces: http://ix.io/2q5e20:39
meenaif you're thinking, isn't this the worst thing i've ever seen, well, yeah, this is unix, baby!20:40
chillysurferi remember last year somebody from upstream showing me scripts that are used for sru validation on cloud providers? i think it was on github20:59
chillysurferanybody know where that could be?20:59
meenachillysurfer: sru validation is happening rn21:04
meenachillysurfer: https://github.com/cloud-init/ubuntu-sru/ ?21:06
blackboxswthnk meena , and that's the right repo21:07
blackboxswI think we are wrapping up the last cloud SRU validtions now. expectation is to finally publish tomorrow21:07
rharperblackboxsw: I didn't yet get to the nfs mount spec verification;  did that get covered?21:17
blackboxswrharper: not yet, but if need be we can grab it.21:18
blackboxswI think the last remaining issue is oracle SRU, an email ping to VMware, and the nfs21:18
rharperblackboxsw: I can work on it but I don't want to hold you up if you needed it done by tonight for tomorrow release21:21
chillysurferawesome thanks!21:22
* meena ➡️ 🛌21:24
blackboxswok lucasmoura done https://github.com/cloud-init/ubuntu-sru/pull/134 with re-run of netplan vs eni test21:27
lucasmourablackboxsw, approved21:32
blackboxswrharper: we are still waitingon a CDO qa run to compete. job is queued, but that and Oracle SRU verification are last remaining bits21:38
blackboxswcomplete even21:38
rharperblackboxsw: hrm;  a quick test with nfs and the default cloud images don't have an nfs client installed, so mounting something with a colon for remote system is complicated locally;  what do we want to do?  THe original bug mentions EC2 nfs (which is type efs or something and maybe the ec2 image has correct client package support in it)21:55
* blackboxsw is reading up on efs setup21:59
blackboxswin aws21:59
* blackboxsw is looking at https://docs.aws.amazon.com/efs/latest/ug/wt1-create-efs-resources.html22:01
blackboxswlooks like their docs are for non-ubuntu (yum install nfs-utils) etc. so image doesn't have utils baked in22:03
rharperok, so looks like it's reboots and cloud-init clean22:04
rharperI can do that22:04
rharperwhat a PITA22:04
blackboxswyeah, pita. rharper we are going to talk about and float new SRU verification process to the mailinglist I think after a retrospective on this round of fun.22:05
blackboxswso watch for that email/proposal folks because we want to reduce the cost of this effort if we can without sacrificing quality.22:05

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