Skip to content

Config diagnostics #501

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Oct 14, 2024
Merged

Config diagnostics #501

merged 4 commits into from
Oct 14, 2024

Conversation

K-Phoen
Copy link
Member

@K-Phoen K-Phoen commented Oct 10, 2024

While helping a first time grizzly user, we ran into some difficulties trying to identify some configuration issues.

The two changes in this PR would have helped us:

  • the first commit introduces a new grr config check command, exposing a quick view of what provider is active or not and whether they're online or not (see screenshot below)
  • the second one fixes a bug where grr config get fails to take environment variables into configuration when printing the current config.

Screenshot from 2024-10-10 17-28-17
@K-Phoen K-Phoen self-assigned this Oct 10, 2024
@K-Phoen K-Phoen requested a review from a team October 10, 2024 15:28
malcolmholmes
malcolmholmes approved these changes Oct 10, 2024
View reviewed changes
Copy link
Contributor

@malcolmholmes malcolmholmes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice
@K-Phoen K-Phoen merged commit 9ba6530 into main Oct 14, 2024
7 checks passed
@K-Phoen K-Phoen deleted the config-diagnostics branch October 14, 2024 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@K-Phoen @malcolmholmes