Skip to content
This repository was archived by the owner on Jul 28, 2026. It is now read-only.

Add HOSTNAME env variable to service file - #811

Merged
rfratto merged 3 commits into
mainfrom
hostname_env_service_file
Aug 11, 2021
Merged

Add HOSTNAME env variable to service file#811
rfratto merged 3 commits into
mainfrom
hostname_env_service_file

Conversation

@dfrankel33

Copy link
Copy Markdown
Contributor

PR Description

Update RPM & Deb service files to allow for expanding the $HOSTNAME variable in the agent config.

Which issue(s) this PR fixes

Notes to the Reviewer

PR Checklist

  • CHANGELOG updated
  • Documentation added
  • Tests updated

@CLAassistant

CLAassistant commented Aug 11, 2021

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@rfratto rfratto left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Before I merge, can you credit yourself in CHANGELOG marking this as an ENHANCEMENT?

@dfrankel33

Copy link
Copy Markdown
Contributor Author

Thank you! Before I merge, can you credit yourself in CHANGELOG marking this as an ENHANCEMENT?

done ✅

@rfratto
rfratto merged commit 5d2877a into main Aug 11, 2021
@rfratto
rfratto deleted the hostname_env_service_file branch August 11, 2021 16:45
@mattdurham mattdurham mentioned this pull request Sep 7, 2021
3 tasks
mattdurham pushed a commit that referenced this pull request Nov 11, 2021
* add HOSTNAME env variable to service file

* changelog

* changelog sort

Co-authored-by: Robert Fratto <robertfratto@gmail.com>
@github-actions github-actions Bot added the frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed. label Apr 11, 2024
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

frozen-due-to-age Locked due to a period of inactivity. Please open new issues or PRs if more discussion is needed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants