Skip to content

Add missing working dir replacement during agent duplication#6979

Merged
bdpiprava merged 1 commit intogocd:masterfrom
SebastianGoral-freenet:fix_multiple_agents
Sep 23, 2019
Merged

Add missing working dir replacement during agent duplication#6979
bdpiprava merged 1 commit intogocd:masterfrom
SebastianGoral-freenet:fix_multiple_agents

Conversation

@SebastianGoral-freenet
Copy link
Contributor

Issue: # none

Description: While using the provided script to duplicate an agent, we noticed that the working dir in the wrapper-properties.conf was not updated. This caused the new agents to fail during startup with the message "Already running agent launcher in this folder.".

@ghost
Copy link

ghost commented Sep 20, 2019

CLA assistant check
All committers have signed the CLA.

@bdpiprava
Copy link
Contributor

@SebastianGoral-md PR LGTM.

Context: This is required if installed agent version < 19.6.0 and then upgraded to version >= 19.6.0. During the upgrade, the wrapper-properties.conf file will be updated with the wrapper_working_dir=/var/lib/go-agent. As wrapper-properties.conf is included at the end in wrapper.conf file, It will override the property(wrapper_working_dir) configured in wrapper.conf.

Earlier script to run multiple agents is only updating the wrapper_working_dir=/var/lib/go-agent-${AGENT_ID} in wrapper.conf and it will work fine except above-mentioned use case.

@bdpiprava bdpiprava merged commit c7019d6 into gocd:master Sep 23, 2019
@maheshp maheshp added this to the Release 19.9.0 milestone Sep 26, 2019
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.

3 participants