Skip to content

Commit

Permalink
Merge pull request openshift#2431 from bgilbert/ignition
Browse files Browse the repository at this point in the history
Bug 1927731: Revert "pkg/daemon: Add IgnitionVersion to Daemon"
  • Loading branch information
openshift-merge-robot authored Feb 25, 2021
2 parents 77b087f + 2203d88 commit 058c02d
Showing 1 changed file with 0 additions and 14 deletions.
14 changes: 0 additions & 14 deletions pkg/daemon/daemon.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,6 @@ type Daemon struct {
// os the operating system the MCD is running on
os OperatingSystem

// IgnitionVersion is the version of the installed Ignition binary on the system
IgnitionVersion string

// mock is set if we're running as non-root, probably under unit tests
mock bool

Expand Down Expand Up @@ -128,8 +125,6 @@ type Daemon struct {
}

const (
// pathIgnition is the path where the ignition binary resides
pathIgnition = "/usr/lib/dracut/modules.d/30ignition/ignition"
// pathSystemd is the path systemd modifiable units, services, etc.. reside
pathSystemd = "/etc/systemd/system"
// pathDevNull is the systems path to and endless blackhole
Expand Down Expand Up @@ -202,7 +197,6 @@ func New(
var (
osImageURL string
osVersion string
ignVersion string
err error
)

Expand All @@ -222,13 +216,6 @@ func New(
return nil, fmt.Errorf("error reading osImageURL from rpm-ostree: %v", err)
}
glog.Infof("Booted osImageURL: %s (%s)", osImageURL, osVersion)

ignVersionBytes, err := exec.Command(pathIgnition, "--version").Output()
if err != nil {
return nil, fmt.Errorf("error getting installed Ignition version: %v", err)
}
ignVersion = strings.SplitAfter(string(ignVersionBytes), " ")[1]
glog.Infof("Installed Ignition binary version: %s", ignVersion)
}

bootID := ""
Expand Down Expand Up @@ -257,7 +244,6 @@ func New(
return &Daemon{
mock: mock,
booting: true,
IgnitionVersion: ignVersion,
os: os,
NodeUpdaterClient: nodeUpdaterClient,
bootedOSImageURL: osImageURL,
Expand Down

0 comments on commit 058c02d

Please sign in to comment.