Skip to content

Commit

Permalink
Adding better warning message to let user know that config file is mi…
Browse files Browse the repository at this point in the history
…ssing and they need to set env parameters. (#347)

* Adding EventHub creation to the deployment script

* Upgrading API versions and fixing the if clause of KV property creation for EH

* Making the warning message more clear

* Minor edit
  • Loading branch information
jainr authored Jun 13, 2022
1 parent d18294c commit dde96da
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion feathr_project/feathr/client.py
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ def __init__(self, config_path:str = "./feathr_config.yaml", local_workspace_dir
self.envutils = envutils

if not os.path.exists(config_path):
self.logger.warning('Configuration path does not exist, you need to set the environment variables explicitly. For all the environment variables, please refer to https://github.com/linkedin/feathr/blob/main/feathr_project/feathrcli/data/feathr_user_workspace/feathr_config.yaml')
self.logger.warning('No Configuration file exist at the user provided config_path or the default config_path (./feathr_config.yaml), you need to set the environment variables explicitly. For all the environment variables that you need to set, please refer to https://github.com/linkedin/feathr/blob/main/feathr_project/feathrcli/data/feathr_user_workspace/feathr_config.yaml')

# Load all configs from yaml at initialization
# DO NOT load any configs from yaml during runtime.
Expand Down

0 comments on commit dde96da

Please sign in to comment.