Skip to content

Commit

Permalink
Merge pull request #9203 from hakman/fix-repo-pkgs
Browse files Browse the repository at this point in the history
Fix repo packages not being installed
  • Loading branch information
k8s-ci-robot authored May 29, 2020
2 parents 2534023 + f2a87f7 commit aa6e09c
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions upup/pkg/fi/nodeup/nodetasks/package.go
Original file line number Diff line number Diff line change
Expand Up @@ -297,7 +297,7 @@ func (_ *Package) RenderLocal(t *local.LocalTarget, a, e, changes *Package) erro

if a == nil || changes.Version != nil {
klog.Infof("Installing package %q (dependencies: %v)", e.Name, e.Deps)
var localPkgs []string
var pkgs []string

if e.Source != nil {
// Install a deb or rpm.
Expand All @@ -317,10 +317,10 @@ func (_ *Package) RenderLocal(t *local.LocalTarget, a, e, changes *Package) erro
}

// Download all the debs/rpms.
localPkgs = make([]string, 1+len(e.Deps))
pkgs = make([]string, 1+len(e.Deps))
for i, pkg := range append([]*Package{e}, e.Deps...) {
local := path.Join(localPackageDir, pkg.Name+ext)
localPkgs[i] = local
pkgs[i] = local
var hash *hashing.Hash
if fi.StringValue(pkg.Hash) != "" {
parsed, err := hashing.FromString(fi.StringValue(pkg.Hash))
Expand All @@ -334,6 +334,8 @@ func (_ *Package) RenderLocal(t *local.LocalTarget, a, e, changes *Package) erro
return err
}
}
} else {
pkgs = append(pkgs, e.Name)
}

var args []string
Expand All @@ -350,7 +352,7 @@ func (_ *Package) RenderLocal(t *local.LocalTarget, a, e, changes *Package) erro
} else {
return fmt.Errorf("unsupported package system")
}
args = append(args, localPkgs...)
args = append(args, pkgs...)

klog.Infof("running command %s", args)
cmd := exec.Command(args[0], args[1:]...)
Expand Down

0 comments on commit aa6e09c

Please sign in to comment.