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

Wrong error message: "nothing to run" #39

Open
Nuru opened this issue Nov 29, 2020 · 3 comments
Open

Wrong error message: "nothing to run" #39

Nuru opened this issue Nov 29, 2020 · 3 comments

Comments

@Nuru
Copy link

Nuru commented Nov 29, 2020

$ VARIANT_DIR=cli variant run helmfile lint nginx-ingress -s ue2-dev
nothing to runError: nothing to run

This appears to be due to subjob "aws eks kubeconfig" error

Error: .variant2/cache/https_git@github_com_cloudposse_atmos.ref=master/modules/kubeconfig/kubeconfig.variant:40,42-73: Unsupported attribute; This object does not have an attribute named "kubeconfig_profile_environment".

Ideally we should see an error message with some kind of stack trace (doesn't have to include line numbers) like

"aws eks kubeconfig": Error:  Unsupported attribute; This object does not have an attribute named "kubeconfig_profile_environment"
"helmfile subcommand"
"helmfile lint"
mumoshu added a commit that referenced this issue Nov 29, 2020
So that we get more descriptive error message in cases like #39
@mumoshu
Copy link
Owner

mumoshu commented Nov 29, 2020

@Nuru Thanks for reporting! Pleaser see #40 - After that we get more descriptive errors like the below:

$ VARIANT_DIR=cli variant run helmfile lint nginx-ingress -s ue2-dev
config "helmfile-command": source 0: job "helmfile-command": job "helmfile-command" is not definedError: job "helmfile subcommand": config "helmfile-command": source 0: job "helmfile-command": job "helmfile-command" is not defined

In this specific case, it seems that the root cause is that job "helmfile-command" is not defined - the helmfile-command job is not defined in any of atomis modules.

mumoshu added a commit that referenced this issue Nov 29, 2020
* Add more context to run error

So that we get more descriptive error message in cases like #39
@mumoshu
Copy link
Owner

mumoshu commented Nov 29, 2020

@Nuru v0.36.0 has been released with #40

@Nuru
Copy link
Author

Nuru commented Nov 29, 2020

@Nuru v0.36.0 has been released with #40

I do not mean to sound ungrateful, but have you fixed #36 yet, or am I still forced to update everywhere now?

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

No branches or pull requests

2 participants