/srv/irclogs.ubuntu.com/2020/08/27/#cloud-init.txt

AnhVoMSFTHi folks, under what circumstance can we run into this error "IsADirectoryError: [Errno 21] Is a directory: '/var/lib/cloud/instance'"12:05
AnhVoMSFTfull output: https://termbin.com/385u12:06
smoserAnhVoMSFT:i thought something went in very recently that intended to fix that.12:23
smoserbut fwiw, you have errors before then12:23
smoserAnhVoMSFT: 0755cff078d5931e1d8e151bdcb84afb92bc0f02.12:25
AnhVoMSFTah thanks - that indeed went in recently12:31
AnhVoMSFTyep, looking at the log I did see an error during writing ssh key12:34
smoseronce htat gets created as a directory, then its done, it wont resolve itself.12:34
smosercloud-init clean will probably fix it, or rm -Rf /var/lib/cloud/instace12:35
AnhVoMSFThow did it actually get created as a directory in the first place?12:35
smosersee the bug for one way12:36
smoserthere may be other ways that it gets created, though. so it could be only same symptom12:37
AnhVoMSFTI have about 10 different cloud-init log with this issue, looking through some of them, the pattern seems to be an error during ssh-authkey-fingerprints12:39
AnhVoMSFTKeyError: "getpwnam(): name not found: 'ubuntu'"12:39
AnhVoMSFTnot related to the last issue, there's another interesting one12:47
AnhVoMSFTFile "/usr/lib/python3/dist-packages/cloudinit/net/dhcp.py", line 162, in maybe_perform_dhcp_discovery12:47
AnhVoMSFT    dhclient_path = subp.which('dhclient')12:47
AnhVoMSFTAttributeError: module 'cloudinit.subp' has no attribute 'which'12:47
smoserAnhVoMSFT: it feels like maybe you have a broken install (or maybe an upgrade or something?)...12:59
smosercloudinit.subp definitely does have which in current13:00
smoserbut it did move recently13:00
AnhVoMSFTthese are just from random cloud-init.log that were collected as part of our telemetry for failed deployments so I don't have more context into that though. The error just seems odd because like you said, cloudinit.subp definitely has which13:39
falcojrblackboxsw_: my brain is in packaging hell right now14:32
falcojrhttps://github.com/canonical/cloud-init/commit/66e114a660c53400e389f119781f378311b65108#diff-4df3321932810ca4e004e72e5587ff28R9514:32
falcojrhow are we getting the caplog error if we're including that?14:32
falcojralso, how did the upstream daily changelog get the latest SRU changelog when the non-daily branch doesn't have it yet?14:38
blackboxsw_minor branch for you falcojr https://github.com/canonical/cloud-init/pull/55815:15
blackboxsw_this should resolve the error because caplog is non-std on xenial's version of pytest15:15
falcojrblackboxsw_: how's this different from the other line I linked?15:16
blackboxsw_reading scrollback15:20
blackboxsw_ahh falcor, so that package dependency path differs from our daily build recipe debuild -S + sbuild-it path which I believe uses the hard/static file from ubuntu/xenial/debian/control during the build process15:32
blackboxsw_falcojr: thx for the review. https://github.com/canonical/cloud-init/pull/558 landed for ubuntu/xenial test pkg deps16:41
blackboxsw_falcojr: looks like master moved again with another commit today. So, we are ready to new-upstream-snapshot 371b392ced518e45be51089b6a67b362957b1dba to each ubuntu/X|B|F branch now16:44
blackboxsw_if you could re-fresh your existing ubuntu/* PRs then we should be good to go16:44
blackboxsw_falcojr: lucasmoura I just kicked off a daily build recipe for xenial to see that it 'works' with current ubuntu/xenial -> ubuntu/daily/xenial branches https://code.launchpad.net/~cloud-init-dev/+recipe/cloud-init-daily-xenial16:54
blackboxsw_once we know the builder works, we can land falcor's new-upstream-snapshots16:54
blackboxsw_knowing that it didn't introduce regressions (which it shouldn't)16:54
blackboxsw_falcojr: case and point about revno in the daily build recipe not quite working as we hoped with occasional build collisions https://launchpadlibrarian.net/495249390/upload_2635183_log.txt17:33
blackboxsw_which seem to only happen when I manually click the build now button in launchpad17:34
beantaxiI notice that cloud-init 20.3 has been announced as Released, and that there is a package for Groovy, but I do not see a package for Bionic or anything else.18:00
beantaxiI am a novice at all this ... but is the idea that the cloud-init team is the cloud-init team, and it releases cloud-init code, and then different teams do the packaging for different distros? Even within Canonical? (that would make perfect sense, but that does not mean it's correct)18:01
blackboxsw_beantaxi: in order for cloud-init to publish into stable Ubuntu releases we have to follow this process for ubuntu https://wiki.ubuntu.com/CloudinitUpdates18:01
blackboxsw_we call this a Stable Release Update (SRU)18:01
blackboxsw_falcojr: and I are working today to resolve and queue a -proposed upload of 20.3 into those ubuntu releases18:02
blackboxsw_beantaxi: expectation is that once we queue uploads, the cloud-init updated packages will be available in the xenial-proposed bionic-proposed apt pockets and will be testable by anyone following these steps https://cloudinit.readthedocs.io/en/latest/topics/debugging.html#stable-release-updates-sru-testing-for-cloud-init18:03
blackboxsw_we'll send out an email to the list as well about this SRU.18:03
blackboxsw_generally when we do an upstream release to groovy 20.3 our team tries to queue an SRU into the stable releases within the next week.18:04
beantaxiGotcha ... so it's still packaged by cloud-iniit, but there are extra steps for making a change to a stable release. That makes perfect sense.18:04
blackboxsw_that SRU testing minimally takes 7 days, but practically takes average of about 1018:04
blackboxsw_yep just so stable releases get more verification (and don't break existing users)18:04
beantaxiIs it generally safe, to grab a source release and install from source, over top of an apt package? (for cloud-init anyway; I imagine there's not a generally correct answer)18:04
blackboxsw_beantaxi: to watch progress on cloud-init SRUs you can subscribe to the SRU process bug we create18:05
blackboxsw_https://bugs.launchpad.net/ubuntu/+source/cloud-init/+bug/189306418:06
ubot5Ubuntu bug 1893064 in cloud-init (Ubuntu Focal) "sru cloud-init (20.2-45 to 20.3-0) Xenial, Bionic, and Focal" [Undecided,New]18:06
blackboxsw_we will update that bug attaching verification logs for every stable release we validate18:06
blackboxsw_and eventually the Xenial/Bionic/Focal items in that bug will go to Fix Released and comments added to that affect18:06
blackboxsw_details on this will come out in the SRU email that will be sent to the mailinglist18:07
blackboxsw_I need to update the IRC topic since we officially "released" 20.3 to groovy 2 days ago18:08
beantaxiThanks! I just subscribed. Should be interesting to get a feel for the process.18:08
=== blackboxsw_ changed the topic of #cloud-init to: pull-requests https://git.io/JeVed | Meeting minutes: https://goo.gl/mrHdaj | Next status meeting Aug 25 16:15 UTC | 20.3 (Aug 25) | https://bugs.launchpad.net/cloud-init/+filebug
blackboxsw_excellent18:08
=== blackboxsw_ changed the topic of #cloud-init to: pull-requests https://git.io/JeVed | Meeting minutes: https://goo.gl/mrHdaj | Next status meeting Aug 25 16:15 UTC | 20.3 (Sep 8) | https://bugs.launchpad.net/cloud-init/+filebug
blackboxsw_also updated the topic for next cloud-init community status meeting: "Next status meeting Aug 25 16:15 UTC "18:09
=== blackboxsw_ changed the topic of #cloud-init to: pull-requests https://git.io/JeVed | Meeting minutes: https://goo.gl/mrHdaj | Next status meeting Sept 816:15 UTC | 20.3 (Aug 25) | https://bugs.launchpad.net/cloud-init/+filebug
blackboxsw_let's try that again: Next status meeting Sept 816:15 UTC18:10
=== blackboxsw_ changed the topic of #cloud-init to: pull-requests https://git.io/JeVed | Meeting minutes: https://goo.gl/mrHdaj | Next status meeting Sept 8 16:15 UTC | 20.3 (Aug 25) | https://bugs.launchpad.net/cloud-init/+filebug
blackboxsw_... ok I really need to lay off the coffe18:10
blackboxsw_e18:10
falcojrblackboxsw_: is there something I can do to help figure out why that upload was rejected?18:38
powersjfalcojr, where are you uploading to?18:54
powersjif it is a ppa you should get an email with a reason18:55
falcojrblackboxsw_ was doing it, so I'm not sure :)18:55
blackboxsw_falcojr: checking upload18:56
blackboxsw_falcojr: you mean groovy?18:56
blackboxsw_powersj: falcojr oooh you mean the daily recipe build for xenial that I linked18:56
falcojrI was asking about18:56
falcojr"falcojr: case and point about revno in the daily build recipe not quite working as we hoped with occasional build collisions https://launchpadlibrarian.net/495249390/upload_2635183_log.txt"18:56
falcojryeah18:56
blackboxsw_right sorry. I'm context switching a bit today and not doing that very well.18:57
blackboxsw_so the daily build recipes show here https://code.launchpad.net/~cloud-init-dev/+recipe/cloud-init-daily-xenial and https://code.launchpad.net/~cloud-init-dev/+recipe/cloud-init-daily-bionic18:57
blackboxsw_you can dive into specific upload log failures clicking int each build18:59
blackboxsw_if needed.18:59
rharperfalcojr: blackboxsw_:  no one uploads to daily pocket, for SRU we upload to the proposed pocket of the cloud-init-dev ppa ;18:59
rharperhttps://launchpad.net/~cloud-init-dev/+archive/ubuntu/proposed18:59
blackboxsw_right, the daily recipe just has occasional upload failures into the daily PPA because the specific version of the deb file already exists in the  daily ppa today (because I clicked the upload button multiple times)19:00
rharperyes19:00
rharperI think we have to wait for another commit, yeah ?19:00
rharperI merged a commit from paride today so tomorrow should get new hash19:00
blackboxsw_rharper: yes, it should just be a timing thing for our next commit19:00
blackboxsw_so I don't think we need to resolve this as falcojr is going to put up PRs for updated ubuntu/xenial|bionic|focal today for 20.3-X19:01
rharperyeah19:02
blackboxsw_once that lands the daily recipe will "just work" because commitish and commit count <revno> will have incremented19:02
blackboxsw_so ultimately there is nothing to do (and I think we may have already introduced the needed commit) because we are just waiting on19:03
blackboxsw_[  ]  cloud-init - 20.3-2566-g1f3a225a-0ubuntu1+1489~trunk~ubuntu16.04.1 which is already queued in the recipe build19:03
blackboxsw_seen at https://code.launchpad.net/~cloud-init-dev/+recipe/cloud-init-daily-xenial19:03
blackboxsw_so falcojr we should be able to proceed with a PR for your ubuntu/xenial with a  new-upstream-snapshot 371b392ced518e45be51089b6a67b362957b1dba19:03
blackboxsw_I think19:03
blackboxsw_that'd sync the latest released commit we published via ubuntu/devel into ubuntu/xenial, bionic and focal19:04
falcojrcool, sounds good19:10
falcojrok, blackboxsw_ , maybe this one is right now??? https://github.com/canonical/cloud-init/pull/55519:47
falcojrI tried the merge and didn't get conflicts this time19:47
blackboxsw_woot falcojr xenial recipe build completed and uploaded https://code.launchpad.net/~cloud-init-dev/+recipe/cloud-init-daily-xenial20:42
blackboxsw_falcojr: awesome getting this PR review now20:42
blackboxsw_falcojr: focal merged https://github.com/canonical/cloud-init/pull/553 :)21:08
blackboxsw_I'll queue an upload for focal -proposed of 20.321:09
blackboxsw_falcojr: lucasmoura over in ubuntu-release channel we get a bot message confirming queued -proposed uploads Unapproved: cloud-init (focal-proposed/main) [20.2-45-g5f7825e2-0ubuntu1~20.04.1 => 20.3-2-g371b392c-0ubuntu1~20.04.1] (core, edubuntu, ubuntu-cloud)21:14
blackboxsw_hhhhhhhrm21:14
blackboxsw_right  20.3-2-g371b392c-0ubuntu1~20.04.1]21:15
falcojrwhat does that mean?21:15
blackboxsw_ok falcojr that means that I have uploaded (because I have "upload" rights to cloud-init)  into the focal-proposed pocket Unapproved queue... which can be found here https://launchpad.net/ubuntu/focal/+queue?queue_state=1&queue_text=cloud-init21:17
falcojrahh, cool...just didn't know if the "unapproved" was significant21:17
blackboxsw_our next SRU step (once we actually have bionic and xenial uploads queued is to ping an SRU vangaurd in #ubuntu-release channel to say, please let our uploads into the official focal-proposed xenial-proposed bionic-proposed apt pockets so that public validation can take place21:18
blackboxsw_once sru vanguards "let the upload into -proposed" it goes from Unapproved -> Accepted I think21:18
falcojrgotcha21:18
blackboxsw_at which point public vms which have focal-proposed apt config enabled would be able to see, download and verify latest cloud-inbit21:19
* blackboxsw_ looks over your bionic and xenial (I'm thinking we still might get into a bit of a pickle with the ubuntu_advantage quilt patches.....)21:20
blackboxsw_falcojr: so, something is wrong with ubuntu/bionic and the quilt refresh route because when we  `quilt push -a` on your branch, we want it to render an cc_ubuntu_advantage.py  that knows how to talk to the old ubuntu-advantage-tools back client, as in accept commands like "ubuntu-advantage disable-livepatch" instead of the new python client which only accepts commands like "ua enable livepatch".22:05
falcojrand quilt refresh doesn't do that after I check out the old version?22:07
blackboxsw_so, if we ran quilt push -f; quilt refresh, that basically drops the existing debian quilt patch, removing the config parsing and documentation that tells you things like https://cloudinit.readthedocs.io/en/18.5/topics/modules.html#ubuntu-advantage22:07
blackboxsw_notice we completely reworked the supported config keys for ubuntu_advantage in tip of cloud-init https://cloudinit.readthedocs.io/en/latest/topics/modules.html#ubuntu-advantage22:08
blackboxsw_falcojr: right, it would only do that if we want the final product of cc_ubuntu_advantage.py to be the value at the commitish git checkout f247dd20ea73f8e153936bee50c57dae9440ecf7^ cloudinit/config/cc_ubuntu_advantage.py22:10
falcojrohhhhh, didn't realize that about the config keys22:11
blackboxsw_so I *think* what we need to do in both bionic and xenial cases is to start the quilt push -f (to apply all quilt patches that can apply) fix the end result to be the file fromgit checkout f247dd20ea73f8e153936bee50c57dae9440ecf7^ cloudinit/config/cc_ubuntu_advantage.py22:11
blackboxsw_we also (painfully) need to apply a minor subp patch (because we migrated subp utilities functions int tip to a new module and that needs to be applied to the result of   git checkout f247dd20ea73f8e153936bee50c57dae9440ecf7^ cloudinit/config/cc_ubuntu_advantage.py22:12
blackboxsw_soooo in short (not really short) to generate your ubuntu/xenial and ubuntu/bionic; you'll need to start over with a new-upstream-snapshot 371b392ced518e45be51089b6a67b362957b1dba22:13
blackboxsw_and then basically perform the following from this PR description22:13
blackboxsw_https://github.com/canonical/cloud-init/pull/43522:13
blackboxsw_it has an applicable ua-subp.patch that we'll need to use22:14
blackboxsw_and again, once we actually get the real python ubuntu-advantage-tools deb released to xenial and bionic , this whole mess goes away22:14
blackboxsw_as we can use tip of master for cc_ubuntu_advantage because the tooling supports it22:14
blackboxsw_"the tooling" == ubuntu-advantage-tools version 25.0 or later22:15
blackboxsw_which is our target shortly22:15
blackboxsw_falcojr: I'll push a PR up for ubuntu/xenial for you to compare against as you are doing it too22:15
falcojrblackboxsw_ is there an additional ubuntu advantage commit I need to cherry pick?22:18
falcojrafter the initial checkout?22:18
blackboxsw_falcojr: the order of commands in the PR description...22:18
blackboxsw_quilt push -f22:18
blackboxsw_(refresh-fix) git checkout f247dd2^ cloudinit/config/cc_ubuntu_advantage.py22:18
blackboxsw_(refresh-fix) git checkout f247dd2^ cloudinit/config/tests/test_ubuntu_advantage.py22:18
falcojrso just the subp one?22:18
blackboxsw_then patch -p1 < ua-subp.patch22:18
falcojrgotcha, I'll follow the PR :)22:19
blackboxsw_falcojr: only diff in that description is the following:22:19
blackboxsw_you aren't working in ubuntu/daily/xenial but ubuntu/xenial22:19
falcojrright22:20
blackboxsw_and your starting with new-upstream-snapshot 371b392ced518e45be51089b6a67b362957b1db instead of new-upstream-snapshot --skip-release  # to update debian/changelog and bump daily version22:20
blackboxsw_the rest I think applies22:20
blackboxsw_or at least I'm going through that now too to confirm22:20
blackboxsw_at the end we want to quilt push -a in your local ubuntu/xenial and grep my-token in cloudinit/config/cc_ubuntu_advantage.py to make sure we have to old config docs22:21
blackboxsw_then quilt pop -a to make sure all patches are popped off the stack22:21
blackboxsw_falcojr: I pushed me ubuntu/xenial branch (but haven't created a PR)22:22
blackboxsw_so, appropriate for diffs22:23
blackboxsw_against yours22:23
* blackboxsw_ is creating a PR documenting what I did for reference22:23
blackboxsw_'cause this is a pain22:23
blackboxsw_... not your fault, our fault for carrying around a quilt patch like this22:24
falcojrblackboxsw_: just pushed mine...I compared ours and the only difference was the names22:29
blackboxsw_sorry excellent falcojr lucasmoura I added a dummy WIP PR just with that context again https://github.com/canonical/cloud-init/pull/55922:32
blackboxsw_which we will reject and take falcor's I believe22:33
blackboxsw_cool, ok I'm trying to build yours and confirm22:34
blackboxsw_falcojr: if you could do the sameish for bionic then I think we are "good"22:34
falcojryep22:36
blackboxsw_It's also EOD, so no worries if you want to grab that tomorrow. we'll probably have to release 2 mondays from now as it's late ubuntu-archive-wise anyway22:37
falcojrI'll finish up this branch real quick and then call it day22:37
blackboxsw_so trying to squeeze the -proposed upload in by this time may be too late to make the cut for "SRU aging" of 7 days22:37
blackboxsw_thanks sir22:37
blackboxsw_build-package/sbuild-it worked on your xenial branch I'm running build-and-push to upload it to -proposed queue now22:39
falcojrgreat, also I'm not able to push tags upstream, so if we want to tag these somebody else will have to do it22:40
blackboxsw_build-and-push does that22:40
falcojrah, great22:40
blackboxsw_so it should work... and we need to sort your ability to push tags  in the future22:41
blackboxsw_ok should be a bot message coming up in #ubuntu-release channel shortly with the xenial upload22:41
blackboxsw_I'm generating my own local ubuntu/bionic now22:42
falcojrahhh, was just gonna say mines different! :)22:43
blackboxsw_just pushed22:43
blackboxsw_oops need --force22:44
blackboxsw_ebbaff6ccf57777f729c386365f6a6969d1b6982 commitish on my branch is what I have22:44
blackboxsw_diffing yours now22:44
blackboxsw_+1 falcojr great work22:44
blackboxsw_ok I think we are good22:44
blackboxsw_I'll sort details if anything else is missing22:44
blackboxsw_I have a bit more before EOD22:45
blackboxsw_thanks again22:45
falcojrwahoo! Cool, thanks22:45
blackboxsw_ok uploads all queued in Unapproved state for xenial bionic and focal. just waiting on Ubuntu SRU vanguard to approve so we can begin testing23:00

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