Closed juliantaylor closed 6 months ago
Yeah, the implementation to save all Puppet environments in a helper struct was pretty naive:
if _, ok := allEnvironments[prefix+renamedBranch]; !ok {
allEnvironments[prefix+renamedBranch] = true
} else {
Fatalf("Renamed environment naming conflict detected with renamed environment " + prefix + renamedBranch)
}
This does not work with same branch names when someone isn't using a prefix for their environments, but different basedir
settings
I've adjusted this to the absolute filepath of the environment:
if _, ok := allEnvironments[filepath.Join(sa.Basedir, prefix+renamedBranch)]; !ok {
allEnvironments[filepath.Join(sa.Basedir, prefix+renamedBranch)] = true
} else {
Fatalf("Renamed environment naming conflict detected with renamed environment " + prefix + renamedBranch)
}
Should be fixed with #222 I'm in the process of releasing a new version.
syncing multiple git sources with the same branch does not work anymore probably since strip_components was added:
gita and gitb contain the same branches and no puppetfile.
This then fails despite no renaming happening at all:
this used to work fine in version 0.8.9