Skip to content

Restart receptor if reload config fails#61

Open
TheRealHaoLiu wants to merge 1 commit intoansible:mainfrom
TheRealHaoLiu:fail-reload-fallback
Open

Restart receptor if reload config fails#61
TheRealHaoLiu wants to merge 1 commit intoansible:mainfrom
TheRealHaoLiu:fail-reload-fallback

Conversation

@TheRealHaoLiu
Copy link
Copy Markdown
Member

@TheRealHaoLiu TheRealHaoLiu commented Oct 23, 2023

RUNNING HANDLER [ansible.receptor.setup : Reload Receptor] ****************************************************************************
fatal: [remote-execution]: FAILED! => {"changed": true, "cmd": ["receptorctl", "--socket", "/var/run/receptor/receptor.sock", "reload"], "delta": "0:00:00.174893", "end": "2023-10-23 20:36:19.259809", "failed_when_result": true, "msg": "non-zero return code", "rc": 3, "start": "2023-10-23 20:36:19.084916", "stderr": "ERROR: a non-reloadable config action 'work-command' was modified or added. Must restart receptor for these changes to take effect ERRORCODE 3", "stderr_lines": ["ERROR: a non-reloadable config action 'work-command' was modified or added. Must restart receptor for these changes to take effect ERRORCODE 3"], "stdout": "", "stdout_lines": []}
...ignoring

this happen when u install receptor as a execution node on a VM that was previously a hop node

fosterseth
fosterseth previously approved these changes Oct 23, 2023
ansible.builtin.systemd:
name: receptor
state: restarted
enabled: true
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Question here. Let say the same run changed receptor systemd override file in /etc/ , do we launch daemon-reload somehow before this reload in that case, or do we need to include the daemon_reload field in this systemd call?

rebeccahhh
rebeccahhh previously approved these changes Oct 24, 2023
Copy link
Copy Markdown
Member

@rebeccahhh rebeccahhh left a comment

Choose a reason for hiding this comment

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

Do you need to

Co-Authored-By: Seth Foster <fosterseth@users.noreply.github.com>
@TheRealHaoLiu TheRealHaoLiu dismissed stale reviews from fosterseth and rebeccahhh via d478237 April 1, 2025 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants