=== harlowja is now known as harlowja_away
=== alexpilotti_ is now known as alexpilotti
[15:47] <smoser> harlowja_away, around ?
[17:09] <smoser> harlowja_away, :-( https://bugs.launchpad.net/cloud-init/+bug/1400793
[17:11] <gholms> Ouch.
=== harlowja_away is now known as harlowja
[18:31] <harlowja> hmmm
[18:35] <harlowja> lets see here
[18:39] <harlowja> dr.josh is on the case, ha
[18:44] <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:45] <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:53] <smoser> harlowja, in  aminute will look
[18:53] <harlowja> k
[19:25] <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:28] <harlowja> although i can probably remove the change in  cloudinit/handlers/cloud_config.py  but anyway
[19:29] <smoser> harlowja, will in a minute. you sir rock.
[19:30] <harlowja> ha
[19:30] <harlowja> damn merging logic
[19:30] <harlowja> lol
[19:36] <smoser> harlowja, right. mergemanydict works as i wanted it to
[19:37] <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:41] <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:42] <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:43] <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:44] <smoser> you ahve to know that its in location 0
[19:45] <harlowja> ya
[19:45] <harlowja> thats a tricky one
=== harlowja is now known as harlowja_away
=== harlowja_away is now known as harlowja