Bug #4908

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

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

Target version:
User visibility:
Effort required:


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 :)


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

PR is ready.


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


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


Updated by Matthieu CERDA almost 6 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:08a21e171ce2ce37d1547a8706d16700486ae30f.


Updated by Jonathan CLARKE almost 6 years ago

Applied in changeset commit:da82add6046a7335051b349dbacc0dfbd0d5aa48.


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:


Updated by Benoît PECCATTE about 5 years ago

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

Also available in: Atom PDF