Adding two host entries; Entries are getting combined into one line on the first run, then properly inserted in the second run

[jberry]

See output below. Trying to add two entries. First run of puppet combines the entries into one /etc/hosts line. Second run properly cleans up /etc/hosts and puts them as two lines. Issue also happens even when both host entries are not coming from the same class, or even the same module. (On another setup, I had 3 host entries specified throughout the environment, and all 3 were combined into one line. Took 2 more puppet runs for it to be clean)

####CLASS###
class requiredhostsentries{

  host{"managementhost2.private.com":
    ensure => present,
    ip => "192.168.10.12",
    host_aliases => ["managementhost2"],
    provider => augeas,
  }

  host{"myrepohost1.private.com":
    ensure => present,
    ip => "192.168.10.192",
    host_aliases => ["myrepohost1"],
    provider => augeas,
  }
}
#####END OF CLASS#####

###PUPPET RUNS###

[root@clientbox40 ~]# puppet agent --test
Info: Retrieving pluginfacts
Info: Retrieving plugin
<snip misc stuff>
Info: Applying configuration version '1400087970'
Notice: /Stage[main]/Required::Required_fmi/Host[managementhost.private.com]/ensure: created
Notice: /Stage[main]/Required::Required_fmi/Host[myrepohost1.private.com]/ensure: created
Notice: Finished catalog run in 7.00 seconds

[root@clientbox40 ~]# cat /etc/hosts
<snip unrelevant hosts>
192.168.10.192  myrepohost1.private.com managementhost2 myrepohost1
<eof>

######Next Run of Puppet######
[root@clientbox40 ~]# puppet agent --test
<snip misc stuff>
Info: Applying configuration version '1400087970'
Notice: /Stage[main]/Required::Required_fmi/Host[managementhost.private.com]/ensure: created
Notice: /Stage[main]/Required::Required_fmi/Host[myrepohost1.private.com]/host_aliases: host_aliases changed 'managementhost2 myrepohost1' to 'myrepohost1'
Notice: Finished catalog run in 6.77 seconds

[root@clientbox40 ~]# cat /etc/hosts
<snip unrelevant hosts>
192.168.10.192  myrepohost1.private.com myrepohost1
192.168.10.12   managementhost2.private.com managementhost2

 ####Information###

Versions of software/modules:
Puppet 3.5.1-1
Foreman 1.5.0-1
Augeasproviders 1.1.0, pulled from git://github.com/hercules-team/augeasproviders

[raphink]

This bug was introduced by commit 7808760 which shares an Augeas handler for Puppet >= 3.4.

[domcleal]

@raphink fyi, I have a WIP branch here to fix it, but it's not quite done... https://github.com/domcleal/augeasproviders/tree/101-seq-shared

[raphink]

Ah great. I hadn't seen how using the 01 trick would be problematic when sharing the handler! We should probably provide a helper in the lib for that, as it may occur in quite a few providers.

[domcleal]

Yeah, a helper to work out the correct seq number based on the current in-memory tree was the way I was going with the fix.

[raphink]

This bug is really problematic. We should probably port is to 1.X and release a bug fix.

[raphink]

PR #107 fixes this issue.

[raphink]

Fixed in cb07321