snadgedoes anyone have a fix for "stages.py[ERROR]: Unable to render networking. Network config is likely broken: No available network renderers found. Searched through list: ['eni', 'sysconfig', 'netplan']"02:30
snadgeF29/F30/F31 all do it.. ci-info shows the eth0: device as down.. then about 5 minutes later, the network finally comes up02:31
snadgethis is on openstack, fedora cloud images.. there's references to bugs from like a year ago, that as far as I can tell don't seem to be resolved02:31
smoserwell... Odd_Bloke or rharper  or blackboxsw . i'm not able to recreate this right now outsize of lxc-pstart03:07
smoser http://paste.ubuntu.com/p/pCwHJgNdyv/03:07
smoserat first i thought it had soemthing to do with the symlink /sbin/ -> usr/sbin gettinb broken03:07
smoserand then i thought it had something to do with psuedo-init deleting itself03:08
smoserbut both of those seem red herring.03:08
snadgeim guessing you guys are all ubuntu devs.. i just want to get some traction on this: https://bugzilla.redhat.com/show_bug.cgi?id=159606203:12
ubot5bugzilla.redhat.com bug 1596062 in distribution "Fedora-Cloud-Base - qcow2 - eth0 is not UP" [Unspecified,Closed: rawhide]03:12
snadgeand the "fix" .. https://pagure.io/fedora-kickstarts/pull-request/40603:13
snadgei realise this is a fedora "issue" .. but i dont seem to be getting much joy from them at the moment.. and this bug is over a year old now03:13
snadgecloud-init obviously supports the sysconfig method.. but this is no longer working since NetworkManager has been removed/deprecated03:47
=== logan_ is now known as logan-
smosersnadge: removed/deprecated in favor of what ?14:03
Odd_Blokeblackboxsw: Thanks for the pointers!14:52
Odd_Blokesnadge: The output of `cloud-init collect-logs` on a failing system would be very helpful for triaging the issue. :)14:53
meenaOdd_Bloke: thanks for finding the refs get_devicelist()14:58
powersjOdd_Bloke, \o/ thanks for the reviews15:53
Odd_Blokepowersj: blackboxsw: rharper: One thing I would like to do is enable some branch protection rules for master.  You should all, I believe, be able to see the options at https://github.com/canonical/cloud-init/settings/branch_protection_rules/new15:59
Odd_BlokeSpecifically, at minimum, I'd like to require PR reviews and status checks.16:00
Odd_BlokeAnd probably "Include administrators" too, as we're all administrators so not checking that wouldn't actually enforce these for anyone but smoser. :p16:01
powersjyou mean you don't want me committing directly to master?16:01
Odd_BlokeI plead the fifth your honor16:02
smosergit push --force16:03
powersjthere is an option to allow force pushes, so maybe it blocks that too?16:04
powersjOdd_Bloke, what is up with the xx-current branch?16:07
powersjBranch protection rules in place for master and ubuntu/* branches16:07
powersjok and squash and merge is now the default16:12
powersjOdd_Bloke, ^16:13
Odd_Blokepowersj: Nice, thank you.  I modified the master rule so that Travis was one of the required checks.16:17
powersjOdd_Bloke, did you see my ping above about xx-current?19:03
Odd_Blokepowersj: Oh, oops.  IIRC, our release process suggests using that as a tag name before using the scripts which automate merging.19:21
Odd_BlokeSo I guess it accidentally ended up pushed.19:21
powersjOdd_Bloke, one we can delete?19:22
blackboxswpuppet manual SRU verification is up https://github.com/cloud-init/ubuntu-sru/pull/7619:24
Odd_Blokepowersj: Yep.19:28
meenaOdd_Bloke: i think i have an idea for get_devicelist() oh19:35
meenai've been down this path before…19:36
meenais there a reason why cloudinit/net/__init__.py contains all those OS specific functions? other than the fact that no other OS mattering except for Linux? :P19:37
Odd_BlokeI believe the networking code was based on code from curtin, which is an installer that specifically targets Linux (and for a while specifically targetted Ubuntu).  So I wouldn't be surprised if it's just an artifact of that, rather than a specific decision.19:39
Odd_Blokerharper might have more insight.19:39
blackboxswyeah I'd agree with that, there have been a couple of discussions about trying to better generalize os-specific modules out of net/__init__.py but not enough bandwidth to make that refactor move19:41
meenaanyone remember this peace of work? https://code.launchpad.net/~i.galic/cloud-init/+git/cloud-init/+merge/358228 ?19:42
blackboxswlike a cloudinit/net/<distro>.py or something that would be sourced in net/__init__.py function implementation. Then it'd likely allow some of the specific Distro classes to drop some of their custom methods19:42
meenablackboxsw: i was trying to, but rand out of steam, esp wrt testing, because, well, i didn't know enough about testing … or, python for that matter.19:43
meenaoh, gosh, i forgot about net vs netinfo ~_~19:44
Odd_Blokemeena: Oh, I hadn't seen that, it predates me being on the cloud-init team.19:44
blackboxswwowo I forgot about that meena19:44
meenablackboxsw: i'm sure we can reuse some /learnings/ from that, if not much of the code.19:45
meenaOdd_Bloke might have better ideas on how to get around recursive … thingies.19:45
blackboxswyeah definitely meena, I knew the 'conversation' had happened, I had forgotten so much work was put into a proposal on that19:45
meenayou're … welcome… maybe.19:47
meenaOdd_Bloke: so, not Old_bloke then. i did think that i didn't remember you from … last year? yeah, it's just a year ago.19:51
meenaa year feels a lot longer with a baby19:51
Odd_Blokemeena: I've been doing cloud-init stuff for ~5 years, but I was on the Ubuntu Cloud Image team until early this year.19:56
meenalet's see the people from Ubuntu i've had irc contact with: Ubuntu Server team, LXD, cloud-init. so, we probably didn't cross paths until now19:57
Odd_BlokeSo my role didn't involve regular upstream work on cloud-init until then.19:57
meenaokay, so, who wants to help me sketch out a design for cloudinit/net** that i a little bit more suitable for… things…20:05
blackboxswpowersj: or Odd_Bloke some SRU script cleanup https://github.com/CanonicalLtd/uss-tableflip/pull/26 and https://github.com/cloud-init/ubuntu-sru/pull/7720:20
blackboxswthey are related20:20
blackboxswto get maas testing logs20:20
blackboxswmeena: that's a bit lift, maybe it's worth either writing up a quick spec that folks can comment on and send it to the mailing list?20:22
blackboxswwe've done that a few times in the past and it'll give folks a chance to think about the approach20:22
meenablackboxsw: i was hoping someone could help draft that in an etherpad20:23
blackboxswwe've used https://hackmd.io/ for cross company collaboration or shared google docs20:23
blackboxswor etherpad :)20:23
meenai forget what the cool etherpad is called, but i use one.20:23
meenai'd much prefer hackmd to … anything from google.20:24
blackboxswmeena: a github gist could work too I suppose20:26
meenacan't work on that at the same time tho, so the back and fork might be very annoying20:27
meenaback and fork — see what i did there??20:27
blackboxswpowersj: Odd_Bloke smoser rharper just added autolinks to our github cloud-init project. so any message, comment or PR description containing LP:<bugid> will auto link to launchpad21:14
Odd_Blokeblackboxsw: Oh, nice.  How is that configured?21:15
powersjblackboxsw, ^ how did you do that?21:15
meenawoooooow. i just ran dead over the repo21:15
blackboxswwhat this means though is that during our squash merge of cloud-init PRs we'll need to remember to restructure that last line to LP: #<bugId>21:15
blackboxswpowersj: Odd_Bloke settings tab, autolink references https://github.com/canonical/cloud-init/settings/key_links21:15
blackboxswwe ca have many21:15
blackboxswif needed, but the 'prefix' can't contain whitespace or # :/21:15
meenablackboxsw: good work!21:15
blackboxswI was tired of cut-n-paste into launchpad :)21:16
powersjwhy no #?21:16
powersjor whitespace :)  since the usual would be "LP: #123"21:16
ubot5Launchpad bug 123 in Launchpad itself "There's no direct way to see the project info when translating it" [Medium,Fix released] https://launchpad.net/bugs/12321:16
blackboxswwe need to ask,request github for that feature. only alpha-nums and :21:16
blackboxswor hyphen21:16
blackboxswbut yeah might be a feature request for github21:17
meenadead: https://gist.github.com/2c7b341c00fc10024605234d51698efc21:19
Odd_Blokeblackboxsw: So I'm not sure we should change how we're formatting things just to get autolinking to work.21:21
Odd_BlokeBecause I think stuff in Launchpad uses the "LP: #" prefix for auto-relating things, and we'd break that.21:22
Odd_Bloke(I'm not sure we _shouldn't_, either, to be clear. :)21:22
blackboxswOdd_Bloke: I'm not suggesting that, since we are already in the process of manually reviewing during squash merge, We'd have to fix the format back in the commit message before merge21:22
meenai don't know how to make more words, https://md.hecke.rs/z17JGX4HT4emH5jTEhMuTA?both21:22
Odd_BlokeOh, I see what you mean.21:22
blackboxswOdd_Bloke: right I agree, I was just wondering we we can do something or automate that merge to fix before we squash merge. But, still have the facility for easy reviews by autolinks21:23
meenaalso, you could have travis check the commit message before (merge?)21:23
Odd_BlokeThe merge message is written into a textbox and the merge happens immediately, I don't think we have a linting window available to us there.21:26
meenahrm… yeah, no, you'd only get a build failed on master after a green build on the pr21:27
blackboxswI've filed a feature request with github21:28
Odd_BlokePotentially using a mergebot like bors, which pulls out of the PR description, would give us a chance.  But then we would need to put the correct formatting into the PR description, so we wouldn't get auto-linking.21:29
meenaso, anyone wanna rewrite https://md.hecke.rs/z17JGX4HT4emH5jTEhMuTA?both to be less blah blah?21:31
Odd_Blokemeena: I just scanned through, and I'm a big +1 on a class-based approach.21:32
Odd_BlokeI'll take a closer look in a few minutes.21:32
meenaOdd_Bloke: then, if you somehow can, maybe suggest how to do that…21:33
meenado we have any api promises to keep towards 3rd party folks, or is everything upstreamed to us?21:33
blackboxswmeena: we try to avoid thrashing class module api in the event that users are plugging into cloud-init with modules that are not upstreamed. We know of a few cases where modules are kept out of upstream due to privacy issues. Mostly though, that'd generally be in the DataSource and config module camp though. not specifically someone calling into cloudinit.network modules that I know of21:36
blackboxswthere are promises that users can drop in there datasource modules and config modules and as long as the module's API surfaces the same entry points, then cloud-init will try to use it. I really don't think it's the same for our net modules.21:37
blackboxswI was thinking primarily of cloud-init's find_source and find_modules stuff in stages.py21:38
meenablackboxsw: Azure uses is_up(); DigitalOcean uses get_devicelist() and OpenNebula and DigitalOcean use is_physical()21:38
meenato name a few from cloudinit.net21:39
blackboxswmeena: internal to cloud-init if we overhaul the net functions we would make sure to update tip of DO Azure and OpenNebula to use those new methods appropriately.21:39
meena*nod *nod21:39
blackboxswhaving a cohesive working tip of cloud-init should suffice I would think21:39
blackboxswfor some reason I can't register to md.hecke.rs21:44
blackboxswinvalid email claims21:45
meena(do you need to register?)21:46
meenablackboxsw: i can move it elsewhere if you want21:46
blackboxswyeah it was rejecting me on registration attempt21:46
blackboxswnot signin21:46
blackboxswobjections to hackmd.io?21:47
meenahttps://hackmd.io/3-YBj1t9TAeKhmfLBQUjXQ?both ← signed in users can edit21:57
Odd_Blokeblackboxsw: rharper: https://github.com/cloud-init/ubuntu-sru/pull/7822:03
meenai'll be  heading towards bed…22:05
Odd_Blokemeena: Sleep well!22:06
meenabed != sleep.22:07
meenait just means i'm reading stuff on my phone rather than trying to write on my laptop.22:07
Odd_Blokemeena: Enjoy being in bed!22:07
meenai do enjoy that, what the heck i'm doing on the couch??22:07
Odd_Blokemeena: 5pm on Friday is the wrong time for me to be thinking about this, so apologies if I don't make sense.  But I think a good way to work out how a class-based approach could work would be to select a single function in cloudinit.net which is Linux-specific (perhaps our old friend get_devicelist), and move that to a method on a Linux-specific network class.  Then see what refactoring you have to22:22
Odd_Blokeperform to get _Linux_ networking functioning again.22:22
Odd_BlokeThat would give us an idea of what a class-based approach would look like, without requiring (a) refactoring _everything_, or (b) working out the right behaviour on FreeBSD.22:24
Odd_Bloke(I'm not saying we would necessarily land such a change, either, it may just serve as an investigative spike.)22:25
meenawhat Odd_Bloke is says.22:25
Odd_Blokemeena: Not sure I follow?  (I guess it's even later for you. :p)22:27
meenaI'm agreeing with you, and yes it is22:27
meenaalso, i thought blackboxsw was saying that first part, that got me notified here,,,22:28
meenaso, currently the easiest way to know which system you on is system_info() unless someone overrode that22:30
Odd_Blokemeena: I'm heading out for the weekend now, I'm afraid.  I'm working all of next week though, so we can put our heads together on Monday?22:31
meenaof course22:31
Odd_BlokeGreat. :)  Have a good weekend!  (And don't stay up too late hacking. ;)22:32
meenaI'll just read horrible news or boring papers22:32
meenaGoneri: https://hackmd.io/3-YBj1t9TAeKhmfLBQUjXQ thinking exercise for the weekend22:38

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