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

Add devspace print to debug patches #893

Closed
shaunc opened this issue Jan 23, 2020 · 6 comments
Closed

Add devspace print to debug patches #893

shaunc opened this issue Jan 23, 2020 · 6 comments
Assignees
Labels
area/config Issues related to the config kind/feature New feature or feature request

Comments

@shaunc
Copy link
Contributor

shaunc commented Jan 23, 2020

Is your feature request related to a problem?

Debugging a series of patches seems unnecessarily cumbersome.

Which solution do you suggest?

A new cli command, devspace print, which prints the configuration for the current or given profile after all patching and variable substitution. Optionally, could merge configuration from dependencies.

Which alternative solutions exist?

Figuring out what you did wrong by staring at your code & the error messages.

Additional context

/kind feature

@FabianKramm
Copy link
Collaborator

FabianKramm commented Jan 23, 2020

@shaunc thanks for opening this issue! You can see the patched configuration in the UI under config, but yes we could also think about an additional command that just prints the loaded configuration.

@FabianKramm FabianKramm added area/config Issues related to the config kind/feature New feature or feature request labels Jan 23, 2020
@shaunc
Copy link
Contributor Author

shaunc commented Jan 23, 2020

Ah -- forgot about it in the UI. I found this feature helpful in serverless. I don't know if the UI even comes up if you have made a big enough blunder? :) But print could just deal with substitution so could be quite robust even if you aren't making sense.

@shaunc
Copy link
Contributor Author

shaunc commented Jan 23, 2020

(Could eventually be combine with other config management... lint? diff?)

@FabianKramm
Copy link
Collaborator

FabianKramm commented Jan 24, 2020

@shaunc yes sounds good, I think we can do this

@jerlam06 jerlam06 self-assigned this Jan 27, 2020
@jerlam06
Copy link
Contributor

The print command was implemented!

@shaunc
Copy link
Contributor Author

shaunc commented Jan 30, 2020

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/config Issues related to the config kind/feature New feature or feature request
Projects
None yet
Development

No branches or pull requests

3 participants