| === harlowja is now known as harlowja_away | ||
| === alexpilotti_ is now known as alexpilotti | ||
| smoser | harlowja_away, around ? | 15:47 |
|---|---|---|
| smoser | harlowja_away, :-( https://bugs.launchpad.net/cloud-init/+bug/1400793 | 17:09 |
| gholms | Ouch. | 17:11 |
| === harlowja_away is now known as harlowja | ||
| harlowja | hmmm | 18:31 |
| harlowja | lets see here | 18:35 |
| harlowja | dr.josh is on the case, ha | 18:39 |
| harlowja | so the existing mergemany dict seems to be operating ok | 18:44 |
| harlowja | http://paste.ubuntu.com/9445284/ seems to be ok there | 18:44 |
| harlowja | smoser can u verify that, just want to make sure thats right/expected | 18:45 |
| harlowja | it outputs | 18:45 |
| harlowja | { | 18:45 |
| harlowja | "snappy": { | 18:45 |
| harlowja | "packages": [ | 18:45 |
| harlowja | "orig_pkg1", | 18:45 |
| harlowja | "orig_pkg2" | 18:45 |
| harlowja | ], | 18:45 |
| harlowja | "ssh_enabled": true | 18:45 |
| harlowja | }, | 18:45 |
| harlowja | "ssh_authorized_keys": [ | 18:45 |
| harlowja | "mykey@smoser" | 18:45 |
| harlowja | ] | 18:45 |
| harlowja | } | 18:45 |
| smoser | harlowja, in aminute will look | 18:53 |
| harlowja | k | 18:53 |
| harlowja | smoser if u get some time, try https://code.launchpad.net/~harlowja/cloud-init/adjusted-merging | 19:25 |
| harlowja | i think that should address this | 19:25 |
| harlowja | http://bazaar.launchpad.net/~harlowja/cloud-init/adjusted-merging/revision/1046 | 19:25 |
| harlowja | mainly that | 19:25 |
| harlowja | although i can probably remove the change in cloudinit/handlers/cloud_config.py but anyway | 19:28 |
| smoser | harlowja, will in a minute. you sir rock. | 19:29 |
| harlowja | ha | 19:30 |
| harlowja | damn merging logic | 19:30 |
| harlowja | lol | 19:30 |
| smoser | harlowja, right. mergemanydict works as i wanted it to | 19:36 |
| smoser | well, as i wanted cloud-config merging to work "by default" | 19:37 |
| harlowja | try that branch, think it will be ok for u | 19:37 |
| smoser | harlowja, yeah, that looks like it fixes it the way i want. | 19:41 |
| harlowja | cools | 19:41 |
| smoser | but the changes to cloud_cofig.py actually dont matter, right ? | 19:41 |
| harlowja | ya | 19:41 |
| smoser | just th m_dict change is what matter.s | 19:41 |
| harlowja | that was just me tweaking stuff | 19:41 |
| harlowja | lol | 19:41 |
| harlowja | the unittest does matter | 19:42 |
| harlowja | check that out | 19:42 |
| smoser | yeah, and thats fine. | 19:42 |
| smoser | right. | 19:42 |
| smoser | ./tools/ccfg-merge-debug works again on trunk | 19:42 |
| smoser | which is nice. for just testing things | 19:42 |
| smoser | i looked at jsonp a while ago | 19:42 |
| smoser | tried to do somethign with it. | 19:42 |
| smoser | the big missing thin gis that you cant take an element out of a list by value | 19:43 |
| smoser | in any ay. | 19:43 |
| smoser | in any way. | 19:43 |
| smoser | ie, if i have: | 19:43 |
| smoser | cloud_config_modules: | 19:43 |
| smoser | ssh | 19:43 |
| smoser | apt-get-config | 19:43 |
| smoser | module-3 | 19:43 |
| harlowja | hmmm, sounds like an new feature, | 19:43 |
| smoser | theres no way to say "remove 'ssh' from cloud_config_modules" | 19:43 |
| smoser | you ahve to know that its in location 0 | 19:44 |
| harlowja | ya | 19:45 |
| harlowja | thats a tricky one | 19:45 |
| === harlowja is now known as harlowja_away | ||
| === harlowja_away is now known as harlowja | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!