Skip to content

Conversation

@Taction
Copy link
Member

@Taction Taction commented Jun 1, 2021

Description

Optimize vault secret component error output

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #908

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

Copy link
Member

@pkedy pkedy left a comment

Choose a reason for hiding this comment

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

LGTM

/cc @halspang

@codecov
Copy link

codecov bot commented Jun 28, 2021

Codecov Report

Merging #909 (e39a0d0) into master (74af5b6) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #909      +/-   ##
==========================================
- Coverage   35.37%   35.36%   -0.01%     
==========================================
  Files         136      136              
  Lines       10728    10730       +2     
==========================================
  Hits         3795     3795              
- Misses       6551     6553       +2     
  Partials      382      382              
Impacted Files Coverage Δ
secretstores/hashicorp/vault/vault.go 32.29% <0.00%> (-0.41%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 74af5b6...e39a0d0. Read the comment docs.

@pkedy pkedy self-assigned this Jun 28, 2021
@yaron2 yaron2 merged commit 61e07bc into dapr:master Jul 8, 2021
Taction added a commit to Taction/components-contrib that referenced this pull request Jul 15, 2021
* master: (23 commits)
  Adds support for TTL in Redis State Store (dapr#990)
  Adds support for TTL in Memcached State Store (PR Fixup) (dapr#1011)
  Adds support for TTL in Cassandra State Store (dapr#996)
  Return blob metadata and add list operation (dapr#999)
  Added new output binding for Cosmos Graph DB - Gremlin (dapr#885)
  Adds support for TTL in CosmosDB State Store (dapr#991)
  Delete unused code and update test (dapr#924)
  Return SystemProperties on events from EventHubs bindings and pubsub (dapr#1009)
  Implement E2E tests for zeebe (dapr#973)
  rebase (dapr#982)
  Update CODEOWNERS (dapr#987)
  ci: add test skip case (dapr#922)
  fix: Mysql should support more data types. dapr#923 (dapr#926)
  fix: Dapr runtime panic when handle Pub/Sub (dapr#3281) (dapr#967)
  switch to golang-jwt (dapr#993)
  unmarshal getBlobRetryCount as int (dapr#919)
  secretstores: support more format of azure key vault (dapr#944)
  Optimize vault secret component error output (dapr#909)
  Convert AZURE_KEYVAULT in conformance.yml a GitHub secret (dapr#1002)
  Use AzureKeyVaultName param for Azure Keyvault conformance test (dapr#975)
  ...

# Conflicts:
#	secretstores/hashicorp/vault/vault.go
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.

Optimize vault secret component error output

4 participants