[8.7](backport #2384) Minor code cleanup (#2464)

* Minor code cleanup (#2384)

* Formatting

* Update ubuntu release

* Revert unsafe change

(cherry picked from commit e4768d5)

# Conflicts:
#	Vagrantfile
#	internal/pkg/agent/application/coordinator/coordinator.go

* Resolving conflicts

---------

Co-authored-by: Shaunak Kashyap <ycombinator@gmail.com>

internal/pkg/agent/application/coordinator/coordinator.go

@@ -828,7 +828,12 @@ func (c *Coordinator) compute() (map[string]interface{}, []component.Component,
 		configInjector = c.monitorMgr.MonitoringConfig
 	}
 
-	comps, err := c.specs.ToComponents(cfg, configInjector, c.state.logLevel, c.agentInfo)
+	comps, err := c.specs.ToComponents(
+		cfg,
+		configInjector,
+		c.state.logLevel,
+		c.agentInfo,
+	)
 	if err != nil {
 		return nil, nil, fmt.Errorf("failed to render components: %w", err)
 	}

pkg/component/runtime/command.go

@@ -148,6 +148,7 @@ func (c *CommandRuntime) Run(ctx context.Context, comm Communicator) error {
 			if sendExpected || c.state.unsettled() {
 				comm.CheckinExpected(c.state.toCheckinExpected(), nil)
 			}
+
 			if changed {
 				c.sendObserved()
 			}

pkg/component/runtime/manager.go

@@ -644,7 +644,7 @@ func (m *Manager) Actions(server proto.ElasticAgent_ActionsServer) error {
 
 // update updates the current state of the running components.
 //
-// This returns as soon as possible, work is performed in the background to
+// This returns as soon as possible, work is performed in the background.
 func (m *Manager) update(components []component.Component, teardown bool) error {
 	m.mx.Lock()
 	defer m.mx.Unlock()
@@ -660,7 +660,8 @@ func (m *Manager) update(components []component.Component, teardown bool) error
 	newComponents := make([]component.Component, 0, len(components))
 	for _, comp := range components {
 		touched[comp.ID] = true
-		if existing, ok := m.current[comp.ID]; ok {
+		existing, ok := m.current[comp.ID]
+		if ok {
 			// existing component; send runtime updated value
 			existing.currComp = comp
 			if err := existing.runtime.Update(comp); err != nil {

pkg/component/runtime/runtime_comm.go

@@ -132,7 +132,10 @@ func (c *runtimeComm) WriteConnInfo(w io.Writer, services ...client.Service) err
 	return nil
 }
 
-func (c *runtimeComm) CheckinExpected(expected *proto.CheckinExpected, observed *proto.CheckinObserved) {
+func (c *runtimeComm) CheckinExpected(
+	expected *proto.CheckinExpected,
+	observed *proto.CheckinObserved,
+) {
 	if c.agentInfo != nil && c.agentInfo.AgentID() != "" {
 		expected.AgentInfo = &proto.CheckinAgentInfo{
 			Id:       c.agentInfo.AgentID(),