Add RDECK_CONFIG_FILE environment variable #2368

Merged
merged 1 commit into from Mar 20, 2017

Conversation

Projects
None yet
2 participants
@dandunckelman
Contributor

dandunckelman commented Mar 14, 2017

This allows the ability to avoid modifying /etc/rundeck/profile and put the override in the OS-specific location:

  • Debian: /etc/default/rundeckd
  • RedHat: /etc/sysconfig/rundeckd

@dandunckelman dandunckelman referenced this pull request in voxpupuli/puppet-rundeck Mar 14, 2017

Merged

Enable installation/configuration of Rundeck v2.8.x. #316

@dandunckelman

This comment has been minimized.

Show comment
Hide comment
@dandunckelman

dandunckelman Mar 17, 2017

Contributor

@gschueler what's your 2 cents on this?

Should tests be added to ensure the expected behavior of the profile with the environment variables being overridden from elsewhere?

Contributor

dandunckelman commented Mar 17, 2017

@gschueler what's your 2 cents on this?

Should tests be added to ensure the expected behavior of the profile with the environment variables being overridden from elsewhere?

@gschueler gschueler added this to the 2.8.0 milestone Mar 20, 2017

@gschueler

This comment has been minimized.

Show comment
Hide comment
@gschueler

gschueler Mar 20, 2017

Member

it lgtm

Member

gschueler commented Mar 20, 2017

it lgtm

@gschueler gschueler merged commit d49b1f2 into rundeck:master Mar 20, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@dandunckelman

This comment has been minimized.

Show comment
Hide comment
@dandunckelman

dandunckelman Mar 20, 2017

Contributor

@gschueler thanks!

Contributor

dandunckelman commented Mar 20, 2017

@gschueler thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment