Skip to content

Add docs for vault monitor#9218

Merged
raskchanky merged 2 commits into
masterfrom
vault-monitor-docs
Jun 29, 2020
Merged

Add docs for vault monitor#9218
raskchanky merged 2 commits into
masterfrom
vault-monitor-docs

Conversation

@raskchanky
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread website/pages/docs/commands/monitor.mdx Outdated
server. This command accepts a log level as an argument, which can be different
from the log level that the Vault server was started with.

The `monitor` command honors the same the variables that the base command
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I don't know that we need to call out standard behaviour with respect to tokens. I do think it makes sense to keep the bit about VAULT_ADDR, since it's relatively unusual that we're behaving differently depending on which node you connect to.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I looked at the docs for a few different commands to get a sense for how our docs are written before writing this up. That particular piece was in the docs for vault debug which is how it got included here. I'll update that part.

@calvn calvn added this to the 1.5 milestone Jun 24, 2020
@raskchanky raskchanky merged commit c62b750 into master Jun 29, 2020
@raskchanky raskchanky deleted the vault-monitor-docs branch June 29, 2020 19:23
andaley pushed a commit that referenced this pull request Jul 17, 2020
eualin pushed a commit to eualin/vault that referenced this pull request Sep 9, 2025
…icorp#9218)

* add fix and tests for importing keys with derivation enabled

* add changelog

* add check to fix kdf for existing imported keys

* only set kdf on derived keys

Co-authored-by: Rachel Culpepper <84159930+rculpepper@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

3 participants