Skip to content

Commit

Permalink
Dev/fix upgrade issue (#2)
Browse files Browse the repository at this point in the history
Summary:
Summary :
Upgrade fails due to wrong ansible user (when it is not root) , Now ansible user / docker user is taken from config.yml file.

Pull Request resolved: #2

Reviewed By: tommyhuynh

Differential Revision: D33551007

Pulled By: aclave1

fbshipit-source-id: a2eb604a88c55392ed2c4becd76987e23e7b1aea
  • Loading branch information
Maoz Maroody authored and facebook-github-bot committed Jan 12, 2022
1 parent b67eb42 commit b42a394
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions nms_stack/nms_cli/nms.py
Original file line number Diff line number Diff line change
Expand Up @@ -251,6 +251,8 @@ def upgrade(ctx, config_file, host, controller, image, tags, verbose, password):

generated_config = generate_common_configs(loaded_config)
if loaded_config:
generated_config["ansible_user"] = loaded_config["ansible_user"]
generated_config["docker_user"] = loaded_config["docker_user"]
controllers_list = loaded_config["controllers_list"]
controllers_list = [
ctrlr for ctrlr in controllers_list if ctrlr["name"] == controller
Expand Down

0 comments on commit b42a394

Please sign in to comment.