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

calicoctl has silent failure if passed in config file does not exist #853

Closed
tmjd opened this issue Jun 14, 2017 · 1 comment
Closed

calicoctl has silent failure if passed in config file does not exist #853

tmjd opened this issue Jun 14, 2017 · 1 comment
Assignees
Labels

Comments

@tmjd
Copy link
Member

tmjd commented Jun 14, 2017

When attempting to pass in a config file to calicoctl node run I had misnamed the file and could not understand why the configuration in my file was being ignored. There is no indication that a passed in file does not exist.

Expected Behavior

If a file is passed with --config to calicoctl and the file does not exist then an error message will be printed. I think it would also be good to just exit in that case too but can understand not exiting to keep the current behavior.

Current Behavior

Passing in a non-existent file to calicoctl --config flag produces no message or error.

Possible Solution

If a non-default value is received from the --config option and the file does not exist then print an error message.

Steps to Reproduce (for bugs)

  1. Use calicoctl where the non-default values do not access the datastore
  2. Run `calicoctl get node --config non-existent-file.yaml
  3. See no indication that file does not exist.

Context

See summary

Your Environment

  • Calicoctl version v1.3.0
@bcreane bcreane self-assigned this Jul 5, 2017
@bcreane
Copy link
Contributor

bcreane commented Jul 6, 2017

Migrated a copy of this issue to projectcalico/calicoctl. See: 1678

@bcreane bcreane closed this as completed Jul 6, 2017
caseydavenport pushed a commit that referenced this issue Dec 14, 2021
Co-authored-by: marvin-tigera <marvin@projectcalico.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants