Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix VCP Install Agent Bug (hotfix 8.1.3) #2851

Merged
merged 6 commits into from
Jan 12, 2022

Conversation

sgilbride
Copy link
Contributor

Description

There was a bug in VOLTTRON Central, preventing agents from being installed properly from the interface. This was caused by the updated install agent functionality removing the method called by VCP (VOLTTRON Central Platform). To fix, VCP now uses the same functionality that Control Services uses, with a client-server structure for installing the agent.

Fixes #2841

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • Tested through manual set up of a VC and VCP instance on a local machine, and verified that a packaged listener agent could be installed, started, stopped, and removed.
  • Tested by creating a separate instance of VOLTTRON on the same machine, and remotely connecting to the VC instance. Verified that a packaged listener agent could be installed, started, stopped, and removed on this remote instance through the VC interface.

craig8 and others added 4 commits December 2, 2021 15:55
…ctionality.

Modified install_agent_vctl to be a wrapper. This allows isntall_agent_remote
to return the agent uuid.
Updated install_agent on VCP to include additional optional fileargs to
account for all install_agent_remote options.
@sgilbride sgilbride changed the base branch from releases/8.1.2 to releases/8.x January 12, 2022 18:06
Updated decodestring to decodebytes in VCP _install_agent method
@craig8 craig8 changed the title Fix VCP Install Agent Bug Fix VCP Install Agent Bug (hotfix 8.1.3) Jan 12, 2022
@craig8 craig8 merged commit 2f1c89d into VOLTTRON:releases/8.x Jan 12, 2022
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.

2 participants