Project

General

Profile

Bug #4908

check-rudder-agent should source /etc/profile before starting

Added by Matthieu CERDA almost 6 years ago. Updated about 5 years ago.

Status:
Released
Priority:
3
Category:
Packaging
Target version:
Severity:
User visibility:
Effort required:
Priority:

Description

We usually source /etc/profile on our script to get environment variables.

check-rudder-agent does not, and we avoid problems by sometimes sourcing /etc/profile from the caller cron job.

This should be avoided, we should explicitely do it in check-rudder-agent :)

#1

Updated by Matthieu CERDA almost 6 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Matthieu CERDA to Nicolas CHARLES
  • % Done changed from 0 to 100
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/330

PR is ready.

#2

Updated by Nicolas CHARLES almost 6 years ago

  • Assignee changed from Nicolas CHARLES to Jonathan CLARKE

This should be linked to a ticket to remove the . /etc/profile in the techniques.
I'm not sure i can merge this, I let Jon decide on this one

#3

Updated by Matthieu CERDA almost 6 years ago

I propose that we simply clean up the cron.d entry we currently use to remove all the legacy crap that is useless now that we have check-rudder-agent. It is here since enough revisions and tester enough to be trusted, let's clean things up a bit :)

However, this has to be done in 2.11

#4

Updated by Matthieu CERDA almost 6 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:08a21e171ce2ce37d1547a8706d16700486ae30f.

#5

Updated by Jonathan CLARKE almost 6 years ago

Applied in changeset commit:da82add6046a7335051b349dbacc0dfbd0d5aa48.

#6

Updated by Vincent MEMBRÉ almost 6 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.10.1, which was released today.
Check out:

#7

Updated by Benoît PECCATTE about 5 years ago

  • Project changed from packages to Rudder
  • Category set to Packaging

Also available in: Atom PDF