| prometheanfire | smoser: btw, I might have a gsoc person do the gentoo cloud-init revamp | 00:58 |
|---|---|---|
| === shardy is now known as shardy_lunch | ||
| === shardy_lunch is now known as shardy | ||
| smoser | prometheanfire, that'd be cool! | 14:32 |
| smoser | Odd_Bloke, re-thinking what i said yesterday. | 14:32 |
| smoser | this is not terribly different than what azure accomplishes in their datasource | 14:33 |
| smoser | they handle re-running the mkfs and mounts | 14:33 |
| === rangerpbzzzz is now known as rangerpb | ||
| rangerpb | smoser, nice email response .... i was just thinking through the use of the ovf file for the hostname, remember that azure ejects the "cdrom" with that file after it completes provisioning ... do you think that is going to be problematic for reboots, etc? | 15:55 |
| smoser | no. it should be ok. it is guaranteed to be there on first boot, and we check to see if the cached instance-id is the same as the current instance id (provided in smbios) to determine if hte cached value is valid | 15:57 |
| smoser | and fwiw... they say they eject it, but i've never seen that happen. | 15:57 |
| rangerpb | :) | 15:58 |
| smoser | (that said, my instances done last more than 30 minutes normally) | 15:58 |
| larsks | smoser: re: the new ci stuff, is it possible to have launchpad take the merge proposal from the commit being merged in order to avoid "No commit message was specified in the merge proposal."? I am going to do a couple of copy-and-paste updates, but that seems like busy work. | 17:00 |
| powersj | larsks: it is not unfortunately. Basically setup to let you propose a new message in the event that you have many commits and assumes you want to squash them all into one with the new message, rather than the last. | 17:02 |
| larsks | Yeah, I understand the reason. I am just spoiled by github, which if there is a single commit will just adopt that commit message for the pr, and only give you an empty field if there are multiple commits. | 17:02 |
| powersj | ah that would be nice :\ | 17:03 |
| smoser | larsks, i know. its a pita. | 17:03 |
| smoser | that is what it shoudl do, like github. | 17:03 |
| larsks | smoser: I've updated those merge requests, so hopefully ci will be happy. Any chance you will have time to look at them? Fixes for #1670052 and #1669504 | 17:08 |
| smoser | just commented on https://code.launchpad.net/~larsks/cloud-init/+git/cloud-init/+merge/318800 | 17:10 |
| smoser | that looks fine other than the flake8 errors... | 17:10 |
| larsks | smoser: fixed and updated... | 17:12 |
| smoser | larsks, didi harlow mention at all where the hard 3 nameserver restriction comes form ? | 17:13 |
| larsks | smoser: I commented in the issue I think: it's referenced in the resolv.conf(5) man page. | 17:13 |
| smoser | and maybe, if it is really 3 per protocol (ipv4 or ipv6) then ideally we'd still check that ? | 17:13 |
| larsks | I don't think it's *per protocol*. | 17:14 |
| larsks | I am pretty sure it is "anything more than three will be ignored". | 17:14 |
| smoser | k | 17:14 |
| smoser | larsks, for the resolvconf, lets at least WARN which ones we're adding. or more clearly state "only the first 3 are will be used" | 17:28 |
| larsks | Aren't we doing that? It logs a warning message with the nameserver that is being ignored. | 17:29 |
| larsks | Line 25 in the review. | 17:29 |
| smoser | if you just read that message, is it clear to you waht its going to do ? or what it did ? | 17:31 |
| smoser | it somewhat makes sense if it raises an exception | 17:31 |
| larsks | I don't think it makes sense at all if it raises an exception. THat means that data in your cloud providers environment can prevent you from logging into your new instance. | 17:32 |
| larsks | That seems like a real problem. | 17:32 |
| smoser | i agree with you there. | 17:32 |
| smoser | i was saying that the *message* makes sense as an exception | 17:32 |
| smoser | but not as a warning | 17:32 |
| smoser | raise Exception("Too many nameservers") | 17:32 |
| smoser | but as a warning: | 17:32 |
| larsks | Okay. | 17:33 |
| larsks | So..."Ignoring nameserver %s because that would exceed the maximum of 3 nameservers"? | 17:33 |
| smoser | well, "using the first 3" or something like that. | 17:33 |
| larsks | smoser: that message will get logged once per nameserver (for each ns beyond 3). Does "using the first 3" make sense (i.e., will someone reading the logs know what "the first 3" are?) | 17:40 |
| larsks | How about that (update I just pushed)? | 17:42 |
| smoser | lookiong | 17:42 |
| smoser | larsks, that looks good, thanks. looking at only the mp and not enough context, i was thinking that 'ns' was the list, not an individual name server. | 17:47 |
| larsks | Okay. New question, unrelated: I am trying to resolve a dependency cycle between cloud-init and os-collect-config (on RHEL). What is the goal of having cloud-final.service run After=multi-user.target? | 17:49 |
| larsks | (I'm not entirely sure what happens during the -final step)' | 17:49 |
| smoser | larsks, that one is merged. the other https://code.launchpad.net/~larsks/cloud-init/+git/cloud-init/+merge/318800 did you think you pushed flake8 fixes ? | 17:53 |
| larsks | smoser: I thought I did, but let me double check. | 17:53 |
| smoser | i just fetched here and 'tox -e flake8' showed errors (0e5610fd08) | 17:54 |
| larsks | There. | 17:56 |
| smoser | having final run after multi-user.target ... we ended up doing that to avoid some issues witih package instaallation | 17:56 |
| smoser | commit 34a26f7f59 talks about it and references 2 bugs that we were hitting. | 17:56 |
| larsks | Thanks, I'll take a look. | 17:57 |
| smoser | if package installation worked fine prior to multi-user.target on redhat (it may well .. due to services not starting by default), then i'm not opposed to it being different there. | 17:58 |
| smoser | the goal really is for '-final' to run "as late as possible" | 17:58 |
| smoser | see the somewhat limited text description i wrote at doc/rtd/topics/boot.rst | 17:59 |
| larsks | I was guessing I would find --no-block somewhere in there, and I did :) | 18:01 |
| rangerpb | smoser, ive been poking around the cloud-init code ... the suggestion to set the hostname from the cdrom metadata is prob worth pursuing. Did you have a place in mind for that? At what point in the cloud-init cycle is the network set up ? | 18:47 |
| smoser | rangerpb, give me a minute, i'll take a quick poke and give you a diff that might helip you get started. | 18:51 |
| rangerpb | awesome | 18:51 |
| smoser | rangerpb, http://paste.ubuntu.com/24132495/ | 19:46 |
| smoser | thats what i have... there is non-trivial work there. | 19:46 |
| smoser | the setting of hostname in this manger is generally i think a good thing (tm) | 19:46 |
| smoser | The other option would be for azure to act more like DigitalOcean and in its local data search, it could dhcp, bring up the network do all the configuration and then take it down, and let the OS bring it up. In its 'get_data()' then that would run in local mode, it would have to read the ovf file, and invoke dhcp and negotiate with the metadata service for the ssh keys. I don't think this is supportable with the legacy "use walinu | 19:48 |
| smoser | xagent path". | 19:48 |
| rangerpb | right | 19:48 |
| rangerpb | getting to the ideal is not going to be trivial | 19:48 |
| smoser | generally speaking though, setting the hostname "centrally" is a win i think | 19:49 |
| rangerpb | yeah big time | 19:50 |
| === Hazelesque_ is now known as Hazelesque | ||
| msaikia | Hi, Just yesterday I got an email from Server Team CI bot about continuous integration failure. I looked into the failed log. Doesnot seem like anything to do with my change. This is my branch https://code.launchpad.net/~msaikia/cloud-init/+git/cloud-init/+merge/305427 | 21:22 |
| msaikia | Any steps that I need to take to fix it? | 21:22 |
| powersj | msaikia: you are right, the issue was fixed in a later version of cloud-init that was trying to access the local maas cfg files | 21:26 |
| === rangerpb is now known as rangerpbzzzz | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!